wikimedia / mediawiki-extensions-Lingo

src/Element.php

Doc Comments +4 added lines, -4 removed lines

@@ -58,8 +58,8 @@ 
 
 	/**
 	 * Lingo\Element constructor.
-	 * @param $term
-	 * @param $definition
+	 * @param string $term
+	 * @param string $definition
 	 */
 	public function __construct( &$term, &$definition = null ) {
 
@@ -152,7 +152,7 @@ 
 
 	/**
 	 * @param $target
-	 * @param $link
+	 * @param DOMElement $link
 	 * @return mixed
 	 */
 	protected function &addClassAttributeToLink( $target, &$link ) {
@@ -203,7 +203,7 @@ 
 
 	/**
 	 * @param StashingDOMDocument $doc
-	 * @return string
+	 * @return DOMElement
 	 * @throws \MWException
 	 */
 	protected function getFullDefinitionAsTooltip( StashingDOMDocument &$doc ) {

Spacing +7 added lines, -7 removed lines

@@ -47,7 +47,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
 
 	const LINK_TEMPLATE_ID = 'LingoLink';
 
@@ -74,14 +74,14 @@ 
 	 * @param $definition
 	 */
 	public function addDefinition( &$definition ) {
-		$this->mDefinitions[] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
+		$this->mDefinitions[ ] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
 	}
 
 	/**
 	 * @param StashingDOMDocument $doc
 	 * @return DOMNode|DOMText
 	 */
-	public function getFullDefinition( StashingDOMDocument &$doc ) {
+	public function getFullDefinition( StashingDOMDocument & $doc ) {
 
 		global $wgexLingoDisplayOnce;
 
@@ -98,7 +98,7 @@ 
 	/**
 	 * @param StashingDOMDocument $doc
 	 */
-	private function buildFullDefinition( StashingDOMDocument &$doc ) {
+	private function buildFullDefinition( StashingDOMDocument & $doc ) {
 
 		// only create if not yet created
 		if ( $this->mFullDefinition === null || $this->mFullDefinition->ownerDocument !== $doc ) {
@@ -125,7 +125,7 @@ 
 	 * @return DOMElement
 	 * @throws \MWException
 	 */
-	protected function getFullDefinitionAsLink( StashingDOMDocument &$doc ) {
+	protected function getFullDefinitionAsLink( StashingDOMDocument & $doc ) {
 
 		// create Title object for target page
 		$target = Title::newFromText( $this->mDefinitions[ 0 ][ self::ELEMENT_LINK ] );
@@ -206,7 +206,7 @@ 
 	 * @return string
 	 * @throws \MWException
 	 */
-	protected function getFullDefinitionAsTooltip( StashingDOMDocument &$doc ) {
+	protected function getFullDefinitionAsTooltip( StashingDOMDocument & $doc ) {
 
 		// Wrap term and definition in <span> tags
 		$span = $doc->createElement( 'span' );
@@ -251,7 +251,7 @@ 
 	 * @param StashingDOMDocument $doc
 	 * @return DOMNode
 	 */
-	private function getLinkTemplate( StashingDOMDocument &$doc ) {
+	private function getLinkTemplate( StashingDOMDocument & $doc ) {
 
 		$mLinkTemplate = $doc->stashGet( self::LINK_TEMPLATE_ID );
 

src/LingoParser.php

Unused Use Statements -2 removed lines

@@ -28,8 +28,6 @@
  */
 namespace Lingo;
 
-use DOMDocument;
-use DOMXPath;
 use Parser;
 
 /**

src/StashingDOMDocument.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 	 * @param $key
 	 * @return bool
 	 */
-	public function isStashed ( $key ) {
+	public function isStashed( $key ) {
 		return isset( $this->mStash[ $key ] );
 	}