Inspection of "Merge "Unify terminology around allowed/valid type..." - wikimedia/mediawiki-extensions-WikibaseQualityConstraints - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bae75b...eee08f )

unknown

created 2021-09-01 11:52 UTC

Status

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch

@@ -62,9 +62,9 @@  discard block
 block discarded – undo
 	 * @throws ConstraintParameterException
 	 * @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);
 		}
 
 		$parameters = [];
@@ -81,33 +81,33 @@  discard block
 block discarded – undo
 		$status = CheckResult::STATUS_COMPLIANCE;
 
 		/** @var Snak $qualifier */
-		foreach ( $context->getSnakStatement()->getQualifiers() as $qualifier ) {
+		foreach ($context->getSnakStatement()->getQualifiers() as $qualifier) {
 			$allowedQualifier = false;
-			foreach ( $properties as $property ) {
-				if ( $qualifier->getPropertyId()->equals( $property ) ) {
+			foreach ($properties as $property) {
+				if ($qualifier->getPropertyId()->equals($property)) {
 					$allowedQualifier = true;
 					break;
 				}
 			}
-			if ( !$allowedQualifier ) {
-				if ( empty( $properties ) || $properties === [ '' ] ) {
-					$message = ( new ViolationMessage( 'wbqc-violation-message-no-qualifiers' ) )
-						->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY );
+			if (!$allowedQualifier) {
+				if (empty($properties) || $properties === ['']) {
+					$message = (new ViolationMessage('wbqc-violation-message-no-qualifiers'))
+						->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY);
 				} else {
-					$message = ( new ViolationMessage( 'wbqc-violation-message-qualifiers' ) )
-						->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY )
-						->withEntityId( $qualifier->getPropertyId(), Role::QUALIFIER_PREDICATE )
-						->withEntityIdList( $properties, Role::QUALIFIER_PREDICATE );
+					$message = (new ViolationMessage('wbqc-violation-message-qualifiers'))
+						->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY)
+						->withEntityId($qualifier->getPropertyId(), Role::QUALIFIER_PREDICATE)
+						->withEntityIdList($properties, Role::QUALIFIER_PREDICATE);
 				}
 				$status = CheckResult::STATUS_VIOLATION;
 				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();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
 				$constraintParameters,
 				$constraintTypeItemId
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/MandatoryQualifiersChecker.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -62,9 +62,9 @@  discard block
 block discarded – undo
 	 * @throws ConstraintParameterException
 	 * @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);
 		}
 
 		$parameters = [];
@@ -75,26 +75,26 @@  discard block
 block discarded – undo
 			$constraintParameters,
 			$constraintTypeItemId
 		);
-		$parameters['property'] = [ $propertyId ];
+		$parameters['property'] = [$propertyId];
 
-		$message = ( new ViolationMessage( 'wbqc-violation-message-mandatory-qualifier' ) )
-			->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY )
-			->withEntityId( $propertyId, Role::QUALIFIER_PREDICATE );
+		$message = (new ViolationMessage('wbqc-violation-message-mandatory-qualifier'))
+			->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY)
+			->withEntityId($propertyId, Role::QUALIFIER_PREDICATE);
 		$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 = null;
 				$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();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
 				$constraintParameters,
 				$constraintTypeItemId
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/OneOfChecker.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -60,9 +60,9 @@  discard block
 block discarded – undo
 	 * @throws ConstraintParameterException
 	 * @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);
 		}
 
 		$parameters = [];
@@ -78,23 +78,23 @@  discard block
 block discarded – undo
 
 		$snak = $context->getSnak();
 
-		$message = ( new ViolationMessage( 'wbqc-violation-message-one-of' ) )
-			->withEntityId( $context->getSnak()->getPropertyId(), Role::PREDICATE )
-			->withItemIdSnakValueList( $items, Role::OBJECT );
+		$message = (new ViolationMessage('wbqc-violation-message-one-of'))
+			->withEntityId($context->getSnak()->getPropertyId(), Role::PREDICATE)
+			->withItemIdSnakValueList($items, Role::OBJECT);
 		$status = CheckResult::STATUS_VIOLATION;
 
