Localisation updates for extension messages from Betawiki

[lhc/web/wiklou.git] / includes / Hooks.php

diff --git a/includes/Hooks.php b/includes/Hooks.php
index 575a28c..046a149 100644 (file)
--- a/includes/Hooks.php
+++ b/includes/Hooks.php
@@ -18,8 +18,8 @@
  *  Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
  *
  * @author Evan Prodromou <evan@wikitravel.org>
- * @package MediaWiki
  * @see hooks.txt
+ * @file
  */
 
 
@@ -28,10 +28,9 @@
  * careful about its contents. So, there's a lot more error-checking
  * in here than would normally be necessary.
  */
-function wfRunHooks($event, $args = null) {
+function wfRunHooks($event, $args = array()) {
 
        global $wgHooks;
-       $fname = 'wfRunHooks';
 
        if (!is_array($wgHooks)) {
                throw new MWException("Global hooks array is not an array!\n");
@@ -104,12 +103,15 @@ function wfRunHooks($event, $args = null) {
                if ( isset( $object ) ) {
                        $func = get_class( $object ) . '::' . $method;
                        $callback = array( $object, $method );
-               } elseif ( false !== ( $pos = strpos( '::', $func ) ) ) {
+               } elseif ( false !== ( $pos = strpos( $func, '::' ) ) ) {
                        $callback = array( substr( $func, 0, $pos ), substr( $func, $pos + 2 ) );
                } else {
                        $callback = $func;
                }
 
+               // Run autoloader (workaround for call_user_func_array bug)
+               is_callable( $callback );
+
                /* Call the hook. */
                wfProfileIn( $func );
                $retval = call_user_func_array( $callback, $hook_args );
@@ -121,6 +123,20 @@ function wfRunHooks($event, $args = null) {
                        global $wgOut;
                        $wgOut->showFatalError($retval);
                        return false;
+               } elseif( $retval === null ) {
+                       if( is_array( $callback ) ) {
+                               if( is_object( $callback[0] ) ) {
+                                       $prettyClass = get_class( $callback[0] );
+                               } else {
+                                       $prettyClass = strval( $callback[0] );
+                               }
+                               $prettyFunc = $prettyClass . '::' . strval( $callback[1] );
+                       } else {
+                               $prettyFunc = strval( $callback );
+                       }
+                       throw new MWException( "Detected bug in an extension! " .
+                               "Hook $prettyFunc failed to return a value; " .
+                               "should return true to continue hook processing or false to abort." );
                } else if (!$retval) {
                        return false;
                }
@@ -128,4 +144,3 @@ function wfRunHooks($event, $args = null) {
 
        return true;
 }
-?>