wmde / WikibaseDataModelServices

src/Diff/TermListPatcher.php

Spacing +15 added lines, -15 removed lines

@@ -27,9 +27,9 @@ 
 	 *
 	 * @throws PatcherException
 	 */
-	public function patchTermList( TermList $terms, Diff $patch ) {
-		foreach ( $patch as $lang => $diffOp ) {
-			$this->patchTerm( $terms, $lang, $diffOp );
+	public function patchTermList(TermList $terms, Diff $patch) {
+		foreach ($patch as $lang => $diffOp) {
+			$this->patchTerm($terms, $lang, $diffOp);
 		}
 	}
 
@@ -42,34 +42,34 @@ 
 	 *
 	 * @throws PatcherException
 	 */
-	private function patchTerm( TermList $terms, $lang, AtomicDiffOp $diffOp ) {
-		$hasLang = $terms->hasTermForLanguage( $lang );
+	private function patchTerm(TermList $terms, $lang, AtomicDiffOp $diffOp) {
+		$hasLang = $terms->hasTermForLanguage($lang);
 
-		switch ( true ) {
+		switch (true) {
 			case $diffOp instanceof DiffOpAdd:
-				if ( !$hasLang ) {
-					$terms->setTextForLanguage( $lang, $diffOp->getNewValue() );
+				if (!$hasLang) {
+					$terms->setTextForLanguage($lang, $diffOp->getNewValue());
 				}
 				break;
 
 			case $diffOp instanceof DiffOpChange:
-				if ( $hasLang
-					&& $terms->getByLanguage( $lang )->getText() === $diffOp->getOldValue()
+				if ($hasLang
+					&& $terms->getByLanguage($lang)->getText() === $diffOp->getOldValue()
 				) {
-					$terms->setTextForLanguage( $lang, $diffOp->getNewValue() );
+					$terms->setTextForLanguage($lang, $diffOp->getNewValue());
 				}
 				break;
 
 			case $diffOp instanceof DiffOpRemove:
-				if ( $hasLang
-					&& $terms->getByLanguage( $lang )->getText() === $diffOp->getOldValue()
+				if ($hasLang
+					&& $terms->getByLanguage($lang)->getText() === $diffOp->getOldValue()
 				) {
-					$terms->removeByLanguage( $lang );
+					$terms->removeByLanguage($lang);
 				}
 				break;
 
 			default:
-				throw new PatcherException( 'Invalid terms diff' );
+				throw new PatcherException('Invalid terms diff');
 		}
 	}
 

src/Diff/EntityPatcherStrategy.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function canPatchEntityType( $entityType );
+	public function canPatchEntityType($entityType);
 
 	/**
 	 * @param EntityDocument $entity
@@ -26,6 +26,6 @@ 
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function patchEntity( EntityDocument $entity, EntityDiff $patch );
+	public function patchEntity(EntityDocument $entity, EntityDiff $patch);
 
 }

src/Statement/V4GuidGenerator.php

Spacing +10 added lines, -10 removed lines

@@ -22,20 +22,20 @@
 	 * @return string
 	 */
 	public function newGuid() {
-		if ( function_exists( 'com_create_guid' ) ) {
-			return trim( com_create_guid(), '{}' );
+		if (function_exists('com_create_guid')) {
+			return trim(com_create_guid(), '{}');
 		}
 
 		return sprintf(
 			'%04X%04X-%04X-%04X-%04X-%04X%04X%04X',
-			mt_rand( 0, 65535 ),
-			mt_rand( 0, 65535 ),
-			mt_rand( 0, 65535 ),
-			mt_rand( 16384, 20479 ),
-			mt_rand( 32768, 49151 ),
-			mt_rand( 0, 65535 ),
-			mt_rand( 0, 65535 ),
-			mt_rand( 0, 65535 )
+			mt_rand(0, 65535),
+			mt_rand(0, 65535),
+			mt_rand(0, 65535),
+			mt_rand(16384, 20479),
+			mt_rand(32768, 49151),
+			mt_rand(0, 65535),
+			mt_rand(0, 65535),
+			mt_rand(0, 65535)
 		);
 	}
 

src/Statement/Grouper/NullStatementGrouper.php

Spacing +2 added lines, -2 removed lines

@@ -21,8 +21,8 @@
 	 * @return StatementList[] An associative array, mapping the default group identifier
 	 *  "statements" to the unmodified StatementList object.
 	 */
-	public function groupStatements( StatementList $statements ) {
-		return [ 'statements' => $statements ];
+	public function groupStatements(StatementList $statements) {
+		return ['statements' => $statements];
 	}
 
 }

src/Statement/Grouper/StatementGrouper.php

Spacing +1 added lines, -1 removed lines

@@ -20,6 +20,6 @@
 	 *  included. The array keys act as identifiers for each group, and can be used as (parts of)
 	 *  message keys.
 	 */
-	public function groupStatements( StatementList $statements );
+	public function groupStatements(StatementList $statements);
 
 }

src/Statement/Grouper/ByPropertyIdStatementGrouper.php

Spacing +4 added lines, -4 removed lines

@@ -18,18 +18,18 @@
 	 * @return StatementList[] An associative array, mapping property id serializations to
 	 *  StatementList objects.
 	 */
-	public function groupStatements( StatementList $statements ) {
+	public function groupStatements(StatementList $statements) {
 		/** @var StatementList[] $groups */
 		$groups = [];
 
-		foreach ( $statements->toArray() as $statement ) {
+		foreach ($statements->toArray() as $statement) {
 			$id = $statement->getPropertyId()->getSerialization();
 
-			if ( !isset( $groups[$id] ) ) {
+			if (!isset($groups[$id])) {
 				$groups[$id] = new StatementList();
 			}
 
-			$groups[$id]->addStatement( $statement );
+			$groups[$id]->addStatement($statement);
 		}
 
 		return $groups;

src/Statement/Grouper/FilteringStatementGrouper.php

Spacing +21 added lines, -21 removed lines

@@ -38,17 +38,17 @@ 
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( array $filters ) {
-		foreach ( $filters as $key => $filter ) {
-			if ( $filter instanceof StatementFilter ) {
+	public function __construct(array $filters) {
+		foreach ($filters as $key => $filter) {
+			if ($filter instanceof StatementFilter) {
 				$this->filters[$key] = $filter;
-			} elseif ( $filter === null ) {
-				$this->setDefaultGroupIdentifier( $key );
+			} elseif ($filter === null) {
+				$this->setDefaultGroupIdentifier($key);
 			} else {
-				throw new InvalidArgumentException( '$filter must be a StatementFilter or null' );
+				throw new InvalidArgumentException('$filter must be a StatementFilter or null');
 			}
 
-			$this->setGroupIdentifier( $key );
+			$this->setGroupIdentifier($key);
 		}
 
 		$this->initializeDefaultGroup();
@@ -57,7 +57,7 @@ 
 	/**
 	 * @param string $key
 	 */
-	private function setGroupIdentifier( $key ) {
+	private function setGroupIdentifier($key) {
 		$this->groupIdentifiers[$key] = $key;
 	}
 
@@ -66,18 +66,18 @@ 
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	private function setDefaultGroupIdentifier( $key ) {
-		if ( $this->defaultGroupIdentifier !== null ) {
-			throw new InvalidArgumentException( 'You must only define one default group' );
+	private function setDefaultGroupIdentifier($key) {
+		if ($this->defaultGroupIdentifier !== null) {
+			throw new InvalidArgumentException('You must only define one default group');
 		}
 
 		$this->defaultGroupIdentifier = $key;
 	}
 
 	private function initializeDefaultGroup() {
-		if ( $this->defaultGroupIdentifier === null ) {
+		if ($this->defaultGroupIdentifier === null) {
 			$this->defaultGroupIdentifier = 'statements';
-			$this->setGroupIdentifier( $this->defaultGroupIdentifier );
+			$this->setGroupIdentifier($this->defaultGroupIdentifier);
 		}
 	}
 
@@ -88,12 +88,12 @@ 
 	 *  StatementList objects. All identifiers given in the constructor are guaranteed to be in the
 	 *  result.
 	 */
-	public function groupStatements( StatementList $statements ) {
+	public function groupStatements(StatementList $statements) {
 		$groups = $this->getEmptyGroups();
 
-		foreach ( $statements->toArray() as $statement ) {
-			$key = $this->getKey( $statement );
-			$groups[$key]->addStatement( $statement );
+		foreach ($statements->toArray() as $statement) {
+			$key = $this->getKey($statement);
+			$groups[$key]->addStatement($statement);
 		}
 
 		return $groups;
@@ -105,7 +105,7 @@ 
 	private function getEmptyGroups() {
 		$groups = [];
 
-		foreach ( $this->groupIdentifiers as $key ) {
+		foreach ($this->groupIdentifiers as $key) {
 			$groups[$key] = new StatementList();
 		}
 
@@ -117,9 +117,9 @@ 
 	 *
 	 * @return string Statement group identifier
 	 */
-	private function getKey( Statement $statement ) {
-		foreach ( $this->filters as $key => $filter ) {
-			if ( $filter->statementMatches( $statement ) ) {
+	private function getKey(Statement $statement) {
+		foreach ($this->filters as $key => $filter) {
+			if ($filter->statementMatches($statement)) {
 				return $key;
 			}
 		}

src/Statement/StatementGuidParser.php

Spacing +8 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace Wikibase\DataModel\Services\Statement;
 
@@ -21,22 +21,22 @@ 
 
 	private EntityIdParser $entityIdParser;
 
-	public function __construct( EntityIdParser $entityIdParser ) {
+	public function __construct(EntityIdParser $entityIdParser) {
 		$this->entityIdParser = $entityIdParser;
 	}
 
 	/**
 	 * @throws StatementGuidParsingException
 	 */
-	public function parse( string $serialization ): StatementGuid {
-		$keyParts = explode( StatementGuid::SEPARATOR, $serialization, 2 );
-		if ( count( $keyParts ) !== 2 ) {
-			throw new StatementGuidParsingException( '$serialization does not have the correct number of parts' );
+	public function parse(string $serialization): StatementGuid {
+		$keyParts = explode(StatementGuid::SEPARATOR, $serialization, 2);
+		if (count($keyParts) !== 2) {
+			throw new StatementGuidParsingException('$serialization does not have the correct number of parts');
 		}
 
 		try {
-			return new StatementGuid( $this->entityIdParser->parse( $keyParts[0] ), $keyParts[1], $serialization );
-		} catch ( EntityIdParsingException $ex ) {
+			return new StatementGuid($this->entityIdParser->parse($keyParts[0]), $keyParts[1], $serialization);
+		} catch (EntityIdParsingException $ex) {
 			throw new StatementGuidParsingException(
 				'$serialization contains invalid EntityId: '
 				. $ex->getMessage()

src/Statement/Filter/DataTypeStatementFilter.php

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
 	 * @param PropertyDataTypeLookup $dataTypeLookup
 	 * @param string[]|string $dataTypes One or more property data type identifiers.
 	 */
-	public function __construct( PropertyDataTypeLookup $dataTypeLookup, $dataTypes ) {
+	public function __construct(PropertyDataTypeLookup $dataTypeLookup, $dataTypes) {
 		$this->dataTypeLookup = $dataTypeLookup;
 		$this->dataTypes = (array)$dataTypes;
 	}
@@ -49,16 +49,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function statementMatches( Statement $statement ) {
+	public function statementMatches(Statement $statement) {
 		$id = $statement->getPropertyId();
 
 		try {
-			$dataType = $this->dataTypeLookup->getDataTypeIdForProperty( $id );
-		} catch ( PropertyDataTypeLookupException $ex ) {
+			$dataType = $this->dataTypeLookup->getDataTypeIdForProperty($id);
+		} catch (PropertyDataTypeLookupException $ex) {
 			return false;
 		}
 
-		return in_array( $dataType, $this->dataTypes );
+		return in_array($dataType, $this->dataTypes);
 	}
 
 }