Fix shown but broken GitLab snippets

SISheogorath · SISheogorath · commit bcb7972607a1 · 2019-03-05T18:16:04.000+01:00
To provide a GitLab integration we need the GitLab integration to be
configured. Otherwise we shouldn't show the Snippet button.

This patch adds the requirement to the variable that decides if the
import from snippets button shows up or not.

Signed-off-by: Sheogorath &lt;sheogorath@shivering-isles.com&gt;
diff --git a/lib/config/index.js b/lib/config/index.js
@@ -128,7 +128,7 @@ if (config.gitlab && config.gitlab.version !== 'v4' && config.gitlab.version !==
   config.gitlab.version = 'v4'
 }
 // If gitlab scope is api, enable snippets Export/import
-config.isGitlabSnippetsEnable = (!config.gitlab.scope || config.gitlab.scope === 'api')
+config.isGitlabSnippetsEnable = (!config.gitlab.scope || config.gitlab.scope === 'api') && config.isGitLabEnable
 
 // Only update i18n files in development setups
 config.updateI18nFiles = (env === Environment.development)

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ if (config.gitlab && config.gitlab.version !== 'v4' && config.gitlab.version !==`
`128`	`128`	`config.gitlab.version = 'v4'`
`129`	`129`	`}`
`130`	`130`	`// If gitlab scope is api, enable snippets Export/import`
`131`		`-config.isGitlabSnippetsEnable = (!config.gitlab.scope \|\| config.gitlab.scope === 'api')`
	`131`	`+config.isGitlabSnippetsEnable = (!config.gitlab.scope \|\| config.gitlab.scope === 'api') && config.isGitLabEnable`
`132`	`132`
`133`	`133`	`// Only update i18n files in development setups`
`134`	`134`	`config.updateI18nFiles = (env === Environment.development)`