| Index: trunk/extensions/NewestPages/NewestPages.page.php |
| — | — | @@ -24,8 +24,6 @@ |
| 25 | 25 | public function execute( $par ) { |
| 26 | 26 | global $wgRequest, $wgOut, $wgContLang, $wgLang; |
| 27 | 27 | |
| 28 | | - |
| 29 | | - |
| 30 | 28 | # Decipher input passed to the page |
| 31 | 29 | $this->decipherParams( $par ); |
| 32 | 30 | $this->setOptions( $wgRequest ); |
| Index: trunk/extensions/ConditionalShowSection/ConditionalShowSection.php |
| — | — | @@ -34,11 +34,21 @@ |
| 35 | 35 | |
| 36 | 36 | $wgHooks['ParserFirstCallInit'][] = 'wfConditionalShowSection'; |
| 37 | 37 | |
| | 38 | +/** |
| | 39 | + * @param $parser Parser |
| | 40 | + * @return bool |
| | 41 | + */ |
| 38 | 42 | function wfConditionalShowSection( $parser ) { |
| 39 | | - $parser->setHook( "cshow", "ConditionalShowSection" ); |
| 40 | | - return true; |
| | 43 | + $parser->setHook( "cshow", "ConditionalShowSection" ); |
| | 44 | + return true; |
| 41 | 45 | } |
| 42 | 46 | |
| | 47 | +/** |
| | 48 | + * @param $input |
| | 49 | + * @param $argv |
| | 50 | + * @param $parser Parser |
| | 51 | + * @return string |
| | 52 | + */ |
| 43 | 53 | function ConditionalShowSection( $input, $argv, &$parser ) { |
| 44 | 54 | # |
| 45 | 55 | # By default, the section is HIDDEN unless the following conditions are met: |
| — | — | @@ -63,7 +73,6 @@ |
| 64 | 74 | global $wgUser; |
| 65 | 75 | |
| 66 | 76 | $userReqLogged = null; # default is "don't care" |
| 67 | | - $userReqGroup = "" ; # assuming no group membership required |
| 68 | 77 | $output = ""; # assuming the section is hidden by default. |
| 69 | 78 | |
| 70 | 79 | $cond1 = false; |
| — | — | @@ -96,7 +105,7 @@ |
| 97 | 106 | $cond1 = true; |
| 98 | 107 | } |
| 99 | 108 | # if both conditions are met, then SHOW else HIDE |
| 100 | | - if ( ( $cond1 === true ) and ( $cond2 === true ) ) { |
| | 109 | + if ( ( $cond1 === true ) && ( $cond2 === true ) ) { |
| 101 | 110 | $output = $parser->recursiveTagParse( $input ); |
| 102 | 111 | } |
| 103 | 112 | |
| Index: trunk/extensions/FeaturedFeeds/FeaturedFeeds.body.php |
| — | — | @@ -9,7 +9,7 @@ |
| 10 | 10 | */ |
| 11 | 11 | public static function getFeeds( $langCode ) { |
| 12 | 12 | global $wgMemc, $wgContLang; |
| 13 | | - |
| | 13 | + |
| 14 | 14 | if ( !$langCode ) { |
| 15 | 15 | $langCode = $wgContLang->getCode(); |
| 16 | 16 | } |
| Index: trunk/extensions/wikihiero/wikihiero.php |
| — | — | @@ -70,13 +70,22 @@ |
| 71 | 71 | |
| 72 | 72 | $wgCompiledFiles[] = MWInit::extCompiledPath( 'wikihiero/data/tables.php' ); |
| 73 | 73 | |
| 74 | | -// Because <hiero> tag is used rarely, we don't need to load its body on every hook call, |
| 75 | | -// so we keep our simple hook handlers here. |
| | 74 | +/** |
| | 75 | + * Because <hiero> tag is used rarely, we don't need to load its body on every hook call, |
| | 76 | + * so we keep our simple hook handlers here. |
| | 77 | + * |
| | 78 | + * @param $parser Parser |
| | 79 | + * @return bool |
| | 80 | + */ |
| 76 | 81 | function wfRegisterWikiHiero( &$parser ) { |
| 77 | 82 | $parser->setHook( 'hiero', 'WikiHiero::parserHook' ); |
| 78 | 83 | return true; |
| 79 | 84 | } |
| 80 | 85 | |
| | 86 | +/** |
| | 87 | + * @param $out OutputPage |
| | 88 | + * @return bool |
| | 89 | + */ |
| 81 | 90 | function wfHieroBeforePageDisplay( $out ) { |
| 82 | 91 | $out->addModuleStyles( 'ext.wikihiero' ); |
| 83 | 92 | return true; |