From: Michael Gapczynski <mtgap@owncloud.com>
Date: Fri, 4 Jan 2013 15:21:33 +0000 (-0500)
Subject: Use hooks to send updates from apps
X-Git-Tag: v5.0.0alpha1~258^2~8
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e7eea3dd2e3eb0f5bd77901167656275d835cfb3;p=nextcloud-server.git

Use hooks to send updates from apps
---

diff --git a/core/ajax/update.php b/core/ajax/update.php
index 6f5398f41cb..0d58f17f097 100644
--- a/core/ajax/update.php
+++ b/core/ajax/update.php
@@ -5,7 +5,11 @@ require_once '../../lib/base.php';
 
 if (OC::checkUpgrade(false)) {
 	$updateEventSource = new OC_EventSource();
-	$updateEventSource->send('success', 'Turned on maintenance mode');
+	$watcher = new UpdateWatcher($updateEventSource);
+	OC_Hook::connect('update', 'success', $watcher, 'success');
+	OC_Hook::connect('update', 'error', $watcher, 'error');
+	OC_Hook::connect('update', 'error', $watcher, 'failure');
+	$watcher->success('Turned on maintenance mode');
 	// Check if the .htaccess is existing - this is needed for upgrades from really old ownCloud versions
 	if (isset($_SERVER['SERVER_SOFTWARE']) && strstr($_SERVER['SERVER_SOFTWARE'], 'Apache')) {
 		if (!OC_Util::ishtaccessworking()) {
@@ -16,11 +20,9 @@ if (OC::checkUpgrade(false)) {
 	}
 	$result = OC_DB::updateDbFromStructure(OC::$SERVERROOT.'/db_structure.xml');
 	if (!$result) {
-		$updateEventSource->send('failure', 'Error updating database');
-		$updateEventSource->close();
-		die();
+		$watcher->failure('Error updating database');
 	}
-	$updateEventSource->send('success', 'Updated database');
+	$watcher->success('Updated database');
 	$minimizerCSS = new OC_Minimizer_CSS();
 	$minimizerCSS->clearCache();
 	$minimizerJS = new OC_Minimizer_JS();
@@ -30,7 +32,43 @@ if (OC::checkUpgrade(false)) {
 	// load all apps to also upgrade enabled apps
 	OC_App::loadApps();
 	OC_Config::setValue('maintenance', false);
-	$updateEventSource->send('success', 'Turned off maintenance mode');
-	$updateEventSource->send('done', 'done');
-	$updateEventSource->close();
+	$watcher->success('Turned off maintenance mode');
+	$watcher->done();
+}
+
+class UpdateWatcher {
+	/**
+	 * @var \OC_EventSource $eventSource;
+	 */
+	private $eventSource;
+
+	public function __construct($eventSource) {
+		$this->eventSource = $eventSource;
+	}
+
+	public function success($message) {
+		OC_Util::obEnd();
+		$this->eventSource->send('success', $message);
+		ob_start();
+	}
+
+	public function error($message) {
+		OC_Util::obEnd();
+		$this->eventSource->send('error', $message);
+		ob_start();
+	}
+
+	public function failure($message) {
+		OC_Util::obEnd();
+		$this->eventSource->send('failure', $message);
+		$this->eventSource->close();
+		die();
+	}
+
+	public function done() {
+		OC_Util::obEnd();
+		$this->eventSource->send('done', '');
+		$this->eventSource->close();
+	}
+
 }
\ No newline at end of file
diff --git a/lib/app.php b/lib/app.php
index be6d5ab3dd3..596f3bf3a79 100755
--- a/lib/app.php
+++ b/lib/app.php
@@ -642,6 +642,7 @@ class OC_App{
 					echo 'Failed to upgrade "'.$app.'". Exception="'.$e->getMessage().'"';
 					die;
 				}
+				OC_Hook::emit('update', 'success', 'Updated '.$app.' app');
 				OC_Appconfig::setValue($app, 'installed_version', OC_App::getAppVersion($app));
 			}
 		}