wikimedia / mediawiki-extensions-Lingo

src/LingoParser.php

Doc Comments +3 added lines

@@ -36,6 +36,9 @@
 		$this->mLingoBackend = new $wgexLingoBackend( $messages );
 	}
 
+	/**
+	 * @return string
+	 */
 	private static function uniqPrefix( Parser &$parser) {
 		if ( defined("Parser::MARKER_PREFIX" )) {
 			return Parser::MARKER_PREFIX;

Spacing +96 added lines, -96 removed lines

@@ -30,14 +30,14 @@ 
 	// The RegEx to split a chunk of text into words
 	public static $regex = null;
 
-	public function __construct( LingoMessageLog &$messages = null ) {
+	public function __construct(LingoMessageLog&$messages = null) {
 		global $wgexLingoBackend;
 
-		$this->mLingoBackend = new $wgexLingoBackend( $messages );
+		$this->mLingoBackend = new $wgexLingoBackend($messages);
 	}
 
-	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();
@@ -50,19 +50,19 @@ 
 	 * @param string $text
 	 * @return Boolean
 	 */
-	static function parse( Parser &$parser, &$text ) {
-		wfProfileIn( __METHOD__ );
-		if ( !self::$parserSingleton ) {
+	static function parse(Parser&$parser, &$text) {
+		wfProfileIn(__METHOD__);
+		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);
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
 		return true;
 	}
@@ -81,14 +81,14 @@ 
 	 * @return Array an array mapping terms (keys) to descriptions (values)
 	 */
 	function getLingoArray() {
-		wfProfileIn( __METHOD__ );
+		wfProfileIn(__METHOD__);
 
 		// build glossary array only once per request
-		if ( !$this->mLingoTree ) {
+		if (!$this->mLingoTree) {
 			$this->buildLingo();
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
 		return $this->mLingoTree->getTermList();
 	}
@@ -99,57 +99,57 @@ 
 	 * @return LingoTree a LingoTree mapping terms (keys) to descriptions (values)
 	 */
 	function getLingoTree() {
-		wfProfileIn( __METHOD__ );
+		wfProfileIn(__METHOD__);
 
 		// 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();
 			}
 
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
 		return $this->mLingoTree;
 	}
 
 	protected function &buildLingo() {
-		wfProfileIn( __METHOD__ );
+		wfProfileIn(__METHOD__);
 
 		$lingoTree = new LingoTree();
 		$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);
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 		return $lingoTree;
 	}
 
@@ -162,101 +162,101 @@ 
 	 * @param $text
 	 * @return Boolean
 	 */
-	protected function realParse( &$parser, &$text ) {
+	protected function realParse(&$parser, &$text) {
 		global $wgRequest;
 
-		wfProfileIn( __METHOD__ );
+		wfProfileIn(__METHOD__);
 
-		$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'])
 		) {
 
-			wfProfileOut( __METHOD__ );
+			wfProfileOut(__METHOD__);
 			return true;
 		}
 
 		// Get array of terms
 		$glossary = $this->getLingoTree();
 
-		if ( $glossary == null ) {
-			wfProfileOut( __METHOD__ );
+		if ($glossary == null) {
+			wfProfileOut(__METHOD__);
 			return true;
 		}
 
 		// Parse HTML from page
-		wfProfileIn( __METHOD__ . ' 1 loadHTML' );
+		wfProfileIn(__METHOD__.' 1 loadHTML');
 		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();
-		wfProfileOut( __METHOD__ . ' 1 loadHTML' );
+		wfProfileOut(__METHOD__.' 1 loadHTML');
 
-		wfProfileIn( __METHOD__ . ' 2 xpath' );
+		wfProfileIn(__METHOD__.' 2 xpath');
 		// 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()"
 		);
-		wfProfileOut( __METHOD__ . ' 2 xpath' );
+		wfProfileOut(__METHOD__.' 2 xpath');
 
 		// Iterate all HTML text matches
 		$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;
 			}
 
-			wfProfileIn( __METHOD__ . ' 3 lexer' );
+			wfProfileIn(__METHOD__.' 3 lexer');
 			$matches = array();
 			preg_match_all(
 				self::$regex,
 				$el->nodeValue,
 				$matches,
-				PREG_OFFSET_CAPTURE | PREG_PATTERN_ORDER
+				PREG_OFFSET_CAPTURE|PREG_PATTERN_ORDER
 			);
-			wfProfileOut( __METHOD__ . ' 3 lexer' );
+			wfProfileOut(__METHOD__.' 3 lexer');
 
-			if ( count( $matches ) == 0 || count( $matches[0] ) == 0 ) {
+			if (count($matches) == 0 || count($matches[0]) == 0) {
 				continue;
 			}
 
 			$lexemes = &$matches[0];
-			$countLexemes = count( $lexemes );
+			$countLexemes = count($lexemes);
 			$parent = &$el->parentNode;
 			$index = 0;
 			$changedElem = false;
 
-			while ( $index < $countLexemes ) {
-				wfProfileIn( __METHOD__ . ' 4 findNextTerm' );
-				list( $skipped, $used, $definition ) =
-					$glossary->findNextTerm( $lexemes, $index, $countLexemes );
-				wfProfileOut( __METHOD__ . ' 4 findNextTerm' );
+			while ($index < $countLexemes) {
+				wfProfileIn(__METHOD__.' 4 findNextTerm');
+				list($skipped, $used, $definition) =
+					$glossary->findNextTerm($lexemes, $index, $countLexemes);
+				wfProfileOut(__METHOD__.' 4 findNextTerm');
 
-				wfProfileIn( __METHOD__ . ' 5 insert' );
-				if ( $used > 0 ) { // found a term
-					if ( $skipped > 0 ) { // skipped some text, insert it as is
+				wfProfileIn(__METHOD__.' 5 insert');
+				if ($used > 0) { // found a term
+					if ($skipped > 0) { // skipped some text, insert it as is
 						$parent->insertBefore(
 							$doc->createTextNode(
-								substr( $el->nodeValue,
+								substr($el->nodeValue,
 									$currLexIndex = $lexemes[$index][1],
-									$lexemes[$index + $skipped][1] - $currLexIndex )
+									$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
@@ -264,76 +264,76 @@ 
 					// 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
 						);
 					} else {
-						wfProfileOut( __METHOD__ . ' 5 insert' );
+						wfProfileOut(__METHOD__.' 5 insert');
 						// In principle superfluous, the loop would run out
 						// anyway. Might save a bit of time.
 						break;
 					}
 				}
-				wfProfileOut( __METHOD__ . ' 5 insert' );
+				wfProfileOut(__METHOD__.' 5 insert');
 
 				$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());
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
 		return true;
 	}
 
-	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");
 			}
 		}
 	}
@@ -341,11 +341,11 @@ 
 	/**
 	 * Purges the lingo tree from the cache.
 	 */
-	public static function purgeCache () {
+	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/LingoTree.php

Doc Comments +5 added lines, -1 removed lines

@@ -61,7 +61,7 @@ 
 	/**
 	 * Adds an element to the Lingo Tree
 	 *
-	 * @param array $path An array containing the constituing lexemes of the term
+	 * @param string[] $path An array containing the constituing lexemes of the term
 	 * @param String $term
 	 * @param String $definition
 	 * @return Array the tree node the element was stored in
@@ -97,6 +97,10 @@ 
 		return $this->mList;
 	}
 
+	/**
+	 * @param integer $index
+	 * @param integer $countLexemes
+	 */
 	function findNextTerm( &$lexemes, $index, $countLexemes ) {
 		wfProfileIn( __METHOD__ );
 

Spacing +25 added lines, -25 removed lines

@@ -39,22 +39,22 @@ 
 	 * Adds a string to the Lingo Tree
 	 * @param String $term
 	 */
-	function addTerm( &$term, $definition ) {
-		if ( !$term ) {
+	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 );
+			$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)));
 		}
 	}
 
@@ -66,24 +66,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] ) ) {
+			if (!isset($tree[$step])) {
 				$tree[$step] = array();
 			}
 
 			$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;
@@ -97,19 +97,19 @@ 
 		return $this->mList;
 	}
 
-	function findNextTerm( &$lexemes, $index, $countLexemes ) {
-		wfProfileIn( __METHOD__ );
+	function findNextTerm(&$lexemes, $index, $countLexemes) {
+		wfProfileIn(__METHOD__);
 
 		$start = $lastindex = $index;
 		$definition = null;
 
 		// skip until the start of a term is found
-		while ( $index < $countLexemes && !$definition ) {
+		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;
@@ -117,23 +117,23 @@ 
 			$index++;
 		}
 
-		wfProfileOut( __METHOD__ );
-		if ( $definition ) {
+		wfProfileOut(__METHOD__);
+		if ($definition) {
 			return array($index - $start - 1, $lastindex - $index + 2, $definition);
 		} else {
 			return array($index - $start, 0, null);
 		}
 	}
 
-	function findNextTermNoSkip( Array &$tree, &$lexemes, $index, $countLexemes ) {
-		wfProfileIn( __METHOD__ );
+	function findNextTermNoSkip(Array &$tree, &$lexemes, $index, $countLexemes) {
+		wfProfileIn(__METHOD__);
 
-		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]);
 		}
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 		return $ret;
 	}
 

src/LingoHooks.php

Spacing +11 added lines, -11 removed lines

@@ -24,17 +24,17 @@ 
 	 * @param String $text
 	 * @return Boolean
 	 */
-	static function parse( &$parser, &$text ) {
+	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
+			!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
 			)
 		) {
@@ -42,8 +42,8 @@ 
 			// 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;
@@ -53,7 +53,7 @@ 
 	 * Creates tag hook(s)
 	 */
 	public static function registerTags(Parser $parser) {
-		$parser->setHook( 'noglossary',  'LingoHooks::noglossaryTagRenderer');
+		$parser->setHook('noglossary', 'LingoHooks::noglossaryTagRenderer');
 		return true;
 	}
 
@@ -66,8 +66,8 @@ 
 	 * @param PPFrame $frame
 	 * @return string
 	 */
-	public static function noglossaryTagRenderer( $input, array $args, Parser $parser, PPFrame $frame ) {
-		$output = $parser->recursiveTagParse( $input, $frame );
+	public static function noglossaryTagRenderer($input, array $args, Parser $parser, PPFrame $frame) {
+		$output = $parser->recursiveTagParse($input, $frame);
 		return '<span class="noglossary">'.$output.'</span>';
 	}
 
@@ -78,12 +78,12 @@ 
 	 *
 	 */
 	public static function initExtension() {
-		MagicWord::$mDoubleUnderscoreIDs[ ] = 'noglossary';
+		MagicWord::$mDoubleUnderscoreIDs[] = 'noglossary';
 
-		foreach ( $GLOBALS['wgExtensionCredits']['parserhook'] as $index => $description ) {
+		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();
+					wfMessage('lingo-desc', $GLOBALS['wgexLingoPage'] ? $GLOBALS['wgexLingoPage'] : wfMessage('lingo-terminologypagename')->inContentLanguage()->text())->text();
 			}
 		}
 	}

src/LingoBackend.php

Spacing +2 added lines, -2 removed lines

@@ -17,9 +17,9 @@
 
 	protected $mMessageLog;
 
-	public function __construct( LingoMessageLog &$messages = null ) {
+	public function __construct(LingoMessageLog&$messages = null) {
 
-		if ( !$messages ) {
+		if (!$messages) {
 			$this->mMessageLog = new LingoMessageLog();
 		} else {
 			$this->mMessageLog = $messages;

src/LingoBasicBackend.php

Spacing +36 added lines, -36 removed lines

@@ -17,18 +17,18 @@ 
 
 	protected $mArticleLines = array();
 
-	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;
 		}
 
@@ -36,15 +36,15 @@ 
 		// 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;
 			}
 
@@ -55,9 +55,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));
 	}
 
 	/**
@@ -70,40 +70,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,
@@ -114,7 +114,7 @@ 
 			}
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
 		return array_pop($ret);
 	}
@@ -125,21 +125,21 @@ 
 	 * @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);
 	}
 
 	/**
@@ -148,12 +148,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/Lingo.i18n.magic.php

Spacing +16 added lines, -16 removed lines

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

src/LingoElement.php

Spacing +67 added lines, -67 removed lines

@@ -22,7 +22,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();
@@ -31,94 +31,94 @@ 
 
 	static private $mLinkTemplate = null;
 
-	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);
 		}
 	}
 
-	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);
 	}
 
-	public function getFullDefinition( DOMDocument &$doc ) {
+	public function getFullDefinition(DOMDocument&$doc) {
 
 		global $wgexLingoDisplayOnce;
 
-		wfProfileIn( __METHOD__ );
+		wfProfileIn(__METHOD__);
 
 		// return textnode if
-		if ( $wgexLingoDisplayOnce && $this->mHasBeenDisplayed ) {
+		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;
 		}
 
-		wfProfileOut( __METHOD__ );
+		wfProfileOut(__METHOD__);
 
-		return $this->mFullDefinition->cloneNode( true );
+		return $this->mFullDefinition->cloneNode(true);
 	}
 
 	public function getCurrentKey() {
-		return key( $this->mDefinitions );
+		return key($this->mDefinitions);
 	}
 
-	public function getTerm( $key ) {
+	public function getTerm($key) {
 		return $this->mDefinitions[$key][self::ELEMENT_TERM];
 	}
 
-	public function getSource( &$key ) {
+	public function getSource(&$key) {
 		return $this->mDefinitions[$key][self::ELEMENT_SOURCE];
 	}
 
-	public function getDefinition( &$key ) {
+	public function getDefinition(&$key) {
 		return $this->mDefinitions[$key][self::ELEMENT_DEFINITION];
 	}
 
-	public function getLink( &$key ) {
+	public function getLink(&$key) {
 		return $this->mDefinitions[$key][self::ELEMENT_LINK];
 	}
 
-	public function getStyle( &$key ) {
+	public function getStyle(&$key) {
 		return $this->mDefinitions[$key][self::ELEMENT_STYLE];
 	}
 
 	public function next() {
-		next( $this->mDefinitions );
+		next($this->mDefinitions);
 	}
 
-	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);
 	}
 
 	/**
@@ -127,18 +127,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;
 	}
@@ -149,42 +149,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;
 	}
 
@@ -192,15 +192,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;
@@ -210,7 +210,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
@@ -218,19 +218,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 +21 added lines, -21 removed lines

@@ -25,58 +25,58 @@
 	const MESSAGE_WARNING = 2;
 	const MESSAGE_NOTICE = 3;
 
-	function addMessage( $message, $severity = self::MESSAGE_NOTICE ) {
+	function addMessage($message, $severity = self::MESSAGE_NOTICE) {
 		$this->mMessages[] = array($message, $severity);
 
 		// log errors and warnings in debug log
-		if ( $severity == self::MESSAGE_WARNING ||
-			$severity == self::MESSAGE_ERROR ) {
-			wfDebug( $message );
+		if ($severity == self::MESSAGE_WARNING ||
+			$severity == self::MESSAGE_ERROR) {
+			wfDebug($message);
 		}
 	}
 
-	function addError( $message ) {
+	function addError($message) {
 		$this->mMessages[] = array($message, self::MESSAGE_ERROR);
-		wfDebug( "Error: $message\n" );
+		wfDebug("Error: $message\n");
 	}
 
-	function addWarning( $message ) {
+	function addWarning($message) {
 		$this->mMessages[] = array($message, self::MESSAGE_WARNING);
-		wfDebug( "Warning: $message\n" );
+		wfDebug("Warning: $message\n");
 	}
 
-	function addNotice( $message ) {
+	function addNotice($message) {
 		$this->mMessages[] = array($message, self::MESSAGE_NOTICE);
 	}
 
-	function getMessagesFormatted( $severity = self::MESSAGE_WARNING, $header = null ) {
+	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
 			);
 
-			$ret = $parser->parse( $ret, $wgTitle, ParserOptions::newFromUser( $wgUser ) );
+			$ret = $parser->parse($ret, $wgTitle, ParserOptions::newFromUser($wgUser));
 		} else {
 			$ret = null;
 		}