Portfolio plugins explicitly requires parent library with superclass definition
authorDavid Mudrak <david.mudrak@gmail.com>
Sun, 27 Jun 2010 14:46:15 +0000 (14:46 +0000)
committerDavid Mudrak <david.mudrak@gmail.com>
Sun, 27 Jun 2010 14:46:15 +0000 (14:46 +0000)
I have discovered a bug with GoogleDocs portfolio in events. An event
handler was defined in googledocs/lib.php and so the cron included just
this library. But the inclusion failed because the superclass
portfolio_plugin_push_base was not known to cron.

IMHO it is generally good habit to require_once the file which defines
the parent classes.

portfolio/boxnet/lib.php
portfolio/flickr/lib.php
portfolio/googledocs/lib.php
portfolio/picasa/lib.php

index d5dac38..784262a 100644 (file)
@@ -1,4 +1,5 @@
 <?php
+require_once($CFG->libdir.'/portfolio/plugin.php');
 require_once($CFG->libdir.'/filelib.php');
 require_once($CFG->libdir.'/boxlib.php');
 
index 7c12b95..977d81a 100755 (executable)
@@ -1,4 +1,5 @@
 <?php
+require_once($CFG->libdir.'/portfolio/plugin.php');
 require_once($CFG->libdir.'/filelib.php');
 require_once($CFG->libdir.'/flickrlib.php');
 
index 7590530..2223f9d 100644 (file)
@@ -5,6 +5,7 @@
  * @author Dan Poltawski <talktodan@gmail.com>
  * @license http://www.gnu.org/copyleft/gpl.html GNU Public License
  */
+require_once($CFG->libdir.'/portfolio/plugin.php');
 require_once($CFG->libdir.'/googleapi.php');
 
 class portfolio_plugin_googledocs extends portfolio_plugin_push_base {
index 5b8551e..a9ba23f 100644 (file)
@@ -5,7 +5,7 @@
  * @author Dan Poltawski <talktodan@gmail.com>
  * @license http://www.gnu.org/copyleft/gpl.html GNU Public License
  */
-
+require_once($CFG->libdir.'/portfolio/plugin.php');
 require_once($CFG->libdir.'/googleapi.php');
 
 class portfolio_plugin_picasa extends portfolio_plugin_push_base {