wikimedia / mediawiki-extensions-Lingo

src/Lingo.i18n.magic.php

Spacing +32 added lines, -32 removed lines

@@ -9,81 +9,81 @@
 $magicWords = array();
 
 /** English (English) */
-$magicWords[ 'en' ] = array(
-	'noglossary' => array( 0, '__NOGLOSSARY__' ),
+$magicWords['en'] = array(
+	'noglossary' => array(0, '__NOGLOSSARY__'),
 );
 
 /** Arabic (العربية) */
-$magicWords[ 'ar' ] = array(
-	'noglossary' => array( 0, '__لا_قاموس__' ),
+$magicWords['ar'] = array(
+	'noglossary' => array(0, '__لا_قاموس__'),
 );
 
 /** Egyptian Arabic (مصرى) */
-$magicWords[ 'arz' ] = array(
-	'noglossary' => array( 0, '__من_غير_قاموس__' ),
+$magicWords['arz'] = array(
+	'noglossary' => array(0, '__من_غير_قاموس__'),
 );
 
 /** German (Deutsch) */
-$magicWords[ 'de' ] = array(
-	'noglossary' => array( 0, '__KEINGLOSSAR__', '__KEIN_GLOSSAR__' ),
+$magicWords['de'] = array(
+	'noglossary' => array(0, '__KEINGLOSSAR__', '__KEIN_GLOSSAR__'),
 );
 
 /** Zazaki (Zazaki) */
-$magicWords[ 'diq' ] = array(
-	'noglossary' => array( 0, 'QISEBENDÇINİYO' ),
+$magicWords['diq'] = array(
+	'noglossary' => array(0, 'QISEBENDÇINİYO'),
 );
 
 /** Esperanto (Esperanto) */
-$magicWords[ 'eo' ] = array(
-	'noglossary' => array( 0, '__NEDIFINO__' ),
+$magicWords['eo'] = array(
+	'noglossary' => array(0, '__NEDIFINO__'),
 );
 
 /** French (français) */
-$magicWords[ 'fr' ] = array(
-	'noglossary' => array( 0, '__SANSGLOSSAIRE__' ),
+$magicWords['fr'] = array(
+	'noglossary' => array(0, '__SANSGLOSSAIRE__'),
 );
 
 /** Italian (italiano) */
-$magicWords[ 'it' ] = array(
-	'noglossary' => array( 0, '__NOGLOSSARIO__' ),
+$magicWords['it'] = array(
+	'noglossary' => array(0, '__NOGLOSSARIO__'),
 );
 
 /** Korean (한국어) */
-$magicWords[ 'ko' ] = array(
-	'noglossary' => array( 0, '__용어집없음__' ),
+$magicWords['ko'] = array(
+	'noglossary' => array(0, '__용어집없음__'),
 );
 
 /** Macedonian (македонски) */
-$magicWords[ 'mk' ] = array(
-	'noglossary' => array( 0, '__БЕЗПОИМНИК__' ),
+$magicWords['mk'] = array(
+	'noglossary' => array(0, '__БЕЗПОИМНИК__'),
 );
 
 /** Malayalam (മലയാളം) */
-$magicWords[ 'ml' ] = array(
-	'noglossary' => array( 0, '__ചുരുക്കംവേണ്ട__' ),
+$magicWords['ml'] = array(
+	'noglossary' => array(0, '__ചുരുക്കംവേണ്ട__'),
 );
 
 /** Low Saxon (Netherlands) (Nedersaksies) */
-$magicWords[ 'nds-nl' ] = array(
-	'noglossary' => array( 0, '__GIEN_GLOSSARIUM__' ),
+$magicWords['nds-nl'] = array(
+	'noglossary' => array(0, '__GIEN_GLOSSARIUM__'),
 );
 
 /** Dutch (Nederlands) */
-$magicWords[ 'nl' ] = array(
-	'noglossary' => array( 0, '___GEENWOORDENLIJST__' ),
+$magicWords['nl'] = array(
+	'noglossary' => array(0, '___GEENWOORDENLIJST__'),
 );
 
 /** Serbian (Latin script) (srpski (latinica)‎) */
-$magicWords[ 'sr-el' ] = array(
-	'noglossary' => array( 0, '__BEZREČNIKA__', '__BEZ_REČNIKA__' ),
+$magicWords['sr-el'] = array(
+	'noglossary' => array(0, '__BEZREČNIKA__', '__BEZ_REČNIKA__'),
 );
 
 /** Turkish (Türkçe) */
-$magicWords[ 'tr' ] = array(
-	'noglossary' => array( 0, '__SÖZLÜKYOK__' ),
+$magicWords['tr'] = array(
+	'noglossary' => array(0, '__SÖZLÜKYOK__'),
 );
 
 /** Simplified Chinese (中文（简体）‎) */
-$magicWords[ 'zh-hans' ] = array(
-	'noglossary' => array( 0, '__无词汇表__' ),
+$magicWords['zh-hans'] = array(
+	'noglossary' => array(0, '__无词汇表__'),
 );

src/LingoTree.php

Spacing +28 added lines, -28 removed lines

@@ -57,22 +57,22 @@ 
 	 * Adds a string to the Lingo Tree
 	 * @param String $term
 	 */
-	public function addTerm( &$term, $definition ) {
-		if ( !$term ) {
+	public function addTerm(&$term, $definition) {
+		if (!$term) {
 			return;
 		}
 
-		if ( isset( $this->mList[ $term ] ) ) { // term exists, store 2nd definition
-			$this->mList[ $term ]->addDefinition( $definition );
+		if (isset($this->mList[$term])) { // term exists, store 2nd definition
+			$this->mList[$term]->addDefinition($definition);
 		} else {
 
 			$matches = array();
-			preg_match_all( LingoParser::$regex, $term, $matches );
+			preg_match_all(LingoParser::$regex, $term, $matches);
 
-			$elt = $this->addElement( $matches[ 0 ], $term, $definition );
-			$this->mList[ $term ] = &$elt[ -1 ];
+			$elt = $this->addElement($matches[0], $term, $definition);
+			$this->mList[$term] = &$elt[ -1];
 
-			$this->mMinLength = min( array( $this->mMinLength, strlen( $term ) ) );
+			$this->mMinLength = min(array($this->mMinLength, strlen($term)));
 		}
 	}
 
@@ -84,24 +84,24 @@ 
 	 * @param String $definition
 	 * @return Array the tree node the element was stored in
 	 */
-	protected function &addElement( Array &$path, &$term, &$definition ) {
+	protected function &addElement(Array &$path, &$term, &$definition) {
 
 		$tree = &$this->mTree;
 
 		// end of path, store description; end of recursion
-		while ( ( $step = array_shift( $path ) ) !== null ) {
+		while (($step = array_shift($path)) !== null) {
 
-			if ( !isset( $tree[ $step ] ) ) {
-				$tree[ $step ] = array();
+			if (!isset($tree[$step])) {
+				$tree[$step] = array();
 			}
 
-			$tree = &$tree[ $step ];
+			$tree = &$tree[$step];
 		}
 
-		if ( isset( $tree[ -1 ] ) ) {
-			$tree[ -1 ]->addDefinition( $definition );
+		if (isset($tree[ -1])) {
+			$tree[ -1]->addDefinition($definition);
 		} else {
-			$tree[ -1 ] = new LingoElement( $term, $definition );
+			$tree[ -1] = new LingoElement($term, $definition);
 		}
 
 		return $tree;
@@ -115,18 +115,18 @@ 
 		return $this->mList;
 	}
 
-	public function findNextTerm( &$lexemes, $index, $countLexemes ) {
+	public function findNextTerm(&$lexemes, $index, $countLexemes) {
 
 		$start = $lastindex = $index;
 		$definition = null;
 
 		// skip until the start of a term is found
-		while ( $index < $countLexemes && !$definition ) {
-			$currLex = &$lexemes[ $index ][ 0 ];
+		while ($index < $countLexemes && !$definition) {
+			$currLex = &$lexemes[$index][0];
 
 			// Did we find the start of a term?
-			if ( array_key_exists( $currLex, $this->mTree ) ) {
-				list( $lastindex, $definition ) = $this->findNextTermNoSkip( $this->mTree[ $currLex ], $lexemes, $index, $countLexemes );
+			if (array_key_exists($currLex, $this->mTree)) {
+				list($lastindex, $definition) = $this->findNextTermNoSkip($this->mTree[$currLex], $lexemes, $index, $countLexemes);
 			}
 
 			// this will increase the index even if we found something;
@@ -134,19 +134,19 @@ 
 			$index++;
 		}
 
-		if ( $definition ) {
-			return array( $index - $start - 1, $lastindex - $index + 2, $definition );
+		if ($definition) {
+			return array($index - $start - 1, $lastindex - $index + 2, $definition);
 		} else {
-			return array( $index - $start, 0, null );
+			return array($index - $start, 0, null);
 		}
 	}
 
-	public function findNextTermNoSkip( Array &$tree, &$lexemes, $index, $countLexemes ) {
+	public function findNextTermNoSkip(Array &$tree, &$lexemes, $index, $countLexemes) {
 
-		if ( $index + 1 < $countLexemes && array_key_exists( $currLex = $lexemes[ $index + 1 ][ 0 ], $tree ) ) {
-			$ret = $this->findNextTermNoSkip( $tree[ $currLex ], $lexemes, $index + 1, $countLexemes );
+		if ($index + 1 < $countLexemes && array_key_exists($currLex = $lexemes[$index + 1][0], $tree)) {
+			$ret = $this->findNextTermNoSkip($tree[$currLex], $lexemes, $index + 1, $countLexemes);
 		} else {
-			$ret = array( $index, &$tree[ -1 ] );
+			$ret = array($index, &$tree[ -1]);
 		}
 
 		return $ret;

src/LingoParser.php

Spacing +74 added lines, -74 removed lines

@@ -52,18 +52,18 @@ 
 	 * LingoParser constructor.
 	 * @param LingoMessageLog|null $messages
 	 */
-	public function __construct( LingoMessageLog &$messages = null ) {
+	public function __construct(LingoMessageLog&$messages = null) {
 		global $wgexLingoBackend;
 
-		$this->mLingoBackend = new $wgexLingoBackend( $messages );
+		$this->mLingoBackend = new $wgexLingoBackend($messages);
 	}
 
 	/**
 	 * @param Parser $parser
 	 * @return string
 	 */
-	private static function uniqPrefix( Parser &$parser ) {
-		if ( defined( "Parser::MARKER_PREFIX" ) ) {
+	private static function uniqPrefix(Parser&$parser) {
+		if (defined("Parser::MARKER_PREFIX")) {
 			return Parser::MARKER_PREFIX;
 		} else {
 			return $parser->uniqPrefix();
@@ -76,17 +76,17 @@ 
 	 * @param string $text
 	 * @return Boolean
 	 */
-	public static function parse( Parser &$parser, &$text ) {
+	public static function parse(Parser&$parser, &$text) {
 
-		if ( !self::$parserSingleton ) {
+		if (!self::$parserSingleton) {
 			self::$parserSingleton = new LingoParser();
 
 			// The RegEx to split a chunk of text into words
 			// Words are: placeholders for stripped items, sequences of letters and numbers, single characters that are neither letter nor number
-			self::$regex = '/' . preg_quote( self::uniqPrefix( $parser ), '/' ) . '.*?' . preg_quote( Parser::MARKER_SUFFIX, '/' ) . '|[\p{L}\p{N}]+|[^\p{L}\p{N}]/u';
+			self::$regex = '/'.preg_quote(self::uniqPrefix($parser), '/').'.*?'.preg_quote(Parser::MARKER_SUFFIX, '/').'|[\p{L}\p{N}]+|[^\p{L}\p{N}]/u';
 		}
 
-		self::$parserSingleton->realParse( $parser, $text );
+		self::$parserSingleton->realParse($parser, $text);
 
 		return true;
 	}
@@ -107,7 +107,7 @@ 
 	public function getLingoArray() {
 
 		// build glossary array only once per request
-		if ( !$this->mLingoTree ) {
+		if (!$this->mLingoTree) {
 			$this->buildLingo();
 		}
 
@@ -122,33 +122,33 @@ 
 	public function getLingoTree() {
 
 		// build glossary array only once per request
-		if ( !$this->mLingoTree ) {
+		if (!$this->mLingoTree) {
 
 			// use cache if enabled
-			if ( $this->mLingoBackend->useCache() ) {
+			if ($this->mLingoBackend->useCache()) {
 
 				// Try cache first
 				global $wgexLingoCacheType;
-				$cache = ( $wgexLingoCacheType !== null ) ? wfGetCache( $wgexLingoCacheType ) : wfGetMainCache();
-				$cachekey = wfMemcKey( 'ext', 'lingo', 'lingotree' );
-				$cachedLingoTree = $cache->get( $cachekey );
+				$cache = ($wgexLingoCacheType !== null) ? wfGetCache($wgexLingoCacheType) : wfGetMainCache();
+				$cachekey = wfMemcKey('ext', 'lingo', 'lingotree');
+				$cachedLingoTree = $cache->get($cachekey);
 
 				// cache hit?
-				if ( $cachedLingoTree !== false && $cachedLingoTree !== null ) {
+				if ($cachedLingoTree !== false && $cachedLingoTree !== null) {
 
-					wfDebug( "Cache hit: Got lingo tree from cache.\n" );
+					wfDebug("Cache hit: Got lingo tree from cache.\n");
 					$this->mLingoTree = &$cachedLingoTree;
 
 				} else {
 
-					wfDebug( "Cache miss: Lingo tree not found in cache.\n" );
-					$this->mLingoTree =& $this->buildLingo();
-					$cache->set( $cachekey, $this->mLingoTree );
-					wfDebug( "Cached lingo tree.\n" );
+					wfDebug("Cache miss: Lingo tree not found in cache.\n");
+					$this->mLingoTree = & $this->buildLingo();
+					$cache->set($cachekey, $this->mLingoTree);
+					wfDebug("Cached lingo tree.\n");
 				}
 			} else {
-				wfDebug( "Caching of lingo tree disabled.\n" );
-				$this->mLingoTree =& $this->buildLingo();
+				wfDebug("Caching of lingo tree disabled.\n");
+				$this->mLingoTree = & $this->buildLingo();
 			}
 
 		}
@@ -162,8 +162,8 @@ 
 		$backend = &$this->mLingoBackend;
 
 		// assemble the result array
-		while ( $elementData = $backend->next() ) {
-			$lingoTree->addTerm( $elementData[ LingoElement::ELEMENT_TERM ], $elementData );
+		while ($elementData = $backend->next()) {
+			$lingoTree->addTerm($elementData[LingoElement::ELEMENT_TERM], $elementData);
 		}
 
 		return $lingoTree;
@@ -178,16 +178,16 @@ 
 	 * @param $text
 	 * @return Boolean
 	 */
-	protected function realParse( &$parser, &$text ) {
+	protected function realParse(&$parser, &$text) {
 		global $wgRequest;
 
-		$action = $wgRequest->getVal( 'action', 'view' );
+		$action = $wgRequest->getVal('action', 'view');
 
-		if ( $text === null ||
+		if ($text === null ||
 			$text === '' ||
 			$action === 'edit' ||
 			$action === 'ajax' ||
-			isset( $_POST[ 'wpPreview' ] )
+			isset($_POST['wpPreview'])
 		) {
 
 			return true;
@@ -196,20 +196,20 @@ 
 		// Get array of terms
 		$glossary = $this->getLingoTree();
 
-		if ( $glossary == null ) {
+		if ($glossary == null) {
 			return true;
 		}
 
 		// Parse HTML from page
 		wfSuppressWarnings();
 
-		$doc = new DOMDocument( '1.0', 'utf-8' );
-		$doc->loadHTML( '<html><head><meta http-equiv="content-type" content="charset=utf-8"/></head><body>' . $text . '</body></html>' );
+		$doc = new DOMDocument('1.0', 'utf-8');
+		$doc->loadHTML('<html><head><meta http-equiv="content-type" content="charset=utf-8"/></head><body>'.$text.'</body></html>');
 
 		wfRestoreWarnings();
 
 		// Find all text in HTML.
-		$xpath = new DOMXpath( $doc );
+		$xpath = new DOMXpath($doc);
 		$elements = $xpath->query(
 			"//*[not(ancestor-or-self::*[@class='noglossary'] or ancestor-or-self::a)][text()!=' ']/text()"
 		);
@@ -218,10 +218,10 @@ 
 		$nb = $elements->length;
 		$changedDoc = false;
 
-		for ( $pos = 0; $pos < $nb; $pos++ ) {
-			$el = $elements->item( $pos );
+		for ($pos = 0; $pos < $nb; $pos++) {
+			$el = $elements->item($pos);
 
-			if ( strlen( $el->nodeValue ) < $glossary->getMinTermLength() ) {
+			if (strlen($el->nodeValue) < $glossary->getMinTermLength()) {
 				continue;
 			}
 
@@ -230,36 +230,36 @@ 
 				self::$regex,
 				$el->nodeValue,
 				$matches,
-				PREG_OFFSET_CAPTURE | PREG_PATTERN_ORDER
+				PREG_OFFSET_CAPTURE|PREG_PATTERN_ORDER
 			);
 
-			if ( count( $matches ) == 0 || count( $matches[ 0 ] ) == 0 ) {
+			if (count($matches) == 0 || count($matches[0]) == 0) {
 				continue;
 			}
 
-			$lexemes = &$matches[ 0 ];
-			$countLexemes = count( $lexemes );
+			$lexemes = &$matches[0];
+			$countLexemes = count($lexemes);
 			$parent = &$el->parentNode;
 			$index = 0;
 			$changedElem = false;
 
-			while ( $index < $countLexemes ) {
-				list( $skipped, $used, $definition ) =
-					$glossary->findNextTerm( $lexemes, $index, $countLexemes );
+			while ($index < $countLexemes) {
+				list($skipped, $used, $definition) =
+					$glossary->findNextTerm($lexemes, $index, $countLexemes);
 
-				if ( $used > 0 ) { // found a term
-					if ( $skipped > 0 ) { // skipped some text, insert it as is
+				if ($used > 0) { // found a term
+					if ($skipped > 0) { // skipped some text, insert it as is
 						$parent->insertBefore(
 							$doc->createTextNode(
-								substr( $el->nodeValue,
-									$currLexIndex = $lexemes[ $index ][ 1 ],
-									$lexemes[ $index + $skipped ][ 1 ] - $currLexIndex )
+								substr($el->nodeValue,
+									$currLexIndex = $lexemes[$index][1],
+									$lexemes[$index + $skipped][1] - $currLexIndex)
 							),
 							$el
 						);
 					}
 
-					$parent->insertBefore( $definition->getFullDefinition( $doc ), $el );
+					$parent->insertBefore($definition->getFullDefinition($doc), $el);
 
 					$changedElem = true;
 				} else { // did not find term, just use the rest of the text
@@ -267,10 +267,10 @@ 
 					// term in the whole element. Might as well not change the
 					// element at all.
 					// Only change element if found term before
-					if ( $changedElem ) {
+					if ($changedElem) {
 						$parent->insertBefore(
 							$doc->createTextNode(
-								substr( $el->nodeValue, $lexemes[ $index ][ 1 ] )
+								substr($el->nodeValue, $lexemes[$index][1])
 							),
 							$el
 						);
@@ -284,17 +284,17 @@ 
 				$index += $used + $skipped;
 			}
 
-			if ( $changedElem ) {
-				$parent->removeChild( $el );
+			if ($changedElem) {
+				$parent->removeChild($el);
 				$changedDoc = true;
 			}
 		}
 
-		if ( $changedDoc ) {
-			$this->loadModules( $parser );
+		if ($changedDoc) {
+			$this->loadModules($parser);
 
 			// U - Ungreedy, D - dollar matches only end of string, s - dot matches newlines
-			$text = preg_replace( '%(^.*<body>)|(</body>.*$)%UDs', '', $doc->saveHTML() );
+			$text = preg_replace('%(^.*<body>)|(</body>.*$)%UDs', '', $doc->saveHTML());
 		}
 
 		return true;
@@ -303,39 +303,39 @@ 
 	/**
 	 * @param $parser
 	 */
-	protected function loadModules( &$parser ) {
+	protected function loadModules(&$parser) {
 		global $wgOut, $wgScriptPath;
 
 		$parserOutput = $parser->getOutput();
 
 		// load scripts
-		if ( defined( 'MW_SUPPORTS_RESOURCE_MODULES' ) ) {
-			$parserOutput->addModules( 'ext.Lingo.Scripts' );
+		if (defined('MW_SUPPORTS_RESOURCE_MODULES')) {
+			$parserOutput->addModules('ext.Lingo.Scripts');
 
-			if ( !$wgOut->isArticle() ) {
-				$wgOut->addModules( 'ext.Lingo.Scripts' );
+			if (!$wgOut->isArticle()) {
+				$wgOut->addModules('ext.Lingo.Scripts');
 			}
 		} else {
 			global $wgStylePath;
-			$parserOutput->addHeadItem( "<script src='$wgStylePath/common/jquery.min.js'></script>\n", 'ext.Lingo.jq' );
-			$parserOutput->addHeadItem( "<script src='$wgScriptPath/extensions/Lingo/libs/Lingo.js'></script>\n", 'ext.Lingo.Scripts' );
+			$parserOutput->addHeadItem("<script src='$wgStylePath/common/jquery.min.js'></script>\n", 'ext.Lingo.jq');
+			$parserOutput->addHeadItem("<script src='$wgScriptPath/extensions/Lingo/libs/Lingo.js'></script>\n", 'ext.Lingo.Scripts');
 
-			if ( !$wgOut->isArticle() ) {
-				$wgOut->addHeadItem( 'ext.Lingo.jq', "<script src='$wgStylePath/common/jquery.min.js'></script>\n" );
-				$wgOut->addHeadItem( 'ext.Lingo.Scripts', "<script src='$wgScriptPath/extensions/Lingo/libs/Lingo.js'></script>\n" );
+			if (!$wgOut->isArticle()) {
+				$wgOut->addHeadItem('ext.Lingo.jq', "<script src='$wgStylePath/common/jquery.min.js'></script>\n");
+				$wgOut->addHeadItem('ext.Lingo.Scripts', "<script src='$wgScriptPath/extensions/Lingo/libs/Lingo.js'></script>\n");
 			}
 		}
 
 		// load styles
-		if ( method_exists( $parserOutput, 'addModuleStyles' ) ) {
-			$parserOutput->addModuleStyles( 'ext.Lingo.Styles' );
-			if ( !$wgOut->isArticle() ) {
-				$wgOut->addModuleStyles( 'ext.Lingo.Styles' );
+		if (method_exists($parserOutput, 'addModuleStyles')) {
+			$parserOutput->addModuleStyles('ext.Lingo.Styles');
+			if (!$wgOut->isArticle()) {
+				$wgOut->addModuleStyles('ext.Lingo.Styles');
 			}
 		} else {
-			$parserOutput->addHeadItem( "<link rel='stylesheet' href='$wgScriptPath/extensions/Lingo/styles/Lingo.css' />\n", 'ext.Lingo.Styles' );
-			if ( !$wgOut->isArticle() ) {
-				$wgOut->addHeadItem( 'ext.Lingo.Styles', "<link rel='stylesheet' href='$wgScriptPath/extensions/Lingo/styles/Lingo.css' />\n" );
+			$parserOutput->addHeadItem("<link rel='stylesheet' href='$wgScriptPath/extensions/Lingo/styles/Lingo.css' />\n", 'ext.Lingo.Styles');
+			if (!$wgOut->isArticle()) {
+				$wgOut->addHeadItem('ext.Lingo.Styles', "<link rel='stylesheet' href='$wgScriptPath/extensions/Lingo/styles/Lingo.css' />\n");
 			}
 		}
 	}
@@ -346,8 +346,8 @@ 
 	public static function purgeCache() {
 
 		global $wgexLingoCacheType;
-		$cache = ( $wgexLingoCacheType !== null ) ? wfGetCache( $wgexLingoCacheType ) : wfGetMainCache();
-		$cache->delete( wfMemcKey( 'ext', 'lingo', 'lingotree' ) );
+		$cache = ($wgexLingoCacheType !== null) ? wfGetCache($wgexLingoCacheType) : wfGetMainCache();
+		$cache->delete(wfMemcKey('ext', 'lingo', 'lingotree'));
 
 	}
 }

src/LingoBasicBackend.php

Spacing +37 added lines, -37 removed lines

@@ -39,18 +39,18 @@ 
 	 * LingoBasicBackend constructor.
 	 * @param LingoMessageLog|null $messages
 	 */
-	public function __construct( LingoMessageLog &$messages = null ) {
+	public function __construct(LingoMessageLog&$messages = null) {
 
 		global $wgexLingoPage, $wgRequest;
 
-		$page = $wgexLingoPage ? $wgexLingoPage : wfMessage( 'lingo-terminologypagename' )->inContentLanguage()->text();
+		$page = $wgexLingoPage ? $wgexLingoPage : wfMessage('lingo-terminologypagename')->inContentLanguage()->text();
 
-		parent::__construct( $messages );
+		parent::__construct($messages);
 
 		// Get Terminology page
-		$title = Title::newFromText( $page );
-		if ( $title->getInterwiki() ) {
-			$this->getMessageLog()->addError( wfMessage( 'lingo-terminologypagenotlocal', $page )->inContentLanguage()->text() );
+		$title = Title::newFromText($page);
+		if ($title->getInterwiki()) {
+			$this->getMessageLog()->addError(wfMessage('lingo-terminologypagenotlocal', $page)->inContentLanguage()->text());
 			return false;
 		}
 
@@ -58,16 +58,16 @@ 
 		// page itself. In this case the Revision is not up to date when we get
 		// here, i.e. $rev->getText() would return outdated Test.
 		// This hack takes the text directly out of the data from the web request.
-		if ( $wgRequest->getVal( 'action', 'view' ) === 'submit'
-			&& Title::newFromText( $wgRequest->getVal( 'title' ) )->getArticleID() === $title->getArticleID()
+		if ($wgRequest->getVal('action', 'view') === 'submit'
+			&& Title::newFromText($wgRequest->getVal('title'))->getArticleID() === $title->getArticleID()
 		) {
 
-			$content = $wgRequest->getVal( 'wpTextbox1' );
+			$content = $wgRequest->getVal('wpTextbox1');
 
 		} else {
-			$rev = $this->getRevision( $title );
-			if ( !$rev ) {
-				$this->getMessageLog()->addWarning( wfMessage( 'lingo-noterminologypage', $page )->inContentLanguage()->text() );
+			$rev = $this->getRevision($title);
+			if (!$rev) {
+				$this->getMessageLog()->addWarning(wfMessage('lingo-noterminologypage', $page)->inContentLanguage()->text());
 				return false;
 			}
 
@@ -78,9 +78,9 @@ 
 		$parser = new Parser;
 		// expand templates and variables in the text, producing valid, static wikitext
 		// have to use a new anonymous user to avoid any leakage as Lingo is caching only one user-independant glossary
-		$content = $parser->preprocess( $content, $title, new ParserOptions( new User() ) );
+		$content = $parser->preprocess($content, $title, new ParserOptions(new User()));
 
-		$this->mArticleLines = array_reverse( explode( "\n", $content ) );
+		$this->mArticleLines = array_reverse(explode("\n", $content));
 	}
 
 	/**
@@ -93,40 +93,40 @@ 
 	 */
 	public function next() {
 
-		wfProfileIn( __METHOD__ );
+		wfProfileIn(__METHOD__);
 
 		static $term = null;
 		static $definitions = array();
 		static $ret = array();
 
 		// find next valid line (yes, the assignation is intended)
-		while ( ( count( $ret ) == 0 ) && ( $entry = each( $this->mArticleLines ) ) ) {
+		while ((count($ret) == 0) && ($entry = each($this->mArticleLines))) {
 
-			if ( empty( $entry[ 1 ] ) || ( $entry[ 1 ][ 0 ] !== ';' && $entry[ 1 ][ 0 ] !== ':' ) ) {
+			if (empty($entry[1]) || ($entry[1][0] !== ';' && $entry[1][0] !== ':')) {
 				continue;
 			}
 
-			$chunks = explode( ':', $entry[ 1 ], 2 );
+			$chunks = explode(':', $entry[1], 2);
 
 			// found a new definition?
-			if ( count( $chunks ) == 2 ) {
+			if (count($chunks) == 2) {
 
 				// wipe the data if its a totaly new term definition
-				if ( !empty( $term ) && count( $definitions ) > 0 ) {
+				if (!empty($term) && count($definitions) > 0) {
 					$definitions = array();
 					$term = null;
 				}
 
-				$definitions[] = trim( $chunks[ 1 ] );
+				$definitions[] = trim($chunks[1]);
 			}
 
 			// found a new term?
-			if ( count( $chunks ) >= 1 && strlen( $chunks[ 0 ] ) >= 1 ) {
-				$term = trim( substr( $chunks[ 0 ], 1 ) );
+			if (count($chunks) >= 1 && strlen($chunks[0]) >= 1) {
+				$term = trim(substr($chunks[0], 1));
 			}
 
-			if ( $term !== null ) {
-				foreach ( $definitions as $definition ) {
+			if ($term !== null) {
+				foreach ($definitions as $definition) {
 					$ret[] = array(
 						LingoElement::ELEMENT_TERM       => $term,
 						LingoElement::ELEMENT_DEFINITION => $definition,
@@ -137,9 +137,9 @@ 
 			}
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
-		return array_pop( $ret );
+		return array_pop($ret);
 	}
 
 	/**
@@ -148,20 +148,20 @@ 
 	 * @param Title $title
 	 * @return Revision
 	 */
-	public function getRevision( $title ) {
+	public function getRevision($title) {
 		global $wgexLingoEnableApprovedRevs;
 
-		if ( $wgexLingoEnableApprovedRevs ) {
+		if ($wgexLingoEnableApprovedRevs) {
 
-			if ( defined( 'APPROVED_REVS_VERSION' ) ) {
-				$rev_id = ApprovedRevs::getApprovedRevID( $title );
-				return Revision::newFromId( $rev_id );
+			if (defined('APPROVED_REVS_VERSION')) {
+				$rev_id = ApprovedRevs::getApprovedRevID($title);
+				return Revision::newFromId($rev_id);
 			} else {
-				wfDebug( 'Support for ApprovedRevs is enabled in Lingo. But ApprovedRevs was not found.\n' );
+				wfDebug('Support for ApprovedRevs is enabled in Lingo. But ApprovedRevs was not found.\n');
 			}
 		}
 
-		return Revision::newFromTitle( $title );
+		return Revision::newFromTitle($title);
 	}
 
 	/**
@@ -170,12 +170,12 @@ 
 	 * @param Page $wikipage
 	 * @return Bool
 	 */
-	public static function purgeCache( &$wikipage ) {
+	public static function purgeCache(&$wikipage) {
 
 		global $wgexLingoPage;
-		$page = $wgexLingoPage ? $wgexLingoPage : wfMessage( 'lingo-terminologypagename' )->inContentLanguage()->text();
+		$page = $wgexLingoPage ? $wgexLingoPage : wfMessage('lingo-terminologypagename')->inContentLanguage()->text();
 
-		if ( !is_null( $wikipage ) && ( $wikipage->getTitle()->getText() === $page ) ) {
+		if (!is_null($wikipage) && ($wikipage->getTitle()->getText() === $page)) {
 
 			LingoParser::purgeCache();
 		}

src/LingoHooks.php

Spacing +15 added lines, -15 removed lines

@@ -42,26 +42,26 @@ 
 	 * @param String $text
 	 * @return Boolean
 	 */
-	public static function parse( &$parser, &$text ) {
+	public static function parse(&$parser, &$text) {
 
 		global $wgexLingoUseNamespaces;
 
 		$title = $parser->getTitle();
 
 		// parse if
-		if ( !isset( $parser->mDoubleUnderscores[ 'noglossary' ] ) && // __NOGLOSSARY__ not present and
+		if (!isset($parser->mDoubleUnderscores['noglossary']) && // __NOGLOSSARY__ not present and
 			(
 				!$title || // title not set or
-				!isset( $wgexLingoUseNamespaces[ $title->getNamespace() ] ) || // namespace not explicitly forbidden (i.e. not in list of namespaces and set to false) or
-				$wgexLingoUseNamespaces[ $title->getNamespace() ] // namespace explicitly allowed
+				!isset($wgexLingoUseNamespaces[$title->getNamespace()]) || // namespace not explicitly forbidden (i.e. not in list of namespaces and set to false) or
+				$wgexLingoUseNamespaces[$title->getNamespace()] // namespace explicitly allowed
 			)
 		) {
 
 			// unstrip strip items of the 'general' group
 			// this will be done again by parse when this hook returns, but it should not hurt to do this twice
 			// Only problem is with other hook handlers that might not expect strip items to be unstripped already
-			$text = $parser->mStripState->unstripGeneral( $text );
-			LingoParser::parse( $parser, $text );
+			$text = $parser->mStripState->unstripGeneral($text);
+			LingoParser::parse($parser, $text);
 		}
 
 		return true;
@@ -70,8 +70,8 @@ 
 	/**
 	 * Creates tag hook(s)
 	 */
-	public static function registerTags( Parser $parser ) {
-		$parser->setHook( 'noglossary', 'LingoHooks::noglossaryTagRenderer' );
+	public static function registerTags(Parser $parser) {
+		$parser->setHook('noglossary', 'LingoHooks::noglossaryTagRenderer');
 		return true;
 	}
 
@@ -84,9 +84,9 @@ 
 	 * @param PPFrame $frame
 	 * @return string
 	 */
-	public static function noglossaryTagRenderer( $input, array $args, Parser $parser, PPFrame $frame ) {
-		$output = $parser->recursiveTagParse( $input, $frame );
-		return '<span class="noglossary">' . $output . '</span>';
+	public static function noglossaryTagRenderer($input, array $args, Parser $parser, PPFrame $frame) {
+		$output = $parser->recursiveTagParse($input, $frame);
+		return '<span class="noglossary">'.$output.'</span>';
 	}
 
 	/**
@@ -97,10 +97,10 @@ 
 	public static function initExtension() {
 		MagicWord::$mDoubleUnderscoreIDs[] = 'noglossary';
 
-		foreach ( $GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ] as $index => $description ) {
-			if ( $GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ][ $index ][ 'name' ] === 'Lingo' ) {
-				$GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ][ $index ][ 'description' ] =
-					wfMessage( 'lingo-desc', $GLOBALS[ 'wgexLingoPage' ] ? $GLOBALS[ 'wgexLingoPage' ] : wfMessage( 'lingo-terminologypagename' )->inContentLanguage()->text() )->text();
+		foreach ($GLOBALS['wgExtensionCredits']['parserhook'] as $index => $description) {
+			if ($GLOBALS['wgExtensionCredits']['parserhook'][$index]['name'] === 'Lingo') {
+				$GLOBALS['wgExtensionCredits']['parserhook'][$index]['description'] =
+					wfMessage('lingo-desc', $GLOBALS['wgexLingoPage'] ? $GLOBALS['wgexLingoPage'] : wfMessage('lingo-terminologypagename')->inContentLanguage()->text())->text();
 			}
 		}
 	}

src/LingoElement.php

Spacing +71 added lines, -71 removed lines

@@ -40,7 +40,7 @@ 
 	const ELEMENT_LINK = 3;
 	const ELEMENT_STYLE = 4;
 
-	const ELEMENT_FIELDCOUNT = 5;  // number of fields stored for each element; (last field's index) + 1
+	const ELEMENT_FIELDCOUNT = 5; // number of fields stored for each element; (last field's index) + 1
 
 	private $mFullDefinition = null;
 	private $mDefinitions = array();
@@ -54,125 +54,125 @@ 
 	 * @param $term
 	 * @param $definition
 	 */
-	public function __construct( &$term, &$definition = null ) {
+	public function __construct(&$term, &$definition = null) {
 
 		$this->mTerm = $term;
 
-		if ( $definition ) {
-			$this->addDefinition( $definition );
+		if ($definition) {
+			$this->addDefinition($definition);
 		}
 	}
 
 	/**
 	 * @param $definition
 	 */
-	public function addDefinition( &$definition ) {
-		$this->mDefinitions[] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
+	public function addDefinition(&$definition) {
+		$this->mDefinitions[] = array_pad($definition, self::ELEMENT_FIELDCOUNT, null);
 	}
 
 	/**
 	 * @param DOMDocument $doc
 	 * @return DOMNode|DOMText
 	 */
-	public function getFullDefinition( DOMDocument &$doc ) {
+	public function getFullDefinition(DOMDocument&$doc) {
 
 		global $wgexLingoDisplayOnce;
 
 		// return textnode if
-		if ( $wgexLingoDisplayOnce && $this->mHasBeenDisplayed ) {
-			return $doc->createTextNode( $this->mTerm );
+		if ($wgexLingoDisplayOnce && $this->mHasBeenDisplayed) {
+			return $doc->createTextNode($this->mTerm);
 		}
 
 		// only create if not yet created
-		if ( $this->mFullDefinition === null || $this->mFullDefinition->ownerDocument !== $doc ) {
+		if ($this->mFullDefinition === null || $this->mFullDefinition->ownerDocument !== $doc) {
 
 			// if there is only one link available, just insert the link
-			if ( count( $this->mDefinitions ) === 1
-				&& !is_string( $this->mDefinitions[ 0 ][ self::ELEMENT_DEFINITION ] )
-				&& is_string( $this->mDefinitions[ 0 ][ self::ELEMENT_LINK ] )
+			if (count($this->mDefinitions) === 1
+				&& !is_string($this->mDefinitions[0][self::ELEMENT_DEFINITION])
+				&& is_string($this->mDefinitions[0][self::ELEMENT_LINK])
 			) {
 
-				$this->mFullDefinition = $this->getFullDefinitionAsLink( $doc );
+				$this->mFullDefinition = $this->getFullDefinitionAsLink($doc);
 
 			} else { // else insert the complete tooltip
 
-				$this->mFullDefinition = $this->getFullDefinitionAsTooltip( $doc );
+				$this->mFullDefinition = $this->getFullDefinitionAsTooltip($doc);
 			}
 
 			$this->mHasBeenDisplayed = true;
 		}
 
-		return $this->mFullDefinition->cloneNode( true );
+		return $this->mFullDefinition->cloneNode(true);
 	}
 
 	/**
 	 * @return mixed
 	 */
 	public function getCurrentKey() {
-		return key( $this->mDefinitions );
+		return key($this->mDefinitions);
 	}
 
 	/**
 	 * @param $key
 	 * @return mixed
 	 */
-	public function getTerm( $key ) {
-		return $this->mDefinitions[ $key ][ self::ELEMENT_TERM ];
+	public function getTerm($key) {
+		return $this->mDefinitions[$key][self::ELEMENT_TERM];
 	}
 
 	/**
 	 * @param $key
 	 * @return mixed
 	 */
-	public function getSource( &$key ) {
-		return $this->mDefinitions[ $key ][ self::ELEMENT_SOURCE ];
+	public function getSource(&$key) {
+		return $this->mDefinitions[$key][self::ELEMENT_SOURCE];
 	}
 
 	/**
 	 * @param $key
 	 * @return mixed
 	 */
-	public function getDefinition( &$key ) {
-		return $this->mDefinitions[ $key ][ self::ELEMENT_DEFINITION ];
+	public function getDefinition(&$key) {
+		return $this->mDefinitions[$key][self::ELEMENT_DEFINITION];
 	}
 
 	/**
 	 * @param $key
 	 * @return mixed
 	 */
-	public function getLink( &$key ) {
-		return $this->mDefinitions[ $key ][ self::ELEMENT_LINK ];
+	public function getLink(&$key) {
+		return $this->mDefinitions[$key][self::ELEMENT_LINK];
 	}
 
 	/**
 	 * @param $key
 	 * @return mixed
 	 */
-	public function getStyle( &$key ) {
-		return $this->mDefinitions[ $key ][ self::ELEMENT_STYLE ];
+	public function getStyle(&$key) {
+		return $this->mDefinitions[$key][self::ELEMENT_STYLE];
 	}
 
 	public function next() {
-		next( $this->mDefinitions );
+		next($this->mDefinitions);
 	}
 
 	/**
 	 * @param DOMDocument $doc
 	 * @return DOMNode
 	 */
-	private function getLinkTemplate( DOMDocument &$doc ) {
+	private function getLinkTemplate(DOMDocument&$doc) {
 		// create template if it does not yet exist
-		if ( !self::$mLinkTemplate || ( self::$mLinkTemplate->ownerDocument !== $doc ) ) {
+		if (!self::$mLinkTemplate || (self::$mLinkTemplate->ownerDocument !== $doc)) {
 			global $wgScriptPath;
 
-			$linkimage = $doc->createElement( 'img' );
-			$linkimage->setAttribute( 'src', $wgScriptPath . '/extensions/Lingo/styles/linkicon.png' );
+			$linkimage = $doc->createElement('img');
+			$linkimage->setAttribute('src', $wgScriptPath.'/extensions/Lingo/styles/linkicon.png');
 
-			self::$mLinkTemplate = $doc->createElement( 'a' );
-			self::$mLinkTemplate->appendChild( $linkimage );
+			self::$mLinkTemplate = $doc->createElement('a');
+			self::$mLinkTemplate->appendChild($linkimage);
 		}
 
-		return self::$mLinkTemplate->cloneNode( true );
+		return self::$mLinkTemplate->cloneNode(true);
 	}
 
 	/**
@@ -181,18 +181,18 @@ 
 	 * @return DOMElement
 	 * @throws MWException
 	 */
-	protected function getFullDefinitionAsLink( DOMDocument &$doc ) {
+	protected function getFullDefinitionAsLink(DOMDocument&$doc) {
 
 		// create Title object for target page
-		$target = Title::newFromText( $this->mDefinitions[ 0 ][ self::ELEMENT_LINK ] );
+		$target = Title::newFromText($this->mDefinitions[0][self::ELEMENT_LINK]);
 
 		// create link element
-		$link = $doc->createElement( 'a', $this->mDefinitions[ 0 ][ self::ELEMENT_TERM ] );
+		$link = $doc->createElement('a', $this->mDefinitions[0][self::ELEMENT_TERM]);
 
 		// set the link target
-		$link->setAttribute( 'href', $target->getLinkUrl() );
-		$link = $this->addClassAttributeToLink( $target, $link );
-		$link = $this->addTitleAttributeToLink( $target, $link );
+		$link->setAttribute('href', $target->getLinkUrl());
+		$link = $this->addClassAttributeToLink($target, $link);
+		$link = $this->addTitleAttributeToLink($target, $link);
 
 		return $link;
 	}
@@ -203,42 +203,42 @@ 
 	 * @return string
 	 * @throws MWException
 	 */
-	protected function getFullDefinitionAsTooltip( DOMDocument &$doc ) {
+	protected function getFullDefinitionAsTooltip(DOMDocument&$doc) {
 
 		// Wrap term and definition in <span> tags
-		$span = $doc->createElement( 'span' );
-		$span->setAttribute( 'class', 'mw-lingo-tooltip ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] );
+		$span = $doc->createElement('span');
+		$span->setAttribute('class', 'mw-lingo-tooltip '.$this->mDefinitions[0][self::ELEMENT_STYLE]);
 
 		// Wrap term in <span> tag, hidden
 		wfSuppressWarnings();
-		$spanTerm = $doc->createElement( 'span', htmlentities( $this->mTerm, ENT_COMPAT, 'UTF-8' ) );
+		$spanTerm = $doc->createElement('span', htmlentities($this->mTerm, ENT_COMPAT, 'UTF-8'));
 
 		wfRestoreWarnings();
-		$spanTerm->setAttribute( 'class', 'mw-lingo-tooltip-abbr' );
+		$spanTerm->setAttribute('class', 'mw-lingo-tooltip-abbr');
 
 		// Wrap definition in a <span> tag
-		$spanDefinition = $doc->createElement( 'span' );
-		$spanDefinition->setAttribute( 'class', 'mw-lingo-tooltip-tip ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] );
+		$spanDefinition = $doc->createElement('span');
+		$spanDefinition->setAttribute('class', 'mw-lingo-tooltip-tip '.$this->mDefinitions[0][self::ELEMENT_STYLE]);
 
-		foreach ( $this->mDefinitions as $definition ) {
+		foreach ($this->mDefinitions as $definition) {
 			wfSuppressWarnings();
-			$element = $doc->createElement( 'span', htmlentities( $definition[ self::ELEMENT_DEFINITION ], ENT_COMPAT, 'UTF-8' ) );
-			$element->setAttribute( 'class', 'mw-lingo-tooltip-definition ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] );
+			$element = $doc->createElement('span', htmlentities($definition[self::ELEMENT_DEFINITION], ENT_COMPAT, 'UTF-8'));
+			$element->setAttribute('class', 'mw-lingo-tooltip-definition '.$this->mDefinitions[0][self::ELEMENT_STYLE]);
 			wfRestoreWarnings();
-			if ( $definition[ self::ELEMENT_LINK ] ) {
-				$linkedTitle = Title::newFromText( $definition[ self::ELEMENT_LINK ] );
-				if ( $linkedTitle ) {
-					$link = $this->getLinkTemplate( $doc );
-					$link->setAttribute( 'href', $linkedTitle->getFullURL() );
-					$element->appendChild( $link );
+			if ($definition[self::ELEMENT_LINK]) {
+				$linkedTitle = Title::newFromText($definition[self::ELEMENT_LINK]);
+				if ($linkedTitle) {
+					$link = $this->getLinkTemplate($doc);
+					$link->setAttribute('href', $linkedTitle->getFullURL());
+					$element->appendChild($link);
 				}
 			}
-			$spanDefinition->appendChild( $element );
+			$spanDefinition->appendChild($element);
 		}
 
 		// insert term and definition
-		$span->appendChild( $spanTerm );
-		$span->appendChild( $spanDefinition );
+		$span->appendChild($spanTerm);
+		$span->appendChild($spanDefinition);
 		return $span;
 	}
 
@@ -246,15 +246,15 @@ 
 	 * @param $target
 	 * @param $link
 	 */
-	protected function &addTitleAttributeToLink( $target, &$link ) {
+	protected function &addTitleAttributeToLink($target, &$link) {
 
-		if ( $target->getPrefixedText() === '' ) {
+		if ($target->getPrefixedText() === '') {
 			// A link like [[#Foo]].  This used to mean an empty title
 			// attribute, but that's silly.  Just don't output a title.
-		} elseif ( $target->isKnown() ) {
-			$link->setAttribute( 'title', $target->getPrefixedText() );
+		} elseif ($target->isKnown()) {
+			$link->setAttribute('title', $target->getPrefixedText());
 		} else {
-			$link->setAttribute( 'title', wfMessage( 'red-link-title', $target->getPrefixedText() )->text() );
+			$link->setAttribute('title', wfMessage('red-link-title', $target->getPrefixedText())->text());
 		}
 
 		return $link;
@@ -264,7 +264,7 @@ 
 	 * @param $target
 	 * @param $link
 	 */
-	protected function &addClassAttributeToLink( $target, &$link ) {
+	protected function &addClassAttributeToLink($target, &$link) {
 
 		// TODO: should this be more elaborate? See Linker::linkAttribs
 		// Cleanest would probably be to use Linker::link and parse it
@@ -272,19 +272,19 @@ 
 		// part here.
 		$classes = '';
 
-		if ( !$target->isKnown() ) {
+		if (!$target->isKnown()) {
 			$classes .= 'new ';
 		}
 
-		if ( $target->isExternal() ) {
+		if ($target->isExternal()) {
 			$classes .= 'extiw ';
 		}
 
 		// set style
-		$classes .= $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ];
+		$classes .= $this->mDefinitions[0][self::ELEMENT_STYLE];
 
-		if ( $classes !== '' ) {
-			$link->setAttribute( 'class', $classes );
+		if ($classes !== '') {
+			$link->setAttribute('class', $classes);
 		}
 
 		return $link;

src/LingoMessageLog.php

Spacing +24 added lines, -24 removed lines

@@ -47,38 +47,38 @@ 
 	 * @param $message
 	 * @param int $severity
 	 */
-	public function addMessage( $message, $severity = self::MESSAGE_NOTICE ) {
-		$this->mMessages[] = array( $message, $severity );
+	public function addMessage($message, $severity = self::MESSAGE_NOTICE) {
+		$this->mMessages[] = array($message, $severity);
 
 		// log errors and warnings in debug log
-		if ( $severity == self::MESSAGE_WARNING ||
+		if ($severity == self::MESSAGE_WARNING ||
 			$severity == self::MESSAGE_ERROR
 		) {
-			wfDebug( $message );
+			wfDebug($message);
 		}
 	}
 
 	/**
 	 * @param $message
 	 */
-	public function addError( $message ) {
-		$this->mMessages[] = array( $message, self::MESSAGE_ERROR );
-		wfDebug( "Error: $message\n" );
+	public function addError($message) {
+		$this->mMessages[] = array($message, self::MESSAGE_ERROR);
+		wfDebug("Error: $message\n");
 	}
 
 	/**
 	 * @param $message
 	 */
-	public function addWarning( $message ) {
-		$this->mMessages[] = array( $message, self::MESSAGE_WARNING );
-		wfDebug( "Warning: $message\n" );
+	public function addWarning($message) {
+		$this->mMessages[] = array($message, self::MESSAGE_WARNING);
+		wfDebug("Warning: $message\n");
 	}
 
 	/**
 	 * @param $message
 	 */
-	public function addNotice( $message ) {
-		$this->mMessages[] = array( $message, self::MESSAGE_NOTICE );
+	public function addNotice($message) {
+		$this->mMessages[] = array($message, self::MESSAGE_NOTICE);
 	}
 
 	/**
@@ -86,36 +86,36 @@ 
 	 * @param null $header
 	 * @return null|string
 	 */
-	public function getMessagesFormatted( $severity = self::MESSAGE_WARNING, $header = null ) {
+	public function getMessagesFormatted($severity = self::MESSAGE_WARNING, $header = null) {
 		global $wgTitle, $wgUser;
 
 		$ret = '';
 
-		foreach ( $this->mMessages as $message ) {
-			if ( $message[ 1 ] <= $severity ) {
-				$ret .= '* ' . $message[ 0 ] . "\n";
+		foreach ($this->mMessages as $message) {
+			if ($message[1] <= $severity) {
+				$ret .= '* '.$message[0]."\n";
 			}
 		}
 
-		if ( $ret != '' ) {
-			if ( !$this->mParser ) {
+		if ($ret != '') {
+			if (!$this->mParser) {
 				$parser = new Parser();
 			}
 
-			if ( $header == null ) {
+			if ($header == null) {
 				$header = '';
-			} elseif ( $header != '' ) {
-				$header = Html::rawElement( 'div', array( 'class' => 'heading' ), $header );
+			} elseif ($header != '') {
+				$header = Html::rawElement('div', array('class' => 'heading'), $header);
 			}
 
-			$ret = Html::rawElement( 'div', array( 'class' => 'messages' ),
-				$header . "\n" .
+			$ret = Html::rawElement('div', array('class' => 'messages'),
+				$header."\n".
 				$ret
 			);
 
 			// FIXME: Variable 'parser' might have not been defined
 			// FIXME: $parser->parse returns ParserOutput, not String
-			$ret = $parser->parse( $ret, $wgTitle, ParserOptions::newFromUser( $wgUser ) );
+			$ret = $parser->parse($ret, $wgTitle, ParserOptions::newFromUser($wgUser));
 		} else {
 			// FIXME: Should probably return '' (and throw an error if necessary)
 			$ret = null;