View GitHub Repository mediawiki-extensions-WikibaseQualityConstraints

Schedule Inspection
Subscribe
Completed
Push — master ( 159567...930599 )
by
unknown
04:27 queued 01:05
created
Status
Category
Toggle Patches
src/ConstraintCheck/Result/CheckResultDeserializer.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch 
@@ -56,13 +56,13 @@  discard block
 
 block discarded – undo
56 56 
 	 * @param array $serialization
57 57 
 	 * @return CheckResult
58 58 
 	 */
59 
-	public function deserialize( array $serialization ) {
59 
+	public function deserialize(array $serialization) {
60 60 
 		$contextCursor = $this->contextCursorDeserializer->deserialize(
61 61 
 			$serialization[CheckResultSerializer::KEY_CONTEXT_CURSOR]
62 62 
 		);
63 63
64 
-		if ( array_key_exists( CheckResultSerializer::KEY_NULL_RESULT, $serialization ) ) {
65 
-			$result = new NullResult( $contextCursor );
64 
+		if (array_key_exists(CheckResultSerializer::KEY_NULL_RESULT, $serialization)) {
65 
+			$result = new NullResult($contextCursor);
66 66 
 			$cachingMetadata = CachingMetadata::fresh();
67 67 
 		} else {
68 68 
 			$constraint = $this->constraintDeserializer->deserialize(
 
@@ -73,7 +73,7 @@  discard block
 
 block discarded – undo
73 73
74 74 
 			$status = $serialization[CheckResultSerializer::KEY_CHECK_RESULT_STATUS];
75 75
76 
-			$violationMessage = $this->getViolationMessageFromSerialization( $serialization );
76 
+			$violationMessage = $this->getViolationMessageFromSerialization($serialization);
77 77
78 78 
 			$result = new CheckResult(
79 79 
 				$contextCursor,
 
@@ -88,13 +88,13 @@  discard block
 
 block discarded – undo
88 88 
 			);
89 89 
 		}
90 90
91 
-		$dependencyMetadata = $this->getDependencyMetadataFromSerialization( $serialization );
91 
+		$dependencyMetadata = $this->getDependencyMetadataFromSerialization($serialization);
92 92
93 93 
 		return $result->withMetadata(
94 
-			Metadata::merge( [
95 
-				Metadata::ofCachingMetadata( $cachingMetadata ),
96 
-				Metadata::ofDependencyMetadata( $dependencyMetadata ),
97 
-			] )
94 
+			Metadata::merge([
95 
+				Metadata::ofCachingMetadata($cachingMetadata),
96 
+				Metadata::ofDependencyMetadata($dependencyMetadata),
97 
+			])
98 98 
 		);
99 99 
 	}
100 100 
 
@@ -102,8 +102,8 @@  discard block
 
 block discarded – undo
102 102 
 	 * @param array $serialization
103 103 
 	 * @return null|ViolationMessage
104 104 
 	 */
105 
-	private function getViolationMessageFromSerialization( array $serialization ) {
106 
-		if ( array_key_exists( CheckResultSerializer::KEY_VIOLATION_MESSAGE, $serialization ) ) {
105 
+	private function getViolationMessageFromSerialization(array $serialization) {
106 
+		if (array_key_exists(CheckResultSerializer::KEY_VIOLATION_MESSAGE, $serialization)) {
107 107 
 			return $this->violationMessageDeserializer->deserialize(
108 108 
 				$serialization[CheckResultSerializer::KEY_VIOLATION_MESSAGE]
109 109 
 			);
 
@@ -116,8 +116,8 @@  discard block
 
 block discarded – undo
116 116 
 	 * @param array $serialization
117 117 
 	 * @return DependencyMetadata
118 118 
 	 */
119 
-	private function getDependencyMetadataFromSerialization( array $serialization ) {
120 
-		if ( array_key_exists( CheckResultSerializer::KEY_DEPENDENCY_METADATA, $serialization ) ) {
119 
+	private function getDependencyMetadataFromSerialization(array $serialization) {
120 
+		if (array_key_exists(CheckResultSerializer::KEY_DEPENDENCY_METADATA, $serialization)) {
121 121 
 			return $this->deserializeDependencyMetadata(
122 122 
 				$serialization[CheckResultSerializer::KEY_DEPENDENCY_METADATA]
123 123 
 			);
 
@@ -130,7 +130,7 @@  discard block
 
 block discarded – undo
130 130 
 	 * @param array $serialization
131 131 
 	 * @return CachingMetadata
132 132 
 	 */
133 
-	private function deserializeCachingMetadata( array $serialization ) {
133 
+	private function deserializeCachingMetadata(array $serialization) {
134 134 
 		if (
135 135 
 			array_key_exists(
136 136 
 				CheckResultSerializer::KEY_CACHING_METADATA_MAX_AGE,
 
@@ -149,7 +149,7 @@  discard block
 
 block discarded – undo
149 149 
 	 * @param array $serialization
150 150 
 	 * @return DependencyMetadata
151 151 
 	 */
152 
-	private function deserializeDependencyMetadata( array $serialization ) {
152 
+	private function deserializeDependencyMetadata(array $serialization) {
153 153 
 		if (
154 154 
 			array_key_exists(
155 155 
 				CheckResultSerializer::KEY_DEPENDENCY_METADATA_FUTURE_TIME,
 
@@ -159,20 +159,20 @@  discard block
 
 block discarded – undo
159 159 
 			$futureTime = TimeValue::newFromArray(
160 160 
 				$serialization[CheckResultSerializer::KEY_DEPENDENCY_METADATA_FUTURE_TIME]
161 161 
 			);
162 
-			$futureTimeDependencyMetadata = DependencyMetadata::ofFutureTime( $futureTime );
162 
+			$futureTimeDependencyMetadata = DependencyMetadata::ofFutureTime($futureTime);
163 163 
 		} else {
164 164 
 			$futureTimeDependencyMetadata = DependencyMetadata::blank();
165 165 
 		}
166 166
167 167 
 		$dependencyMetadata = array_reduce(
168 168 
 			$serialization[CheckResultSerializer::KEY_DEPENDENCY_METADATA_ENTITY_IDS],
169 
-			function ( DependencyMetadata $metadata, $entityIdSerialization ) {
170 
-				$entityId = $this->entityIdParser->parse( $entityIdSerialization );
169 
+			function(DependencyMetadata $metadata, $entityIdSerialization) {
170 
+				$entityId = $this->entityIdParser->parse($entityIdSerialization);
171 171
172 
-				return DependencyMetadata::merge( [
172 
+				return DependencyMetadata::merge([
173 173 
 					$metadata,
174 
-					DependencyMetadata::ofEntityId( $entityId )
175 
-				] );
174 
+					DependencyMetadata::ofEntityId($entityId)
175 
+				]);
176 176 
 			},
177 177 
 			$futureTimeDependencyMetadata
178 178 
 		);
Please login to merge, or discard this patch.