@@ -41,7 +41,7 @@ |
||
| 41 | 41 | * Extensions\Lingo\LingoBackend constructor. |
| 42 | 42 | * @param LingoMessageLog|null $messages |
| 43 | 43 | */ |
| 44 | - public function __construct( LingoMessageLog &$messages = null ) { |
|
| 44 | + public function __construct( LingoMessageLog & $messages = null ) { |
|
| 45 | 45 | |
| 46 | 46 | if ( !$messages ) { |
| 47 | 47 | $this->mMessageLog = new LingoMessageLog(); |
@@ -48,7 +48,7 @@ discard block |
||
| 48 | 48 | * Extensions\Lingo\LingoBasicBackend constructor. |
| 49 | 49 | * @param LingoMessageLog|null $messages |
| 50 | 50 | */ |
| 51 | - public function __construct( LingoMessageLog &$messages = null ) { |
|
| 51 | + public function __construct( LingoMessageLog & $messages = null ) { |
|
| 52 | 52 | |
| 53 | 53 | global $wgexLingoPage, $wgRequest; |
| 54 | 54 | |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | $term = null; |
| 127 | 127 | } |
| 128 | 128 | |
| 129 | - $definitions[] = trim( $chunks[ 1 ] ); |
|
| 129 | + $definitions[ ] = trim( $chunks[ 1 ] ); |
|
| 130 | 130 | } |
| 131 | 131 | |
| 132 | 132 | // found a new term? |
@@ -136,7 +136,7 @@ discard block |
||
| 136 | 136 | |
| 137 | 137 | if ( $term !== null ) { |
| 138 | 138 | foreach ( $definitions as $definition ) { |
| 139 | - $ret[] = array( |
|
| 139 | + $ret[ ] = array( |
|
| 140 | 140 | LingoElement::ELEMENT_TERM => $term, |
| 141 | 141 | LingoElement::ELEMENT_DEFINITION => $definition, |
| 142 | 142 | LingoElement::ELEMENT_LINK => null, |
@@ -100,7 +100,7 @@ |
||
| 100 | 100 | * - extension description shown on Special:Version |
| 101 | 101 | */ |
| 102 | 102 | public static function initExtension() { |
| 103 | - MagicWord::$mDoubleUnderscoreIDs[] = 'noglossary'; |
|
| 103 | + MagicWord::$mDoubleUnderscoreIDs[ ] = 'noglossary'; |
|
| 104 | 104 | |
| 105 | 105 | foreach ( $GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ] as $index => $description ) { |
| 106 | 106 | if ( $GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ][ $index ][ 'name' ] === 'Lingo' ) { |
@@ -53,7 +53,7 @@ discard block |
||
| 53 | 53 | * @param int $severity |
| 54 | 54 | */ |
| 55 | 55 | public function addMessage( $message, $severity = self::MESSAGE_NOTICE ) { |
| 56 | - $this->mMessages[] = array( $message, $severity ); |
|
| 56 | + $this->mMessages[ ] = array( $message, $severity ); |
|
| 57 | 57 | |
| 58 | 58 | // log errors and warnings in debug log |
| 59 | 59 | if ( $severity == self::MESSAGE_WARNING || |
@@ -67,7 +67,7 @@ discard block |
||
| 67 | 67 | * @param $message |
| 68 | 68 | */ |
| 69 | 69 | public function addError( $message ) { |
| 70 | - $this->mMessages[] = array( $message, self::MESSAGE_ERROR ); |
|
| 70 | + $this->mMessages[ ] = array( $message, self::MESSAGE_ERROR ); |
|
| 71 | 71 | wfDebug( "Error: $message\n" ); |
| 72 | 72 | } |
| 73 | 73 | |
@@ -75,7 +75,7 @@ discard block |
||
| 75 | 75 | * @param $message |
| 76 | 76 | */ |
| 77 | 77 | public function addWarning( $message ) { |
| 78 | - $this->mMessages[] = array( $message, self::MESSAGE_WARNING ); |
|
| 78 | + $this->mMessages[ ] = array( $message, self::MESSAGE_WARNING ); |
|
| 79 | 79 | wfDebug( "Warning: $message\n" ); |
| 80 | 80 | } |
| 81 | 81 | |
@@ -83,7 +83,7 @@ discard block |
||
| 83 | 83 | * @param $message |
| 84 | 84 | */ |
| 85 | 85 | public function addNotice( $message ) { |
| 86 | - $this->mMessages[] = array( $message, self::MESSAGE_NOTICE ); |
|
| 86 | + $this->mMessages[ ] = array( $message, self::MESSAGE_NOTICE ); |
|
| 87 | 87 | } |
| 88 | 88 | |
| 89 | 89 | /** |
@@ -85,7 +85,7 @@ discard block |
||
| 85 | 85 | * @param String $definition |
| 86 | 86 | * @return Array the tree node the element was stored in |
| 87 | 87 | */ |
| 88 | - protected function &addElement( Array &$path, &$term, &$definition ) { |
|
| 88 | + protected function &addElement( array &$path, &$term, &$definition ) { |
|
| 89 | 89 | |
| 90 | 90 | $tree = &$this->mTree; |
| 91 | 91 | |
@@ -142,7 +142,7 @@ discard block |
||
| 142 | 142 | } |
| 143 | 143 | } |
| 144 | 144 | |
| 145 | - public function findNextTermNoSkip( Array &$tree, &$lexemes, $index, $countLexemes ) { |
|
| 145 | + public function findNextTermNoSkip( array &$tree, &$lexemes, $index, $countLexemes ) { |
|
| 146 | 146 | |
| 147 | 147 | if ( $index + 1 < $countLexemes && array_key_exists( $currLex = $lexemes[ $index + 1 ][ 0 ], $tree ) ) { |
| 148 | 148 | $ret = $this->findNextTermNoSkip( $tree[ $currLex ], $lexemes, $index + 1, $countLexemes ); |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | * Extensions\Lingo\LingoParser constructor. |
| 58 | 58 | * @param LingoMessageLog|null $messages |
| 59 | 59 | */ |
| 60 | - public function __construct( LingoMessageLog &$messages = null ) { |
|
| 60 | + public function __construct( LingoMessageLog & $messages = null ) { |
|
| 61 | 61 | global $wgexLingoBackend; |
| 62 | 62 | |
| 63 | 63 | $this->mLingoBackend = new $wgexLingoBackend( $messages ); |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | * @param string $text |
| 70 | 70 | * @return Boolean |
| 71 | 71 | */ |
| 72 | - public static function parse( Parser &$parser, &$text ) { |
|
| 72 | + public static function parse( Parser & $parser, &$text ) { |
|
| 73 | 73 | |
| 74 | 74 | if ( !self::$parserSingleton ) { |
| 75 | 75 | self::$parserSingleton = new LingoParser(); |
@@ -135,13 +135,13 @@ discard block |
||
| 135 | 135 | } else { |
| 136 | 136 | |
| 137 | 137 | wfDebug( "Cache miss: Lingo tree not found in cache.\n" ); |
| 138 | - $this->mLingoTree =& $this->buildLingo(); |
|
| 138 | + $this->mLingoTree = & $this->buildLingo(); |
|
| 139 | 139 | $cache->set( $cachekey, $this->mLingoTree ); |
| 140 | 140 | wfDebug( "Cached lingo tree.\n" ); |
| 141 | 141 | } |
| 142 | 142 | } else { |
| 143 | 143 | wfDebug( "Caching of lingo tree disabled.\n" ); |
| 144 | - $this->mLingoTree =& $this->buildLingo(); |
|
| 144 | + $this->mLingoTree = & $this->buildLingo(); |
|
| 145 | 145 | } |
| 146 | 146 | |
| 147 | 147 | } |
@@ -48,7 +48,7 @@ discard block |
||
| 48 | 48 | const ELEMENT_LINK = 3; |
| 49 | 49 | const ELEMENT_STYLE = 4; |
| 50 | 50 | |
| 51 | - const ELEMENT_FIELDCOUNT = 5; // number of fields stored for each element; (last field's index) + 1 |
|
| 51 | + const ELEMENT_FIELDCOUNT = 5; // number of fields stored for each element; (last field's index) + 1 |
|
| 52 | 52 | static private $mLinkTemplate = null; |
| 53 | 53 | private $mFullDefinition = null; |
| 54 | 54 | private $mDefinitions = array(); |
@@ -73,14 +73,14 @@ discard block |
||
| 73 | 73 | * @param $definition |
| 74 | 74 | */ |
| 75 | 75 | public function addDefinition( &$definition ) { |
| 76 | - $this->mDefinitions[] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null ); |
|
| 76 | + $this->mDefinitions[ ] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null ); |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | /** |
| 80 | 80 | * @param DOMDocument $doc |
| 81 | 81 | * @return DOMNode|DOMText |
| 82 | 82 | */ |
| 83 | - public function getFullDefinition( DOMDocument &$doc ) { |
|
| 83 | + public function getFullDefinition( DOMDocument & $doc ) { |
|
| 84 | 84 | |
| 85 | 85 | global $wgexLingoDisplayOnce; |
| 86 | 86 | |
@@ -98,7 +98,7 @@ discard block |
||
| 98 | 98 | * @param DOMDocument $doc |
| 99 | 99 | * @return DOMDocument |
| 100 | 100 | */ |
| 101 | - private function buildFullDefinition( DOMDocument &$doc ) { |
|
| 101 | + private function buildFullDefinition( DOMDocument & $doc ) { |
|
| 102 | 102 | |
| 103 | 103 | // only create if not yet created |
| 104 | 104 | if ( $this->mFullDefinition === null || $this->mFullDefinition->ownerDocument !== $doc ) { |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | * @return DOMElement |
| 127 | 127 | * @throws MWException |
| 128 | 128 | */ |
| 129 | - protected function getFullDefinitionAsLink( DOMDocument &$doc ) { |
|
| 129 | + protected function getFullDefinitionAsLink( DOMDocument & $doc ) { |
|
| 130 | 130 | |
| 131 | 131 | // create Title object for target page |
| 132 | 132 | $target = Title::newFromText( $this->mDefinitions[ 0 ][ self::ELEMENT_LINK ] ); |
@@ -196,7 +196,7 @@ discard block |
||
| 196 | 196 | * @return string |
| 197 | 197 | * @throws MWException |
| 198 | 198 | */ |
| 199 | - protected function getFullDefinitionAsTooltip( DOMDocument &$doc ) { |
|
| 199 | + protected function getFullDefinitionAsTooltip( DOMDocument & $doc ) { |
|
| 200 | 200 | |
| 201 | 201 | // Wrap term and definition in <span> tags |
| 202 | 202 | $span = $doc->createElement( 'span' ); |
@@ -239,7 +239,7 @@ discard block |
||
| 239 | 239 | * @param DOMDocument $doc |
| 240 | 240 | * @return DOMNode |
| 241 | 241 | */ |
| 242 | - private function getLinkTemplate( DOMDocument &$doc ) { |
|
| 242 | + private function getLinkTemplate( DOMDocument & $doc ) { |
|
| 243 | 243 | // create template if it does not yet exist |
| 244 | 244 | if ( !self::$mLinkTemplate || ( self::$mLinkTemplate->ownerDocument !== $doc ) ) { |
| 245 | 245 | global $wgScriptPath; |