wmde / WikibaseDataModelServices

src/Lookup/ReferencedEntityIdLookup.php

Spacing +1 added lines, -1 removed lines

@@ -36,6 +36,6 @@
 	 * @return EntityId|null Returns null in case none of the target entities are referenced.
 	 * @throws ReferencedEntityIdLookupException
 	 */
-	public function getReferencedEntityId( EntityId $fromId, PropertyId $propertyId, array $toIds );
+	public function getReferencedEntityId(EntityId $fromId, PropertyId $propertyId, array $toIds);
 
 }

src/Lookup/PropertyDataTypeLookupException.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 	 * @param string|null $message
 	 * @param Exception|null $previous
 	 */
-	public function __construct( PropertyId $propertyId, $message = null, ?Exception $previous = null ) {
+	public function __construct(PropertyId $propertyId, $message = null, ?Exception $previous = null) {
 		$this->propertyId = $propertyId;
 
 		parent::__construct(

src/Lookup/PropertyDataTypeLookup.php

Spacing +1 added lines, -1 removed lines

@@ -22,6 +22,6 @@
 	 * @return string
 	 * @throws PropertyDataTypeLookupException
 	 */
-	public function getDataTypeIdForProperty( PropertyId $propertyId );
+	public function getDataTypeIdForProperty(PropertyId $propertyId);
 
 }

src/Lookup/MaxReferencedEntityVisitsExhaustedException.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 		$message = $message ?: 'Referenced entity id lookup failed: Maximum number of entity visits (' .
 				$maxEntityVisits . ') exhausted.';
 
-		parent::__construct( $fromId, $propertyId, $toIds, $message, $previous );
+		parent::__construct($fromId, $propertyId, $toIds, $message, $previous);
 	}
 
 	/**

src/ByPropertyIdGrouper.php

Spacing +20 added lines, -20 removed lines

@@ -28,21 +28,21 @@ 
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( $propertyIdProviders ) {
-		$this->assertArePropertyIdProviders( $propertyIdProviders );
-		$this->indexPropertyIdProviders( $propertyIdProviders );
+	public function __construct($propertyIdProviders) {
+		$this->assertArePropertyIdProviders($propertyIdProviders);
+		$this->indexPropertyIdProviders($propertyIdProviders);
 	}
 
 	/**
 	 * @param PropertyIdProvider[]|Traversable $propertyIdProviders
 	 */
-	private function assertArePropertyIdProviders( $propertyIdProviders ) {
-		if ( !is_iterable( $propertyIdProviders ) ) {
-			throw new InvalidArgumentException( '$propertyIdProviders must be an array or an instance of Traversable' );
+	private function assertArePropertyIdProviders($propertyIdProviders) {
+		if (!is_iterable($propertyIdProviders)) {
+			throw new InvalidArgumentException('$propertyIdProviders must be an array or an instance of Traversable');
 		}
 
-		foreach ( $propertyIdProviders as $propertyIdProvider ) {
-			if ( !( $propertyIdProvider instanceof PropertyIdProvider ) ) {
+		foreach ($propertyIdProviders as $propertyIdProvider) {
+			if (!($propertyIdProvider instanceof PropertyIdProvider)) {
 				throw new InvalidArgumentException(
 					'Every element in $propertyIdProviders must be an instance of PropertyIdProvider'
 				);
@@ -53,13 +53,13 @@ 
 	/**
 	 * @param PropertyIdProvider[]|Traversable $propertyIdProviders
 	 */
-	private function indexPropertyIdProviders( $propertyIdProviders ) {
-		foreach ( $propertyIdProviders as $propertyIdProvider ) {
-			$this->addPropertyIdProvider( $propertyIdProvider );
+	private function indexPropertyIdProviders($propertyIdProviders) {
+		foreach ($propertyIdProviders as $propertyIdProvider) {
+			$this->addPropertyIdProvider($propertyIdProvider);
 		}
 	}
 
-	private function addPropertyIdProvider( PropertyIdProvider $propertyIdProvider ) {
+	private function addPropertyIdProvider(PropertyIdProvider $propertyIdProvider) {
 		$idSerialization = $propertyIdProvider->getPropertyId()->getSerialization();
 		$this->byPropertyId[$idSerialization][] = $propertyIdProvider;
 	}
@@ -73,10 +73,10 @@ 
 	 */
 	public function getPropertyIds() {
 		return array_map(
-			static function( $propertyId ) {
-				return new NumericPropertyId( $propertyId );
+			static function($propertyId) {
+				return new NumericPropertyId($propertyId);
 			},
-			array_keys( $this->byPropertyId )
+			array_keys($this->byPropertyId)
 		);
 	}
 
@@ -90,11 +90,11 @@ 
 	 * @throws OutOfBoundsException
 	 * @return PropertyIdProvider[]
 	 */
-	public function getByPropertyId( NumericPropertyId $propertyId ) {
+	public function getByPropertyId(NumericPropertyId $propertyId) {
 		$idSerialization = $propertyId->getSerialization();
 
-		if ( !isset( $this->byPropertyId[$idSerialization] ) ) {
-			throw new OutOfBoundsException( 'PropertyIdProvider with propertyId "' . $idSerialization . '" not found' );
+		if (!isset($this->byPropertyId[$idSerialization])) {
+			throw new OutOfBoundsException('PropertyIdProvider with propertyId "' . $idSerialization . '" not found');
 		}
 
 		return $this->byPropertyId[$idSerialization];
@@ -109,8 +109,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function hasPropertyId( NumericPropertyId $propertyId ) {
-		return isset( $this->byPropertyId[$propertyId->getSerialization()] );
+	public function hasPropertyId(NumericPropertyId $propertyId) {
+		return isset($this->byPropertyId[$propertyId->getSerialization()]);
 	}
 
 }

src/Diff/Internal/SiteLinkListPatcher.php

Spacing +14 added lines, -14 removed lines

@@ -24,7 +24,7 @@ 
 	private $patcher;
 
 	public function __construct() {
-		$this->patcher = new MapPatcher( false, new ListPatcher() );
+		$this->patcher = new MapPatcher(false, new ListPatcher());
 	}
 
 	/**
@@ -34,18 +34,18 @@ 
 	 * @return SiteLinkList
 	 * @throws InvalidArgumentException
 	 */
-	public function getPatchedSiteLinkList( SiteLinkList $siteLinks, Diff $patch ) {
-		$baseData = $this->getSiteLinksInDiffFormat( $siteLinks );
-		$patchedData = $this->patcher->patch( $baseData, $patch );
+	public function getPatchedSiteLinkList(SiteLinkList $siteLinks, Diff $patch) {
+		$baseData = $this->getSiteLinksInDiffFormat($siteLinks);
+		$patchedData = $this->patcher->patch($baseData, $patch);
 
 		$patchedSiteLinks = new SiteLinkList();
 
-		foreach ( $patchedData as $siteId => $siteLinkData ) {
-			if ( array_key_exists( 'name', $siteLinkData ) ) {
+		foreach ($patchedData as $siteId => $siteLinkData) {
+			if (array_key_exists('name', $siteLinkData)) {
 				$patchedSiteLinks->addNewSiteLink(
 					$siteId,
 					$siteLinkData['name'],
-					$this->getBadgesFromSiteLinkData( $siteLinkData )
+					$this->getBadgesFromSiteLinkData($siteLinkData)
 				);
 			}
 		}
@@ -58,27 +58,27 @@ 
 	 *
 	 * @return ItemId[]|null
 	 */
-	private function getBadgesFromSiteLinkData( array $siteLinkData ) {
-		if ( !array_key_exists( 'badges', $siteLinkData ) || !is_array( $siteLinkData['badges'] ) ) {
+	private function getBadgesFromSiteLinkData(array $siteLinkData) {
+		if (!array_key_exists('badges', $siteLinkData) || !is_array($siteLinkData['badges'])) {
 			return null;
 		}
 
 		return array_map(
-			static function( $idSerialization ) {
-				return new ItemId( $idSerialization );
+			static function($idSerialization) {
+				return new ItemId($idSerialization);
 			},
 			$siteLinkData['badges']
 		);
 	}
 
-	private function getSiteLinksInDiffFormat( SiteLinkList $siteLinks ): array {
+	private function getSiteLinksInDiffFormat(SiteLinkList $siteLinks): array {
 		$linksInDiffFormat = [];
 
-		foreach ( $siteLinks->toArray() as $siteLink ) {
+		foreach ($siteLinks->toArray() as $siteLink) {
 			$linksInDiffFormat[$siteLink->getSiteId()] = [
 				'name' => $siteLink->getPageName(),
 				'badges' => array_map(
-					static function( ItemId $id ) {
+					static function(ItemId $id) {
 						return $id->getSerialization();
 					},
 					$siteLink->getBadges()

src/Diff/Internal/FingerprintPatcher.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	 *
 	 * @throws PatcherException
 	 */
-	public function patchFingerprint( Fingerprint $fingerprint, EntityDiff $patch ) {
+	public function patchFingerprint(Fingerprint $fingerprint, EntityDiff $patch) {
 		$this->termListPatcher->patchTermList(
 			$fingerprint->getLabels(),
 			$patch->getLabelsDiff()

src/Diff/Internal/AliasGroupListPatcher.php

Spacing +35 added lines, -35 removed lines

@@ -29,9 +29,9 @@ 
 	 *
 	 * @throws PatcherException
 	 */
-	public function patchAliasGroupList( AliasGroupList $groups, Diff $patch ) {
-		foreach ( $patch as $lang => $diffOp ) {
-			$this->patchAliasGroup( $groups, $lang, $diffOp );
+	public function patchAliasGroupList(AliasGroupList $groups, Diff $patch) {
+		foreach ($patch as $lang => $diffOp) {
+			$this->patchAliasGroup($groups, $lang, $diffOp);
 		}
 	}
 
@@ -44,34 +44,34 @@ 
 	 *
 	 * @throws PatcherException
 	 */
-	private function patchAliasGroup( AliasGroupList $groups, $lang, DiffOp $diffOp ) {
-		$hasLang = $groups->hasGroupForLanguage( $lang );
+	private function patchAliasGroup(AliasGroupList $groups, $lang, DiffOp $diffOp) {
+		$hasLang = $groups->hasGroupForLanguage($lang);
 
-		switch ( true ) {
+		switch (true) {
 			case $diffOp instanceof DiffOpAdd:
-				if ( !$hasLang ) {
-					$groups->setAliasesForLanguage( $lang, $diffOp->getNewValue() );
+				if (!$hasLang) {
+					$groups->setAliasesForLanguage($lang, $diffOp->getNewValue());
 				}
 				break;
 
 			case $diffOp instanceof DiffOpChange:
-				$this->applyAliasGroupChange( $groups, $lang, $diffOp );
+				$this->applyAliasGroupChange($groups, $lang, $diffOp);
 				break;
 
 			case $diffOp instanceof DiffOpRemove:
-				if ( $hasLang
-					&& $groups->getByLanguage( $lang )->getAliases() === $diffOp->getOldValue()
+				if ($hasLang
+					&& $groups->getByLanguage($lang)->getAliases() === $diffOp->getOldValue()
 				) {
-					$groups->removeByLanguage( $lang );
+					$groups->removeByLanguage($lang);
 				}
 				break;
 
 			case $diffOp instanceof Diff:
-				$this->applyAliasGroupDiff( $groups, $lang, $diffOp );
+				$this->applyAliasGroupDiff($groups, $lang, $diffOp);
 				break;
 
 			default:
-				throw new PatcherException( 'Invalid aliases diff' );
+				throw new PatcherException('Invalid aliases diff');
 		}
 	}
 
@@ -80,11 +80,11 @@ 
 	 * @param string $lang
 	 * @param DiffOpChange $patch
 	 */
-	private function applyAliasGroupChange( AliasGroupList $groups, $lang, DiffOpChange $patch ) {
-		if ( $groups->hasGroupForLanguage( $lang )
-			&& $groups->getByLanguage( $lang )->getAliases() === $patch->getOldValue()
+	private function applyAliasGroupChange(AliasGroupList $groups, $lang, DiffOpChange $patch) {
+		if ($groups->hasGroupForLanguage($lang)
+			&& $groups->getByLanguage($lang)->getAliases() === $patch->getOldValue()
 		) {
-			$groups->setAliasesForLanguage( $lang, $patch->getNewValue() );
+			$groups->setAliasesForLanguage($lang, $patch->getNewValue());
 		}
 	}
 
@@ -93,13 +93,13 @@ 
 	 * @param string $lang
 	 * @param Diff $patch
 	 */
-	private function applyAliasGroupDiff( AliasGroupList $groups, $lang, Diff $patch ) {
-		$hasLang = $groups->hasGroupForLanguage( $lang );
+	private function applyAliasGroupDiff(AliasGroupList $groups, $lang, Diff $patch) {
+		$hasLang = $groups->hasGroupForLanguage($lang);
 
-		if ( $hasLang || !$this->containsOperationsOnOldValues( $patch ) ) {
-			$aliases = $hasLang ? $groups->getByLanguage( $lang )->getAliases() : [];
-			$aliases = $this->getPatchedAliases( $aliases, $patch );
-			$groups->setAliasesForLanguage( $lang, $aliases );
+		if ($hasLang || !$this->containsOperationsOnOldValues($patch)) {
+			$aliases = $hasLang ? $groups->getByLanguage($lang)->getAliases() : [];
+			$aliases = $this->getPatchedAliases($aliases, $patch);
+			$groups->setAliasesForLanguage($lang, $aliases);
 		}
 	}
 
@@ -108,7 +108,7 @@ 
 	 *
 	 * @return bool
 	 */
-	private function containsOperationsOnOldValues( Diff $diff ) {
+	private function containsOperationsOnOldValues(Diff $diff) {
 		return $diff->getChanges() !== []
 			|| $diff->getRemovals() !== [];
 	}
@@ -122,30 +122,30 @@ 
 	 * @throws PatcherException
 	 * @return string[]
 	 */
-	private function getPatchedAliases( array $aliases, Diff $patch ) {
-		foreach ( $patch as $diffOp ) {
-			switch ( true ) {
+	private function getPatchedAliases(array $aliases, Diff $patch) {
+		foreach ($patch as $diffOp) {
+			switch (true) {
 				case $diffOp instanceof DiffOpAdd:
 					$aliases[] = $diffOp->getNewValue();
 					break;
 
 				case $diffOp instanceof DiffOpChange:
-					$key = array_search( $diffOp->getOldValue(), $aliases, true );
-					if ( $key !== false ) {
-						unset( $aliases[$key] );
+					$key = array_search($diffOp->getOldValue(), $aliases, true);
+					if ($key !== false) {
+						unset($aliases[$key]);
 						$aliases[] = $diffOp->getNewValue();
 					}
 					break;
 
 				case $diffOp instanceof DiffOpRemove:
-					$key = array_search( $diffOp->getOldValue(), $aliases, true );
-					if ( $key !== false ) {
-						unset( $aliases[$key] );
+					$key = array_search($diffOp->getOldValue(), $aliases, true);
+					if ($key !== false) {
+						unset($aliases[$key]);
 					}
 					break;
 
 				default:
-					throw new PatcherException( 'Invalid aliases diff' );
+					throw new PatcherException('Invalid aliases diff');
 			}
 		}
 

src/Diff/StatementListDiffer.php

Spacing +7 added lines, -7 removed lines

@@ -26,29 +26,29 @@
 	 * @return Diff
 	 * @throws UnexpectedValueException
 	 */
-	public function getDiff( StatementList $fromStatements, StatementList $toStatements ) {
+	public function getDiff(StatementList $fromStatements, StatementList $toStatements) {
 		return new Diff(
 			$this->newDiffer()->doDiff(
-				$this->toDiffArray( $fromStatements ),
-				$this->toDiffArray( $toStatements )
+				$this->toDiffArray($fromStatements),
+				$this->toDiffArray($toStatements)
 			),
 			true
 		);
 	}
 
 	private function newDiffer(): MapDiffer {
-		return new MapDiffer( false, null, new ComparableComparer() );
+		return new MapDiffer(false, null, new ComparableComparer());
 	}
 
-	private function toDiffArray( StatementList $statementList ): array {
+	private function toDiffArray(StatementList $statementList): array {
 		$statementArray = [];
 
 		/**
 		 * @var Statement $statement
 		 */
-		foreach ( $statementList as $statement ) {
+		foreach ($statementList as $statement) {
 			$guid = $statement->getGuid();
-			if ( $guid === null ) {
+			if ($guid === null) {
 				$statementArray[] = $statement;
 			} else {
 				$statementArray[$guid] = $statement;