feat(files/user-config): make configs admin-configurable

Signed-off-by: Kai Henseler <[email protected]>
IONOS-Productivity · Dec 11, 2024 · 9800c9e · 9800c9e
1 parent 50c5af4
commit 9800c9e
Show file tree

Hide file tree

Showing 2 changed files with 63 additions and 10 deletions.
diff --git a/apps/files/lib/Service/UserConfig.php b/apps/files/lib/Service/UserConfig.php
@@ -6,6 +6,7 @@
 namespace OCA\Files\Service;
 
 use OCA\Files\AppInfo\Application;
+use OCP\AppFramework\Services\IAppConfig;
 use OCP\IConfig;
 use OCP\IUser;
 use OCP\IUserSession;
@@ -53,7 +54,7 @@ class UserConfig {
 	protected IConfig $config;
 	protected ?IUser $user = null;
 
-	public function __construct(IConfig $config, IUserSession $userSession) {
+	public function __construct(IConfig $config, IUserSession $userSession, protected IAppConfig $appConfig) {
 		$this->config = $config;
 		$this->user = $userSession->getUser();
 	}
@@ -139,9 +140,11 @@ public function getConfigs(): array {
 
 		$userId = $this->user->getUID();
 		$userConfigs = array_map(function (string $key) use ($userId) {
-			$value = $this->config->getUserValue($userId, Application::APP_ID, $key, $this->getDefaultConfigValue($key));
-			// If the default is expected to be a boolean, we need to cast the value
-			if (is_bool($this->getDefaultConfigValue($key)) && is_string($value)) {
+			$value = $this->config->getUserValue($userId, Application::APP_ID, $key, null);
+			if ($value === null) {
+				$value = $this->appConfig->getAppValueBool($key, $this->getDefaultConfigValue($key));
+			} else if (is_bool($this->getDefaultConfigValue($key)) && is_string($value)) {
+				// If the default value is expected to be a boolean, we need to cast the value
 				return $value === '1';
 			}
 			return $value;

diff --git a/apps/files/tests/Service/UserConfigTest.php b/apps/files/tests/Service/UserConfigTest.php
@@ -9,6 +9,7 @@
 
 use OCA\Files\AppInfo\Application;
 use OCA\Files\Service\UserConfig;
+use OCP\AppFramework\Services\IAppConfig;
 use OCP\IConfig;
 use OCP\IUser;
 use OCP\IUserSession;
@@ -34,6 +35,9 @@ class UserConfigTest extends \Test\TestCase {
 	/** @var IUserSession|\PHPUnit\Framework\MockObject\MockObject */
 	private $userSessionMock;
 
+	/** @var IAppConfig|\PHPUnit\Framework\MockObject\MockObject */
+	private $appConfigMock;
+
 	/**
 	 * @var UserConfig|\PHPUnit\Framework\MockObject\MockObject
 	 */
@@ -42,6 +46,7 @@ class UserConfigTest extends \Test\TestCase {
 	protected function setUp(): void {
 		parent::setUp();
 		$this->configMock = $this->createMock(IConfig::class);
+		$this->appConfigMock = $this->createMock(IAppConfig::class);
 
 		$this->userUID = static::getUniqueID('user_id-');
 		\OC::$server->getUserManager()->createUser($this->userUID, 'test');
@@ -67,6 +72,7 @@ protected function getUserConfigService(array $methods = []) {
 			->setConstructorArgs([
 				$this->configMock,
 				$this->userSessionMock,
+				$this->appConfigMock,
 			])
 			->setMethods($methods)
 			->getMock();
@@ -100,31 +106,31 @@ public function testThrowsExceptionWhenNoUserLoggedInForSetConfig(): void {
 		$this->expectException(\Exception::class);
 		$this->expectExceptionMessage('No user logged in');
 
-		$userConfig = new UserConfig($this->configMock, $this->userSessionMock);
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
 		$userConfig->setConfig('crop_image_previews', true);
 	}
 
 	public function testThrowsInvalidArgumentExceptionForUnknownConfigKey(): void {
 		$this->expectException(\InvalidArgumentException::class);
 		$this->expectExceptionMessage('Unknown config key');
 
-		$userConfig = new UserConfig($this->configMock, $this->userSessionMock);
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
 		$userConfig->setConfig('unknown_key', true);
 	}
 
 	public function testThrowsInvalidArgumentExceptionForInvalidConfigValue(): void {
 		$this->expectException(\InvalidArgumentException::class);
 		$this->expectExceptionMessage('Invalid config value');
 
-		$userConfig = new UserConfig($this->configMock, $this->userSessionMock);
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
 		$userConfig->setConfig('crop_image_previews', 'foo');
 	}
 
 	public function testSetsConfigSuccessfully(): void {
 		$this->configMock->expects($this->once())
 			->method('setUserValue')
 			->with($this->userUID, Application::APP_ID, 'crop_image_previews', '1');
-		$userConfig = new UserConfig($this->configMock, $this->userSessionMock);
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
 		$userConfig->setConfig('crop_image_previews', true);
 	}
 
@@ -135,7 +141,13 @@ public function testGetsConfigsWithDefaultValuesSuccessfully(): void {
 				return $default;
 			});
 
-		$userConfig = new UserConfig($this->configMock, $this->userSessionMock);
+		// pass the default app settings unchanged
+		$this->appConfigMock->method('getAppValueBool')
+			->willReturnCallback(function ($key, $default) {
+				return $default;
+			});
+
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
 		$configs = $userConfig->getConfigs();
 		$this->assertEquals([
 			'crop_image_previews' => true,
@@ -162,7 +174,13 @@ public function testGetsConfigsOverrideWithUserValuesSuccessfully(): void {
 				return $default;
 			});
 
-		$userConfig = new UserConfig($this->configMock, $this->userSessionMock);
+		// pass the default app settings unchanged
+		$this->appConfigMock->method('getAppValueBool')
+			->willReturnCallback(function ($key, $default) {
+				return $default;
+			});
+
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
 		$configs = $userConfig->getConfigs();
 		$this->assertEquals([
 			'crop_image_previews' => false,
@@ -173,4 +191,36 @@ public function testGetsConfigsOverrideWithUserValuesSuccessfully(): void {
 			'folder_tree' => true,
 		], $configs);
 	}
+
+	public function testGetsConfigsOverrideWithAppsValuesSuccessfully(): void {
+		$this->userSessionMock->method('getUser')->willReturn($this->userMock);
+
+		// set all user values to true
+		$this->configMock->method('getUserValue')
+			->willReturnCallback(function () {
+				return true;
+			});
+
+		// emulate override by the app config values
+		$this->appConfigMock->method('getAppValueBool')
+			->willReturnCallback(function ($key, $default) {
+				if ($key === 'crop_image_previews') {
+					return false;
+				} elseif ($key === 'show_hidden') {
+					return false;
+				}
+				return $default;
+			});
+
+		$userConfig = new UserConfig($this->configMock, $this->userSessionMock, $this->appConfigMock);
+		$configs = $userConfig->getConfigs();
+		$this->assertEquals([
+			'crop_image_previews' => false,
+			'show_hidden' => false,
+			'sort_favorites_first' => true,
+			'sort_folders_first' => true,
+			'grid_view' => true,
+			'folder_tree' => true,
+		], $configs);
+	}
 }