diff options
author | Dries Buytaert <dries@buytaert.net> | 2010-04-10 09:49:49 +0000 |
---|---|---|
committer | Dries Buytaert <dries@buytaert.net> | 2010-04-10 09:49:49 +0000 |
commit | c80481634f6a7e00b43eb63ed5b0c8a2feb959c8 (patch) | |
tree | 6e0ccd059e2520123bf9e8f45851f95b215f27d4 /modules | |
parent | 459d623467a23d8deda42620eb9be7992d5cfd24 (diff) | |
download | brdo-c80481634f6a7e00b43eb63ed5b0c8a2feb959c8.tar.gz brdo-c80481634f6a7e00b43eb63ed5b0c8a2feb959c8.tar.bz2 |
- Patch #606190 by dww, noahb, sun: improve handling of database schema updates in update manager workflow.
Diffstat (limited to 'modules')
-rw-r--r-- | modules/system/system.updater.inc | 6 | ||||
-rw-r--r-- | modules/update/update.authorize.inc | 8 |
2 files changed, 9 insertions, 5 deletions
diff --git a/modules/system/system.updater.inc b/modules/system/system.updater.inc index fa265d551..079fb0af1 100644 --- a/modules/system/system.updater.inc +++ b/modules/system/system.updater.inc @@ -81,10 +81,8 @@ class ModuleUpdater extends Updater implements DrupalUpdaterInterface { } public function postUpdateTasks() { - // @todo: If there are schema updates. - return array( - l(t('Run database updates for !project', array('!project' => $this->title)), 'update.php'), - ); + // We don't want to check for DB updates here, we do that once for all + // updated modules on the landing page. } } diff --git a/modules/update/update.authorize.inc b/modules/update/update.authorize.inc index 0a5514d9d..7c675bf37 100644 --- a/modules/update/update.authorize.inc +++ b/modules/update/update.authorize.inc @@ -154,7 +154,9 @@ function update_authorize_batch_copy_project($project, $updater_name, $local_url } _update_batch_create_message($context['results']['log'][$project], t('Installed %project_name successfully', array('%project_name' => $project))); - $context['results']['tasks'] += $tasks; + if (!empty($tasks)) { + $context['results']['tasks'] += $tasks; + } // This particular operation is now complete, even though the batch might // have other operations to perform. @@ -206,6 +208,10 @@ function update_authorize_update_batch_finished($success, $results) { 'type' => 'error', ); } + // Since we're doing an update of existing code, always add a task for + // running update.php. + $results['tasks'][] = t('Your modules have been downloaded and updated.'); + $results['tasks'][] = t('<a href="@update">Run database updates</a>', array('@update' => base_path() . 'update.php')); // Set all these values into the SESSION so authorize.php can display them. $_SESSION['authorize_results']['success'] = $success; |