-		foreach ( $items as $item ) {
-			if ( $item->matchesSnak( $snak ) ) {
+		foreach ($items as $item) {
+			if ($item->matchesSnak($snak)) {
 				$message = null;
 				$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();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
 				$constraintTypeItemId,
 				true
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

src/Html/HtmlTableCellBuilder.php 1 patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -38,9 +38,9 @@  discard block
 block discarded – undo
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( $content, array $attributes = [], $isRawContent = false ) {
-		Assert::parameterType( 'string', $content, '$content' );
-		Assert::parameterType( 'boolean', $isRawContent, '$isRawContent' );
+	public function __construct($content, array $attributes = [], $isRawContent = false) {
+		Assert::parameterType('string', $content, '$content');
+		Assert::parameterType('boolean', $isRawContent, '$isRawContent');
 
 		$this->content = $content;
 		$this->attributes = $attributes;
@@ -66,10 +66,10 @@  discard block
 block discarded – undo
 	 * @suppress SecurityCheck-DoubleEscaped
 	 */
 	public function toHtml() {
-		if ( $this->isRawContent ) {
-			return Html::rawElement( 'td', $this->getAttributes(), $this->content );
+		if ($this->isRawContent) {
+			return Html::rawElement('td', $this->getAttributes(), $this->content);
 		} else {
-			return Html::element( 'td', $this->getAttributes(), $this->content );
+			return Html::element('td', $this->getAttributes(), $this->content);
 		}
 	}
 

Please login to merge, or discard this patch.

src/Html/HtmlTableHeaderBuilder.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -41,10 +41,10 @@  discard block
 block discarded – undo
 	 *
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( $content, $isSortable = false, $isRawContent = false ) {
-		Assert::parameterType( 'string', $content, '$content' );
-		Assert::parameterType( 'boolean', $isSortable, '$isSortable' );
-		Assert::parameterType( 'boolean', $isRawContent, '$isRawContent' );
+	public function __construct($content, $isSortable = false, $isRawContent = false) {
+		Assert::parameterType('string', $content, '$content');
+		Assert::parameterType('boolean', $isSortable, '$isSortable');
+		Assert::parameterType('boolean', $isRawContent, '$isRawContent');
 
 		$this->content = $content;
 		$this->isSortable = $isSortable;
@@ -71,20 +71,20 @@  discard block
 block discarded – undo
 	 * @return string HTML
 	 */
 	public function toHtml() {
-		$attributes = [ 'role' => 'columnheader button' ];
+		$attributes = ['role' => 'columnheader button'];
 
-		if ( !$this->isSortable ) {
+		if (!$this->isSortable) {
 			$attributes['class'] = 'unsortable';
 		}
 
-		if ( !$this->isRawContent ) {
+		if (!$this->isRawContent) {
 			// @phan-suppress-next-line SecurityCheck-DoubleEscaped
-			$content = htmlspecialchars( $this->content );
+			$content = htmlspecialchars($this->content);
 		} else {
 			$content = $this->content;
 		}
 
-		return Html::rawElement( 'th', $attributes, $content );
+		return Html::rawElement('th', $attributes, $content);
 	}
 
 }

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/SymmetricChecker.php 1 patch

Spacing +23 added lines, -23 removed lines patch added patch discarded remove patch

@@ -74,9 +74,9 @@  discard block
 block discarded – undo
 	 *
 	 * @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);
 		}
 
 		$parameters = [];
@@ -84,9 +84,9 @@  discard block
 block discarded – undo
 		$snak = $context->getSnak();
 		$propertyId = $context->getSnak()->getPropertyId();
 
-		if ( !$snak instanceof PropertyValueSnak ) {
+		if (!$snak instanceof PropertyValueSnak) {
 			// nothing to check
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE );
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE);
 		}
 
 		$dataValue = $snak->getDataValue();
@@ -95,18 +95,18 @@  discard block
 block discarded – undo
 		 * error handling:
 		 *   type of $dataValue for properties with 'Symmetric' constraint has to be 'wikibase-entityid'
 		 */
-		if ( !$dataValue instanceof EntityIdValue ) {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
-				->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM )
-				->withDataValueType( 'wikibase-entityid' );
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		if (!$dataValue instanceof EntityIdValue) {
+			$message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
+				->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM)
+				->withDataValueType('wikibase-entityid');
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		$targetEntityId = $dataValue->getEntityId();
-		$targetEntity = $this->entityLookup->getEntity( $targetEntityId );
-		if ( !$targetEntity instanceof StatementListProvider ) {
-			$message = new ViolationMessage( 'wbqc-violation-message-target-entity-must-exist' );
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		$targetEntity = $this->entityLookup->getEntity($targetEntityId);
+		if (!$targetEntity instanceof StatementListProvider) {
+			$message = new ViolationMessage('wbqc-violation-message-target-entity-must-exist');
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		$symmetricStatement = $this->connectionCheckerHelper->findStatementWithPropertyAndEntityIdValue(
@@ -114,23 +114,23 @@  discard block
 block discarded – undo
 			$propertyId,
 			$context->getEntity()->getId()
 		);
-		if ( $symmetricStatement !== null ) {
+		if ($symmetricStatement !== null) {
 			$message = null;
 			$status = CheckResult::STATUS_COMPLIANCE;
 		} else {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-symmetric' ) )
-				->withEntityId( $targetEntityId, Role::SUBJECT )
-				->withEntityId( $propertyId, Role::PREDICATE )
-				->withEntityId( $context->getEntity()->getId(), Role::OBJECT );
+			$message = (new ViolationMessage('wbqc-violation-message-symmetric'))
+				->withEntityId($targetEntityId, Role::SUBJECT)
+				->withEntityId($propertyId, Role::PREDICATE)
+				->withEntityId($context->getEntity()->getId(), Role::OBJECT);
 			$status = CheckResult::STATUS_VIOLATION;
 		}
 
-		return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
-			->withMetadata( Metadata::ofDependencyMetadata(
-				DependencyMetadata::ofEntityId( $targetEntityId ) ) );
+		return (new CheckResult($context, $constraint, $parameters, $status, $message))
+			->withMetadata(Metadata::ofDependencyMetadata(
+				DependencyMetadata::ofEntityId($targetEntityId) ));
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		// no parameters
 		return [];
 	}

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/UniqueValueChecker.php 1 patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch

@@ -65,24 +65,24 @@  discard block
 block discarded – undo
 	 * @throws SparqlHelperException if the checker uses SPARQL and the query times out or some other error occurs
 	 * @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);
 		}
 
 		$parameters = [];
 
-		if ( !( $this->sparqlHelper instanceof DummySparqlHelper ) ) {
-			if ( $context->getType() === 'statement' ) {
+		if (!($this->sparqlHelper instanceof DummySparqlHelper)) {
+			if ($context->getType() === 'statement') {
 				$result = $this->sparqlHelper->findEntitiesWithSameStatement(
 					$context->getSnakStatement(),
 					true // ignore deprecated statements
 				);
 			} else {
 				$snak = $context->getSnak();
-				if ( !$snak instanceof PropertyValueSnak ) {
+				if (!$snak instanceof PropertyValueSnak) {
 					// nothing to check
-					return new CheckResult( $context, $constraint, [], CheckResult::STATUS_COMPLIANCE );
+					return new CheckResult($context, $constraint, [], CheckResult::STATUS_COMPLIANCE);
 				}
 				$result = $this->sparqlHelper->findEntitiesWithSameQualifierOrReference(
 					$context->getEntity()->getId(),
@@ -95,27 +95,27 @@  discard block
 block discarded – undo
 			$otherEntities = $result->getArray();
 			$metadata = $result->getMetadata();
 
-			if ( $otherEntities === [] ) {
+			if ($otherEntities === []) {
 				$status = CheckResult::STATUS_COMPLIANCE;
 				$message = null;
 			} else {
-				$otherEntities = array_values( array_filter( $otherEntities ) ); // remove nulls
+				$otherEntities = array_values(array_filter($otherEntities)); // remove nulls
 				$status = CheckResult::STATUS_VIOLATION;
-				$message = ( new ViolationMessage( 'wbqc-violation-message-unique-value' ) )
-					->withEntityIdList( $otherEntities, Role::SUBJECT );
+				$message = (new ViolationMessage('wbqc-violation-message-unique-value'))
+					->withEntityIdList($otherEntities, Role::SUBJECT);
 			}
 		} else {
 			$status = CheckResult::STATUS_TODO;
-			$message = ( new ViolationMessage( 'wbqc-violation-message-not-yet-implemented' ) )
-				->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM );
+			$message = (new ViolationMessage('wbqc-violation-message-not-yet-implemented'))
+				->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM);
 			$metadata = Metadata::blank();
 		}
 
-		return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
-			->withMetadata( $metadata );
+		return (new CheckResult($context, $constraint, $parameters, $status, $message))
+			->withMetadata($metadata);
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		// no parameters
 		return [];
 	}

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/InverseChecker.php 1 patch

Spacing +25 added lines, -25 removed lines patch added patch discarded remove patch

@@ -84,9 +84,9 @@  discard block
 block discarded – undo
 	 * @throws ConstraintParameterException
 	 * @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);
 		}
 
 		$parameters = [];
@@ -97,13 +97,13 @@  discard block
 block discarded – undo
 			$constraintParameters,
 			$constraintTypeItemId
 		);
-		$parameters['property'] = [ $propertyId ];
+		$parameters['property'] = [$propertyId];
 
 		$snak = $context->getSnak();
 
-		if ( !$snak instanceof PropertyValueSnak ) {
+		if (!$snak instanceof PropertyValueSnak) {
 			// nothing to check
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE );
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE);
 		}
 
 		$dataValue = $snak->getDataValue();
@@ -112,18 +112,18 @@  discard block
 block discarded – undo
 		 * error handling:
 		 *   type of $dataValue for properties with 'Inverse' constraint has to be 'wikibase-entityid'
 		 */
-		if ( !$dataValue instanceof EntityIdValue ) {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
-				->withEntityId( new ItemId( $constraintTypeItemId ), Role::CONSTRAINT_TYPE_ITEM )
-				->withDataValueType( 'wikibase-entityid' );
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		if (!$dataValue instanceof EntityIdValue) {
+			$message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
+				->withEntityId(new ItemId($constraintTypeItemId), Role::CONSTRAINT_TYPE_ITEM)
+				->withDataValueType('wikibase-entityid');
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		$targetEntityId = $dataValue->getEntityId();
-		$targetEntity = $this->entityLookup->getEntity( $targetEntityId );
-		if ( !$targetEntity instanceof StatementListProvider ) {
-			$message = new ViolationMessage( 'wbqc-violation-message-target-entity-must-exist' );
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		$targetEntity = $this->entityLookup->getEntity($targetEntityId);
+		if (!$targetEntity instanceof StatementListProvider) {
+			$message = new ViolationMessage('wbqc-violation-message-target-entity-must-exist');
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		$inverseStatement = $this->connectionCheckerHelper->findStatementWithPropertyAndEntityIdValue(
@@ -131,23 +131,23 @@  discard block
 block discarded – undo
 			$propertyId,
 			$context->getEntity()->getId()
 		);
-		if ( $inverseStatement !== null ) {
+		if ($inverseStatement !== null) {
 			$message = null;
 			$status = CheckResult::STATUS_COMPLIANCE;
 		} else {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-inverse' ) )
-				->withEntityId( $targetEntityId, Role::SUBJECT )
-				->withEntityId( $propertyId, Role::PREDICATE )
-				->withEntityId( $context->getEntity()->getId(), Role::OBJECT );
+			$message = (new ViolationMessage('wbqc-violation-message-inverse'))
+				->withEntityId($targetEntityId, Role::SUBJECT)
+				->withEntityId($propertyId, Role::PREDICATE)
+				->withEntityId($context->getEntity()->getId(), Role::OBJECT);
 			$status = CheckResult::STATUS_VIOLATION;
 		}
 
-		return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
-			->withMetadata( Metadata::ofDependencyMetadata(
-				DependencyMetadata::ofEntityId( $targetEntityId ) ) );
+		return (new CheckResult($context, $constraint, $parameters, $status, $message))
+			->withMetadata(Metadata::ofDependencyMetadata(
+				DependencyMetadata::ofEntityId($targetEntityId) ));
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
 				$constraintParameters,
 				$constraintTypeItemId
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/TargetRequiredClaimChecker.php 1 patch

Spacing +27 added lines, -27 removed lines patch added patch discarded remove patch

@@ -82,9 +82,9 @@  discard block
 block discarded – undo
 	 * @throws ConstraintParameterException
 	 * @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);
 		}
 
 		$parameters = [];
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
 			$constraintParameters,
 			$constraintTypeItemId
 		);
-		$parameters['property'] = [ $propertyId ];
+		$parameters['property'] = [$propertyId];
 
 		$items = $this->constraintParameterParser->parseItemsParameter(
 			$constraintParameters,
@@ -106,9 +106,9 @@  discard block
 block discarded – undo
 
 		$snak = $context->getSnak();
 
-		if ( !$snak instanceof PropertyValueSnak ) {
+		if (!$snak instanceof PropertyValueSnak) {
 			// nothing to check
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE );
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE);
 		}
 
 		$dataValue = $snak->getDataValue();
@@ -117,18 +117,18 @@  discard block
 block discarded – undo
 		 * error handling:
 		 *   type of $dataValue for properties with 'Target required claim' constraint has to be 'wikibase-entityid'
 		 */
-		if ( !$dataValue instanceof EntityIdValue ) {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
-				->withEntityId( new ItemId( $constraintTypeItemId ), Role::CONSTRAINT_TYPE_ITEM )
-				->withDataValueType( 'wikibase-entityid' );
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		if (!$dataValue instanceof EntityIdValue) {
+			$message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
+				->withEntityId(new ItemId($constraintTypeItemId), Role::CONSTRAINT_TYPE_ITEM)
+				->withDataValueType('wikibase-entityid');
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		$targetEntityId = $dataValue->getEntityId();
-		$targetEntity = $this->entityLookup->getEntity( $targetEntityId );
-		if ( !$targetEntity instanceof StatementListProvider ) {
-			$message = new ViolationMessage( 'wbqc-violation-message-target-entity-must-exist' );
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		$targetEntity = $this->entityLookup->getEntity($targetEntityId);
+		if (!$targetEntity instanceof StatementListProvider) {
+			$message = new ViolationMessage('wbqc-violation-message-target-entity-must-exist');
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		/*
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
 		 *   a) a property only
 		 *   b) a property and a number of items (each combination forming an individual claim)
 		 */
-		if ( $items === [] ) {
+		if ($items === []) {
 			$requiredStatement = $this->connectionCheckerHelper->findStatementWithProperty(
 				$targetEntity->getStatements(),
 				$propertyId
@@ -149,23 +149,23 @@  discard block
 block discarded – undo
 			);
 		}
 
-		if ( $requiredStatement !== null ) {
+		if ($requiredStatement !== null) {
 			$status = CheckResult::STATUS_COMPLIANCE;
 			$message = null;
 		} else {
 			$status = CheckResult::STATUS_VIOLATION;
-			$message = ( new ViolationMessage( 'wbqc-violation-message-target-required-claim' ) )
-				->withEntityId( $targetEntityId, Role::SUBJECT )
-				->withEntityId( $propertyId, Role::PREDICATE )
-				->withItemIdSnakValueList( $items, Role::OBJECT );
+			$message = (new ViolationMessage('wbqc-violation-message-target-required-claim'))
+				->withEntityId($targetEntityId, Role::SUBJECT)
+				->withEntityId($propertyId, Role::PREDICATE)
+				->withItemIdSnakValueList($items, Role::OBJECT);
 		}
 
-		return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
-			->withMetadata( Metadata::ofDependencyMetadata(
-				DependencyMetadata::ofEntityId( $targetEntityId ) ) );
+		return (new CheckResult($context, $constraint, $parameters, $status, $message))
+			->withMetadata(Metadata::ofDependencyMetadata(
+				DependencyMetadata::ofEntityId($targetEntityId) ));
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
 				$constraintParameters,
 				$constraintTypeItemId
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		try {
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
 				$constraintTypeItemId,
 				false
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

		@@ -82,9 +82,9 @@ discard block
		block discarded – undo
82	82	* @throws ConstraintParameterException
83	83	* @return CheckResult
84	84	*/
85		- public function checkConstraint( Context $context, Constraint $constraint ) {
86		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
87		- return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
	85	+ public function checkConstraint(Context $context, Constraint $constraint) {
	86	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	87	+ return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
88	88	}
89	89
90	90	$parameters = [];
		@@ -95,7 +95,7 @@ discard block
		block discarded – undo
95	95	$constraintParameters,
96	96	$constraintTypeItemId
97	97	);
98		- $parameters['property'] = [ $propertyId ];
	98	+ $parameters['property'] = [$propertyId];
99	99
100	100	$items = $this->constraintParameterParser->parseItemsParameter(
101	101	$constraintParameters,
		@@ -106,9 +106,9 @@ discard block
		block discarded – undo
106	106
107	107	$snak = $context->getSnak();
108	108
109		- if ( !$snak instanceof PropertyValueSnak ) {
	109	+ if (!$snak instanceof PropertyValueSnak) {
110	110	// nothing to check
111		- return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE );
	111	+ return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE);
112	112	}
113	113
114	114	$dataValue = $snak->getDataValue();
		@@ -117,18 +117,18 @@ discard block
		block discarded – undo
117	117	* error handling:
118	118	* type of $dataValue for properties with 'Target required claim' constraint has to be 'wikibase-entityid'
119	119	*/
120		- if ( !$dataValue instanceof EntityIdValue ) {
121		- $message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
122		- ->withEntityId( new ItemId( $constraintTypeItemId ), Role::CONSTRAINT_TYPE_ITEM )
123		- ->withDataValueType( 'wikibase-entityid' );
124		- return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
	120	+ if (!$dataValue instanceof EntityIdValue) {
	121	+ $message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
	122	+ ->withEntityId(new ItemId($constraintTypeItemId), Role::CONSTRAINT_TYPE_ITEM)
	123	+ ->withDataValueType('wikibase-entityid');
	124	+ return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
125	125	}
126	126
127	127	$targetEntityId = $dataValue->getEntityId();
128		- $targetEntity = $this->entityLookup->getEntity( $targetEntityId );
129		- if ( !$targetEntity instanceof StatementListProvider ) {
130		- $message = new ViolationMessage( 'wbqc-violation-message-target-entity-must-exist' );
131		- return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
	128	+ $targetEntity = $this->entityLookup->getEntity($targetEntityId);
	129	+ if (!$targetEntity instanceof StatementListProvider) {
	130	+ $message = new ViolationMessage('wbqc-violation-message-target-entity-must-exist');
	131	+ return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
132	132	}
133	133
134	134	/*
		@@ -136,7 +136,7 @@ discard block
		block discarded – undo
136	136	* a) a property only
137	137	* b) a property and a number of items (each combination forming an individual claim)
138	138	*/
139		- if ( $items === [] ) {
	139	+ if ($items === []) {
140	140	$requiredStatement = $this->connectionCheckerHelper->findStatementWithProperty(
141	141	$targetEntity->getStatements(),
142	142	$propertyId
		@@ -149,23 +149,23 @@ discard block
		block discarded – undo
149	149	);
150	150	}
151	151
152		- if ( $requiredStatement !== null ) {
	152	+ if ($requiredStatement !== null) {
153	153	$status = CheckResult::STATUS_COMPLIANCE;
154	154	$message = null;
155	155	} else {
156	156	$status = CheckResult::STATUS_VIOLATION;
157		- $message = ( new ViolationMessage( 'wbqc-violation-message-target-required-claim' ) )
158		- ->withEntityId( $targetEntityId, Role::SUBJECT )
159		- ->withEntityId( $propertyId, Role::PREDICATE )
160		- ->withItemIdSnakValueList( $items, Role::OBJECT );
	157	+ $message = (new ViolationMessage('wbqc-violation-message-target-required-claim'))
	158	+ ->withEntityId($targetEntityId, Role::SUBJECT)
	159	+ ->withEntityId($propertyId, Role::PREDICATE)
	160	+ ->withItemIdSnakValueList($items, Role::OBJECT);
161	161	}
162	162
163		- return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
164		- ->withMetadata( Metadata::ofDependencyMetadata(
165		- DependencyMetadata::ofEntityId( $targetEntityId ) ) );
	163	+ return (new CheckResult($context, $constraint, $parameters, $status, $message))
	164	+ ->withMetadata(Metadata::ofDependencyMetadata(
	165	+ DependencyMetadata::ofEntityId($targetEntityId) ));
166	166	}
167	167
168		- public function checkConstraintParameters( Constraint $constraint ) {
	168	+ public function checkConstraintParameters(Constraint $constraint) {
169	169	$constraintParameters = $constraint->getConstraintParameters();
170	170	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
171	171	$exceptions = [];
		@@ -174,7 +174,7 @@ discard block
		block discarded – undo
174	174	$constraintParameters,
175	175	$constraintTypeItemId
176	176	);
177		- } catch ( ConstraintParameterException $e ) {
	177	+ } catch (ConstraintParameterException $e) {
178	178	$exceptions[] = $e;
179	179	}
180	180	try {
		@@ -183,7 +183,7 @@ discard block
		block discarded – undo
183	183	$constraintTypeItemId,
184	184	false
185	185	);
186		- } catch ( ConstraintParameterException $e ) {
	186	+ } catch (ConstraintParameterException $e) {
187	187	$exceptions[] = $e;
188	188	}
189	189	return $exceptions;

		@@ -62,9 +62,9 @@ discard block
		block discarded – undo
62	62	* @throws ConstraintParameterException
63	63	* @return CheckResult
64	64	*/
65		- public function checkConstraint( Context $context, Constraint $constraint ) {
66		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
67		- return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
	65	+ public function checkConstraint(Context $context, Constraint $constraint) {
	66	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	67	+ return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
68	68	}
69	69
70	70	$parameters = [];
		@@ -81,33 +81,33 @@ discard block
		block discarded – undo
81	81	$status = CheckResult::STATUS_COMPLIANCE;
82	82
83	83	/** @var Snak $qualifier */
84		- foreach ( $context->getSnakStatement()->getQualifiers() as $qualifier ) {
	84	+ foreach ($context->getSnakStatement()->getQualifiers() as $qualifier) {
85	85	$allowedQualifier = false;
86		- foreach ( $properties as $property ) {
87		- if ( $qualifier->getPropertyId()->equals( $property ) ) {
	86	+ foreach ($properties as $property) {
	87	+ if ($qualifier->getPropertyId()->equals($property)) {
88	88	$allowedQualifier = true;
89	89	break;
90	90	}
91	91	}
92		- if ( !$allowedQualifier ) {
93		- if ( empty( $properties ) \|\| $properties === [ '' ] ) {
94		- $message = ( new ViolationMessage( 'wbqc-violation-message-no-qualifiers' ) )
95		- ->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY );
	92	+ if (!$allowedQualifier) {
	93	+ if (empty($properties) \|\| $properties === ['']) {
	94	+ $message = (new ViolationMessage('wbqc-violation-message-no-qualifiers'))
	95	+ ->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY);
96	96	} else {
97		- $message = ( new ViolationMessage( 'wbqc-violation-message-qualifiers' ) )
98		- ->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY )
99		- ->withEntityId( $qualifier->getPropertyId(), Role::QUALIFIER_PREDICATE )
100		- ->withEntityIdList( $properties, Role::QUALIFIER_PREDICATE );
	97	+ $message = (new ViolationMessage('wbqc-violation-message-qualifiers'))
	98	+ ->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY)
	99	+ ->withEntityId($qualifier->getPropertyId(), Role::QUALIFIER_PREDICATE)
	100	+ ->withEntityIdList($properties, Role::QUALIFIER_PREDICATE);
101	101	}
102	102	$status = CheckResult::STATUS_VIOLATION;
103	103	break;
104	104	}
105	105	}
106	106
107		- return new CheckResult( $context, $constraint, $parameters, $status, $message );
	107	+ return new CheckResult($context, $constraint, $parameters, $status, $message);
108	108	}
109	109
110		- public function checkConstraintParameters( Constraint $constraint ) {
	110	+ public function checkConstraintParameters(Constraint $constraint) {
111	111	$constraintParameters = $constraint->getConstraintParameters();
112	112	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
113	113	$exceptions = [];
		@@ -116,7 +116,7 @@ discard block
		block discarded – undo
116	116	$constraintParameters,
117	117	$constraintTypeItemId
118	118	);
119		- } catch ( ConstraintParameterException $e ) {
	119	+ } catch (ConstraintParameterException $e) {
120	120	$exceptions[] = $e;
121	121	}
122	122	return $exceptions;

		@@ -60,9 +60,9 @@ discard block
		block discarded – undo
60	60	* @throws ConstraintParameterException
61	61	* @return CheckResult
62	62	*/
63		- public function checkConstraint( Context $context, Constraint $constraint ) {
64		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
65		- return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
	63	+ public function checkConstraint(Context $context, Constraint $constraint) {
	64	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	65	+ return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
66	66	}
67	67
68	68	$parameters = [];
		@@ -78,23 +78,23 @@ discard block
		block discarded – undo
78	78
79	79	$snak = $context->getSnak();
80	80
81		- $message = ( new ViolationMessage( 'wbqc-violation-message-one-of' ) )
82		- ->withEntityId( $context->getSnak()->getPropertyId(), Role::PREDICATE )
83		- ->withItemIdSnakValueList( $items, Role::OBJECT );
	81	+ $message = (new ViolationMessage('wbqc-violation-message-one-of'))
	82	+ ->withEntityId($context->getSnak()->getPropertyId(), Role::PREDICATE)
	83	+ ->withItemIdSnakValueList($items, Role::OBJECT);
84	84	$status = CheckResult::STATUS_VIOLATION;
85	85
86		- foreach ( $items as $item ) {
87		- if ( $item->matchesSnak( $snak ) ) {
	86	+ foreach ($items as $item) {
	87	+ if ($item->matchesSnak($snak)) {
88	88	$message = null;
89	89	$status = CheckResult::STATUS_COMPLIANCE;
90	90	break;
91	91	}
92	92	}
93	93
94		- return new CheckResult( $context, $constraint, $parameters, $status, $message );
	94	+ return new CheckResult($context, $constraint, $parameters, $status, $message);
95	95	}
96	96
97		- public function checkConstraintParameters( Constraint $constraint ) {
	97	+ public function checkConstraintParameters(Constraint $constraint) {
98	98	$constraintParameters = $constraint->getConstraintParameters();
99	99	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
100	100	$exceptions = [];
		@@ -104,7 +104,7 @@ discard block
		block discarded – undo
104	104	$constraintTypeItemId,
105	105	true
106	106	);
107		- } catch ( ConstraintParameterException $e ) {
	107	+ } catch (ConstraintParameterException $e) {
108	108	$exceptions[] = $e;
109	109	}
110	110	return $exceptions;

		@@ -38,9 +38,9 @@ discard block
		block discarded – undo
38	38	*
39	39	* @throws InvalidArgumentException
40	40	*/
41		- public function __construct( $content, array $attributes = [], $isRawContent = false ) {
42		- Assert::parameterType( 'string', $content, '$content' );
43		- Assert::parameterType( 'boolean', $isRawContent, '$isRawContent' );
	41	+ public function __construct($content, array $attributes = [], $isRawContent = false) {
	42	+ Assert::parameterType('string', $content, '$content');
	43	+ Assert::parameterType('boolean', $isRawContent, '$isRawContent');
44	44
45	45	$this->content = $content;
46	46	$this->attributes = $attributes;
		@@ -66,10 +66,10 @@ discard block
		block discarded – undo
66	66	* @suppress SecurityCheck-DoubleEscaped
67	67	*/
68	68	public function toHtml() {
69		- if ( $this->isRawContent ) {
70		- return Html::rawElement( 'td', $this->getAttributes(), $this->content );
	69	+ if ($this->isRawContent) {
	70	+ return Html::rawElement('td', $this->getAttributes(), $this->content);
71	71	} else {
72		- return Html::element( 'td', $this->getAttributes(), $this->content );
	72	+ return Html::element('td', $this->getAttributes(), $this->content);
73	73	}
74	74	}
75	75

		@@ -41,10 +41,10 @@ discard block
		block discarded – undo
41	41	*
42	42	* @throws InvalidArgumentException
43	43	*/
44		- public function __construct( $content, $isSortable = false, $isRawContent = false ) {
45		- Assert::parameterType( 'string', $content, '$content' );
46		- Assert::parameterType( 'boolean', $isSortable, '$isSortable' );
47		- Assert::parameterType( 'boolean', $isRawContent, '$isRawContent' );
	44	+ public function __construct($content, $isSortable = false, $isRawContent = false) {
	45	+ Assert::parameterType('string', $content, '$content');
	46	+ Assert::parameterType('boolean', $isSortable, '$isSortable');
	47	+ Assert::parameterType('boolean', $isRawContent, '$isRawContent');
48	48
49	49	$this->content = $content;
50	50	$this->isSortable = $isSortable;
		@@ -71,20 +71,20 @@ discard block
		block discarded – undo
71	71	* @return string HTML
72	72	*/
73	73	public function toHtml() {
74		- $attributes = [ 'role' => 'columnheader button' ];
	74	+ $attributes = ['role' => 'columnheader button'];
75	75
76		- if ( !$this->isSortable ) {
	76	+ if (!$this->isSortable) {
77	77	$attributes['class'] = 'unsortable';
78	78	}
79	79
80		- if ( !$this->isRawContent ) {
	80	+ if (!$this->isRawContent) {
81	81	// @phan-suppress-next-line SecurityCheck-DoubleEscaped
82		- $content = htmlspecialchars( $this->content );
	82	+ $content = htmlspecialchars($this->content);
83	83	} else {
84	84	$content = $this->content;
85	85	}
86	86
87		- return Html::rawElement( 'th', $attributes, $content );
	87	+ return Html::rawElement('th', $attributes, $content);
88	88	}
89	89
90	90	}

		@@ -74,9 +74,9 @@ discard block
		block discarded – undo
74	74	*
75	75	* @return CheckResult
76	76	*/
77		- public function checkConstraint( Context $context, Constraint $constraint ) {
78		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
79		- return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
	77	+ public function checkConstraint(Context $context, Constraint $constraint) {
	78	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	79	+ return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
80	80	}
81	81
82	82	$parameters = [];
		@@ -84,9 +84,9 @@ discard block
		block discarded – undo
84	84	$snak = $context->getSnak();
85	85	$propertyId = $context->getSnak()->getPropertyId();
86	86
87		- if ( !$snak instanceof PropertyValueSnak ) {
	87	+ if (!$snak instanceof PropertyValueSnak) {
88	88	// nothing to check
89		- return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE );
	89	+ return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE);
90	90	}
91	91
92	92	$dataValue = $snak->getDataValue();
		@@ -95,18 +95,18 @@ discard block
		block discarded – undo
95	95	* error handling:
96	96	* type of $dataValue for properties with 'Symmetric' constraint has to be 'wikibase-entityid'
97	97	*/
98		- if ( !$dataValue instanceof EntityIdValue ) {
99		- $message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
100		- ->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM )
101		- ->withDataValueType( 'wikibase-entityid' );
102		- return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
	98	+ if (!$dataValue instanceof EntityIdValue) {
	99	+ $message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
	100	+ ->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM)
	101	+ ->withDataValueType('wikibase-entityid');
	102	+ return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
103	103	}
104	104
105	105	$targetEntityId = $dataValue->getEntityId();
106		- $targetEntity = $this->entityLookup->getEntity( $targetEntityId );
107		- if ( !$targetEntity instanceof StatementListProvider ) {
108		- $message = new ViolationMessage( 'wbqc-violation-message-target-entity-must-exist' );
109		- return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
	106	+ $targetEntity = $this->entityLookup->getEntity($targetEntityId);
	107	+ if (!$targetEntity instanceof StatementListProvider) {
	108	+ $message = new ViolationMessage('wbqc-violation-message-target-entity-must-exist');
	109	+ return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
110	110	}
111	111
112	112	$symmetricStatement = $this->connectionCheckerHelper->findStatementWithPropertyAndEntityIdValue(
		@@ -114,23 +114,23 @@ discard block
		block discarded – undo
114	114	$propertyId,
115	115	$context->getEntity()->getId()
116	116	);
117		- if ( $symmetricStatement !== null ) {
	117	+ if ($symmetricStatement !== null) {
118	118	$message = null;
119	119	$status = CheckResult::STATUS_COMPLIANCE;
120	120	} else {
121		- $message = ( new ViolationMessage( 'wbqc-violation-message-symmetric' ) )
122		- ->withEntityId( $targetEntityId, Role::SUBJECT )
123		- ->withEntityId( $propertyId, Role::PREDICATE )
124		- ->withEntityId( $context->getEntity()->getId(), Role::OBJECT );
	121	+ $message = (new ViolationMessage('wbqc-violation-message-symmetric'))
	122	+ ->withEntityId($targetEntityId, Role::SUBJECT)
	123	+ ->withEntityId($propertyId, Role::PREDICATE)
	124	+ ->withEntityId($context->getEntity()->getId(), Role::OBJECT);
125	125	$status = CheckResult::STATUS_VIOLATION;
126	126	}
127	127
128		- return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
129		- ->withMetadata( Metadata::ofDependencyMetadata(
130		- DependencyMetadata::ofEntityId( $targetEntityId ) ) );
	128	+ return (new CheckResult($context, $constraint, $parameters, $status, $message))
	129	+ ->withMetadata(Metadata::ofDependencyMetadata(
	130	+ DependencyMetadata::ofEntityId($targetEntityId) ));
131	131	}
132	132
133		- public function checkConstraintParameters( Constraint $constraint ) {
	133	+ public function checkConstraintParameters(Constraint $constraint) {
134	134	// no parameters
135	135	return [];
136	136	}

		@@ -65,24 +65,24 @@ discard block
		block discarded – undo
65	65	* @throws SparqlHelperException if the checker uses SPARQL and the query times out or some other error occurs
66	66	* @return CheckResult
67	67	*/
68		- public function checkConstraint( Context $context, Constraint $constraint ) {
69		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
70		- return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
	68	+ public function checkConstraint(Context $context, Constraint $constraint) {
	69	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	70	+ return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
71	71	}
72	72
73	73	$parameters = [];
74	74
75		- if ( !( $this->sparqlHelper instanceof DummySparqlHelper ) ) {
76		- if ( $context->getType() === 'statement' ) {
	75	+ if (!($this->sparqlHelper instanceof DummySparqlHelper)) {
	76	+ if ($context->getType() === 'statement') {
77	77	$result = $this->sparqlHelper->findEntitiesWithSameStatement(
78	78	$context->getSnakStatement(),
79	79	true // ignore deprecated statements
80	80	);
81	81	} else {
82	82	$snak = $context->getSnak();
83		- if ( !$snak instanceof PropertyValueSnak ) {
	83	+ if (!$snak instanceof PropertyValueSnak) {
84	84	// nothing to check
85		- return new CheckResult( $context, $constraint, [], CheckResult::STATUS_COMPLIANCE );
	85	+ return new CheckResult($context, $constraint, [], CheckResult::STATUS_COMPLIANCE);
86	86	}
87	87	$result = $this->sparqlHelper->findEntitiesWithSameQualifierOrReference(
88	88	$context->getEntity()->getId(),
		@@ -95,27 +95,27 @@ discard block
		block discarded – undo
95	95	$otherEntities = $result->getArray();
96	96	$metadata = $result->getMetadata();
97	97
98		- if ( $otherEntities === [] ) {
	98	+ if ($otherEntities === []) {
99	99	$status = CheckResult::STATUS_COMPLIANCE;
100	100	$message = null;
101	101	} else {
102		- $otherEntities = array_values( array_filter( $otherEntities ) ); // remove nulls
	102	+ $otherEntities = array_values(array_filter($otherEntities)); // remove nulls
103	103	$status = CheckResult::STATUS_VIOLATION;
104		- $message = ( new ViolationMessage( 'wbqc-violation-message-unique-value' ) )
105		- ->withEntityIdList( $otherEntities, Role::SUBJECT );
	104	+ $message = (new ViolationMessage('wbqc-violation-message-unique-value'))
	105	+ ->withEntityIdList($otherEntities, Role::SUBJECT);
106	106	}
107	107	} else {
108	108	$status = CheckResult::STATUS_TODO;
109		- $message = ( new ViolationMessage( 'wbqc-violation-message-not-yet-implemented' ) )
110		- ->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM );
	109	+ $message = (new ViolationMessage('wbqc-violation-message-not-yet-implemented'))
	110	+ ->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM);
111	111	$metadata = Metadata::blank();
112	112	}
113	113
114		- return ( new CheckResult( $context, $constraint, $parameters, $status, $message ) )
115		- ->withMetadata( $metadata );
	114	+ return (new CheckResult($context, $constraint, $parameters, $status, $message))
	115	+ ->withMetadata($metadata);
116	116	}
117	117
118		- public function checkConstraintParameters( Constraint $constraint ) {
	118	+ public function checkConstraintParameters(Constraint $constraint) {
119	119	// no parameters
120	120	return [];
121	121	}

wikimedia / mediawiki-extensions-WikibaseQualityConstraints

Push — master ( bae75b...eee08f )

Status

Category

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +23 added lines, -23 removed lines patch added patch discarded remove patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch

Spacing +25 added lines, -25 removed lines patch added patch discarded remove patch

Spacing +27 added lines, -27 removed lines patch added patch discarded remove patch