MDL-33552 - portfolio api E_STRICT fixes

admin_config_form and admin_config_validation are now static
sehaj-sk · Jun 12, 2012 · c17ec77 · c17ec77
1 parent f8dfdb5
commit c17ec77
Show file tree

Hide file tree

Showing 8 changed files with 20 additions and 20 deletions.
diff --git a/lib/portfolio/forms.php b/lib/portfolio/forms.php
@@ -199,13 +199,9 @@ public function definition() {
 
         // let the plugin add the fields they want (either statically or not)
         if (portfolio_static_function($this->plugin, 'has_admin_config')) {
-            if (!$this->instance) {
-                require_once($CFG->libdir . '/portfolio/plugin.php');
-                require_once($CFG->dirroot . '/portfolio/' . $this->plugin .  '/lib.php');
-                call_user_func(array('portfolio_plugin_' . $this->plugin, 'admin_config_form'), $mform);
-            } else {
-                $this->instance->admin_config_form($mform);
-            }
+            require_once($CFG->libdir . '/portfolio/plugin.php');
+            require_once($CFG->dirroot . '/portfolio/' . $this->plugin .  '/lib.php');
+            call_user_func(array('portfolio_plugin_' . $this->plugin, 'admin_config_form'), $mform);
         }
 
         // and set the data if we have some.
@@ -237,12 +233,7 @@ public function validation($data, $files) {
         }
 
         $pluginerrors = array();
-        if ($this->instance) {
-            $pluginerrors = $this->instance->admin_config_validation($data);
-        }
-        else {
-            $pluginerrors = portfolio_static_function($this->plugin, 'admin_config_validation', $data);
-        }
+        $pluginerrors = portfolio_static_function($this->plugin, 'admin_config_validation', $data);
         if (is_array($pluginerrors)) {
             $errors = array_merge($errors, $pluginerrors);
         }

diff --git a/lib/portfolio/plugin.php b/lib/portfolio/plugin.php
@@ -347,7 +347,7 @@ public function user_config_form(&$mform) {}
      *
      * @param moodleform $mform passed by reference, add elements to it.
      */
-    public function admin_config_form(&$mform) {}
+    public static function admin_config_form(&$mform) {}
 
     /**
      * Just like the moodle form validation function,
@@ -356,7 +356,7 @@ public function admin_config_form(&$mform) {}
      *
      * @param array $data data from form.
      */
-    public function admin_config_validation($data) {}
+    public static function admin_config_validation($data) {}
 
     /**
      * mform to display to the user exporting data using this plugin.

diff --git a/portfolio/boxnet/lib.php b/portfolio/boxnet/lib.php
@@ -106,7 +106,7 @@ public function export_config_validation(array $data) {
         }
     }
 
-    public function admin_config_form(&$mform) {
+    public static function admin_config_form(&$mform) {
         global $CFG;
 
         $mform->addElement('text', 'apikey', get_string('apikey', 'portfolio_boxnet'));

diff --git a/portfolio/flickr/lib.php b/portfolio/flickr/lib.php
@@ -96,7 +96,7 @@ public static function has_admin_config() {
         return true;
     }
 
-    public function admin_config_form(&$mform) {
+    public static function admin_config_form(&$mform) {
         global $CFG;
 
         $strrequired = get_string('required');

diff --git a/portfolio/googledocs/lib.php b/portfolio/googledocs/lib.php
@@ -100,7 +100,7 @@ public static function get_allowed_config() {
         return array('clientid', 'secret');
     }
 
-    public function admin_config_form(&$mform) {
+    public static function admin_config_form(&$mform) {
         $a = new stdClass;
         $a->docsurl = get_docs_url('Google_OAuth2_Setup');
         $a->callbackurl = google_oauth::callback_url()->out(false);

diff --git a/portfolio/mahara/lib.php b/portfolio/mahara/lib.php
@@ -84,7 +84,7 @@ public static function has_admin_config() {
         return true;
     }
 
-    public function admin_config_form(&$mform) {
+    public static function admin_config_form(&$mform) {
         $strrequired = get_string('required');
         $hosts = self::get_mnet_hosts(); // this is called by sanity check but it's ok because it's cached
         foreach ($hosts as $host) {

diff --git a/portfolio/picasa/lib.php b/portfolio/picasa/lib.php
@@ -100,7 +100,7 @@ public static function get_allowed_config() {
         return array('clientid', 'secret');
     }
 
-    public function admin_config_form(&$mform) {
+    public static function admin_config_form(&$mform) {
         $a = new stdClass;
         $a->docsurl = get_docs_url('Google_OAuth2_Setup');
         $a->callbackurl = google_oauth::callback_url()->out(false);

diff --git a/portfolio/upgrade.txt b/portfolio/upgrade.txt
@@ -0,0 +1,9 @@
+This files describes API changes in /portfolio/ portfolio system,
+information provided here is intended especially for developers.
+
+=== 2.3 ===
+
+required changes:
+* The following methods must now be declared static for php5 compatibility:
+    - admin_config_form
+    - admin_config_validation