Inspection of "build: Updating mediawiki/mediawiki-codesniffer to..." - wikimedia/mediawiki-extensions-WikibaseQualityConstraints - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 12d883...691e21 )

unknown

created 2025-02-01 09:47 UTC

Status

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

@@ -36,17 +36,17 @@  discard block
 block discarded – undo
 		return self::ALL_ENTITY_TYPES_SUPPORTED;
 	}
 
-	public function checkConstraint( Context $context, Constraint $constraint ) {
+	public function checkConstraint(Context $context, Constraint $constraint) {
 		$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);
 		}
 
-		if ( $snak->getDataValue() instanceof QuantityValue ) {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-noBounds' ) )
-				->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY );
+		if ($snak->getDataValue() instanceof QuantityValue) {
+			$message = (new ViolationMessage('wbqc-violation-message-noBounds'))
+				->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY);
 			return new CheckResult(
 				$context,
 				$constraint,
@@ -55,10 +55,10 @@  discard block
 block discarded – undo
 			);
 		}
 
-		return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
+		return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		// no parameters
 		return [];
 	}

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/MandatoryQualifiersChecker.php 1 patch

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

@@ -67,9 +67,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);
 		}
 
 		$constraintParameters = $constraint->getConstraintParameters();
@@ -80,24 +80,24 @@  discard block
 block discarded – undo
 			$constraintTypeItemId
 		);
 
-		$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, $status, $message );
+		return new CheckResult($context, $constraint, $status, $message);
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -106,7 +106,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/IntegerChecker.php 1 patch

Spacing +21 added lines, -22 removed lines patch added patch discarded remove patch

@@ -38,22 +38,21 @@  discard block
 block discarded – undo
 		return self::ALL_ENTITY_TYPES_SUPPORTED;
 	}
 
-	public function checkConstraint( Context $context, Constraint $constraint ) {
+	public function checkConstraint(Context $context, Constraint $constraint) {
 		$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);
 		}
 
-		$violationMessage = $this->checkSnak( $snak );
+		$violationMessage = $this->checkSnak($snak);
 
 		return new CheckResult(
 			$context,
 			$constraint,
 			$violationMessage === null ?
-				CheckResult::STATUS_COMPLIANCE :
-				CheckResult::STATUS_VIOLATION,
+				CheckResult::STATUS_COMPLIANCE : CheckResult::STATUS_VIOLATION,
 			$violationMessage
 		);
 	}
@@ -62,23 +61,23 @@  discard block
 block discarded – undo
 	 * @param PropertyValueSnak $snak
 	 * @return ViolationMessage|null
 	 */
