Merge pull request #588 from wmde/mwFixes

Fix remaining minor whitespace issues

Author: Benestar

src/SiteLinkList.php

@@ -123,7 +123,7 @@ public function count() {
 	 */
 	public function getBySiteId( $siteId ) {
 		if ( !$this->hasLinkWithSiteId( $siteId ) ) {
-			throw new OutOfBoundsException( 'SiteLink with siteId "' . $siteId . '" not found'  );
+			throw new OutOfBoundsException( 'SiteLink with siteId "' . $siteId . '" not found' );
 		}
 
 		return $this->siteLinks[$siteId];

tests/unit/ByPropertyIdArrayTest.php

@@ -257,7 +257,7 @@ public function moveProvider() {
 		$argLists[] = array( $c, $c[1], 0, array( $c[1], $c[0], $c[2], $c[3], $c[4], $c[5] ) );
 		$argLists[] = array( $c, $c[1], 1, $c );
 		$argLists[] = array( $c, $c[1], 2, $c );
-		$argLists[] = array( $c, $c[1], 3, array( $c[2], $c[3], $c[4], $c[0], $c[1], $c[5] ) ); //
+		$argLists[] = array( $c, $c[1], 3, array( $c[2], $c[3], $c[4], $c[0], $c[1], $c[5] ) );
 		$argLists[] = array( $c, $c[1], 4, array( $c[2], $c[3], $c[4], $c[0], $c[1], $c[5] ) );
 		$argLists[] = array( $c, $c[1], 5, array( $c[2], $c[3], $c[4], $c[0], $c[1], $c[5] ) );
 		$argLists[] = array( $c, $c[1], 6, array( $c[2], $c[3], $c[4], $c[5], $c[0], $c[1] ) );

tests/unit/Entity/EntityTest.php

@@ -218,8 +218,8 @@ public function testSetEmptyAlias( array $aliasesLists ) {
 				$entity->setAliases( $langCode, $aliases );
 			}
 		}
-		$entity->setAliases( 'zh', array( 'wind', 'air', '', 'fire') );
-		$entity->setAliases( 'zu', array( '', '') );
+		$entity->setAliases( 'zh', array( 'wind', 'air', '', 'fire' ) );
+		$entity->setAliases( 'zu', array( '', '' ) );
 
 		foreach ( $aliasesLists as $langCode => $aliasesList ) {
 			$expected = array_values( array_unique( array_pop( $aliasesList ) ) );

tests/unit/SiteLinkListTest.php

@@ -245,7 +245,7 @@ public function testAddNewSiteLink() {
 		$list->addNewSiteLink( 'enwiki', 'cats' );
 		$list->addNewSiteLink( 'dewiki', 'katzen', array( new ItemId( 'Q1' ) ) );
 
-		$this->assertTrue( $list->equals( new SiteLinkList( array (
+		$this->assertTrue( $list->equals( new SiteLinkList( array(
 			new SiteLink( 'enwiki', 'cats' ),
 			new SiteLink( 'dewiki', 'katzen', array( new ItemId( 'Q1' ) ) ),
 		) ) ) );
@@ -257,7 +257,7 @@ public function testAddSiteLink() {
 		$list->addSiteLink( new SiteLink( 'enwiki', 'cats' ) );
 		$list->addSiteLink( new SiteLink( 'dewiki', 'katzen' ) );
 
-		$this->assertTrue( $list->equals( new SiteLinkList( array (
+		$this->assertTrue( $list->equals( new SiteLinkList( array(
 			new SiteLink( 'enwiki', 'cats' ),
 			new SiteLink( 'dewiki', 'katzen' ),
 		) ) ) );

tests/unit/Statement/StatementGuidTest.php

@@ -29,7 +29,7 @@ public function testConstructor( EntityId $entityId, $guid, $expected ) {
 		$statementGuid = new StatementGuid( $entityId, $guid );
 
 		$this->assertEquals( $expected, $statementGuid->getSerialization() );
-		$this->assertEquals( $entityId, $statementGuid->getEntityId());
+		$this->assertEquals( $entityId, $statementGuid->getEntityId() );
 	}
 
 	public function provideConstructionData() {