@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | * Lingo\LingoParser constructor. |
58 | 58 | * @param MessageLog|null $messages |
59 | 59 | */ |
60 | - public function __construct( MessageLog &$messages = null ) { |
|
60 | + public function __construct( MessageLog & $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 | * Lingo\BasicBackend constructor. |
49 | 49 | * @param MessageLog|null $messages |
50 | 50 | */ |
51 | - public function __construct( MessageLog &$messages = null ) { |
|
51 | + public function __construct( MessageLog & $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 | Element::ELEMENT_TERM => $term, |
141 | 141 | Element::ELEMENT_DEFINITION => $definition, |
142 | 142 | Element::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' ) { |
@@ -41,7 +41,7 @@ |
||
41 | 41 | * Lingo\Backend constructor. |
42 | 42 | * @param MessageLog|null $messages |
43 | 43 | */ |
44 | - public function __construct( MessageLog &$messages = null ) { |
|
44 | + public function __construct( MessageLog & $messages = null ) { |
|
45 | 45 | |
46 | 46 | if ( !$messages ) { |
47 | 47 | $this->mMessageLog = new MessageLog(); |
@@ -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; |
@@ -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 ); |
@@ -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 | /** |