wmde / WikibaseDataModelServices

src/Statement/Filter/PropertySetStatementFilter.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 	/**
 	 * @param string[]|string $propertyIds One or more property id serializations.
 	 */
-	public function __construct( $propertyIds ) {
+	public function __construct($propertyIds) {
 		$this->propertyIds = (array)$propertyIds;
 	}
 
@@ -40,9 +40,9 @@ 
 	 *
 	 * @return bool
 	 */
-	public function statementMatches( Statement $statement ) {
+	public function statementMatches(Statement $statement) {
 		$id = $statement->getPropertyId()->getSerialization();
-		return in_array( $id, $this->propertyIds );
+		return in_array($id, $this->propertyIds);
 	}
 
 }

src/Statement/GuidGenerator.php

Spacing +3 added lines, -3 removed lines

@@ -33,12 +33,12 @@
 	 *
 	 * @return string
 	 */
-	public function newGuid( EntityId $entityId ) {
+	public function newGuid(EntityId $entityId) {
 		return $entityId->getSerialization() . StatementGuid::SEPARATOR . $this->baseGenerator->newGuid();
 	}
 
-	public function newStatementId( EntityId $entityId ): StatementGuid {
-		return new StatementGuid( $entityId, $this->baseGenerator->newGuid() );
+	public function newStatementId(EntityId $entityId): StatementGuid {
+		return new StatementGuid($entityId, $this->baseGenerator->newGuid());
 	}
 
 }

src/Statement/StatementGuidValidator.php

Spacing +13 added lines, -13 removed lines

@@ -19,7 +19,7 @@ 
 	 */
 	private $entityIdParser;
 
-	public function __construct( EntityIdParser $entityIdParser ) {
+	public function __construct(EntityIdParser $entityIdParser) {
 		$this->entityIdParser = $entityIdParser;
 	}
 
@@ -32,14 +32,14 @@ 
 	 *
 	 * @return bool
 	 */
-	public function validate( $guid ) {
-		if ( !$this->validateFormat( $guid ) ) {
+	public function validate($guid) {
+		if (!$this->validateFormat($guid)) {
 			return false;
 		}
 
-		$guidParts = explode( StatementGuid::SEPARATOR, $guid );
+		$guidParts = explode(StatementGuid::SEPARATOR, $guid);
 
-		if ( !$this->validateStatementGuidPrefix( $guidParts[0] ) || !$this->validateGuid( $guidParts[1] ) ) {
+		if (!$this->validateStatementGuidPrefix($guidParts[0]) || !$this->validateGuid($guidParts[1])) {
 			return false;
 		}
 
@@ -55,14 +55,14 @@ 
 	 *
 	 * @return bool
 	 */
-	public function validateFormat( $guid ) {
-		if ( !is_string( $guid ) ) {
+	public function validateFormat($guid) {
+		if (!is_string($guid)) {
 			return false;
 		}
 
-		$keyParts = explode( '$', $guid );
+		$keyParts = explode('$', $guid);
 
-		if ( count( $keyParts ) !== 2 ) {
+		if (count($keyParts) !== 2) {
 			return false;
 		}
 
@@ -78,7 +78,7 @@ 
 	 *
 	 * @return bool
 	 */
-	private function validateGuid( $guid ) {
+	private function validateGuid($guid) {
 		return (bool)preg_match(
 			'/^\{?[A-Z\d]{8}-[A-Z\d]{4}-[A-Z\d]{4}-[A-Z\d]{4}-[A-Z\d]{12}\}?\z/i',
 			$guid
@@ -94,11 +94,11 @@ 
 	 *
 	 * @return bool
 	 */
-	private function validateStatementGuidPrefix( $prefixedId ) {
+	private function validateStatementGuidPrefix($prefixedId) {
 		try {
-			$this->entityIdParser->parse( $prefixedId );
+			$this->entityIdParser->parse($prefixedId);
 			return true;
-		} catch ( EntityIdParsingException $ex ) {
+		} catch (EntityIdParsingException $ex) {
 			return false;
 		}
 	}

src/DataValue/ValuesFinder.php

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 	 */
 	private $propertyDataTypeLookup;
 
-	public function __construct( PropertyDataTypeLookup $propertyDataTypeLookup ) {
+	public function __construct(PropertyDataTypeLookup $propertyDataTypeLookup) {
 		$this->propertyDataTypeLookup = $propertyDataTypeLookup;
 	}
 
@@ -36,15 +36,15 @@ 
 	 *
 	 * @return DataValue[]
 	 */
-	public function findFromSnaks( array $snaks, $dataType ) {
+	public function findFromSnaks(array $snaks, $dataType) {
 		$found = [];
 
-		foreach ( $snaks as $snak ) {
-			if ( $snak instanceof PropertyValueSnak &&
-				$this->isMatchingDataType( $snak->getPropertyId(), $dataType )
+		foreach ($snaks as $snak) {
+			if ($snak instanceof PropertyValueSnak &&
+				$this->isMatchingDataType($snak->getPropertyId(), $dataType)
 			) {
 				$dataValue = $snak->getDataValue();
-				if ( method_exists( $dataValue, 'getHash' ) ) {
+				if (method_exists($dataValue, 'getHash')) {
 					// We can only tag values as found that are
 					// hashable. DataValueObject should, but `getDataValue`
 					// only returns the `DataValue` interface, which doesn't
@@ -64,10 +64,10 @@ 
 	 *
 	 * @return bool
 	 */
-	private function isMatchingDataType( PropertyId $propertyId, $dataType ) {
+	private function isMatchingDataType(PropertyId $propertyId, $dataType) {
 		try {
-			return $this->propertyDataTypeLookup->getDataTypeIdForProperty( $propertyId ) === $dataType;
-		} catch ( PropertyDataTypeLookupException $ex ) {
+			return $this->propertyDataTypeLookup->getDataTypeIdForProperty($propertyId) === $dataType;
+		} catch (PropertyDataTypeLookupException $ex) {
 			return false;
 		}
 	}

src/EntityId/InMemoryEntityIdPager.php

Spacing +6 added lines, -6 removed lines

@@ -29,11 +29,11 @@ 
 	/**
 	 * @param EntityId ...$ids
 	 */
-	public function __construct( ...$ids ) {
+	public function __construct(...$ids) {
 		$this->entityIds = $ids;
 	}
 
-	public function addEntityId( EntityId $entityId ) {
+	public function addEntityId(EntityId $entityId) {
 		$this->entityIds[] = $entityId;
 	}
 
@@ -44,9 +44,9 @@ 
 	 *
 	 * @return EntityId[]
 	 */
-	public function fetchIds( $limit ) {
-		$entityIds = array_slice( $this->entityIds, $this->offset, $limit );
-		$this->offset += count( $entityIds );
+	public function fetchIds($limit) {
+		$entityIds = array_slice($this->entityIds, $this->offset, $limit);
+		$this->offset += count($entityIds);
 		return $entityIds;
 	}
 
@@ -56,7 +56,7 @@ 
 	}
 
 	/** @inheritDoc */
-	public function setPosition( $position ) {
+	public function setPosition($position) {
 		$this->offset = $position;
 	}
 

src/EntityId/EscapingEntityIdFormatter.php

Spacing +6 added lines, -6 removed lines

@@ -32,9 +32,9 @@ 
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( EntityIdFormatter $formatter, $escapeCallback ) {
-		if ( !is_callable( $escapeCallback ) ) {
-			throw new InvalidArgumentException( '$escapeCallback must be callable' );
+	public function __construct(EntityIdFormatter $formatter, $escapeCallback) {
+		if (!is_callable($escapeCallback)) {
+			throw new InvalidArgumentException('$escapeCallback must be callable');
 		}
 
 		$this->formatter = $formatter;
@@ -48,9 +48,9 @@ 
 	 *
 	 * @return string Typically wikitext or HTML, depending on the $escapeCallback provided.
 	 */
-	public function formatEntityId( EntityId $value ) {
-		$text = $this->formatter->formatEntityId( $value );
-		$escaped = ( $this->escapeCallback )( $text );
+	public function formatEntityId(EntityId $value) {
+		$text = $this->formatter->formatEntityId($value);
+		$escaped = ($this->escapeCallback)($text);
 		return $escaped;
 	}
 

src/EntityId/EntityIdComposer.php

Spacing +10 added lines, -10 removed lines

@@ -33,10 +33,10 @@ 
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( array $composers ) {
-		foreach ( $composers as $entityType => $composer ) {
-			if ( !is_string( $entityType ) || $entityType === '' || !is_callable( $composer ) ) {
-				throw new InvalidArgumentException( '$composers must map non-empty strings to callables' );
+	public function __construct(array $composers) {
+		foreach ($composers as $entityType => $composer) {
+			if (!is_string($entityType) || $entityType === '' || !is_callable($composer)) {
+				throw new InvalidArgumentException('$composers must map non-empty strings to callables');
 			}
 		}
 
@@ -53,15 +53,15 @@ 
 	 *  object.
 	 * @return EntityId
 	 */
-	public function composeEntityId( $entityType, $uniquePart ) {
-		if ( !isset( $this->composers[$entityType] ) ) {
-			throw new InvalidArgumentException( 'Unknown entity type ' . $entityType );
+	public function composeEntityId($entityType, $uniquePart) {
+		if (!isset($this->composers[$entityType])) {
+			throw new InvalidArgumentException('Unknown entity type ' . $entityType);
 		}
 
-		$id = $this->composers[$entityType]( $uniquePart );
+		$id = $this->composers[$entityType]($uniquePart);
 
-		if ( !( $id instanceof EntityId ) ) {
-			throw new UnexpectedValueException( 'Composer for ' . $entityType . ' is invalid' );
+		if (!($id instanceof EntityId)) {
+			throw new UnexpectedValueException('Composer for ' . $entityType . ' is invalid');
 		}
 
 		return $id;

src/EntityId/PlainEntityIdFormatter.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 	 *
 	 * @return string Plain text
 	 */
-	public function formatEntityId( EntityId $entityId ) {
+	public function formatEntityId(EntityId $entityId) {
 		return $entityId->getSerialization();
 	}
 

src/EntityId/SuffixEntityIdParser.php

Spacing +8 added lines, -8 removed lines

@@ -31,7 +31,7 @@ 
 	 * @param string $prefix The prefix to be stripped. Stripping is case sensitive.
 	 * @param EntityIdParser $idParser
 	 */
-	public function __construct( $prefix, EntityIdParser $idParser ) {
+	public function __construct($prefix, EntityIdParser $idParser) {
 		$this->prefix = $prefix;
 		$this->idParser = $idParser;
 	}
@@ -47,16 +47,16 @@ 
 	 *         or the remaining suffix is not a valid entity ID string.
 	 * @return EntityId
 	 */
-	public function parse( $idSerialization ) {
-		$prefixLength = strlen( $this->prefix );
+	public function parse($idSerialization) {
+		$prefixLength = strlen($this->prefix);
 
-		if ( strncmp( $idSerialization, $this->prefix, $prefixLength ) !== 0 ) {
-			throw new EntityIdParsingException( 'Missing expected prefix "' . $this->prefix
-				. '" in "' . $idSerialization . '"' );
+		if (strncmp($idSerialization, $this->prefix, $prefixLength) !== 0) {
+			throw new EntityIdParsingException('Missing expected prefix "' . $this->prefix
+				. '" in "' . $idSerialization . '"');
 		}
 
-		$idSerialization = substr( $idSerialization, $prefixLength );
-		return $this->idParser->parse( $idSerialization );
+		$idSerialization = substr($idSerialization, $prefixLength);
+		return $this->idParser->parse($idSerialization);
 	}
 
 }