Comment and typing improvements

To make the StatusNet::addPlugin() accept only arrays,
the lib/default.php had to be changed because all plugins
had 'null' as default value instead of an array.

diff --git a/lib/default.php b/lib/default.php
index 55ef65e81f4ee3f8456c30fe7ac8285e7ad9930b..005a3683d815d1009775b4d3fd519c059731150d 100644 (file)
--- a/lib/default.php
+++ b/lib/default.php
@@ -298,15 +298,15 @@ $default =
                             'StrictTransportSecurity' => null,
                         ),
               'default' => array(
-                            'Activity' => null,
-                            'Bookmark' => null,
-                            'ClientSideShorten' => null,
-                            'Event' => null,
-                            'OpenID' => null,
-                            'Poll' => null,
-                            'QnA' => null,
-                            'SearchSub' => null,
-                            'TagSub' => null,
+                            'Activity' => array(),
+                            'Bookmark' => array(),
+                            'ClientSideShorten' => array(),
+                            'Event' => array(),
+                            'OpenID' => array(),
+                            'Poll' => array(),
+                            'QnA' => array(),
+                            'SearchSub' => array(),
+                            'TagSub' => array(),
                         ),
               'locale_path' => false, // Set to a path to use *instead of* each plugin's own locale subdirectories
               'server' => null,

diff --git a/lib/framework.php b/lib/framework.php
index 3ef76ea4b5296981f07ca3c7a3823f1808f247c6..76ba549098effb2f27abee3607bbba55b3356763 100644 (file)
--- a/lib/framework.php
+++ b/lib/framework.php
@@ -95,7 +95,7 @@ require_once(INSTALLDIR.'/lib/language.php');
 require_once(INSTALLDIR.'/lib/event.php');
 require_once(INSTALLDIR.'/lib/plugin.php');
 
-function addPlugin($name, $attrs = null)
+function addPlugin($name, array $attrs=array())
 {
     return StatusNet::addPlugin($name, $attrs);
 }

diff --git a/lib/statusnet.php b/lib/statusnet.php
index cf2069c1a6114a9c378c6ad70896d97aa9d288d1..e596dd96d13a07d2979f565658b019567b1a2f7f 100644 (file)
--- a/lib/statusnet.php
+++ b/lib/statusnet.php
@@ -44,7 +44,7 @@ class StatusNet
      *
      * @throws ServerException if plugin can't be found
      */
-    public static function addPlugin($name, $attrs = null)
+    public static function addPlugin($name, array $attrs=array())
     {
         $name = ucfirst($name);
 
@@ -78,11 +78,11 @@ class StatusNet
             }
         }
 
+        // Doesn't this $inst risk being garbage collected or something?
+        // TODO: put into a static array that makes sure $inst isn't lost.
         $inst = new $pluginclass();
-        if (!empty($attrs)) {
-            foreach ($attrs as $aname => $avalue) {
-                $inst->$aname = $avalue;
-            }
+        foreach ($attrs as $aname => $avalue) {
+            $inst->$aname = $avalue;
         }
 
         // Record activated plugins for later display/config dump

diff --git a/scripts/showplugins.php b/scripts/showplugins.php
index 3c03646227aae73c8f089f0e6c26bd4839fa4f58..e79b4e72242888f4661eedba444473e27caf6101 100755 (executable)
--- a/scripts/showplugins.php
+++ b/scripts/showplugins.php
@@ -24,7 +24,7 @@ require_once INSTALLDIR.'/scripts/commandline.inc';
 
 foreach (StatusNet::getActivePlugins() as $plugin=>$args) {
     echo "$plugin: ";
-    if ($args === null) {
+    if (empty($args)) {
         echo "(no args)\n";
     } else {
         foreach ($args as $arg => $val) {