wikimedia / mediawiki-extensions-WikibaseQualityConstraints

src/ConstraintCheck/ItemIdSnakValue.php

Spacing +12 added lines, -12 removed lines

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

src/Api/CheckConstraintsRdf.php

Spacing +28 added lines, -28 removed lines

@@ -47,7 +47,7 @@ 
 		EntityIdLookup $entityIdLookup,
 		RdfVocabulary $rdfVocabulary
 	) {
-		parent::__construct( $page, $context );
+		parent::__construct($page, $context);
 		$this->resultsSource = $resultsSource;
 		$this->entityIdLookup = $entityIdLookup;
 		$this->rdfVocabulary = $rdfVocabulary;
@@ -108,8 +108,8 @@ 
 	 * @param string $guid
 	 * @return string
 	 */
-	private function cleanupGuid( $guid ) {
-		return preg_replace( '/[^\w-]/', '-', $guid );
+	private function cleanupGuid($guid) {
+		return preg_replace('/[^\w-]/', '-', $guid);
 	}
 
 	/**
@@ -121,60 +121,60 @@ 
 		$response = $this->getRequest()->response();
 		$this->getOutput()->disable();
 
-		if ( !$this->resultsSource instanceof CachingResultsSource ) {
+		if (!$this->resultsSource instanceof CachingResultsSource) {
 			// TODO: make configurable whether only cached results are returned
-			$response->statusHeader( 501 ); // Not Implemented
+			$response->statusHeader(501); // Not Implemented
 			return null;
 		}
 
-		$entityId = $this->entityIdLookup->getEntityIdForTitle( $this->getTitle() );
-		if ( $entityId === null ) {
-			$response->statusHeader( 404 ); // Not Found
+		$entityId = $this->entityIdLookup->getEntityIdForTitle($this->getTitle());
+		if ($entityId === null) {
+			$response->statusHeader(404); // Not Found
 			return null;
 		}
-		$revId = $this->getRequest()->getInt( 'revision' );
+		$revId = $this->getRequest()->getInt('revision');
 
-		$results = $this->resultsSource->getStoredResults( $entityId, $revId );
-		if ( $results === null ) {
-			$response->statusHeader( 204 ); // No Content
+		$results = $this->resultsSource->getStoredResults($entityId, $revId);
+		if ($results === null) {
+			$response->statusHeader(204); // No Content
 			return null;
 		}
 
 		$format = 'ttl'; // TODO: make format an option
 
 		$writerFactory = new RdfWriterFactory();
-		$formatName = $writerFactory->getFormatName( $format );
-		$contentType = $writerFactory->getMimeTypes( $formatName )[0];
+		$formatName = $writerFactory->getFormatName($format);
+		$contentType = $writerFactory->getMimeTypes($formatName)[0];
 
-		$writer = $writerFactory->getWriter( $formatName );
-		foreach ( [ RdfVocabulary::NS_STATEMENT, RdfVocabulary::NS_ONTOLOGY ] as $ns ) {
-			$writer->prefix( $ns, $this->rdfVocabulary->getNamespaceURI( $ns ) );
+		$writer = $writerFactory->getWriter($formatName);
+		foreach ([RdfVocabulary::NS_STATEMENT, RdfVocabulary::NS_ONTOLOGY] as $ns) {
+			$writer->prefix($ns, $this->rdfVocabulary->getNamespaceURI($ns));
 		}
 		$writer->start();
 		$writtenAny = false;
 
-		foreach ( $results->getArray() as $checkResult ) {
-			if ( $checkResult instanceof NullResult ) {
+		foreach ($results->getArray() as $checkResult) {
+			if ($checkResult instanceof NullResult) {
 				continue;
 			}
-			if ( $checkResult->getStatus() === CheckResult::STATUS_BAD_PARAMETERS ) {
+			if ($checkResult->getStatus() === CheckResult::STATUS_BAD_PARAMETERS) {
 				continue;
 			}
 			$writtenAny = true;
-			$writer->about( RdfVocabulary::NS_STATEMENT,
-				$this->cleanupGuid( $checkResult->getContextCursor()->getStatementGuid() ) )
-				->say( RdfVocabulary::NS_ONTOLOGY, 'hasViolationForConstraint' )
-				->is( RdfVocabulary::NS_STATEMENT,
-					$this->cleanupGuid( $checkResult->getConstraint()->getConstraintId() ) );
+			$writer->about(RdfVocabulary::NS_STATEMENT,
+				$this->cleanupGuid($checkResult->getContextCursor()->getStatementGuid()))
+				->say(RdfVocabulary::NS_ONTOLOGY, 'hasViolationForConstraint')
+				->is(RdfVocabulary::NS_STATEMENT,
+					$this->cleanupGuid($checkResult->getConstraint()->getConstraintId()));
 		}
 		$writer->finish();
-		if ( $writtenAny ) {
-			$response->header( "Content-Type: $contentType; charset=UTF-8" );
+		if ($writtenAny) {
+			$response->header("Content-Type: $contentType; charset=UTF-8");
 			echo $writer->drain();
 		} else {
 			// Do not output RDF if we haven't written any actual statements. Output 204 instead
 			$writer->drain();
-			$response->statusHeader( 204 ); // No Content
+			$response->statusHeader(204); // No Content
 		}
 		return null;
 	}

src/Job/CheckConstraintsJob.php

Spacing +13 added lines, -13 removed lines

@@ -37,25 +37,25 @@ 
 	 * @param Title $title
 	 * @param string[] $params should contain 'entityId' => 'Q1234'
 	 */
-	public function __construct( Title $title, array $params ) {
-		parent::__construct( self::COMMAND, $title, $params );
+	public function __construct(Title $title, array $params) {
+		parent::__construct(self::COMMAND, $title, $params);
 		$this->removeDuplicates = true;
 
-		Assert::parameterType( 'string', $params['entityId'], '$params[\'entityId\']' );
+		Assert::parameterType('string', $params['entityId'], '$params[\'entityId\']');
 
-		$resultSource = ConstraintsServices::getResultsSource( MediaWikiServices::getInstance() );
+		$resultSource = ConstraintsServices::getResultsSource(MediaWikiServices::getInstance());
 		'@phan-var CachingResultsSource $resultSource';
 		// This job should only ever be used when caching result sources are used.
-		$this->setResultsSource( $resultSource );
+		$this->setResultsSource($resultSource);
 
-		$this->setEntityIdParser( WikibaseRepo::getEntityIdParser() );
+		$this->setEntityIdParser(WikibaseRepo::getEntityIdParser());
 	}
 
-	public function setResultsSource( CachingResultsSource $resultsSource ) {
+	public function setResultsSource(CachingResultsSource $resultsSource) {
 		$this->resultsSource = $resultsSource;
 	}
 
-	public function setEntityIdParser( EntityIdParser $parser ) {
+	public function setEntityIdParser(EntityIdParser $parser) {
 		$this->entityIdParser = $parser;
 	}
 
@@ -66,19 +66,19 @@ 
 	 */
 	public function run() {
 		try {
-			$entityId = $this->entityIdParser->parse( $this->params['entityId'] );
-		} catch ( EntityIdParsingException $e ) {
+			$entityId = $this->entityIdParser->parse($this->params['entityId']);
+		} catch (EntityIdParsingException $e) {
 			return false;
 		}
 
-		$this->checkConstraints( $entityId );
+		$this->checkConstraints($entityId);
 
 		return true;
 	}
 
-	private function checkConstraints( EntityId $entityId ) {
+	private function checkConstraints(EntityId $entityId) {
 		$this->resultsSource->getResults(
-			[ $entityId ],
+			[$entityId],
 			[],
 			null,
 			[]

src/ConstraintCheck/Helper/ValueCountCheckerHelper.php

Spacing +5 added lines, -5 removed lines

@@ -20,13 +20,13 @@
 	 * @param PropertyId $propertyId
 	 * @return int
 	 */
-	public function getPropertyCount( array $snaks, PropertyId $propertyId ) {
-		return count( array_filter(
+	public function getPropertyCount(array $snaks, PropertyId $propertyId) {
+		return count(array_filter(
 			$snaks,
-			static function ( Snak $snak ) use ( $propertyId ) {
-				return $snak->getPropertyId()->equals( $propertyId );
+			static function(Snak $snak) use ($propertyId) {
+				return $snak->getPropertyId()->equals($propertyId);
 			}
-		) );
+		));
 	}
 
 }

src/ConstraintCheck/Message/ViolationMessage.php

Spacing +37 added lines, -37 removed lines

@@ -128,17 +128,17 @@ 
 	public function __construct(
 		$messageKey
 	) {
-		if ( strpos( $messageKey, self::MESSAGE_KEY_PREFIX ) !== 0 ) {
+		if (strpos($messageKey, self::MESSAGE_KEY_PREFIX) !== 0) {
 			throw new InvalidArgumentException(
-				'ViolationMessage key ⧼' .
-				$messageKey .
-				'⧽ should start with "' .
-				self::MESSAGE_KEY_PREFIX .
+				'ViolationMessage key ⧼'.
+				$messageKey.
+				'⧽ should start with "'.
+				self::MESSAGE_KEY_PREFIX.
 				'".'
 			);
 		}
 
-		$this->messageKeySuffix = substr( $messageKey, strlen( self::MESSAGE_KEY_PREFIX ) );
+		$this->messageKeySuffix = substr($messageKey, strlen(self::MESSAGE_KEY_PREFIX));
 		$this->arguments = [];
 	}
 
@@ -147,7 +147,7 @@ 
 	 * @return string
 	 */
 	public function getMessageKey() {
-		return self::MESSAGE_KEY_PREFIX . $this->messageKeySuffix;
+		return self::MESSAGE_KEY_PREFIX.$this->messageKeySuffix;
 	}
 
 	/**
@@ -167,9 +167,9 @@ 
 	 * @param mixed $value the value, which should match the $type
 	 * @return ViolationMessage
 	 */
-	public function withArgument( $type, $role, $value ) {
+	public function withArgument($type, $role, $value) {
 		$ret = clone $this;
-		$ret->arguments[] = [ 'type' => $type, 'role' => $role, 'value' => $value ];
+		$ret->arguments[] = ['type' => $type, 'role' => $role, 'value' => $value];
 		return $ret;
 	}
 
@@ -181,8 +181,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withEntityId( EntityId $entityId, $role = null ) {
-		return $this->withArgument( self::TYPE_ENTITY_ID, $role, $entityId );
+	public function withEntityId(EntityId $entityId, $role = null) {
+		return $this->withArgument(self::TYPE_ENTITY_ID, $role, $entityId);
 	}
 
 	/**
@@ -199,8 +199,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withEntityIdList( array $entityIdList, $role = null ) {
-		return $this->withArgument( self::TYPE_ENTITY_ID_LIST, $role, $entityIdList );
+	public function withEntityIdList(array $entityIdList, $role = null) {
+		return $this->withArgument(self::TYPE_ENTITY_ID_LIST, $role, $entityIdList);
 	}
 
 	/**
@@ -211,8 +211,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withItemIdSnakValue( ItemIdSnakValue $value, $role = null ) {
-		return $this->withArgument( self::TYPE_ITEM_ID_SNAK_VALUE, $role, $value );
+	public function withItemIdSnakValue(ItemIdSnakValue $value, $role = null) {
+		return $this->withArgument(self::TYPE_ITEM_ID_SNAK_VALUE, $role, $value);
 	}
 
 	/**
@@ -229,8 +229,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withItemIdSnakValueList( array $valueList, $role = null ) {
-		return $this->withArgument( self::TYPE_ITEM_ID_SNAK_VALUE_LIST, $role, $valueList );
+	public function withItemIdSnakValueList(array $valueList, $role = null) {
+		return $this->withArgument(self::TYPE_ITEM_ID_SNAK_VALUE_LIST, $role, $valueList);
 	}
 
 	/**
@@ -241,8 +241,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withDataValue( DataValue $dataValue, $role = null ) {
-		return $this->withArgument( self::TYPE_DATA_VALUE, $role, $dataValue );
+	public function withDataValue(DataValue $dataValue, $role = null) {
+		return $this->withArgument(self::TYPE_DATA_VALUE, $role, $dataValue);
 	}
 
 	/**
@@ -257,8 +257,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withDataValueType( $dataValueType, $role = null ) {
-		return $this->withArgument( self::TYPE_DATA_VALUE_TYPE, $role, $dataValueType );
+	public function withDataValueType($dataValueType, $role = null) {
+		return $this->withArgument(self::TYPE_DATA_VALUE_TYPE, $role, $dataValueType);
 	}
 
 	/**
@@ -269,8 +269,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withInlineCode( $code, $role = null ) {
-		return $this->withArgument( self::TYPE_INLINE_CODE, $role, $code );
+	public function withInlineCode($code, $role = null) {
+		return $this->withArgument(self::TYPE_INLINE_CODE, $role, $code);
 	}
 
 	/**
@@ -281,8 +281,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withConstraintScope( $scope, $role = null ) {
-		return $this->withArgument( self::TYPE_CONSTRAINT_SCOPE, $role, $scope );
+	public function withConstraintScope($scope, $role = null) {
+		return $this->withArgument(self::TYPE_CONSTRAINT_SCOPE, $role, $scope);
 	}
 
 	/**
@@ -293,8 +293,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withConstraintScopeList( array $scopeList, $role = null ) {
-		return $this->withArgument( self::TYPE_CONSTRAINT_SCOPE_LIST, $role, $scopeList );
+	public function withConstraintScopeList(array $scopeList, $role = null) {
+		return $this->withArgument(self::TYPE_CONSTRAINT_SCOPE_LIST, $role, $scopeList);
 	}
 
 	/**
@@ -305,8 +305,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withPropertyScope( $scope, $role = null ) {
-		return $this->withArgument( self::TYPE_PROPERTY_SCOPE, $role, $scope );
+	public function withPropertyScope($scope, $role = null) {
+		return $this->withArgument(self::TYPE_PROPERTY_SCOPE, $role, $scope);
 	}
 
 	/**
@@ -317,8 +317,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withPropertyScopeList( array $scopeList, $role = null ) {
-		return $this->withArgument( self::TYPE_PROPERTY_SCOPE_LIST, $role, $scopeList );
+	public function withPropertyScopeList(array $scopeList, $role = null) {
+		return $this->withArgument(self::TYPE_PROPERTY_SCOPE_LIST, $role, $scopeList);
 	}
 
 	/**
@@ -333,8 +333,8 @@ 
 	 * @param string $languageCode
 	 * @return ViolationMessage
 	 */
-	public function withLanguage( $languageCode ) {
-		return $this->withArgument( self::TYPE_LANGUAGE, null, $languageCode );
+	public function withLanguage($languageCode) {
+		return $this->withArgument(self::TYPE_LANGUAGE, null, $languageCode);
 	}
 
 	/**
@@ -349,8 +349,8 @@ 
 	 * @param string[] $languageCodes
 	 * @return ViolationMessage
 	 */
-	public function withLanguages( $languageCodes ) {
-		return $this->withArgument( self::TYPE_LANGUAGE_LIST, null, $languageCodes );
+	public function withLanguages($languageCodes) {
+		return $this->withArgument(self::TYPE_LANGUAGE_LIST, null, $languageCodes);
 	}
 
 	/**
@@ -365,8 +365,8 @@ 
 	 * @param string|null $role one of the Role::* constants
 	 * @return ViolationMessage
 	 */
-	public function withMultilingualText( MultilingualTextValue $text, $role = null ) {
-		return $this->withArgument( self::TYPE_MULTILINGUAL_TEXT, $role, $text );
+	public function withMultilingualText(MultilingualTextValue $text, $role = null) {
+		return $this->withArgument(self::TYPE_MULTILINGUAL_TEXT, $role, $text);
 	}
 
 }

src/Role.php

Spacing +1 added lines, -1 removed lines

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

src/ConstraintDeserializer.php

Spacing +4 added lines, -5 removed lines

@@ -10,14 +10,13 @@
  */
 class ConstraintDeserializer {
 
-	public function deserialize( array $serialization ) {
+	public function deserialize(array $serialization) {
 		return new Constraint(
 			$serialization['id'],
-			new NumericPropertyId( $serialization['pid'] ),
+			new NumericPropertyId($serialization['pid']),
 			$serialization['qid'],
-			array_key_exists( 'params', $serialization ) ?
-			$serialization['params'] :
-		[]
+			array_key_exists('params', $serialization) ?
+			$serialization['params'] : []
 			);
 	}
 

src/CachingConstraintLookup.php

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
 	/**
 	 * @param ConstraintLookup $lookup The lookup to which all queries are delegated.
 	 */
-	public function __construct( ConstraintLookup $lookup ) {
+	public function __construct(ConstraintLookup $lookup) {
 		$this->lookup = $lookup;
 	}
 
@@ -35,10 +35,10 @@ 
 	 *
 	 * @return Constraint[]
 	 */
-	public function queryConstraintsForProperty( NumericPropertyId $propertyId ) {
+	public function queryConstraintsForProperty(NumericPropertyId $propertyId) {
 		$id = $propertyId->getSerialization();
-		if ( !array_key_exists( $id, $this->cache ) ) {
-			$this->cache[$id] = $this->lookup->queryConstraintsForProperty( $propertyId );
+		if (!array_key_exists($id, $this->cache)) {
+			$this->cache[$id] = $this->lookup->queryConstraintsForProperty($propertyId);
 		}
 		return $this->cache[$id];
 	}

src/ConstraintLookup.php

Spacing +1 added lines, -1 removed lines

@@ -14,6 +14,6 @@
 	 *
 	 * @return Constraint[]
 	 */
-	public function queryConstraintsForProperty( NumericPropertyId $propertyId );
+	public function queryConstraintsForProperty(NumericPropertyId $propertyId);
 
 }