Keep group restrictions when reenabling apps after an update

Signed-off-by: Joas Schilling <[email protected]>

Author: nickvergessen

lib/private/App/AppManager.php

@@ -391,7 +391,11 @@ public function disableApp($appId, $automaticDisabled = false) {
 		}
 
 		if ($automaticDisabled) {
-			$this->autoDisabledApps[] = $appId;
+			$previousSetting = $this->appConfig->getValue($appId, 'enabled', 'yes');
+			if ($previousSetting !== 'yes' && $previousSetting !== 'no')  {
+				$previousSetting = json_decode($previousSetting, true);
+			}
+			$this->autoDisabledApps[$appId] = $previousSetting;
 		}
 
 		unset($this->installedAppsCache[$appId]);

lib/private/Updater.php

@@ -400,7 +400,7 @@ private function isCodeUpgrade(): bool {
 	 * @throws \Exception
 	 */
 	private function upgradeAppStoreApps(array $disabledApps, bool $reenable = false): void {
-		foreach ($disabledApps as $app) {
+		foreach ($disabledApps as $app => $previousEnableSetting) {
 			try {
 				$this->emit('\OC\Updater', 'checkAppStoreAppBefore', [$app]);
 				if ($this->installer->isUpdateAvailable($app)) {
@@ -411,7 +411,11 @@ private function upgradeAppStoreApps(array $disabledApps, bool $reenable = false
 
 				if ($reenable) {
 					$ocApp = new \OC_App();
-					$ocApp->enable($app);
+					if (!empty($previousEnableSetting)) {
+						$ocApp->enable($app, $previousEnableSetting);
+					} else {
+						$ocApp->enable($app);
+					}
 				}
 			} catch (\Exception $ex) {
 				$this->log->error($ex->getMessage(), [

lib/public/App/IAppManager.php

@@ -182,12 +182,6 @@ public function getAlwaysEnabledApps();
 	 */
 	public function getEnabledAppsForGroup(IGroup $group): array;
 
-	/**
-	 * @return array
-	 * @since 17.0.0
-	 */
-	public function getAutoDisabledApps(): array;
-
 	/**
 	 * @param String $appId
 	 * @return string[]