wikimedia / mediawiki-extensions-WikibaseQualityConstraints

src/ConstraintCheck/Helper/ConstraintParameterException.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
 	/**
 	 * @param string $message HTML
 	 */
-	public function __construct( $message ) {
-		parent::__construct( $message );
+	public function __construct($message) {
+		parent::__construct($message);
 	}
 
 }

src/ConstraintCheck/Helper/ValueCountCheckerHelper.php

Spacing +4 added lines, -4 removed lines

@@ -24,14 +24,14 @@
 	 *
 	 * @return int[]
 	 */
-	public function getPropertyCount( StatementList $statements ) {
-		if ( !isset( $this->propertyCount ) ) {
+	public function getPropertyCount(StatementList $statements) {
+		if (!isset($this->propertyCount)) {
 			$this->propertyCount = [];
 
 			/** @var Statement $statement */
-			foreach ( $statements as $statement ) {
+			foreach ($statements as $statement) {
 				$counter = $statement->getRank() === Statement::RANK_DEPRECATED ? 0 : 1;
-				if ( array_key_exists( $statement->getPropertyId()->getSerialization(), $this->propertyCount ) ) {
+				if (array_key_exists($statement->getPropertyId()->getSerialization(), $this->propertyCount)) {
 					$this->propertyCount[$statement->getPropertyId()->getSerialization()] += $counter;
 				} else {
 					$this->propertyCount[$statement->getPropertyId()->getSerialization()] = $counter;

src/ConstraintCheck/Helper/ConnectionCheckerHelper.php

Spacing +10 added lines, -10 removed lines

@@ -28,8 +28,8 @@ 
 		StatementList $statementList,
 		PropertyId $propertyId
 	) {
-		$statementListByPropertyId = $statementList->getByPropertyId( $propertyId );
-		if ( $statementListByPropertyId->isEmpty() ) {
+		$statementListByPropertyId = $statementList->getByPropertyId($propertyId);
+		if ($statementListByPropertyId->isEmpty()) {
 			return null;
 		} else {
 			return $statementListByPropertyId->toArray()[0];
@@ -50,15 +50,15 @@ 
 		PropertyId $propertyId,
 		EntityId $value
 	) {
-		$statementListByPropertyId = $statementList->getByPropertyId( $propertyId );
+		$statementListByPropertyId = $statementList->getByPropertyId($propertyId);
 		/** @var Statement $statement */
-		foreach ( $statementListByPropertyId as $statement ) {
+		foreach ($statementListByPropertyId as $statement) {
 			$snak = $statement->getMainSnak();
-			if ( $snak->getType() === 'value' ) {
+			if ($snak->getType() === 'value') {
 				$dataValue = $snak->getDataValue();
 				if (
 					$dataValue->getType() === 'wikibase-entityid' &&
-					$dataValue->getEntityId()->equals( $value )
+					$dataValue->getEntityId()->equals($value)
 				) {
 					return $statement;
 				}
@@ -81,12 +81,12 @@ 
 		PropertyId $propertyId,
 		array $values
 	) {
-		$statementListByPropertyId = $statementList->getByPropertyId( $propertyId );
+		$statementListByPropertyId = $statementList->getByPropertyId($propertyId);
 		/** @var Statement $statement */
-		foreach ( $statementListByPropertyId as $statement ) {
+		foreach ($statementListByPropertyId as $statement) {
 			$snak = $statement->getMainSnak();
-			foreach ( $values as $value ) {
-				if ( $value->matchesSnak( $snak ) ) {
+			foreach ($values as $value) {
+				if ($value->matchesSnak($snak)) {
 					return $statement;
 				}
 			}

src/ConstraintCheck/Helper/LoggingHelper.php

Spacing +12 added lines, -12 removed lines

@@ -48,8 +48,8 @@ 
 		$this->dataFactory = $dataFactory;
 		$this->logger = $logger;
 		$this->constraintCheckDurationLimits = [
-			'info' => $config->get( 'WBQualityConstraintsCheckDurationInfoSeconds' ),
-			'warning' => $config->get( 'WBQualityConstraintsCheckDurationWarningSeconds' ),
+			'info' => $config->get('WBQualityConstraintsCheckDurationInfoSeconds'),
+			'warning' => $config->get('WBQualityConstraintsCheckDurationWarningSeconds'),
 		];
 	}
 
@@ -75,41 +75,41 @@ 
 		$durationSeconds,
 		$method
 	) {
-		$constraintCheckerClassShortName = substr( strrchr( $constraintCheckerClass, '\\' ), 1 );
+		$constraintCheckerClassShortName = substr(strrchr($constraintCheckerClass, '\\'), 1);
 		$constraintTypeItemId = $constraint->getConstraintTypeItemId();
 
 		$this->dataFactory->timing(
-			'wikibase.quality.constraints.check.timing.' .
-				$constraintTypeItemId . '-' .
+			'wikibase.quality.constraints.check.timing.'.
+				$constraintTypeItemId.'-'.
 				$constraintCheckerClassShortName,
 			$durationSeconds * 1000
 		);
 
 		// find the longest limit (and associated log level) that the duration exceeds
-		foreach ( $this->constraintCheckDurationLimits as $level => $limit ) {
+		foreach ($this->constraintCheckDurationLimits as $level => $limit) {
 			if (
 				// duration exceeds this limit
-				isset( $limit ) && $durationSeconds > $limit &&
+				isset($limit) && $durationSeconds > $limit &&
 				// this limit is longer than previous longest limit
-				( !isset( $limitSeconds ) || $limit > $limitSeconds )
+				(!isset($limitSeconds) || $limit > $limitSeconds)
 			) {
 				$limitSeconds = $limit;
 				$logLevel = $level;
 			}
 		}
 
-		if ( !isset( $limitSeconds ) ) {
+		if (!isset($limitSeconds)) {
 			return;
 		}
-		if ( $context->getType() !== Context::TYPE_STATEMENT ) {
+		if ($context->getType() !== Context::TYPE_STATEMENT) {
 			// TODO log less details but still log something
 			return;
 		}
 
 		$this->logger->log(
 			$logLevel,
-			'Constraint check with {constraintCheckerClassShortName} ' .
-			'took longer than {limitSeconds} second(s) ' .
+			'Constraint check with {constraintCheckerClassShortName} '.
+			'took longer than {limitSeconds} second(s) '.
 			'(duration: {durationSeconds} seconds).',
 			[
 				'method' => $method,

src/ConstraintCheck/Result/CheckResult.php

Spacing +5 added lines, -5 removed lines

@@ -146,11 +146,11 @@ 
 	public function getDataValue() {
 		$mainSnak = $this->context->getSnak();
 
-		if ( $mainSnak instanceof PropertyValueSnak ) {
+		if ($mainSnak instanceof PropertyValueSnak) {
 			return $mainSnak->getDataValue();
 		}
 
-		throw new LogicException( 'Cannot get DataValue, Snak is of type ' . $this->getSnakType() . '.' );
+		throw new LogicException('Cannot get DataValue, Snak is of type '.$this->getSnakType().'.');
 	}
 
 	/**
@@ -178,7 +178,7 @@ 
 	 * @param string $key
 	 * @param string $value
 	 */
-	public function addParameter( $key, $value ) {
+	public function addParameter($key, $value) {
 		$this->parameters[$key][] = $value;
 	}
 
@@ -192,7 +192,7 @@ 
 	/**
 	 * @param string $status
 	 */
-	public function setStatus( $status ) {
+	public function setStatus($status) {
 		$this->status = $status;
 	}
 
@@ -207,7 +207,7 @@ 
 	 * @param Metadata $cm
 	 * @return self
 	 */
-	public function withMetadata( Metadata $cm ) {
+	public function withMetadata(Metadata $cm) {
 		$this->metadata = $cm;
 		return $this;
 	}

src/ConstraintCheck/Checker/SymmetricChecker.php

Spacing +24 added lines, -24 removed lines

@@ -60,12 +60,12 @@ 
 	 *
 	 * @return CheckResult
 	 */
-	public function checkConstraint( Context $context, Constraint $constraint ) {
-		if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
-			return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
+	public function checkConstraint(Context $context, Constraint $constraint) {
+		if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
+			return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
 		}
-		if ( $context->getType() !== Context::TYPE_STATEMENT ) {
-			return new CheckResult( $context, $constraint, [], CheckResult::STATUS_NOT_MAIN_SNAK );
+		if ($context->getType() !== Context::TYPE_STATEMENT) {
+			return new CheckResult($context, $constraint, [], CheckResult::STATUS_NOT_MAIN_SNAK);
 		}
 
 		$parameters = [];
@@ -73,9 +73,9 @@ 
 		$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();
@@ -84,22 +84,22 @@ 
 		 * error handling:
 		 *   type of $dataValue for properties with 'Symmetric' constraint has to be 'wikibase-entityid'
 		 */
-		if ( $dataValue->getType() !== 'wikibase-entityid' ) {
-			$message = wfMessage( "wbqc-violation-message-value-needed-of-type" )
+		if ($dataValue->getType() !== 'wikibase-entityid') {
+			$message = wfMessage("wbqc-violation-message-value-needed-of-type")
 					 ->rawParams(
-						 $this->constraintParameterRenderer->formatItemId( $constraint->getConstraintTypeItemId(), Role::CONSTRAINT_TYPE_ITEM ),
+						 $this->constraintParameterRenderer->formatItemId($constraint->getConstraintTypeItemId(), Role::CONSTRAINT_TYPE_ITEM),
 						 'wikibase-entityid' // TODO is there a message for this type so we can localize it?
 					 )
 					 ->escaped();
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 		/** @var EntityIdValue $dataValue */
 
 		$targetEntityId = $dataValue->getEntityId();
-		$targetEntity = $this->entityLookup->getEntity( $targetEntityId );
-		if ( $targetEntity === null ) {
-			$message = wfMessage( "wbqc-violation-message-target-entity-must-exist" )->escaped();
-			return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message );
+		$targetEntity = $this->entityLookup->getEntity($targetEntityId);
+		if ($targetEntity === null) {
+			$message = wfMessage("wbqc-violation-message-target-entity-must-exist")->escaped();
+			return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message);
 		}
 
 		$symmetricStatement = $this->connectionCheckerHelper->findStatementWithPropertyAndEntityIdValue(
@@ -107,26 +107,26 @@ 
 			$propertyId,
 			$context->getEntity()->getId()
 		);
-		if ( $symmetricStatement !== null ) {
+		if ($symmetricStatement !== null) {
 			$message = '';
 			$status = CheckResult::STATUS_COMPLIANCE;
 		} else {
-			$message = wfMessage( 'wbqc-violation-message-symmetric' )
+			$message = wfMessage('wbqc-violation-message-symmetric')
 					 ->rawParams(
-						 $this->constraintParameterRenderer->formatEntityId( $targetEntityId, Role::SUBJECT ),
-						 $this->constraintParameterRenderer->formatEntityId( $propertyId, Role::PREDICATE ),
-						 $this->constraintParameterRenderer->formatEntityId( $context->getEntity()->getId(), Role::OBJECT )
+						 $this->constraintParameterRenderer->formatEntityId($targetEntityId, Role::SUBJECT),
+						 $this->constraintParameterRenderer->formatEntityId($propertyId, Role::PREDICATE),
+						 $this->constraintParameterRenderer->formatEntityId($context->getEntity()->getId(), Role::OBJECT)
 					 )
 					 ->escaped();
 			$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 [];
 	}

src/ConstraintCheck/Cache/CachedArray.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 	 */
 	private $metadata;
 
-	public function __construct( array $array, Metadata $metadata ) {
+	public function __construct(array $array, Metadata $metadata) {
 		$this->array = $array;
 		$this->metadata = $metadata;
 	}

src/ConstraintCheck/Cache/DependencyMetadata.php

Spacing +5 added lines, -5 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param EntityId $entityId An entity ID from which the value was derived.
 	 * @return self Indication that a value is was derived from the entity with the given ID.
 	 */
-	public static function ofEntityId( EntityId $entityId ) {
+	public static function ofEntityId(EntityId $entityId) {
 		$ret = new self;
 		$ret->entityIds[] = $entityId;
 		return $ret;
@@ -39,11 +39,11 @@ 
 	 * @param self[] $metadatas
 	 * @return self
 	 */
-	public static function merge( array $metadatas ) {
-		Assert::parameterElementType( self::class, $metadatas, '$metadatas' );
+	public static function merge(array $metadatas) {
+		Assert::parameterElementType(self::class, $metadatas, '$metadatas');
 		$ret = new self;
-		foreach ( $metadatas as $metadata ) {
-			$ret->entityIds = array_merge( $ret->entityIds, $metadata->entityIds );
+		foreach ($metadatas as $metadata) {
+			$ret->entityIds = array_merge($ret->entityIds, $metadata->entityIds);
 		}
 		return $ret;
 	}

src/ConstraintCheck/Cache/CachedBool.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 	 * @param bool $bool
 	 * @param Metadata $metadata
 	 */
-	public function __construct( $bool, Metadata $metadata ) {
+	public function __construct($bool, Metadata $metadata) {
 		$this->bool = $bool;
 		$this->metadata = $metadata;
 	}