Vishnunk90 has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/104490


Change subject: wfRunHooks() return value no longer implies whether variable 
value was found
......................................................................

wfRunHooks() return value no longer implies whether variable value was found

Bug: 12837
Change-Id: If53b6f196489e630b29ddf62617ddb0d7164479c
---
M includes/parser/Parser.php
1 file changed, 2 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/90/104490/1

diff --git a/includes/parser/Parser.php b/includes/parser/Parser.php
index 9be75ae..31638ce 100644
--- a/includes/parser/Parser.php
+++ b/includes/parser/Parser.php
@@ -3034,11 +3034,8 @@
                                return $wgLanguageCode;
                        default:
                                $ret = null;
-                               if ( wfRunHooks( 
'ParserGetVariableValueSwitch', array( &$this, &$this->mVarCache, &$index, 
&$ret, &$frame ) ) ) {
-                                       return $ret;
-                               } else {
-                                       return null;
-                               }
+                               wfRunHooks( 'ParserGetVariableValueSwitch', 
array( &$this, &$this->mVarCache, &$index, &$ret, &$frame ) );
+                               return $ret;
                }
 
                if ( $index ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/104490
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If53b6f196489e630b29ddf62617ddb0d7164479c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Vishnunk90 <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to