diff options
author | Pawel Boguslawski <pawel.boguslawski@ib.pl> | 2025-01-16 14:47:14 +0100 |
---|---|---|
committer | Andy Scherzinger <info@andy-scherzinger.de> | 2025-01-27 16:08:53 +0100 |
commit | c7bcfbf8649c01af19bf516b46b16d1fcf0eba5d (patch) | |
tree | 9e474030a4e6d924d608ab8450983914bb89b9c4 | |
parent | b70654aeed6bec4befd65641717a6d167d6ddbd0 (diff) | |
download | nextcloud-server-c7bcfbf8649c01af19bf516b46b16d1fcf0eba5d.tar.gz nextcloud-server-c7bcfbf8649c01af19bf516b46b16d1fcf0eba5d.zip |
fix: Hide "Create templates folder" option if templates are disabled in configuration
When both `skeletondirectory` and `templatedirectory` are set to empty
strings in configuration, templates folder creation should be disabled
and no Create templates folder option should be present
in new folder menu.
Related: https://github.com/nextcloud/server/issues/39266
Related: https://github.com/nextcloud/server/issues/46455
Author-Change-Id: IB#1156403
Signed-off-by: Pawel Boguslawski <pawel.boguslawski@ib.pl>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
-rw-r--r-- | apps/files/lib/Controller/ViewController.php | 1 | ||||
-rw-r--r-- | apps/files/src/newMenu/newTemplatesFolder.ts | 6 | ||||
-rw-r--r-- | config/config.sample.php | 13 |
3 files changed, 11 insertions, 9 deletions
diff --git a/apps/files/lib/Controller/ViewController.php b/apps/files/lib/Controller/ViewController.php index 4dd9f4141e2..2c242040731 100644 --- a/apps/files/lib/Controller/ViewController.php +++ b/apps/files/lib/Controller/ViewController.php @@ -194,6 +194,7 @@ class ViewController extends Controller { $this->eventDispatcher->dispatchTyped(new LoadViewer()); } + $this->initialState->provideInitialState('templates_enabled', ($this->config->getSystemValueString('skeletondirectory', '') !== '') || ($this->config->getSystemValueString('templatedirectory', '') !== '')); $this->initialState->provideInitialState('templates_path', $this->templateManager->hasTemplateDirectory() ? $this->templateManager->getTemplatePath() : false); $this->initialState->provideInitialState('templates', $this->templateManager->listCreators()); diff --git a/apps/files/src/newMenu/newTemplatesFolder.ts b/apps/files/src/newMenu/newTemplatesFolder.ts index 43a6f08b525..bf6862bda08 100644 --- a/apps/files/src/newMenu/newTemplatesFolder.ts +++ b/apps/files/src/newMenu/newTemplatesFolder.ts @@ -17,7 +17,9 @@ import PlusSvg from '@mdi/svg/svg/plus.svg?raw' import axios from '@nextcloud/axios' import logger from '../logger.ts' +const templatesEnabled = loadState<boolean>('files', 'templates_enabled', true) let templatesPath = loadState<string|false>('files', 'templates_path', false) +logger.debug('Templates folder enabled', { templatesEnabled }) logger.debug('Initial templates folder', { templatesPath }) /** @@ -57,8 +59,8 @@ export const entry = { iconSvgInline: PlusSvg, order: 30, enabled(context: Folder): boolean { - // Templates folder already initialized - if (templatesPath) { + // Templates disabled or templates folder already initialized + if (!templatesEnabled || templatesPath) { return false } // Allow creation on your own folders only diff --git a/config/config.sample.php b/config/config.sample.php index b0b0f21853b..f30bd92adbc 100644 --- a/config/config.sample.php +++ b/config/config.sample.php @@ -288,8 +288,9 @@ $CONFIG = [ /** * The directory where the skeleton files are located. These files will be - * copied to the data directory of new users. Leave empty to not copy any - * skeleton files. + * copied to the data directory of new users. Set empty string to not copy any + * skeleton files. If unset and templatedirectory is empty string, shipped + * templates will be used to create a template directory for the user. * ``{lang}`` can be used as a placeholder for the language of the user. * If the directory does not exist, it falls back to non dialect (from ``de_DE`` * to ``de``). If that does not exist either, it falls back to ``default`` @@ -298,18 +299,16 @@ $CONFIG = [ */ 'skeletondirectory' => '/path/to/nextcloud/core/skeleton', - /** * The directory where the template files are located. These files will be - * copied to the template directory of new users. Leave empty to not copy any + * copied to the template directory of new users. Set empty string to not copy any * template files. * ``{lang}`` can be used as a placeholder for the language of the user. * If the directory does not exist, it falls back to non dialect (from ``de_DE`` * to ``de``). If that does not exist either, it falls back to ``default`` * - * If this is not set creating a template directory will only happen if no custom - * ``skeletondirectory`` is defined, otherwise the shipped templates will be used - * to create a template directory for the user. + * To disable creating a template directory, set both skeletondirectory and + * templatedirectory to empty strings. */ 'templatedirectory' => '/path/to/nextcloud/templates', |