wikimedia / mediawiki-extensions-WikibaseQualityConstraints

src/ConstraintCheck/Result/CheckResultSerializer.php

Spacing +13 added lines, -13 removed lines

@@ -72,27 +72,27 @@ 
 	 * @param CheckResult $checkResult
 	 * @return array
 	 */
-	public function serialize( CheckResult $checkResult ) {
+	public function serialize(CheckResult $checkResult) {
 		$contextCursor = $checkResult->getContextCursor();
 		$constraint = $checkResult->getConstraint();
 		$violationMessage = $checkResult->getMessage();
 		$cachingMetadata = $checkResult->getMetadata()->getCachingMetadata();
 
 		$serialization = [
-			self::KEY_CONTEXT_CURSOR => $this->contextCursorSerializer->serialize( $contextCursor ),
-			self::KEY_CONSTRAINT => $this->constraintSerializer->serialize( $constraint ),
+			self::KEY_CONTEXT_CURSOR => $this->contextCursorSerializer->serialize($contextCursor),
+			self::KEY_CONSTRAINT => $this->constraintSerializer->serialize($constraint),
 			self::KEY_CHECK_RESULT_STATUS => $checkResult->getStatus(),
-			self::KEY_CACHING_METADATA => $this->serializeCachingMetadata( $cachingMetadata ),
+			self::KEY_CACHING_METADATA => $this->serializeCachingMetadata($cachingMetadata),
 		];
 
-		if ( $violationMessage !== null ) {
+		if ($violationMessage !== null) {
 			$serialization[self::KEY_VIOLATION_MESSAGE] =
-				$this->violationMessageSerializer->serialize( $violationMessage );
+				$this->violationMessageSerializer->serialize($violationMessage);
 		}
 
-		if ( $this->serializeDependencyMetadata ) {
+		if ($this->serializeDependencyMetadata) {
 			$serialization[self::KEY_DEPENDENCY_METADATA] =
-				$this->serializeDependencyMetadata( $checkResult );
+				$this->serializeDependencyMetadata($checkResult);
 		}
 
 		return $serialization;
@@ -102,12 +102,12 @@ 
 	 * @param CachingMetadata $cachingMetadata
 	 * @return array
 	 */
-	private function serializeCachingMetadata( CachingMetadata $cachingMetadata ) {
+	private function serializeCachingMetadata(CachingMetadata $cachingMetadata) {
 		$maximumAge = $cachingMetadata->getMaximumAgeInSeconds();
 
 		$serialization = [];
 
-		if ( $maximumAge > 0 ) {
+		if ($maximumAge > 0) {
 			$serialization[self::KEY_CACHING_METADATA_MAX_AGE] = $maximumAge;
 		}
 
@@ -118,21 +118,21 @@ 
 	 * @param CheckResult $checkResult
 	 * @return array
 	 */
-	private function serializeDependencyMetadata( CheckResult $checkResult ) {
+	private function serializeDependencyMetadata(CheckResult $checkResult) {
 		$dependencyMetadata = $checkResult->getMetadata()->getDependencyMetadata();
 		$entityIds = $dependencyMetadata->getEntityIds();
 		$futureTime = $dependencyMetadata->getFutureTime();
 
 		$serialization = [
 			self::KEY_DEPENDENCY_METADATA_ENTITY_IDS => array_map(
-				function ( EntityId $entityId ) {
+				function(EntityId $entityId) {
 					return $entityId->getSerialization();
 				},
 				$entityIds
 			),
 		];
 
-		if ( $futureTime !== null ) {
+		if ($futureTime !== null) {
 			$serialization[self::KEY_DEPENDENCY_METADATA_FUTURE_TIME] =
 				$futureTime->getArrayValue();
 		}

src/ConstraintCheck/Result/CheckResultDeserializer.php

Spacing +17 added lines, -17 removed lines

@@ -55,7 +55,7 @@ 
 	 * @param array $serialization
 	 * @return CheckResult
 	 */
-	public function deserialize( array $serialization ) {
+	public function deserialize(array $serialization) {
 		$contextCursor = $this->contextCursorDeserializer->deserialize(
 			$serialization[CheckResultSerializer::KEY_CONTEXT_CURSOR]
 		);
@@ -68,7 +68,7 @@ 
 
 		$status = $serialization[CheckResultSerializer::KEY_CHECK_RESULT_STATUS];
 
-		if ( array_key_exists( CheckResultSerializer::KEY_VIOLATION_MESSAGE, $serialization ) ) {
+		if (array_key_exists(CheckResultSerializer::KEY_VIOLATION_MESSAGE, $serialization)) {
 			$violationMessage = $this->violationMessageDeserializer->deserialize(
 				$serialization[CheckResultSerializer::KEY_VIOLATION_MESSAGE]
 			);
@@ -80,7 +80,7 @@ 
 			$serialization[CheckResultSerializer::KEY_CACHING_METADATA]
 		);
 
-		if ( array_key_exists( CheckResultSerializer::KEY_DEPENDENCY_METADATA, $serialization ) ) {
+		if (array_key_exists(CheckResultSerializer::KEY_DEPENDENCY_METADATA, $serialization)) {
 			$dependencyMetadata = $this->deserializeDependencyMetadata(
 				$serialization[CheckResultSerializer::KEY_DEPENDENCY_METADATA]
 			);
@@ -88,17 +88,17 @@ 
 			$dependencyMetadata = DependencyMetadata::blank();
 		}
 
-		return ( new CheckResult(
+		return (new CheckResult(
 			$contextCursor,
 			$constraint,
 			$parameters,
 			$status,
 			$violationMessage
-		) )->withMetadata(
-			Metadata::merge( [
-				Metadata::ofCachingMetadata( $cachingMetadata ),
-				Metadata::ofDependencyMetadata( $dependencyMetadata ),
-			] )
+		))->withMetadata(
+			Metadata::merge([
+				Metadata::ofCachingMetadata($cachingMetadata),
+				Metadata::ofDependencyMetadata($dependencyMetadata),
+			])
 		);
 	}
 
@@ -106,7 +106,7 @@ 
 	 * @param array $serialization
 	 * @return CachingMetadata
 	 */
-	private function deserializeCachingMetadata( array $serialization ) {
+	private function deserializeCachingMetadata(array $serialization) {
 		if (
 			array_key_exists(
 				CheckResultSerializer::KEY_CACHING_METADATA_MAX_AGE,
@@ -125,7 +125,7 @@ 
 	 * @param array $serialization
 	 * @return DependencyMetadata
 	 */
-	private function deserializeDependencyMetadata( array $serialization ) {
+	private function deserializeDependencyMetadata(array $serialization) {
 		if (
 			array_key_exists(
 				CheckResultSerializer::KEY_DEPENDENCY_METADATA_FUTURE_TIME,
@@ -135,20 +135,20 @@ 
 			$futureTime = TimeValue::newFromArray(
 				$serialization[CheckResultSerializer::KEY_DEPENDENCY_METADATA_FUTURE_TIME]
 			);
-			$futureTimeDependencyMetadata = DependencyMetadata::ofFutureTime( $futureTime );
+			$futureTimeDependencyMetadata = DependencyMetadata::ofFutureTime($futureTime);
 		} else {
 			$futureTimeDependencyMetadata = DependencyMetadata::blank();
 		}
 
 		$dependencyMetadata = array_reduce(
 			$serialization[CheckResultSerializer::KEY_DEPENDENCY_METADATA_ENTITY_IDS],
-			function ( DependencyMetadata $metadata, $entityIdSerialization ) {
-				$entityId = $this->entityIdParser->parse( $entityIdSerialization );
+			function(DependencyMetadata $metadata, $entityIdSerialization) {
+				$entityId = $this->entityIdParser->parse($entityIdSerialization);
 
-				return DependencyMetadata::merge( [
+				return DependencyMetadata::merge([
 					$metadata,
-					DependencyMetadata::ofEntityId( $entityId )
-				] );
+					DependencyMetadata::ofEntityId($entityId)
+				]);
 			},
 			$futureTimeDependencyMetadata
 		);