MDL-59562 core: updated type hint of new function
authorPeter Burnett <peterburnett@catalyst-au.net>
Thu, 16 Jan 2020 01:58:31 +0000 (11:58 +1000)
committerPeter Burnett <peterburnett@catalyst-au.net>
Thu, 16 Jan 2020 03:05:30 +0000 (13:05 +1000)
lib/classes/plugin_manager.php
lib/classes/plugininfo/base.php
lib/tests/plugininfo/base_test.php

index 55fcf18..7ac652d 100644 (file)
@@ -927,9 +927,9 @@ class core_plugin_manager {
      *
      * @param \core\plugininfo\base $plugin the plugin we are checking
      * @param int $branch the moodle branch to check support for
-     * @return bool
+     * @return string
      */
-    public function check_explicitly_supported($plugin, $branch) : bool {
+    public function check_explicitly_supported($plugin, $branch) : string {
         // Check for correctly formed supported.
         if (isset($plugin->pluginsupported)) {
             // Broken apart for readability.
index 71e5a70..75d980f 100644 (file)
@@ -254,7 +254,7 @@ abstract class base {
             if (is_array($plugin->supported) && $isint && $isrange) {
                 $this->pluginsupported = $plugin->supported;
             } else {
-                throw new coding_exception('Incorrect syntax in $plugin->supported in '."$this->name");
+                throw new coding_exception('Incorrect syntax in plugin supported declaration in '."$this->name");
             }
         }
 
@@ -262,7 +262,7 @@ abstract class base {
             if ((ctype_digit($plugin->incompatible) || is_int($plugin->incompatible)) && (int) $plugin->incompatible > 0) {
                 $this->pluginincompatible = intval($plugin->incompatible);
             } else {
-                throw new coding_exception('Incorrect syntax in $plugin->incompatible in '."$this->name");
+                throw new coding_exception('Incorrect syntax in plugin incompatible declaration in '."$this->name");
             }
         }
 
index 508c14d..d5b805f 100644 (file)
@@ -81,7 +81,7 @@ class base_testcase extends \advanced_testcase {
         $pluginman->add_fake_plugin_info($plugininfo);
 
         $this->expectException(\coding_exception::class);
-        $this->expectExceptionMessage('Incorrect syntax in $plugin->supported in example');
+        $this->expectExceptionMessage('Incorrect syntax in plugin supported declaration in example');
         $plugininfo->load_disk_version();
     }
 
@@ -147,7 +147,7 @@ class base_testcase extends \advanced_testcase {
         $pluginman->add_fake_plugin_info($plugininfo);
 
         $this->expectException(\coding_exception::class);
-        $this->expectExceptionMessage('Incorrect syntax in $plugin->incompatible in example');
+        $this->expectExceptionMessage('Incorrect syntax in plugin incompatible declaration in example');
         $plugininfo->load_disk_version();
     }