fixup! fix(profile): make sure the app cannot be disabled as it was i…

…n core Signed-off-by: skjnldsv <[email protected]>
nextcloud · Nov 13, 2024 · d9909aa · d9909aa
1 parent a0aaaf2
commit d9909aa
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 5 deletions.
diff --git a/apps/profile/appinfo/info.xml b/apps/profile/appinfo/info.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <!--
-  - SPDX-FileCopyrightText: 2018-2024 Nextcloud GmbH and Nextcloud contributors
-  - SPDX-License-Identifier: AGPL-3.0-only
+  - SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
+  - SPDX-License-Identifier: AGPL-3.0-or-later
 -->
 <info xmlns:xsi= "http://www.w3.org/2001/XMLSchema-instance"
 	  xsi:noNamespaceSchemaLocation="https://apps.nextcloud.com/schema/apps/info.xsd">
@@ -11,7 +11,7 @@
 	<description>Provides a customisable user profile interface.</description>
 	<version>1.0.0</version>
 	<licence>agpl</licence>
-	<author>Chris Ng</author>
+	<author>Nextcloud GmbH</author>
 	<namespace>Profile</namespace>
 	<category>social</category>
 	<bugs>https://github.com/nextcloud/server/issues</bugs>

diff --git a/build/integration/features/provisioning-v1.feature b/build/integration/features/provisioning-v1.feature
@@ -599,6 +599,7 @@ Feature: provisioning
       | files_trashbin |
       | files_versions |
       | lookup_server_connector |
+      | profile |
       | provisioning_api |
       | settings |
       | sharebymail |

diff --git a/tests/lib/App/AppManagerTest.php b/tests/lib/App/AppManagerTest.php
@@ -605,6 +605,7 @@ public function testGetAppsNeedingUpgrade(): void {
 			'dav' => ['id' => 'dav'],
 			'files' => ['id' => 'files'],
 			'federatedfilesharing' => ['id' => 'federatedfilesharing'],
+			'profile' => ['id' => 'profile'],
 			'provisioning_api' => ['id' => 'provisioning_api'],
 			'lookup_server_connector' => ['id' => 'lookup_server_connector'],
 			'test1' => ['id' => 'test1', 'version' => '1.0.1', 'requiremax' => '9.0.0'],

diff --git a/tests/lib/AppTest.php b/tests/lib/AppTest.php
@@ -345,6 +345,7 @@ public function appConfigValuesProvider() {
 					'federatedfilesharing',
 					'lookup_server_connector',
 					'oauth2',
+					'profile',
 					'provisioning_api',
 					'settings',
 					'theming',
@@ -368,6 +369,7 @@ public function appConfigValuesProvider() {
 					'federatedfilesharing',
 					'lookup_server_connector',
 					'oauth2',
+					'profile',
 					'provisioning_api',
 					'settings',
 					'theming',
@@ -392,6 +394,7 @@ public function appConfigValuesProvider() {
 					'federatedfilesharing',
 					'lookup_server_connector',
 					'oauth2',
+					'profile',
 					'provisioning_api',
 					'settings',
 					'theming',
@@ -416,6 +419,7 @@ public function appConfigValuesProvider() {
 					'federatedfilesharing',
 					'lookup_server_connector',
 					'oauth2',
+					'profile',
 					'provisioning_api',
 					'settings',
 					'theming',
@@ -524,11 +528,11 @@ public function testEnabledAppsCache(): void {
 			);
 
 		$apps = \OC_App::getEnabledApps();
-		$this->assertEquals(['files', 'app3', 'cloud_federation_api', 'dav', 'federatedfilesharing', 'lookup_server_connector', 'oauth2', 'provisioning_api', 'settings', 'theming', 'twofactor_backupcodes', 'viewer', 'workflowengine'], $apps);
+		$this->assertEquals(['files', 'app3', 'cloud_federation_api', 'dav', 'federatedfilesharing', 'lookup_server_connector', 'oauth2', 'profile', 'provisioning_api', 'settings', 'theming', 'twofactor_backupcodes', 'viewer', 'workflowengine'], $apps);
 
 		// mock should not be called again here
 		$apps = \OC_App::getEnabledApps();
-		$this->assertEquals(['files', 'app3', 'cloud_federation_api', 'dav', 'federatedfilesharing', 'lookup_server_connector', 'oauth2', 'provisioning_api', 'settings', 'theming', 'twofactor_backupcodes', 'viewer', 'workflowengine'], $apps);
+		$this->assertEquals(['files', 'app3', 'cloud_federation_api', 'dav', 'federatedfilesharing', 'lookup_server_connector', 'oauth2', 'profile', 'provisioning_api', 'settings', 'theming', 'twofactor_backupcodes', 'viewer', 'workflowengine'], $apps);
 
 		$this->restoreAppConfig();
 		\OC_User::setUserId(null);