[Pkg-owncloud-commits] [owncloud] 09/95: on ownCloud upgrade: upgrade all apps in order, load important ones

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to annotated tag v8.0.1
in repository owncloud.

commit 61eaf0e8326b57168138a3533ad1bcfa2c2a1740
Author: Arthur Schiwon <blizzz at owncloud.com>
Date:   Tue Feb 3 19:02:25 2015 +0100

    on ownCloud upgrade: upgrade all apps in order, load important ones
---
 lib/private/updater.php | 40 +++++++++++++++++++++++++++++++++++++---
 1 file changed, 37 insertions(+), 3 deletions(-)

diff --git a/lib/private/updater.php b/lib/private/updater.php
index fb41e2d..20b140d 100644
--- a/lib/private/updater.php
+++ b/lib/private/updater.php
@@ -291,13 +291,47 @@ class Updater extends BasicEmitter {
 		include \OC_App::getAppPath($appId) . '/appinfo/preupdate.php';
 	}
 
+
+	/**
+	 * upgrades all apps within a major ownCloud upgrade. Also loads "priority"
+	 * (types authentication, filesystem, logging, in that order) afterwards.
+	 *
+	 * @throws NeedsUpdateException
+	 */
 	protected function doAppUpgrade() {
 		$apps = \OC_App::getEnabledApps();
+		$priorityTypes = array('authentication', 'filesystem', 'logging');
+		$pseudoOtherType = 'other';
+		$stacks = array($pseudoOtherType => array());
 
 		foreach ($apps as $appId) {
-			if (\OC_App::shouldUpgrade($appId)) {
-				\OC_App::updateApp($appId);
-				$this->emit('\OC\Updater', 'appUpgrade', array($appId, \OC_App::getAppVersion($appId)));
+			$priorityType = false;
+			foreach ($priorityTypes as $type) {
+				if(!isset($stacks[$type])) {
+					$stacks[$type] = array();
+				}
+				if (\OC_App::isType($appId, $type)) {
+					$stacks[$type][] = $appId;
+					$priorityType = true;
+					break;
+				}
+			}
+			if (!$priorityType) {
+				$stacks[$pseudoOtherType] = $appId;
+			}
+		}
+		foreach ($stacks as $type => $stack) {
+			foreach ($stack as $appId) {
+				if (\OC_App::shouldUpgrade($appId)) {
+					\OC_App::updateApp($appId);
+					$this->emit('\OC\Updater', 'appUpgrade', array($appId, \OC_App::getAppVersion($appId)));
+				}
+				if($type !== $pseudoOtherType) {
+					// load authentication, filesystem and logging apps after
+					// upgrading them. Other apps my need to rely on modifying
+					// user and/or filesystem aspects.
+					\OC_App::loadApp($appId, false);
+				}
 			}
 		}
 	}

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud.git