-	public function checkSnak( PropertyValueSnak $snak ) {
+	public function checkSnak(PropertyValueSnak $snak) {
 		$dataValue = $snak->getDataValue();
 
-		if ( $dataValue instanceof DecimalValue ) {
-			if ( !$this->isInteger( $dataValue ) ) {
-				return $this->getViolationMessage( 'wbqc-violation-message-integer', $snak );
+		if ($dataValue instanceof DecimalValue) {
+			if (!$this->isInteger($dataValue)) {
+				return $this->getViolationMessage('wbqc-violation-message-integer', $snak);
 			}
-		} elseif ( $dataValue instanceof UnboundedQuantityValue ) {
-			if ( !$this->isInteger( $dataValue->getAmount() ) ) {
-				return $this->getViolationMessage( 'wbqc-violation-message-integer', $snak );
+		} elseif ($dataValue instanceof UnboundedQuantityValue) {
+			if (!$this->isInteger($dataValue->getAmount())) {
+				return $this->getViolationMessage('wbqc-violation-message-integer', $snak);
 			} elseif (
 				$dataValue instanceof QuantityValue && (
-					!$this->isInteger( $dataValue->getLowerBound() ) ||
-					!$this->isInteger( $dataValue->getUpperBound() )
+					!$this->isInteger($dataValue->getLowerBound()) ||
+					!$this->isInteger($dataValue->getUpperBound())
 				)
 			) {
-				return $this->getViolationMessage( 'wbqc-violation-message-integer-bounds', $snak );
+				return $this->getViolationMessage('wbqc-violation-message-integer-bounds', $snak);
 			}
 		}
 
@@ -89,7 +88,7 @@  discard block
 block discarded – undo
 	 * @param DecimalValue $decimalValue
 	 * @return bool
 	 */
-	private function isInteger( DecimalValue $decimalValue ) {
+	private function isInteger(DecimalValue $decimalValue) {
 		return $decimalValue->getTrimmed()->getFractionalPart() === '';
 	}
 
@@ -98,13 +97,13 @@  discard block
 block discarded – undo
 	 * @param PropertyValueSnak $snak
 	 * @return ViolationMessage
 	 */
-	private function getViolationMessage( $messageKey, PropertyValueSnak $snak ) {
-		return ( new ViolationMessage( $messageKey ) )
-			->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY )
-			->withDataValue( $snak->getDataValue() );
+	private function getViolationMessage($messageKey, PropertyValueSnak $snak) {
+		return (new ViolationMessage($messageKey))
+			->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY)
+			->withDataValue($snak->getDataValue());
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		// no parameters
 		return [];
 	}

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/AllowedUnitsChecker.php 1 patch

Spacing +34 added lines, -35 removed lines patch added patch discarded remove patch

@@ -72,7 +72,7 @@  discard block
 block discarded – undo
 	 * @throws ConstraintParameterException
 	 * @return CheckResult
 	 */
-	public function checkConstraint( Context $context, Constraint $constraint ) {
+	public function checkConstraint(Context $context, Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$unitsParameter = $this->constraintParameterParser
 			->parseUnitsParameter(
@@ -81,50 +81,49 @@  discard block
 block discarded – undo
 			);
 
 		$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);
 		}
 
 		$dataValue = $snak->getDataValue();
-		if ( !$dataValue instanceof UnboundedQuantityValue ) {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
-				->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM )
-				->withDataValueType( 'quantity' );
-			return new CheckResult( $context, $constraint, CheckResult::STATUS_VIOLATION, $message );
+		if (!$dataValue instanceof UnboundedQuantityValue) {
+			$message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
+				->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM)
+				->withDataValueType('quantity');
+			return new CheckResult($context, $constraint, CheckResult::STATUS_VIOLATION, $message);
 		}
 
-		if ( $dataValue->getUnit() === '1' ) {
-			return $this->checkUnitless( $context, $constraint, $unitsParameter, $snak );
+		if ($dataValue->getUnit() === '1') {
+			return $this->checkUnitless($context, $constraint, $unitsParameter, $snak);
 		}
 
 		$status = CheckResult::STATUS_VIOLATION;
-		$actualUnit = $this->standardize( $dataValue )->getUnit();
-		foreach ( $unitsParameter->getUnitQuantities() as $unitQuantity ) {
-			$allowedUnit = $this->standardize( $unitQuantity )->getUnit();
-			if ( $actualUnit === $allowedUnit ) {
+		$actualUnit = $this->standardize($dataValue)->getUnit();
+		foreach ($unitsParameter->getUnitQuantities() as $unitQuantity) {
+			$allowedUnit = $this->standardize($unitQuantity)->getUnit();
+			if ($actualUnit === $allowedUnit) {
 				$status = CheckResult::STATUS_COMPLIANCE;
 				break;
 			}
 		}
 
-		if ( $status === CheckResult::STATUS_VIOLATION ) {
-			if ( $unitsParameter->getUnitItemIds() === [] ) {
-				$message = ( new ViolationMessage( 'wbqc-violation-message-units-none' ) )
-					->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY );
+		if ($status === CheckResult::STATUS_VIOLATION) {
+			if ($unitsParameter->getUnitItemIds() === []) {
+				$message = (new ViolationMessage('wbqc-violation-message-units-none'))
+					->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY);
 			} else {
 				$messageKey = $unitsParameter->getUnitlessAllowed() ?
-					'wbqc-violation-message-units-or-none' :
-					'wbqc-violation-message-units';
-				$message = ( new ViolationMessage( $messageKey ) )
-					->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY )
-					->withEntityIdList( $unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE );
+					'wbqc-violation-message-units-or-none' : 'wbqc-violation-message-units';
+				$message = (new ViolationMessage($messageKey))
+					->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY)
+					->withEntityIdList($unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE);
 			}
 		} else {
 			$message = null;
 		}
 
-		return new CheckResult( $context, $constraint, $status, $message );
+		return new CheckResult($context, $constraint, $status, $message);
 	}
 
 	/**
@@ -140,17 +139,17 @@  discard block
 block discarded – undo
 		UnitsParameter $unitsParameter,
 		PropertyValueSnak $snak
 	) {
-		if ( $unitsParameter->getUnitlessAllowed() ) {
+		if ($unitsParameter->getUnitlessAllowed()) {
 			$message = null;
 			$status = CheckResult::STATUS_COMPLIANCE;
 		} else {
-			$message = ( new ViolationMessage( 'wbqc-violation-message-units' ) )
-				->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY )
-				->withEntityIdList( $unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE );
+			$message = (new ViolationMessage('wbqc-violation-message-units'))
+				->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY)
+				->withEntityIdList($unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE);
 			$status = CheckResult::STATUS_VIOLATION;
 		}
 
-		return new CheckResult( $context, $constraint, $status, $message );
+		return new CheckResult($context, $constraint, $status, $message);
 	}
 
 	/**
@@ -159,20 +158,20 @@  discard block
 block discarded – undo
 	 * @param UnboundedQuantityValue $value
 	 * @return UnboundedQuantityValue
 	 */
-	private function standardize( UnboundedQuantityValue $value ) {
-		if ( $this->unitConverter === null ) {
+	private function standardize(UnboundedQuantityValue $value) {
+		if ($this->unitConverter === null) {
 			return $value;
 		}
 
-		$standard = $this->unitConverter->toStandardUnits( $value );
-		if ( $standard !== null ) {
+		$standard = $this->unitConverter->toStandardUnits($value);
+		if ($standard !== null) {
 			return $standard;
 		} else {
 			return $value;
 		}
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$exceptions = [];
 		try {
@@ -181,7 +180,7 @@  discard block
 block discarded – undo
 				$constraint->getConstraintTypeItemId(),
 				true
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/ItemChecker.php 1 patch

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

@@ -75,9 +75,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);
 		}
 
 		$constraintParameters = $constraint->getConstraintParameters();
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
 		 *   a) a property only
 		 *   b) a property and a number of items (each combination of property and item forming an individual claim)
 		 */
-		if ( $items === [] ) {
+		if ($items === []) {
 			$requiredStatement = $this->connectionCheckerHelper->findStatementWithProperty(
 				$context->getEntity()->getStatements(),
 				$propertyId
@@ -112,21 +112,21 @@  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-item' ) )
-				->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY )
-				->withEntityId( $propertyId, Role::PREDICATE )
-				->withItemIdSnakValueList( $items, Role::OBJECT );
+			$message = (new ViolationMessage('wbqc-violation-message-item'))
+				->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY)
+				->withEntityId($propertyId, Role::PREDICATE)
+				->withItemIdSnakValueList($items, Role::OBJECT);
 		}
 
-		return new CheckResult( $context, $constraint, $status, $message );
+		return new CheckResult($context, $constraint, $status, $message);
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
 				$constraintParameters,
 				$constraintTypeItemId
 			);
-		} catch ( ConstraintParameterException $e ) {
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		try {
@@ -144,7 +144,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.

src/ConstraintCheck/Checker/QualifierChecker.php 1 patch

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

@@ -44,16 +44,16 @@
 block discarded – undo
 	 *
 	 * @return CheckResult
 	 */
-	public function checkConstraint( Context $context, Constraint $constraint ) {
-		if ( $context->getType() === Context::TYPE_QUALIFIER ) {
-			return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
+	public function checkConstraint(Context $context, Constraint $constraint) {
+		if ($context->getType() === Context::TYPE_QUALIFIER) {
+			return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
 		} else {
-			$message = new ViolationMessage( 'wbqc-violation-message-qualifier' );
-			return new CheckResult( $context, $constraint, CheckResult::STATUS_VIOLATION, $message );
+			$message = new ViolationMessage('wbqc-violation-message-qualifier');
+			return new CheckResult($context, $constraint, CheckResult::STATUS_VIOLATION, $message);
 		}
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		// no parameters
 		return [];
 	}

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/PropertyScopeChecker.php 1 patch

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

@@ -46,7 +46,7 @@  discard block
 block discarded – undo
 		return self::ALL_ENTITY_TYPES_SUPPORTED;
 	}
 
-	public function checkConstraint( Context $context, Constraint $constraint ) {
+	public function checkConstraint(Context $context, Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
 			$constraintTypeItemId
 		);
 
-		if ( in_array( $context->getType(), $allowedContextTypes ) ) {
+		if (in_array($context->getType(), $allowedContextTypes)) {
 			return new CheckResult(
 				$context->getCursor(),
 				$constraint,
@@ -66,15 +66,15 @@  discard block
 block discarded – undo
 				$context->getCursor(),
 				$constraint,
 				CheckResult::STATUS_VIOLATION,
-				( new ViolationMessage( 'wbqc-violation-message-property-scope' ) )
-					->withEntityId( $context->getSnak()->getPropertyId() )
-					->withPropertyScope( $context->getType() )
-					->withPropertyScopeList( $allowedContextTypes )
+				(new ViolationMessage('wbqc-violation-message-property-scope'))
+					->withEntityId($context->getSnak()->getPropertyId())
+					->withPropertyScope($context->getType())
+					->withPropertyScopeList($allowedContextTypes)
 			);
 		}
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 		$exceptions = [];
@@ -83,7 +83,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/SingleValueChecker.php 1 patch

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

@@ -64,9 +64,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);
 		}
 
 		$separators = $this->constraintParameterParser->parseSeparatorsParameter(
@@ -75,19 +75,19 @@  discard block
 block discarded – undo
 
 		$propertyId = $context->getSnak()->getPropertyId();
 		$propertyCount = $this->valueCountCheckerHelper->getPropertyCount(
-			$context->getSnakGroup( Context::GROUP_NON_DEPRECATED, $separators ),
+			$context->getSnakGroup(Context::GROUP_NON_DEPRECATED, $separators),
 			$propertyId
 		);
 
-		if ( $propertyCount > 1 ) {
-			$message = $this->getViolationMessage( $separators, $propertyId );
+		if ($propertyCount > 1) {
+			$message = $this->getViolationMessage($separators, $propertyId);
 			$status = CheckResult::STATUS_VIOLATION;
 		} else {
 			$message = null;
 			$status = CheckResult::STATUS_COMPLIANCE;
 		}
 
-		return new CheckResult( $context, $constraint, $status, $message );
+		return new CheckResult($context, $constraint, $status, $message);
 	}
 
 	/**
@@ -95,22 +95,21 @@  discard block
 block discarded – undo
 	 * @param PropertyId $propertyId
 	 * @return ViolationMessage
 	 */
-	private function getViolationMessage( array $separators, PropertyId $propertyId ) {
+	private function getViolationMessage(array $separators, PropertyId $propertyId) {
 		$messageKey = $separators === [] ?
-			'wbqc-violation-message-single-value' :
-			'wbqc-violation-message-single-value-separators';
+			'wbqc-violation-message-single-value' : 'wbqc-violation-message-single-value-separators';
 
-		return ( new ViolationMessage( $messageKey ) )
-			->withEntityId( $propertyId )
-			->withEntityIdList( $separators );
+		return (new ViolationMessage($messageKey))
+			->withEntityId($propertyId)
+			->withEntityIdList($separators);
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		$constraintParameters = $constraint->getConstraintParameters();
 		$exceptions = [];
 		try {
-			$this->constraintParameterParser->parseSeparatorsParameter( $constraintParameters );
-		} catch ( ConstraintParameterException $e ) {
+			$this->constraintParameterParser->parseSeparatorsParameter($constraintParameters);
+		} catch (ConstraintParameterException $e) {
 			$exceptions[] = $e;
 		}
 		return $exceptions;

Please login to merge, or discard this patch.

src/ConstraintCheck/Checker/ReferenceChecker.php 1 patch

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

@@ -33,16 +33,16 @@
 block discarded – undo
 		return self::ALL_ENTITY_TYPES_SUPPORTED;
 	}
 
-	public function checkConstraint( Context $context, Constraint $constraint ) {
-		if ( $context->getType() === Context::TYPE_REFERENCE ) {
-			return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
+	public function checkConstraint(Context $context, Constraint $constraint) {
+		if ($context->getType() === Context::TYPE_REFERENCE) {
+			return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
 		} else {
-			$message = new ViolationMessage( 'wbqc-violation-message-reference' );
-			return new CheckResult( $context, $constraint, CheckResult::STATUS_VIOLATION, $message );
+			$message = new ViolationMessage('wbqc-violation-message-reference');
+			return new CheckResult($context, $constraint, CheckResult::STATUS_VIOLATION, $message);
 		}
 	}
 
-	public function checkConstraintParameters( Constraint $constraint ) {
+	public function checkConstraintParameters(Constraint $constraint) {
 		// no parameters
 		return [];
 	}

Please login to merge, or discard this patch.

		@@ -72,7 +72,7 @@ discard block
		block discarded – undo
72	72	* @throws ConstraintParameterException
73	73	* @return CheckResult
74	74	*/
75		- public function checkConstraint( Context $context, Constraint $constraint ) {
	75	+ public function checkConstraint(Context $context, Constraint $constraint) {
76	76	$constraintParameters = $constraint->getConstraintParameters();
77	77	$unitsParameter = $this->constraintParameterParser
78	78	->parseUnitsParameter(
		@@ -81,50 +81,49 @@ discard block
		block discarded – undo
81	81	);
82	82
83	83	$snak = $context->getSnak();
84		- if ( !$snak instanceof PropertyValueSnak ) {
	84	+ if (!$snak instanceof PropertyValueSnak) {
85	85	// nothing to check
86		- return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
	86	+ return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
87	87	}
88	88
89	89	$dataValue = $snak->getDataValue();
90		- if ( !$dataValue instanceof UnboundedQuantityValue ) {
91		- $message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) )
92		- ->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM )
93		- ->withDataValueType( 'quantity' );
94		- return new CheckResult( $context, $constraint, CheckResult::STATUS_VIOLATION, $message );
	90	+ if (!$dataValue instanceof UnboundedQuantityValue) {
	91	+ $message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type'))
	92	+ ->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM)
	93	+ ->withDataValueType('quantity');
	94	+ return new CheckResult($context, $constraint, CheckResult::STATUS_VIOLATION, $message);
95	95	}
96	96
97		- if ( $dataValue->getUnit() === '1' ) {
98		- return $this->checkUnitless( $context, $constraint, $unitsParameter, $snak );
	97	+ if ($dataValue->getUnit() === '1') {
	98	+ return $this->checkUnitless($context, $constraint, $unitsParameter, $snak);
99	99	}
100	100
101	101	$status = CheckResult::STATUS_VIOLATION;
102		- $actualUnit = $this->standardize( $dataValue )->getUnit();
103		- foreach ( $unitsParameter->getUnitQuantities() as $unitQuantity ) {
104		- $allowedUnit = $this->standardize( $unitQuantity )->getUnit();
105		- if ( $actualUnit === $allowedUnit ) {
	102	+ $actualUnit = $this->standardize($dataValue)->getUnit();
	103	+ foreach ($unitsParameter->getUnitQuantities() as $unitQuantity) {
	104	+ $allowedUnit = $this->standardize($unitQuantity)->getUnit();
	105	+ if ($actualUnit === $allowedUnit) {
106	106	$status = CheckResult::STATUS_COMPLIANCE;
107	107	break;
108	108	}
109	109	}
110	110
111		- if ( $status === CheckResult::STATUS_VIOLATION ) {
112		- if ( $unitsParameter->getUnitItemIds() === [] ) {
113		- $message = ( new ViolationMessage( 'wbqc-violation-message-units-none' ) )
114		- ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY );
	111	+ if ($status === CheckResult::STATUS_VIOLATION) {
	112	+ if ($unitsParameter->getUnitItemIds() === []) {
	113	+ $message = (new ViolationMessage('wbqc-violation-message-units-none'))
	114	+ ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY);
115	115	} else {
116	116	$messageKey = $unitsParameter->getUnitlessAllowed() ?
117		- 'wbqc-violation-message-units-or-none' :
118		- 'wbqc-violation-message-units';
119		- $message = ( new ViolationMessage( $messageKey ) )
120		- ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY )
121		- ->withEntityIdList( $unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE );
	117	+ 'wbqc-violation-message-units-or-none' : 'wbqc-violation-message-units';
	118	+ $message = (new ViolationMessage($messageKey))
	119	+ ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY)
	120	+ ->withEntityIdList($unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE);
122	121	}
123	122	} else {
124	123	$message = null;
125	124	}
126	125
127		- return new CheckResult( $context, $constraint, $status, $message );
	126	+ return new CheckResult($context, $constraint, $status, $message);
128	127	}
129	128
130	129	/**
		@@ -140,17 +139,17 @@ discard block
		block discarded – undo
140	139	UnitsParameter $unitsParameter,
141	140	PropertyValueSnak $snak
142	141	) {
143		- if ( $unitsParameter->getUnitlessAllowed() ) {
	142	+ if ($unitsParameter->getUnitlessAllowed()) {
144	143	$message = null;
145	144	$status = CheckResult::STATUS_COMPLIANCE;
146	145	} else {
147		- $message = ( new ViolationMessage( 'wbqc-violation-message-units' ) )
148		- ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY )
149		- ->withEntityIdList( $unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE );
	146	+ $message = (new ViolationMessage('wbqc-violation-message-units'))
	147	+ ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY)
	148	+ ->withEntityIdList($unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE);
150	149	$status = CheckResult::STATUS_VIOLATION;
151	150	}
152	151
153		- return new CheckResult( $context, $constraint, $status, $message );
	152	+ return new CheckResult($context, $constraint, $status, $message);
154	153	}
155	154
156	155	/**
		@@ -159,20 +158,20 @@ discard block
		block discarded – undo
159	158	* @param UnboundedQuantityValue $value
160	159	* @return UnboundedQuantityValue
161	160	*/
162		- private function standardize( UnboundedQuantityValue $value ) {
163		- if ( $this->unitConverter === null ) {
	161	+ private function standardize(UnboundedQuantityValue $value) {
	162	+ if ($this->unitConverter === null) {
164	163	return $value;
165	164	}
166	165
167		- $standard = $this->unitConverter->toStandardUnits( $value );
168		- if ( $standard !== null ) {
	166	+ $standard = $this->unitConverter->toStandardUnits($value);
	167	+ if ($standard !== null) {
169	168	return $standard;
170	169	} else {
171	170	return $value;
172	171	}
173	172	}
174	173
175		- public function checkConstraintParameters( Constraint $constraint ) {
	174	+ public function checkConstraintParameters(Constraint $constraint) {
176	175	$constraintParameters = $constraint->getConstraintParameters();
177	176	$exceptions = [];
178	177	try {
		@@ -181,7 +180,7 @@ discard block
		block discarded – undo
181	180	$constraint->getConstraintTypeItemId(),
182	181	true
183	182	);
184		- } catch ( ConstraintParameterException $e ) {
	183	+ } catch (ConstraintParameterException $e) {
185	184	$exceptions[] = $e;
186	185	}
187	186	return $exceptions;

		@@ -36,17 +36,17 @@ discard block
		block discarded – undo
36	36	return self::ALL_ENTITY_TYPES_SUPPORTED;
37	37	}
38	38
39		- public function checkConstraint( Context $context, Constraint $constraint ) {
	39	+ public function checkConstraint(Context $context, Constraint $constraint) {
40	40	$snak = $context->getSnak();
41	41
42		- if ( !$snak instanceof PropertyValueSnak ) {
	42	+ if (!$snak instanceof PropertyValueSnak) {
43	43	// nothing to check
44		- return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
	44	+ return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
45	45	}
46	46
47		- if ( $snak->getDataValue() instanceof QuantityValue ) {
48		- $message = ( new ViolationMessage( 'wbqc-violation-message-noBounds' ) )
49		- ->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY );
	47	+ if ($snak->getDataValue() instanceof QuantityValue) {
	48	+ $message = (new ViolationMessage('wbqc-violation-message-noBounds'))
	49	+ ->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY);
50	50	return new CheckResult(
51	51	$context,
52	52	$constraint,
		@@ -55,10 +55,10 @@ discard block
		block discarded – undo
55	55	);
56	56	}
57	57
58		- return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
	58	+ return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
59	59	}
60	60
61		- public function checkConstraintParameters( Constraint $constraint ) {
	61	+ public function checkConstraintParameters(Constraint $constraint) {
62	62	// no parameters
63	63	return [];
64	64	}

		@@ -67,9 +67,9 @@ discard block
		block discarded – undo
67	67	* @throws ConstraintParameterException
68	68	* @return CheckResult
69	69	*/
70		- public function checkConstraint( Context $context, Constraint $constraint ) {
71		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
72		- return new CheckResult( $context, $constraint, CheckResult::STATUS_DEPRECATED );
	70	+ public function checkConstraint(Context $context, Constraint $constraint) {
	71	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	72	+ return new CheckResult($context, $constraint, CheckResult::STATUS_DEPRECATED);
73	73	}
74	74
75	75	$constraintParameters = $constraint->getConstraintParameters();
		@@ -80,24 +80,24 @@ discard block
		block discarded – undo
80	80	$constraintTypeItemId
81	81	);
82	82
83		- $message = ( new ViolationMessage( 'wbqc-violation-message-mandatory-qualifier' ) )
84		- ->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY )
85		- ->withEntityId( $propertyId, Role::QUALIFIER_PREDICATE );
	83	+ $message = (new ViolationMessage('wbqc-violation-message-mandatory-qualifier'))
	84	+ ->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY)
	85	+ ->withEntityId($propertyId, Role::QUALIFIER_PREDICATE);
86	86	$status = CheckResult::STATUS_VIOLATION;
87	87
88	88	/** @var Snak $qualifier */
89		- foreach ( $context->getSnakStatement()->getQualifiers() as $qualifier ) {
90		- if ( $propertyId->equals( $qualifier->getPropertyId() ) ) {
	89	+ foreach ($context->getSnakStatement()->getQualifiers() as $qualifier) {
	90	+ if ($propertyId->equals($qualifier->getPropertyId())) {
91	91	$message = null;
92	92	$status = CheckResult::STATUS_COMPLIANCE;
93	93	break;
94	94	}
95	95	}
96	96
97		- return new CheckResult( $context, $constraint, $status, $message );
	97	+ return new CheckResult($context, $constraint, $status, $message);
98	98	}
99	99
100		- public function checkConstraintParameters( Constraint $constraint ) {
	100	+ public function checkConstraintParameters(Constraint $constraint) {
101	101	$constraintParameters = $constraint->getConstraintParameters();
102	102	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
103	103	$exceptions = [];
		@@ -106,7 +106,7 @@ discard block
		block discarded – undo
106	106	$constraintParameters,
107	107	$constraintTypeItemId
108	108	);
109		- } catch ( ConstraintParameterException $e ) {
	109	+ } catch (ConstraintParameterException $e) {
110	110	$exceptions[] = $e;
111	111	}
112	112	return $exceptions;

		@@ -38,22 +38,21 @@ discard block
		block discarded – undo
38	38	return self::ALL_ENTITY_TYPES_SUPPORTED;
39	39	}
40	40
41		- public function checkConstraint( Context $context, Constraint $constraint ) {
	41	+ public function checkConstraint(Context $context, Constraint $constraint) {
42	42	$snak = $context->getSnak();
43	43
44		- if ( !$snak instanceof PropertyValueSnak ) {
	44	+ if (!$snak instanceof PropertyValueSnak) {
45	45	// nothing to check
46		- return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
	46	+ return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
47	47	}
48	48
49		- $violationMessage = $this->checkSnak( $snak );
	49	+ $violationMessage = $this->checkSnak($snak);
50	50
51	51	return new CheckResult(
52	52	$context,
53	53	$constraint,
54	54	$violationMessage === null ?
55		- CheckResult::STATUS_COMPLIANCE :
56		- CheckResult::STATUS_VIOLATION,
	55	+ CheckResult::STATUS_COMPLIANCE : CheckResult::STATUS_VIOLATION,
57	56	$violationMessage
58	57	);
59	58	}
		@@ -62,23 +61,23 @@ discard block
		block discarded – undo
62	61	* @param PropertyValueSnak $snak
63	62	* @return ViolationMessage\|null
64	63	*/
65		- public function checkSnak( PropertyValueSnak $snak ) {
	64	+ public function checkSnak(PropertyValueSnak $snak) {
66	65	$dataValue = $snak->getDataValue();
67	66
68		- if ( $dataValue instanceof DecimalValue ) {
69		- if ( !$this->isInteger( $dataValue ) ) {
70		- return $this->getViolationMessage( 'wbqc-violation-message-integer', $snak );
	67	+ if ($dataValue instanceof DecimalValue) {
	68	+ if (!$this->isInteger($dataValue)) {
	69	+ return $this->getViolationMessage('wbqc-violation-message-integer', $snak);
71	70	}
72		- } elseif ( $dataValue instanceof UnboundedQuantityValue ) {
73		- if ( !$this->isInteger( $dataValue->getAmount() ) ) {
74		- return $this->getViolationMessage( 'wbqc-violation-message-integer', $snak );
	71	+ } elseif ($dataValue instanceof UnboundedQuantityValue) {
	72	+ if (!$this->isInteger($dataValue->getAmount())) {
	73	+ return $this->getViolationMessage('wbqc-violation-message-integer', $snak);
75	74	} elseif (
76	75	$dataValue instanceof QuantityValue && (
77		- !$this->isInteger( $dataValue->getLowerBound() ) \|\|
78		- !$this->isInteger( $dataValue->getUpperBound() )
	76	+ !$this->isInteger($dataValue->getLowerBound()) \|\|
	77	+ !$this->isInteger($dataValue->getUpperBound())
79	78	)
80	79	) {
81		- return $this->getViolationMessage( 'wbqc-violation-message-integer-bounds', $snak );
	80	+ return $this->getViolationMessage('wbqc-violation-message-integer-bounds', $snak);
82	81	}
83	82	}
84	83
		@@ -89,7 +88,7 @@ discard block
		block discarded – undo
89	88	* @param DecimalValue $decimalValue
90	89	* @return bool
91	90	*/
92		- private function isInteger( DecimalValue $decimalValue ) {
	91	+ private function isInteger(DecimalValue $decimalValue) {
93	92	return $decimalValue->getTrimmed()->getFractionalPart() === '';
94	93	}
95	94
		@@ -98,13 +97,13 @@ discard block
		block discarded – undo
98	97	* @param PropertyValueSnak $snak
99	98	* @return ViolationMessage
100	99	*/
101		- private function getViolationMessage( $messageKey, PropertyValueSnak $snak ) {
102		- return ( new ViolationMessage( $messageKey ) )
103		- ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY )
104		- ->withDataValue( $snak->getDataValue() );
	100	+ private function getViolationMessage($messageKey, PropertyValueSnak $snak) {
	101	+ return (new ViolationMessage($messageKey))
	102	+ ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY)
	103	+ ->withDataValue($snak->getDataValue());
105	104	}
106	105
107		- public function checkConstraintParameters( Constraint $constraint ) {
	106	+ public function checkConstraintParameters(Constraint $constraint) {
108	107	// no parameters
109	108	return [];
110	109	}

		@@ -75,9 +75,9 @@ discard block
		block discarded – undo
75	75	* @throws ConstraintParameterException
76	76	* @return CheckResult
77	77	*/
78		- public function checkConstraint( Context $context, Constraint $constraint ) {
79		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
80		- return new CheckResult( $context, $constraint, CheckResult::STATUS_DEPRECATED );
	78	+ public function checkConstraint(Context $context, Constraint $constraint) {
	79	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	80	+ return new CheckResult($context, $constraint, CheckResult::STATUS_DEPRECATED);
81	81	}
82	82
83	83	$constraintParameters = $constraint->getConstraintParameters();
		@@ -99,7 +99,7 @@ discard block
		block discarded – undo
99	99	* a) a property only
100	100	* b) a property and a number of items (each combination of property and item forming an individual claim)
101	101	*/
102		- if ( $items === [] ) {
	102	+ if ($items === []) {
103	103	$requiredStatement = $this->connectionCheckerHelper->findStatementWithProperty(
104	104	$context->getEntity()->getStatements(),
105	105	$propertyId
		@@ -112,21 +112,21 @@ discard block
		block discarded – undo
112	112	);
113	113	}
114	114
115		- if ( $requiredStatement !== null ) {
	115	+ if ($requiredStatement !== null) {
116	116	$status = CheckResult::STATUS_COMPLIANCE;
117	117	$message = null;
118	118	} else {
119	119	$status = CheckResult::STATUS_VIOLATION;
120		- $message = ( new ViolationMessage( 'wbqc-violation-message-item' ) )
121		- ->withEntityId( $context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY )
122		- ->withEntityId( $propertyId, Role::PREDICATE )
123		- ->withItemIdSnakValueList( $items, Role::OBJECT );
	120	+ $message = (new ViolationMessage('wbqc-violation-message-item'))
	121	+ ->withEntityId($context->getSnak()->getPropertyId(), Role::CONSTRAINT_PROPERTY)
	122	+ ->withEntityId($propertyId, Role::PREDICATE)
	123	+ ->withItemIdSnakValueList($items, Role::OBJECT);
124	124	}
125	125
126		- return new CheckResult( $context, $constraint, $status, $message );
	126	+ return new CheckResult($context, $constraint, $status, $message);
127	127	}
128	128
129		- public function checkConstraintParameters( Constraint $constraint ) {
	129	+ public function checkConstraintParameters(Constraint $constraint) {
130	130	$constraintParameters = $constraint->getConstraintParameters();
131	131	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
132	132	$exceptions = [];
		@@ -135,7 +135,7 @@ discard block
		block discarded – undo
135	135	$constraintParameters,
136	136	$constraintTypeItemId
137	137	);
138		- } catch ( ConstraintParameterException $e ) {
	138	+ } catch (ConstraintParameterException $e) {
139	139	$exceptions[] = $e;
140	140	}
141	141	try {
		@@ -144,7 +144,7 @@ discard block
		block discarded – undo
144	144	$constraintTypeItemId,
145	145	false
146	146	);
147		- } catch ( ConstraintParameterException $e ) {
	147	+ } catch (ConstraintParameterException $e) {
148	148	$exceptions[] = $e;
149	149	}
150	150	return $exceptions;

		@@ -44,16 +44,16 @@
		block discarded – undo
44	44	*
45	45	* @return CheckResult
46	46	*/
47		- public function checkConstraint( Context $context, Constraint $constraint ) {
48		- if ( $context->getType() === Context::TYPE_QUALIFIER ) {
49		- return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
	47	+ public function checkConstraint(Context $context, Constraint $constraint) {
	48	+ if ($context->getType() === Context::TYPE_QUALIFIER) {
	49	+ return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
50	50	} else {
51		- $message = new ViolationMessage( 'wbqc-violation-message-qualifier' );
52		- return new CheckResult( $context, $constraint, CheckResult::STATUS_VIOLATION, $message );
	51	+ $message = new ViolationMessage('wbqc-violation-message-qualifier');
	52	+ return new CheckResult($context, $constraint, CheckResult::STATUS_VIOLATION, $message);
53	53	}
54	54	}
55	55
56		- public function checkConstraintParameters( Constraint $constraint ) {
	56	+ public function checkConstraintParameters(Constraint $constraint) {
57	57	// no parameters
58	58	return [];
59	59	}

		@@ -46,7 +46,7 @@ discard block
		block discarded – undo
46	46	return self::ALL_ENTITY_TYPES_SUPPORTED;
47	47	}
48	48
49		- public function checkConstraint( Context $context, Constraint $constraint ) {
	49	+ public function checkConstraint(Context $context, Constraint $constraint) {
50	50	$constraintParameters = $constraint->getConstraintParameters();
51	51	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
52	52
		@@ -55,7 +55,7 @@ discard block
		block discarded – undo
55	55	$constraintTypeItemId
56	56	);
57	57
58		- if ( in_array( $context->getType(), $allowedContextTypes ) ) {
	58	+ if (in_array($context->getType(), $allowedContextTypes)) {
59	59	return new CheckResult(
60	60	$context->getCursor(),
61	61	$constraint,
		@@ -66,15 +66,15 @@ discard block
		block discarded – undo
66	66	$context->getCursor(),
67	67	$constraint,
68	68	CheckResult::STATUS_VIOLATION,
69		- ( new ViolationMessage( 'wbqc-violation-message-property-scope' ) )
70		- ->withEntityId( $context->getSnak()->getPropertyId() )
71		- ->withPropertyScope( $context->getType() )
72		- ->withPropertyScopeList( $allowedContextTypes )
	69	+ (new ViolationMessage('wbqc-violation-message-property-scope'))
	70	+ ->withEntityId($context->getSnak()->getPropertyId())
	71	+ ->withPropertyScope($context->getType())
	72	+ ->withPropertyScopeList($allowedContextTypes)
73	73	);
74	74	}
75	75	}
76	76
77		- public function checkConstraintParameters( Constraint $constraint ) {
	77	+ public function checkConstraintParameters(Constraint $constraint) {
78	78	$constraintParameters = $constraint->getConstraintParameters();
79	79	$constraintTypeItemId = $constraint->getConstraintTypeItemId();
80	80	$exceptions = [];
		@@ -83,7 +83,7 @@ discard block
		block discarded – undo
83	83	$constraintParameters,
84	84	$constraintTypeItemId
85	85	);
86		- } catch ( ConstraintParameterException $e ) {
	86	+ } catch (ConstraintParameterException $e) {
87	87	$exceptions[] = $e;
88	88	}
89	89	return $exceptions;

		@@ -64,9 +64,9 @@ discard block
		block discarded – undo
64	64	* @throws ConstraintParameterException
65	65	* @return CheckResult
66	66	*/
67		- public function checkConstraint( Context $context, Constraint $constraint ) {
68		- if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
69		- return new CheckResult( $context, $constraint, CheckResult::STATUS_DEPRECATED );
	67	+ public function checkConstraint(Context $context, Constraint $constraint) {
	68	+ if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
	69	+ return new CheckResult($context, $constraint, CheckResult::STATUS_DEPRECATED);
70	70	}
71	71
72	72	$separators = $this->constraintParameterParser->parseSeparatorsParameter(
		@@ -75,19 +75,19 @@ discard block
		block discarded – undo
75	75
76	76	$propertyId = $context->getSnak()->getPropertyId();
77	77	$propertyCount = $this->valueCountCheckerHelper->getPropertyCount(
78		- $context->getSnakGroup( Context::GROUP_NON_DEPRECATED, $separators ),
	78	+ $context->getSnakGroup(Context::GROUP_NON_DEPRECATED, $separators),
79	79	$propertyId
80	80	);
81	81
82		- if ( $propertyCount > 1 ) {
83		- $message = $this->getViolationMessage( $separators, $propertyId );
	82	+ if ($propertyCount > 1) {
	83	+ $message = $this->getViolationMessage($separators, $propertyId);
84	84	$status = CheckResult::STATUS_VIOLATION;
85	85	} else {
86	86	$message = null;
87	87	$status = CheckResult::STATUS_COMPLIANCE;
88	88	}
89	89
90		- return new CheckResult( $context, $constraint, $status, $message );
	90	+ return new CheckResult($context, $constraint, $status, $message);
91	91	}
92	92
93	93	/**
		@@ -95,22 +95,21 @@ discard block
		block discarded – undo
95	95	* @param PropertyId $propertyId
96	96	* @return ViolationMessage
97	97	*/
98		- private function getViolationMessage( array $separators, PropertyId $propertyId ) {
	98	+ private function getViolationMessage(array $separators, PropertyId $propertyId) {
99	99	$messageKey = $separators === [] ?
100		- 'wbqc-violation-message-single-value' :
101		- 'wbqc-violation-message-single-value-separators';
	100	+ 'wbqc-violation-message-single-value' : 'wbqc-violation-message-single-value-separators';
102	101
103		- return ( new ViolationMessage( $messageKey ) )
104		- ->withEntityId( $propertyId )
105		- ->withEntityIdList( $separators );
	102	+ return (new ViolationMessage($messageKey))
	103	+ ->withEntityId($propertyId)
	104	+ ->withEntityIdList($separators);
106	105	}
107	106
108		- public function checkConstraintParameters( Constraint $constraint ) {
	107	+ public function checkConstraintParameters(Constraint $constraint) {
109	108	$constraintParameters = $constraint->getConstraintParameters();
110	109	$exceptions = [];
111	110	try {
112		- $this->constraintParameterParser->parseSeparatorsParameter( $constraintParameters );
113		- } catch ( ConstraintParameterException $e ) {
	111	+ $this->constraintParameterParser->parseSeparatorsParameter($constraintParameters);
	112	+ } catch (ConstraintParameterException $e) {
114	113	$exceptions[] = $e;
115	114	}
116	115	return $exceptions;

		@@ -33,16 +33,16 @@
		block discarded – undo
33	33	return self::ALL_ENTITY_TYPES_SUPPORTED;
34	34	}
35	35
36		- public function checkConstraint( Context $context, Constraint $constraint ) {
37		- if ( $context->getType() === Context::TYPE_REFERENCE ) {
38		- return new CheckResult( $context, $constraint, CheckResult::STATUS_COMPLIANCE );
	36	+ public function checkConstraint(Context $context, Constraint $constraint) {
	37	+ if ($context->getType() === Context::TYPE_REFERENCE) {
	38	+ return new CheckResult($context, $constraint, CheckResult::STATUS_COMPLIANCE);
39	39	} else {
40		- $message = new ViolationMessage( 'wbqc-violation-message-reference' );
41		- return new CheckResult( $context, $constraint, CheckResult::STATUS_VIOLATION, $message );
	40	+ $message = new ViolationMessage('wbqc-violation-message-reference');
	41	+ return new CheckResult($context, $constraint, CheckResult::STATUS_VIOLATION, $message);
42	42	}
43	43	}
44	44
45		- public function checkConstraintParameters( Constraint $constraint ) {
	45	+ public function checkConstraintParameters(Constraint $constraint) {
46	46	// no parameters
47	47	return [];
48	48	}

wikimedia / mediawiki-extensions-WikibaseQualityConstraints

Push — master ( 12d883...691e21 )

Status

Category

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

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

Spacing +21 added lines, -22 removed lines patch added patch discarded remove patch

Spacing +34 added lines, -35 removed lines patch added patch discarded remove patch

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

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

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

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

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