MDL-49329 admin: Fix checking for available updates
authorDavid Mudrák <david@moodle.com>
Fri, 9 Oct 2015 22:24:59 +0000 (00:24 +0200)
committerDavid Mudrák <david@moodle.com>
Sat, 10 Oct 2015 07:15:27 +0000 (09:15 +0200)
This was a regression of my recent improvement of rendering the "Check
for updates" button. There is now unified parameter ?fetchupdates=1 that
can be used on either admin/index.php or admin/plugins.php, so that we
can use a common UI widget for both locations (without the need to pass
the URL explicitly).

admin/index.php
admin/plugins.php
admin/renderer.php

index 6c3469f..3972513 100644 (file)
@@ -103,7 +103,7 @@ $confirmrelease = optional_param('confirmrelease', 0, PARAM_BOOL); // Core relea
 $confirmplugins = optional_param('confirmplugincheck', 0, PARAM_BOOL); // Plugins check page confirmed?
 $showallplugins = optional_param('showallplugins', 0, PARAM_BOOL); // Show all plugins on the plugins check page?
 $agreelicense = optional_param('agreelicense', 0, PARAM_BOOL); // GPL license confirmed for installation?
-$fetchupdates = optional_param('fetchremote', 0, PARAM_BOOL); // Should check for available updates?
+$fetchupdates = optional_param('fetchupdates', 0, PARAM_BOOL); // Should check for available updates?
 $newaddonreq = optional_param('installaddonrequest', null, PARAM_RAW); // Plugin installation requested at moodle.org/plugins.
 $upgradekeyhash = optional_param('upgradekeyhash', null, PARAM_ALPHANUM); // Hash of provided upgrade key.
 $installdep = optional_param('installdep', null, PARAM_COMPONENT); // Install given missing dependency (required plugin).
index 9c9e2bc..39510cc 100644 (file)
@@ -28,7 +28,7 @@ require_once(dirname(dirname(__FILE__)) . '/config.php');
 require_once($CFG->libdir . '/adminlib.php');
 require_once($CFG->libdir . '/filelib.php');
 
-$fetchremote = optional_param('fetchremote', false, PARAM_BOOL); // Check for available plugins updates.
+$fetchupdates = optional_param('fetchupdates', false, PARAM_BOOL); // Check for available plugins updates.
 $updatesonly = optional_param('updatesonly', false, PARAM_BOOL); // Show updateable plugins only.
 $contribonly = optional_param('contribonly', false, PARAM_BOOL); // Show additional plugins only.
 $uninstall = optional_param('uninstall', '', PARAM_COMPONENT); // Uninstall the plugin.
@@ -207,7 +207,7 @@ $output = $PAGE->get_renderer('core', 'admin');
 
 $checker = \core\update\checker::instance();
 
-if ($fetchremote) {
+if ($fetchupdates) {
     require_sesskey();
     $checker->fetch();
     redirect($PAGE->url);
index 11e0b6d..14f116d 100644 (file)
@@ -346,7 +346,7 @@ class core_admin_renderer extends plugin_renderer_base {
         if ($checker->enabled()) {
             $output .= $this->container_start('checkforupdates');
             $output .= $this->single_button(
-                new moodle_url($reloadurl, array('fetchremote' => 1)),
+                new moodle_url($reloadurl, array('fetchupdates' => 1)),
                 get_string('checkforupdates', 'core_plugin')
             );
             if ($timefetched = $checker->get_last_timefetched()) {