wmde / WikibaseDataModelServices

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 ) {
+			return $this->propertyDataTypeLookup->getDataTypeIdForProperty($propertyId) === $dataType;
+		} catch (PropertyDataTypeLookupException) {
 			return false;
 		}
 	}

src/Lookup/ExceptionIgnoringEntityLookup.php

Spacing +6 added lines, -6 removed lines

@@ -19,7 +19,7 @@ 
 	 */
 	private $lookup;
 
-	public function __construct( EntityLookup $lookup ) {
+	public function __construct(EntityLookup $lookup) {
 		$this->lookup = $lookup;
 	}
 
@@ -30,10 +30,10 @@ 
 	 * @param EntityId $entityId
 	 * @return EntityDocument|null
 	 */
-	public function getEntity( EntityId $entityId ) {
+	public function getEntity(EntityId $entityId) {
 		try {
-			return $this->lookup->getEntity( $entityId );
-		} catch ( EntityLookupException ) {
+			return $this->lookup->getEntity($entityId);
+		} catch (EntityLookupException) {
 			return null;
 		}
 	}
@@ -46,8 +46,8 @@ 
 	 * @param EntityId $entityId
 	 * @return bool
 	 */
-	public function hasEntity( EntityId $entityId ) {
-		return $this->lookup->hasEntity( $entityId );
+	public function hasEntity(EntityId $entityId) {
+		return $this->lookup->hasEntity($entityId);
 	}
 
 }

src/EntityId/EntityIdLabelFormatter.php

Spacing +7 added lines, -7 removed lines

@@ -22,7 +22,7 @@ 
 	 */
 	private $labelLookup;
 
-	public function __construct( LabelLookup $labelLookup ) {
+	public function __construct(LabelLookup $labelLookup) {
 		$this->labelLookup = $labelLookup;
 	}
 
@@ -33,9 +33,9 @@ 
 	 *
 	 * @return string Plain text
 	 */
-	public function formatEntityId( EntityId $entityId ) {
-		$term = $this->lookupEntityLabel( $entityId );
-		if ( $term !== null ) {
+	public function formatEntityId(EntityId $entityId) {
+		$term = $this->lookupEntityLabel($entityId);
+		if ($term !== null) {
 			return $term->getText();
 		}
 
@@ -48,10 +48,10 @@ 
 	 *
 	 * @return Term|null Null if no label was found or the entity does not exist
 	 */
-	protected function lookupEntityLabel( EntityId $entityId ) {
+	protected function lookupEntityLabel(EntityId $entityId) {
 		try {
-			return $this->labelLookup->getLabel( $entityId );
-		} catch ( LabelDescriptionLookupException ) {
+			return $this->labelLookup->getLabel($entityId);
+		} catch (LabelDescriptionLookupException) {
 			return null;
 		}
 	}

src/Entity/PropertyDataTypeMatcher.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	private $propertyDataTypeLookup;
 
-	public function __construct( PropertyDataTypeLookup $propertyDataTypeLookup ) {
+	public function __construct(PropertyDataTypeLookup $propertyDataTypeLookup) {
 		$this->propertyDataTypeLookup = $propertyDataTypeLookup;
 	}
 
@@ -33,10 +33,10 @@ 
 	 *
 	 * @return bool
 	 */
-	public function isMatchingDataType( PropertyId $propertyId, $dataType ) {
+	public function isMatchingDataType(PropertyId $propertyId, $dataType) {
 		try {
-			return $this->findDataTypeIdForProperty( $propertyId ) === $dataType;
-		} catch ( PropertyDataTypeLookupException ) {
+			return $this->findDataTypeIdForProperty($propertyId) === $dataType;
+		} catch (PropertyDataTypeLookupException) {
 			return false;
 		}
 	}
@@ -47,8 +47,8 @@ 
 	 * @return string
 	 * @throws PropertyDataTypeLookupException
 	 */
-	private function findDataTypeIdForProperty( PropertyId $propertyId ) {
-		return $this->propertyDataTypeLookup->getDataTypeIdForProperty( $propertyId );
+	private function findDataTypeIdForProperty(PropertyId $propertyId) {
+		return $this->propertyDataTypeLookup->getDataTypeIdForProperty($propertyId);
 	}
 
 }

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 ) {
+			$dataType = $this->dataTypeLookup->getDataTypeIdForProperty($id);
+		} catch (PropertyDataTypeLookupException) {
 			return false;
 		}
 
-		return in_array( $dataType, $this->dataTypes );
+		return in_array($dataType, $this->dataTypes);
 	}
 
 }

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 ) {
+		} catch (EntityIdParsingException) {
 			return false;
 		}
 	}