Simplify a bit by adding the result inside the wfRunHooks() check
authorAlexandre Emsenhuber <ialex@users.mediawiki.org>
Tue, 4 Oct 2011 14:32:07 +0000 (14:32 +0000)
committerAlexandre Emsenhuber <ialex@users.mediawiki.org>
Tue, 4 Oct 2011 14:32:07 +0000 (14:32 +0000)
includes/LogEventsList.php

index 284e537..d2deeea 100644 (file)
@@ -707,16 +707,15 @@ class LogEventsList {
                }
 
                /* hook can return false, if we don't want the message to be emitted (Wikia BugId:7093) */
-               if ( !wfRunHooks( 'LogEventsListShowLogExtract', array( &$s, $types, $page, $user, $param ) ) ) {
-                       return $pager->getNumRows();
+               if ( wfRunHooks( 'LogEventsListShowLogExtract', array( &$s, $types, $page, $user, $param ) ) ) {
+                       // $out can be either an OutputPage object or a String-by-reference
+                       if ( $out instanceof OutputPage ){
+                               $out->addHTML( $s );
+                       } else {
+                               $out = $s;
+                       }
                }
 
-               // $out can be either an OutputPage object or a String-by-reference
-               if( $out instanceof OutputPage ){
-                       $out->addHTML( $s );
-               } else {
-                       $out = $s;
-               }
                return $pager->getNumRows();
        }