wikimedia / mediawiki-extensions-WikibaseQualityConstraints

src/ConstraintParameterRenderer.php

Spacing +85 added lines, -85 removed lines

@@ -60,20 +60,20 @@ 
 	 *
 	 * @return string HTML
 	 */
-	public function formatValue( $value ) {
-		if ( is_string( $value ) ) {
+	public function formatValue($value) {
+		if (is_string($value)) {
 			// Cases like 'Format' 'pattern' or 'minimum'/'maximum' values, which we have stored as
 			// strings
-			return htmlspecialchars( $value );
-		} elseif ( $value instanceof EntityId ) {
+			return htmlspecialchars($value);
+		} elseif ($value instanceof EntityId) {
 			// Cases like 'Conflicts with' 'property', to which we can link
-			return $this->formatEntityId( $value );
-		} elseif ( $value instanceof ItemIdSnakValue ) {
+			return $this->formatEntityId($value);
+		} elseif ($value instanceof ItemIdSnakValue) {
 			// Cases like EntityId but can also be somevalue or novalue
-			return $this->formatItemIdSnakValue( $value );
+			return $this->formatItemIdSnakValue($value);
 		} else {
 			// Cases where we format a DataValue
-			return $this->formatDataValue( $value );
+			return $this->formatDataValue($value);
 		}
 	}
 
@@ -84,23 +84,23 @@ 
 	 *
 	 * @return string HTML
 	 */
-	public function formatParameters( $parameters ) {
-		if ( $parameters === null || count( $parameters ) == 0 ) {
+	public function formatParameters($parameters) {
+		if ($parameters === null || count($parameters) == 0) {
 			return null;
 		}
 
-		$valueFormatter = function ( $value ) {
-			return $this->formatValue( $value );
+		$valueFormatter = function($value) {
+			return $this->formatValue($value);
 		};
 
 		$formattedParameters = [];
-		foreach ( $parameters as $parameterName => $parameterValue ) {
-			$formattedParameterValues = implode( ', ',
-				$this->limitArrayLength( array_map( $valueFormatter, $parameterValue ) ) );
-			$formattedParameters[] = sprintf( '%s: %s', $parameterName, $formattedParameterValues );
+		foreach ($parameters as $parameterName => $parameterValue) {
+			$formattedParameterValues = implode(', ',
+				$this->limitArrayLength(array_map($valueFormatter, $parameterValue)));
+			$formattedParameters[] = sprintf('%s: %s', $parameterName, $formattedParameterValues);
 		}
 
-		return implode( '; ', $formattedParameters );
+		return implode('; ', $formattedParameters);
 	}
 
 	/**
@@ -110,10 +110,10 @@ 
 	 *
 	 * @return array
 	 */
-	private function limitArrayLength( array $array ) {
-		if ( count( $array ) > self::MAX_PARAMETER_ARRAY_LENGTH ) {
-			$array = array_slice( $array, 0, self::MAX_PARAMETER_ARRAY_LENGTH );
-			array_push( $array, '...' );
+	private function limitArrayLength(array $array) {
+		if (count($array) > self::MAX_PARAMETER_ARRAY_LENGTH) {
+			$array = array_slice($array, 0, self::MAX_PARAMETER_ARRAY_LENGTH);
+			array_push($array, '...');
 		}
 
 		return $array;
@@ -126,12 +126,12 @@ 
 	 * @param string $value HTML
 	 * @return string HTML
 	 */
-	public static function formatByRole( $role, $value ) {
-		if ( $role === null ) {
+	public static function formatByRole($role, $value) {
+		if ($role === null) {
 			return $value;
 		}
 
-		return '<span class="wbqc-role wbqc-role-' . htmlspecialchars( $role ) . '">'
+		return '<span class="wbqc-role wbqc-role-'.htmlspecialchars($role).'">'
 			. $value
 			. '</span>';
 	}
@@ -141,9 +141,9 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string HTML
 	 */
-	public function formatDataValue( DataValue $value, $role = null ) {
-		return self::formatByRole( $role,
-			$this->dataValueFormatter->format( $value ) );
+	public function formatDataValue(DataValue $value, $role = null) {
+		return self::formatByRole($role,
+			$this->dataValueFormatter->format($value));
 	}
 
 	/**
@@ -151,9 +151,9 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string HTML
 	 */
-	public function formatEntityId( EntityId $entityId, $role = null ) {
-		return self::formatByRole( $role,
-			$this->entityIdLabelFormatter->formatEntityId( $entityId ) );
+	public function formatEntityId(EntityId $entityId, $role = null) {
+		return self::formatByRole($role,
+			$this->entityIdLabelFormatter->formatEntityId($entityId));
 	}
 
 	/**
@@ -165,18 +165,18 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string HTML
 	 */
-	public function formatPropertyId( $propertyId, $role = null ) {
-		if ( $propertyId instanceof PropertyId ) {
-			return $this->formatEntityId( $propertyId, $role );
-		} elseif ( is_string( $propertyId ) ) {
+	public function formatPropertyId($propertyId, $role = null) {
+		if ($propertyId instanceof PropertyId) {
+			return $this->formatEntityId($propertyId, $role);
+		} elseif (is_string($propertyId)) {
 			try {
-				return $this->formatEntityId( new PropertyId( $propertyId ), $role );
-			} catch ( InvalidArgumentException $e ) {
-				return self::formatByRole( $role,
-					htmlspecialchars( $propertyId ) );
+				return $this->formatEntityId(new PropertyId($propertyId), $role);
+			} catch (InvalidArgumentException $e) {
+				return self::formatByRole($role,
+					htmlspecialchars($propertyId));
 			}
 		} else {
-			throw new InvalidArgumentException( '$propertyId must be either PropertyId or string' );
+			throw new InvalidArgumentException('$propertyId must be either PropertyId or string');
 		}
 	}
 
@@ -189,18 +189,18 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string HTML
 	 */
-	public function formatItemId( $itemId, $role = null ) {
-		if ( $itemId instanceof ItemId ) {
-			return $this->formatEntityId( $itemId, $role );
-		} elseif ( is_string( $itemId ) ) {
+	public function formatItemId($itemId, $role = null) {
+		if ($itemId instanceof ItemId) {
+			return $this->formatEntityId($itemId, $role);
+		} elseif (is_string($itemId)) {
 			try {
-				return $this->formatEntityId( new ItemId( $itemId ), $role );
-			} catch ( InvalidArgumentException $e ) {
-				return self::formatByRole( $role,
-					htmlspecialchars( $itemId ) );
+				return $this->formatEntityId(new ItemId($itemId), $role);
+			} catch (InvalidArgumentException $e) {
+				return self::formatByRole($role,
+					htmlspecialchars($itemId));
 			}
 		} else {
-			throw new InvalidArgumentException( '$itemId must be either ItemId or string' );
+			throw new InvalidArgumentException('$itemId must be either ItemId or string');
 		}
 	}
 
@@ -211,20 +211,20 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string HTML
 	 */
-	public function formatItemIdSnakValue( ItemIdSnakValue $value, $role = null ) {
-		switch ( true ) {
+	public function formatItemIdSnakValue(ItemIdSnakValue $value, $role = null) {
+		switch (true) {
 			case $value->isValue():
-				return $this->formatEntityId( $value->getItemId(), $role );
+				return $this->formatEntityId($value->getItemId(), $role);
 			case $value->isSomeValue():
-				return self::formatByRole( $role,
+				return self::formatByRole($role,
 					'<span class="wikibase-snakview-variation-somevaluesnak">'
-						. wfMessage( 'wikibase-snakview-snaktypeselector-somevalue' )->escaped()
-						. '</span>' );
+						. wfMessage('wikibase-snakview-snaktypeselector-somevalue')->escaped()
+						. '</span>');
 			case $value->isNoValue():
-				return self::formatByRole( $role,
+				return self::formatByRole($role,
 					'<span class="wikibase-snakview-variation-novaluesnak">'
-						. wfMessage( 'wikibase-snakview-snaktypeselector-novalue' )->escaped()
-						. '</span>' );
+						. wfMessage('wikibase-snakview-snaktypeselector-novalue')->escaped()
+						. '</span>');
 		}
 	}
 
@@ -238,15 +238,15 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string[] HTML
 	 */
-	public function formatPropertyIdList( array $propertyIds, $role = null ) {
-		if ( empty( $propertyIds ) ) {
-			return [ '<ul></ul>' ];
+	public function formatPropertyIdList(array $propertyIds, $role = null) {
+		if (empty($propertyIds)) {
+			return ['<ul></ul>'];
 		}
-		$propertyIds = $this->limitArrayLength( $propertyIds );
-		$formattedPropertyIds = array_map( [ $this, "formatPropertyId" ], $propertyIds, array_fill( 0, count( $propertyIds ), $role ) );
+		$propertyIds = $this->limitArrayLength($propertyIds);
+		$formattedPropertyIds = array_map([$this, "formatPropertyId"], $propertyIds, array_fill(0, count($propertyIds), $role));
 		array_unshift(
 			$formattedPropertyIds,
-			'<ul><li>' . implode( '</li><li>', $formattedPropertyIds ) . '</li></ul>'
+			'<ul><li>'.implode('</li><li>', $formattedPropertyIds).'</li></ul>'
 		);
 		return $formattedPropertyIds;
 	}
@@ -261,15 +261,15 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string[] HTML
 	 */
-	public function formatItemIdList( array $itemIds, $role = null ) {
-		if ( empty( $itemIds ) ) {
-			return [ '<ul></ul>' ];
+	public function formatItemIdList(array $itemIds, $role = null) {
+		if (empty($itemIds)) {
+			return ['<ul></ul>'];
 		}
-		$itemIds = $this->limitArrayLength( $itemIds );
-		$formattedItemIds = array_map( [ $this, "formatItemId" ], $itemIds, array_fill( 0, count( $itemIds ), $role ) );
+		$itemIds = $this->limitArrayLength($itemIds);
+		$formattedItemIds = array_map([$this, "formatItemId"], $itemIds, array_fill(0, count($itemIds), $role));
 		array_unshift(
 			$formattedItemIds,
-			'<ul><li>' . implode( '</li><li>', $formattedItemIds ) . '</li></ul>'
+			'<ul><li>'.implode('</li><li>', $formattedItemIds).'</li></ul>'
 		);
 		return $formattedItemIds;
 	}
@@ -284,23 +284,23 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string[] HTML
 	 */
-	public function formatEntityIdList( array $entityIds, $role = null ) {
-		if ( empty( $entityIds ) ) {
-			return [ '<ul></ul>' ];
+	public function formatEntityIdList(array $entityIds, $role = null) {
+		if (empty($entityIds)) {
+			return ['<ul></ul>'];
 		}
 		$formattedEntityIds = [];
-		foreach ( $entityIds as $entityId ) {
-			if ( count( $formattedEntityIds ) >= self::MAX_PARAMETER_ARRAY_LENGTH ) {
+		foreach ($entityIds as $entityId) {
+			if (count($formattedEntityIds) >= self::MAX_PARAMETER_ARRAY_LENGTH) {
 				$formattedEntityIds[] = '...';
 				break;
 			}
-			if ( $entityId !== null ) {
-				$formattedEntityIds[] = $this->formatEntityId( $entityId, $role );
+			if ($entityId !== null) {
+				$formattedEntityIds[] = $this->formatEntityId($entityId, $role);
 			}
 		}
 		array_unshift(
 			$formattedEntityIds,
-			'<ul><li>' . implode( '</li><li>', $formattedEntityIds ) . '</li></ul>'
+			'<ul><li>'.implode('</li><li>', $formattedEntityIds).'</li></ul>'
 		);
 		return $formattedEntityIds;
 	}
@@ -315,24 +315,24 @@ 
 	 * @param string|null $role one of the Role constants or null
 	 * @return string[] HTML
 	 */
-	public function formatItemIdSnakValueList( array $values, $role = null ) {
-		if ( empty( $values ) ) {
-			return [ '<ul></ul>' ];
+	public function formatItemIdSnakValueList(array $values, $role = null) {
+		if (empty($values)) {
+			return ['<ul></ul>'];
 		}
-		$values = $this->limitArrayLength( $values );
+		$values = $this->limitArrayLength($values);
 		$formattedValues = array_map(
-			function( $value ) use ( $role ) {
-				if ( $value === '...' ) {
+			function($value) use ($role) {
+				if ($value === '...') {
 					return '...';
 				} else {
-					return $this->formatItemIdSnakValue( $value, $role );
+					return $this->formatItemIdSnakValue($value, $role);
 				}
 			},
 			$values
 		);
 		array_unshift(
 			$formattedValues,
-			'<ul><li>' . implode( '</li><li>', $formattedValues ) . '</li></ul>'
+			'<ul><li>'.implode('</li><li>', $formattedValues).'</li></ul>'
 		);
 		return $formattedValues;
 	}

src/Role.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 	 * @codeCoverageIgnore
 	 */
 	private function __construct() {
-		throw new LogicException( 'This class should never be instantiated.' );
+		throw new LogicException('This class should never be instantiated.');
 	}
 
 }

src/ConstraintCheck/ItemIdSnakValue.php

Spacing +10 added lines, -10 removed lines

@@ -44,7 +44,7 @@ 
 	 * @param ItemId $itemId
 	 * @return self
 	 */
-	public static function fromItemId( ItemId $itemId ) {
+	public static function fromItemId(ItemId $itemId) {
 		$ret = new self;
 		$ret->itemId = $itemId;
 		return $ret;
@@ -80,16 +80,16 @@ 
 	 * @throws InvalidArgumentException
 	 * @return self
 	 */
-	public static function fromSnak( Snak $snak ) {
-		switch ( true ) {
+	public static function fromSnak(Snak $snak) {
+		switch (true) {
 			case $snak instanceof PropertyValueSnak:
 				if (
 					$snak->getDataValue() instanceof EntityIdValue &&
 					$snak->getDataValue()->getEntityId() instanceof ItemId
 				) {
-					return self::fromItemId( $snak->getDataValue()->getEntityId() );
+					return self::fromItemId($snak->getDataValue()->getEntityId());
 				} else {
-					throw new InvalidArgumentException( 'Snak must contain item ID value or be a somevalue / novalue snak' );
+					throw new InvalidArgumentException('Snak must contain item ID value or be a somevalue / novalue snak');
 				}
 			case $snak instanceof PropertySomeValueSnak:
 				return self::someValue();
@@ -132,8 +132,8 @@ 
 	 * @return ItemId
 	 */
 	public function getItemId() {
-		if ( ! $this->isValue() ) {
-			throw new DomainException( 'This value does not contain an item ID.' );
+		if (!$this->isValue()) {
+			throw new DomainException('This value does not contain an item ID.');
 		}
 		return $this->itemId;
 	}
@@ -145,13 +145,13 @@ 
 	 * @param Snak $snak
 	 * @return bool
 	 */
-	public function matchesSnak( Snak $snak ) {
-		switch ( true ) {
+	public function matchesSnak(Snak $snak) {
+		switch (true) {
 			case $snak instanceof PropertyValueSnak:
 				return $this->isValue() &&
 					$snak->getDataValue() instanceof EntityIdValue &&
 					$snak->getDataValue()->getEntityId() instanceof ItemId &&
-					$snak->getDataValue()->getEntityId()->equals( $this->getItemId() );
+					$snak->getDataValue()->getEntityId()->equals($this->getItemId());
 			case $snak instanceof PropertySomeValueSnak:
 				return $this->isSomeValue();
 			case $snak instanceof PropertyNoValueSnak:

src/ConstraintCheck/Context/MainSnakContext.php

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
 		EntityDocument $entity,
 		Statement $statement
 	) {
-		parent::__construct( $entity, $statement->getMainSnak() );
+		parent::__construct($entity, $statement->getMainSnak());
 		$this->statement = $statement;
 	}
 
@@ -39,7 +39,7 @@ 
 		return $this->statement;
 	}
 
-	protected function &getMainArray( array &$container ) {
+	protected function &getMainArray(array &$container) {
 		$statementArray = &$this->getStatementArray(
 			$container,
 			$this->entity->getId()->getSerialization(),
@@ -47,8 +47,8 @@ 
 			$this->statement->getGuid()
 		);
 
-		if ( !array_key_exists( 'mainsnak', $statementArray ) ) {
-			$statementArray['mainsnak'] = [ 'hash' => $this->statement->getMainSnak()->getHash() ];
+		if (!array_key_exists('mainsnak', $statementArray)) {
+			$statementArray['mainsnak'] = ['hash' => $this->statement->getMainSnak()->getHash()];
 		}
 		$mainsnakArray = &$statementArray['mainsnak'];
 

src/ConstraintCheck/Context/QualifierContext.php

Spacing +8 added lines, -8 removed lines

@@ -21,7 +21,7 @@ 
 		Statement $statement,
 		Snak $snak
 	) {
-		parent::__construct( $entity, $snak );
+		parent::__construct($entity, $snak);
 		$this->statement = $statement;
 	}
 
@@ -29,7 +29,7 @@ 
 		return self::TYPE_QUALIFIER;
 	}
 
-	protected function &getMainArray( array &$container ) {
+	protected function &getMainArray(array &$container) {
 		$statementArray = &$this->getStatementArray(
 			$container,
 			$this->entity->getId()->getSerialization(),
@@ -37,25 +37,25 @@ 
 			$this->statement->getGuid()
 		);
 
-		if ( !array_key_exists( 'qualifiers', $statementArray ) ) {
+		if (!array_key_exists('qualifiers', $statementArray)) {
 			$statementArray['qualifiers'] = [];
 		}
 		$qualifiersArray = &$statementArray['qualifiers'];
 
 		$propertyId = $this->getSnak()->getPropertyId()->getSerialization();
-		if ( !array_key_exists( $propertyId, $qualifiersArray ) ) {
+		if (!array_key_exists($propertyId, $qualifiersArray)) {
 			$qualifiersArray[$propertyId] = [];
 		}
 		$propertyArray = &$qualifiersArray[$propertyId];
 
-		foreach ( $propertyArray as &$potentialQualifierArray ) {
-			if ( $potentialQualifierArray['hash'] === $this->getSnak()->getHash() ) {
+		foreach ($propertyArray as &$potentialQualifierArray) {
+			if ($potentialQualifierArray['hash'] === $this->getSnak()->getHash()) {
 				$qualifierArray = &$potentialQualifierArray;
 				break;
 			}
 		}
-		if ( !isset( $qualifierArray ) ) {
-			$qualifierArray = [ 'hash' => $this->getSnak()->getHash() ];
+		if (!isset($qualifierArray)) {
+			$qualifierArray = ['hash' => $this->getSnak()->getHash()];
 			$propertyArray[] = &$qualifierArray;
 		}
 

src/ConstraintCheck/Context/ReferenceContext.php

Spacing +12 added lines, -12 removed lines

@@ -28,7 +28,7 @@ 
 		Reference $reference,
 		Snak $snak
 	) {
-		parent::__construct( $entity, $snak );
+		parent::__construct($entity, $snak);
 		$this->statement = $statement;
 		$this->reference = $reference;
 	}
@@ -37,7 +37,7 @@ 
 		return self::TYPE_REFERENCE;
 	}
 
-	protected function &getMainArray( array &$container ) {
+	protected function &getMainArray(array &$container) {
 		$statementArray = &$this->getStatementArray(
 			$container,
 			$this->entity->getId()->getSerialization(),
@@ -45,38 +45,38 @@ 
 			$this->statement->getGuid()
 		);
 
-		if ( !array_key_exists( 'references', $statementArray ) ) {
+		if (!array_key_exists('references', $statementArray)) {
 			$statementArray['references'] = [];
 		}
 		$referencesArray = &$statementArray['references'];
 
-		foreach ( $referencesArray as &$potentialReferenceArray ) {
-			if ( $potentialReferenceArray['hash'] === $this->reference->getHash() ) {
+		foreach ($referencesArray as &$potentialReferenceArray) {
+			if ($potentialReferenceArray['hash'] === $this->reference->getHash()) {
 				$referenceArray = &$potentialReferenceArray;
 				break;
 			}
 		}
-		if ( !isset( $referenceArray ) ) {
-			$referenceArray = [ 'hash' => $this->reference->getHash(), 'snaks' => [] ];
+		if (!isset($referenceArray)) {
+			$referenceArray = ['hash' => $this->reference->getHash(), 'snaks' => []];
 			$referencesArray[] = &$referenceArray;
 		}
 
 		$snaksArray = &$referenceArray['snaks'];
 
 		$propertyId = $this->getSnak()->getPropertyId()->getSerialization();
-		if ( !array_key_exists( $propertyId, $snaksArray ) ) {
+		if (!array_key_exists($propertyId, $snaksArray)) {
 			$snaksArray[$propertyId] = [];
 		}
 		$propertyArray = &$snaksArray[$propertyId];
 
-		foreach ( $propertyArray as &$potentialSnakArray ) {
-			if ( $potentialSnakArray['hash'] === $this->getSnak()->getHash() ) {
+		foreach ($propertyArray as &$potentialSnakArray) {
+			if ($potentialSnakArray['hash'] === $this->getSnak()->getHash()) {
 				$snakArray = &$potentialSnakArray;
 				break;
 			}
 		}
-		if ( !isset( $snakArray ) ) {
-			$snakArray = [ 'hash' => $this->getSnak()->getHash() ];
+		if (!isset($snakArray)) {
+			$snakArray = ['hash' => $this->getSnak()->getHash()];
 			$propertyArray[] = &$snakArray;
 		}
 

src/ConstraintCheck/Result/NullResult.php

Spacing +5 added lines, -5 removed lines

@@ -25,22 +25,22 @@
 	 */
 	const NULL_PROPERTY_ID = 'P2147483647';
 
-	public function __construct( Context $context ) {
+	public function __construct(Context $context) {
 		$constraint = new Constraint(
 			'null',
-			new PropertyId( self::NULL_PROPERTY_ID ),
+			new PropertyId(self::NULL_PROPERTY_ID),
 			'none',
 			[]
 		);
-		parent::__construct( $context, $constraint );
+		parent::__construct($context, $constraint);
 	}
 
 	public function getConstraint() {
-		throw new DomainException( 'NullResult holds no constraint' );
+		throw new DomainException('NullResult holds no constraint');
 	}
 
 	public function getConstraintId() {
-		throw new DomainException( 'NullResult holds no constraint' );
+		throw new DomainException('NullResult holds no constraint');
 	}
 
 }

src/ConstraintCheck/ConstraintChecker.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 	 * @throws ConstraintParameterException if the constraint parameters are invalid
 	 * @throws SparqlHelperException if the checker uses SPARQL and the query times out or some other error occurs
 	 */
-	public function checkConstraint( Context $context, Constraint $constraint );
+	public function checkConstraint(Context $context, Constraint $constraint);
 
 	/**
 	 * Check if the constraint parameters of $constraint are valid.
@@ -33,6 +33,6 @@ 
 	 *
 	 * @return ConstraintParameterException[]
 	 */
-	public function checkConstraintParameters( Constraint $constraint );
+	public function checkConstraintParameters(Constraint $constraint);
 
 }

src/ConstraintCheck/Checker/MandatoryQualifiersChecker.php

Spacing +16 added lines, -16 removed lines

@@ -52,46 +52,46 @@
 	 *
 	 * @return CheckResult
 	 */
-	public function checkConstraint( Context $context, Constraint $constraint ) {
-		if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
-			return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
+	public function checkConstraint(Context $context, Constraint $constraint) {
+		if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
+			return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
 		}
-		if ( $context->getType() !== Context::TYPE_STATEMENT ) {
-			return new CheckResult( $context, $constraint, [], CheckResult::STATUS_NOT_MAIN_SNAK );
+		if ($context->getType() !== Context::TYPE_STATEMENT) {
+			return new CheckResult($context, $constraint, [], CheckResult::STATUS_NOT_MAIN_SNAK);
 		}
 
 		$parameters = [];
 		$constraintParameters = $constraint->getConstraintParameters();
 
-		$propertyId = $this->constraintParameterParser->parsePropertyParameter( $constraintParameters, $constraint->getConstraintTypeItemId() );
-		$parameters['property'] = [ $propertyId ];
+		$propertyId = $this->constraintParameterParser->parsePropertyParameter($constraintParameters, $constraint->getConstraintTypeItemId());
+		$parameters['property'] = [$propertyId];
 
-		$message = wfMessage( "wbqc-violation-message-mandatory-qualifier" )
+		$message = wfMessage("wbqc-violation-message-mandatory-qualifier")
 				 ->rawParams(
-					 $this->constraintParameterRenderer->formatEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY ),
-					 $this->constraintParameterRenderer->formatEntityId( $propertyId, Role::QUALIFIER_PREDICATE )
+					 $this->constraintParameterRenderer->formatEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY),
+					 $this->constraintParameterRenderer->formatEntityId($propertyId, Role::QUALIFIER_PREDICATE)
 				 )
 				 ->escaped();
 		$status = CheckResult::STATUS_VIOLATION;
 
 		/** @var Snak $qualifier */
-		foreach ( $context->getSnakStatement()->getQualifiers() as $qualifier ) {
-			if ( $propertyId->equals( $qualifier->getPropertyId() ) ) {
+		foreach ($context->getSnakStatement()->getQualifiers() as $qualifier) {
+			if ($propertyId->equals($qualifier->getPropertyId())) {
 				$message = '';
 				$status = CheckResult::STATUS_COMPLIANCE;
 				break;
 			}
 		}
 
-		return new CheckResult( $context, $constraint, $parameters, $status, $message );
+		return new CheckResult($context, $constraint, $parameters, $status, $message);
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$exceptions = [];
 		try {
-			$this->constraintParameterParser->parsePropertyParameter( $constraintParameters, $constraint->getConstraintTypeItemId() );
-		} catch ( ConstraintParameterException $e ) {
+			$this->constraintParameterParser->parsePropertyParameter($constraintParameters, $constraint->getConstraintTypeItemId());
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;