Completed
Push — master ( 296d48...58f7ad )
by
unknown
02:05
created
src/ConstraintParameterRenderer.php 1 patch
Spacing   +85 added lines, -85 removed lines patch added patch discarded remove patch
@@ -60,20 +60,20 @@  discard block
 block discarded – undo
60 60
 	 *
61 61
 	 * @return string HTML
62 62
 	 */
63
-	public function formatValue( $value ) {
64
-		if ( is_string( $value ) ) {
63
+	public function formatValue($value) {
64
+		if (is_string($value)) {
65 65
 			// Cases like 'Format' 'pattern' or 'minimum'/'maximum' values, which we have stored as
66 66
 			// strings
67
-			return htmlspecialchars( $value );
68
-		} elseif ( $value instanceof EntityId ) {
67
+			return htmlspecialchars($value);
68
+		} elseif ($value instanceof EntityId) {
69 69
 			// Cases like 'Conflicts with' 'property', to which we can link
70
-			return $this->formatEntityId( $value );
71
-		} elseif ( $value instanceof ItemIdSnakValue ) {
70
+			return $this->formatEntityId($value);
71
+		} elseif ($value instanceof ItemIdSnakValue) {
72 72
 			// Cases like EntityId but can also be somevalue or novalue
73
-			return $this->formatItemIdSnakValue( $value );
73
+			return $this->formatItemIdSnakValue($value);
74 74
 		} else {
75 75
 			// Cases where we format a DataValue
76
-			return $this->formatDataValue( $value );
76
+			return $this->formatDataValue($value);
77 77
 		}
78 78
 	}
79 79
 
@@ -84,23 +84,23 @@  discard block
 block discarded – undo
84 84
 	 *
85 85
 	 * @return string HTML
86 86
 	 */
87
-	public function formatParameters( $parameters ) {
88
-		if ( $parameters === null || count( $parameters ) == 0 ) {
87
+	public function formatParameters($parameters) {
88
+		if ($parameters === null || count($parameters) == 0) {
89 89
 			return null;
90 90
 		}
91 91
 
92
-		$valueFormatter = function ( $value ) {
93
-			return $this->formatValue( $value );
92
+		$valueFormatter = function($value) {
93
+			return $this->formatValue($value);
94 94
 		};
95 95
 
96 96
 		$formattedParameters = [];
97
-		foreach ( $parameters as $parameterName => $parameterValue ) {
98
-			$formattedParameterValues = implode( ', ',
99
-				$this->limitArrayLength( array_map( $valueFormatter, $parameterValue ) ) );
100
-			$formattedParameters[] = sprintf( '%s: %s', $parameterName, $formattedParameterValues );
97
+		foreach ($parameters as $parameterName => $parameterValue) {
98
+			$formattedParameterValues = implode(', ',
99
+				$this->limitArrayLength(array_map($valueFormatter, $parameterValue)));
100
+			$formattedParameters[] = sprintf('%s: %s', $parameterName, $formattedParameterValues);
101 101
 		}
102 102
 
103
-		return implode( '; ', $formattedParameters );
103
+		return implode('; ', $formattedParameters);
104 104
 	}
105 105
 
106 106
 	/**
@@ -110,10 +110,10 @@  discard block
 block discarded – undo
110 110
 	 *
111 111
 	 * @return array
112 112
 	 */
113
-	private function limitArrayLength( array $array ) {
114
-		if ( count( $array ) > self::MAX_PARAMETER_ARRAY_LENGTH ) {
115
-			$array = array_slice( $array, 0, self::MAX_PARAMETER_ARRAY_LENGTH );
116
-			array_push( $array, '...' );
113
+	private function limitArrayLength(array $array) {
114
+		if (count($array) > self::MAX_PARAMETER_ARRAY_LENGTH) {
115
+			$array = array_slice($array, 0, self::MAX_PARAMETER_ARRAY_LENGTH);
116
+			array_push($array, '...');
117 117
 		}
118 118
 
119 119
 		return $array;
@@ -126,12 +126,12 @@  discard block
 block discarded – undo
126 126
 	 * @param string $value HTML
127 127
 	 * @return string HTML
128 128
 	 */
129
-	public static function formatByRole( $role, $value ) {
130
-		if ( $role === null ) {
129
+	public static function formatByRole($role, $value) {
130
+		if ($role === null) {
131 131
 			return $value;
132 132
 		}
133 133
 
134
-		return '<span class="wbqc-role wbqc-role-' . htmlspecialchars( $role ) . '">'
134
+		return '<span class="wbqc-role wbqc-role-'.htmlspecialchars($role).'">'
135 135
 			. $value
136 136
 			. '</span>';
137 137
 	}
@@ -141,9 +141,9 @@  discard block
 block discarded – undo
141 141
 	 * @param string|null $role one of the Role constants or null
142 142
 	 * @return string HTML
143 143
 	 */
144
-	public function formatDataValue( DataValue $value, $role = null ) {
145
-		return self::formatByRole( $role,
146
-			$this->dataValueFormatter->format( $value ) );
144
+	public function formatDataValue(DataValue $value, $role = null) {
145
+		return self::formatByRole($role,
146
+			$this->dataValueFormatter->format($value));
147 147
 	}
148 148
 
149 149
 	/**
@@ -151,9 +151,9 @@  discard block
 block discarded – undo
151 151
 	 * @param string|null $role one of the Role constants or null
152 152
 	 * @return string HTML
153 153
 	 */
154
-	public function formatEntityId( EntityId $entityId, $role = null ) {
155
-		return self::formatByRole( $role,
156
-			$this->entityIdLabelFormatter->formatEntityId( $entityId ) );
154
+	public function formatEntityId(EntityId $entityId, $role = null) {
155
+		return self::formatByRole($role,
156
+			$this->entityIdLabelFormatter->formatEntityId($entityId));
157 157
 	}
158 158
 
159 159
 	/**
@@ -165,18 +165,18 @@  discard block
 block discarded – undo
165 165
 	 * @param string|null $role one of the Role constants or null
166 166
 	 * @return string HTML
167 167
 	 */
168
-	public function formatPropertyId( $propertyId, $role = null ) {
169
-		if ( $propertyId instanceof PropertyId ) {
170
-			return $this->formatEntityId( $propertyId, $role );
171
-		} elseif ( is_string( $propertyId ) ) {
168
+	public function formatPropertyId($propertyId, $role = null) {
169
+		if ($propertyId instanceof PropertyId) {
170
+			return $this->formatEntityId($propertyId, $role);
171
+		} elseif (is_string($propertyId)) {
172 172
 			try {
173
-				return $this->formatEntityId( new PropertyId( $propertyId ), $role );
174
-			} catch ( InvalidArgumentException $e ) {
175
-				return self::formatByRole( $role,
176
-					htmlspecialchars( $propertyId ) );
173
+				return $this->formatEntityId(new PropertyId($propertyId), $role);
174
+			} catch (InvalidArgumentException $e) {
175
+				return self::formatByRole($role,
176
+					htmlspecialchars($propertyId));
177 177
 			}
178 178
 		} else {
179
-			throw new InvalidArgumentException( '$propertyId must be either PropertyId or string' );
179
+			throw new InvalidArgumentException('$propertyId must be either PropertyId or string');
180 180
 		}
181 181
 	}
182 182
 
@@ -189,18 +189,18 @@  discard block
 block discarded – undo
189 189
 	 * @param string|null $role one of the Role constants or null
190 190
 	 * @return string HTML
191 191
 	 */
192
-	public function formatItemId( $itemId, $role = null ) {
193
-		if ( $itemId instanceof ItemId ) {
194
-			return $this->formatEntityId( $itemId, $role );
195
-		} elseif ( is_string( $itemId ) ) {
192
+	public function formatItemId($itemId, $role = null) {
193
+		if ($itemId instanceof ItemId) {
194
+			return $this->formatEntityId($itemId, $role);
195
+		} elseif (is_string($itemId)) {
196 196
 			try {
197
-				return $this->formatEntityId( new ItemId( $itemId ), $role );
198
-			} catch ( InvalidArgumentException $e ) {
199
-				return self::formatByRole( $role,
200
-					htmlspecialchars( $itemId ) );
197
+				return $this->formatEntityId(new ItemId($itemId), $role);
198
+			} catch (InvalidArgumentException $e) {
199
+				return self::formatByRole($role,
200
+					htmlspecialchars($itemId));
201 201
 			}
202 202
 		} else {
203
-			throw new InvalidArgumentException( '$itemId must be either ItemId or string' );
203
+			throw new InvalidArgumentException('$itemId must be either ItemId or string');
204 204
 		}
205 205
 	}
206 206
 
@@ -211,20 +211,20 @@  discard block
 block discarded – undo
211 211
 	 * @param string|null $role one of the Role constants or null
212 212
 	 * @return string HTML
213 213
 	 */
214
-	public function formatItemIdSnakValue( ItemIdSnakValue $value, $role = null ) {
215
-		switch ( true ) {
214
+	public function formatItemIdSnakValue(ItemIdSnakValue $value, $role = null) {
215
+		switch (true) {
216 216
 			case $value->isValue():
217
-				return $this->formatEntityId( $value->getItemId(), $role );
217
+				return $this->formatEntityId($value->getItemId(), $role);
218 218
 			case $value->isSomeValue():
219
-				return self::formatByRole( $role,
219
+				return self::formatByRole($role,
220 220
 					'<span class="wikibase-snakview-variation-somevaluesnak">'
221
-						. wfMessage( 'wikibase-snakview-snaktypeselector-somevalue' )->escaped()
222
-						. '</span>' );
221
+						. wfMessage('wikibase-snakview-snaktypeselector-somevalue')->escaped()
222
+						. '</span>');
223 223
 			case $value->isNoValue():
224
-				return self::formatByRole( $role,
224
+				return self::formatByRole($role,
225 225
 					'<span class="wikibase-snakview-variation-novaluesnak">'
226
-						. wfMessage( 'wikibase-snakview-snaktypeselector-novalue' )->escaped()
227
-						. '</span>' );
226
+						. wfMessage('wikibase-snakview-snaktypeselector-novalue')->escaped()
227
+						. '</span>');
228 228
 		}
229 229
 	}
230 230
 
@@ -238,15 +238,15 @@  discard block
 block discarded – undo
238 238
 	 * @param string|null $role one of the Role constants or null
239 239
 	 * @return string[] HTML
240 240
 	 */
241
-	public function formatPropertyIdList( array $propertyIds, $role = null ) {
242
-		if ( empty( $propertyIds ) ) {
243
-			return [ '<ul></ul>' ];
241
+	public function formatPropertyIdList(array $propertyIds, $role = null) {
242
+		if (empty($propertyIds)) {
243
+			return ['<ul></ul>'];
244 244
 		}
245
-		$propertyIds = $this->limitArrayLength( $propertyIds );
246
-		$formattedPropertyIds = array_map( [ $this, "formatPropertyId" ], $propertyIds, array_fill( 0, count( $propertyIds ), $role ) );
245
+		$propertyIds = $this->limitArrayLength($propertyIds);
246
+		$formattedPropertyIds = array_map([$this, "formatPropertyId"], $propertyIds, array_fill(0, count($propertyIds), $role));
247 247
 		array_unshift(
248 248
 			$formattedPropertyIds,
249
-			'<ul><li>' . implode( '</li><li>', $formattedPropertyIds ) . '</li></ul>'
249
+			'<ul><li>'.implode('</li><li>', $formattedPropertyIds).'</li></ul>'
250 250
 		);
251 251
 		return $formattedPropertyIds;
252 252
 	}
@@ -261,15 +261,15 @@  discard block
 block discarded – undo
261 261
 	 * @param string|null $role one of the Role constants or null
262 262
 	 * @return string[] HTML
263 263
 	 */
264
-	public function formatItemIdList( array $itemIds, $role = null ) {
265
-		if ( empty( $itemIds ) ) {
266
-			return [ '<ul></ul>' ];
264
+	public function formatItemIdList(array $itemIds, $role = null) {
265
+		if (empty($itemIds)) {
266
+			return ['<ul></ul>'];
267 267
 		}
268
-		$itemIds = $this->limitArrayLength( $itemIds );
269
-		$formattedItemIds = array_map( [ $this, "formatItemId" ], $itemIds, array_fill( 0, count( $itemIds ), $role ) );
268
+		$itemIds = $this->limitArrayLength($itemIds);
269
+		$formattedItemIds = array_map([$this, "formatItemId"], $itemIds, array_fill(0, count($itemIds), $role));
270 270
 		array_unshift(
271 271
 			$formattedItemIds,
272
-			'<ul><li>' . implode( '</li><li>', $formattedItemIds ) . '</li></ul>'
272
+			'<ul><li>'.implode('</li><li>', $formattedItemIds).'</li></ul>'
273 273
 		);
274 274
 		return $formattedItemIds;
275 275
 	}
@@ -284,23 +284,23 @@  discard block
 block discarded – undo
284 284
 	 * @param string|null $role one of the Role constants or null
285 285
 	 * @return string[] HTML
286 286
 	 */
287
-	public function formatEntityIdList( array $entityIds, $role = null ) {
288
-		if ( empty( $entityIds ) ) {
289
-			return [ '<ul></ul>' ];
287
+	public function formatEntityIdList(array $entityIds, $role = null) {
288
+		if (empty($entityIds)) {
289
+			return ['<ul></ul>'];
290 290
 		}
291 291
 		$formattedEntityIds = [];
292
-		foreach ( $entityIds as $entityId ) {
293
-			if ( count( $formattedEntityIds ) >= self::MAX_PARAMETER_ARRAY_LENGTH ) {
292
+		foreach ($entityIds as $entityId) {
293
+			if (count($formattedEntityIds) >= self::MAX_PARAMETER_ARRAY_LENGTH) {
294 294
 				$formattedEntityIds[] = '...';
295 295
 				break;
296 296
 			}
297
-			if ( $entityId !== null ) {
298
-				$formattedEntityIds[] = $this->formatEntityId( $entityId, $role );
297
+			if ($entityId !== null) {
298
+				$formattedEntityIds[] = $this->formatEntityId($entityId, $role);
299 299
 			}
300 300
 		}
301 301
 		array_unshift(
302 302
 			$formattedEntityIds,
303
-			'<ul><li>' . implode( '</li><li>', $formattedEntityIds ) . '</li></ul>'
303
+			'<ul><li>'.implode('</li><li>', $formattedEntityIds).'</li></ul>'
304 304
 		);
305 305
 		return $formattedEntityIds;
306 306
 	}
@@ -315,24 +315,24 @@  discard block
 block discarded – undo
315 315
 	 * @param string|null $role one of the Role constants or null
316 316
 	 * @return string[] HTML
317 317
 	 */
318
-	public function formatItemIdSnakValueList( array $values, $role = null ) {
319
-		if ( empty( $values ) ) {
320
-			return [ '<ul></ul>' ];
318
+	public function formatItemIdSnakValueList(array $values, $role = null) {
319
+		if (empty($values)) {
320
+			return ['<ul></ul>'];
321 321
 		}
322
-		$values = $this->limitArrayLength( $values );
322
+		$values = $this->limitArrayLength($values);
323 323
 		$formattedValues = array_map(
324
-			function( $value ) use ( $role ) {
325
-				if ( $value === '...' ) {
324
+			function($value) use ($role) {
325
+				if ($value === '...') {
326 326
 					return '...';
327 327
 				} else {
328
-					return $this->formatItemIdSnakValue( $value, $role );
328
+					return $this->formatItemIdSnakValue($value, $role);
329 329
 				}
330 330
 			},
331 331
 			$values
332 332
 		);
333 333
 		array_unshift(
334 334
 			$formattedValues,
335
-			'<ul><li>' . implode( '</li><li>', $formattedValues ) . '</li></ul>'
335
+			'<ul><li>'.implode('</li><li>', $formattedValues).'</li></ul>'
336 336
 		);
337 337
 		return $formattedValues;
338 338
 	}
Please login to merge, or discard this patch.
src/Role.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@
 block discarded – undo
72 72
 	 * @codeCoverageIgnore
73 73
 	 */
74 74
 	private function __construct() {
75
-		throw new LogicException( 'This class should never be instantiated.' );
75
+		throw new LogicException('This class should never be instantiated.');
76 76
 	}
77 77
 
78 78
 }
Please login to merge, or discard this patch.
src/ConstraintCheck/ItemIdSnakValue.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 	 * @param ItemId $itemId
45 45
 	 * @return self
46 46
 	 */
47
-	public static function fromItemId( ItemId $itemId ) {
47
+	public static function fromItemId(ItemId $itemId) {
48 48
 		$ret = new self;
49 49
 		$ret->itemId = $itemId;
50 50
 		return $ret;
@@ -80,16 +80,16 @@  discard block
 block discarded – undo
80 80
 	 * @throws InvalidArgumentException
81 81
 	 * @return self
82 82
 	 */
83
-	public static function fromSnak( Snak $snak ) {
84
-		switch ( true ) {
83
+	public static function fromSnak(Snak $snak) {
84
+		switch (true) {
85 85
 			case $snak instanceof PropertyValueSnak:
86 86
 				if (
87 87
 					$snak->getDataValue() instanceof EntityIdValue &&
88 88
 					$snak->getDataValue()->getEntityId() instanceof ItemId
89 89
 				) {
90
-					return self::fromItemId( $snak->getDataValue()->getEntityId() );
90
+					return self::fromItemId($snak->getDataValue()->getEntityId());
91 91
 				} else {
92
-					throw new InvalidArgumentException( 'Snak must contain item ID value or be a somevalue / novalue snak' );
92
+					throw new InvalidArgumentException('Snak must contain item ID value or be a somevalue / novalue snak');
93 93
 				}
94 94
 			case $snak instanceof PropertySomeValueSnak:
95 95
 				return self::someValue();
@@ -132,8 +132,8 @@  discard block
 block discarded – undo
132 132
 	 * @return ItemId
133 133
 	 */
134 134
 	public function getItemId() {
135
-		if ( ! $this->isValue() ) {
136
-			throw new DomainException( 'This value does not contain an item ID.' );
135
+		if (!$this->isValue()) {
136
+			throw new DomainException('This value does not contain an item ID.');
137 137
 		}
138 138
 		return $this->itemId;
139 139
 	}
@@ -145,13 +145,13 @@  discard block
 block discarded – undo
145 145
 	 * @param Snak $snak
146 146
 	 * @return bool
147 147
 	 */
148
-	public function matchesSnak( Snak $snak ) {
149
-		switch ( true ) {
148
+	public function matchesSnak(Snak $snak) {
149
+		switch (true) {
150 150
 			case $snak instanceof PropertyValueSnak:
151 151
 				return $this->isValue() &&
152 152
 					$snak->getDataValue() instanceof EntityIdValue &&
153 153
 					$snak->getDataValue()->getEntityId() instanceof ItemId &&
154
-					$snak->getDataValue()->getEntityId()->equals( $this->getItemId() );
154
+					$snak->getDataValue()->getEntityId()->equals($this->getItemId());
155 155
 			case $snak instanceof PropertySomeValueSnak:
156 156
 				return $this->isSomeValue();
157 157
 			case $snak instanceof PropertyNoValueSnak:
Please login to merge, or discard this patch.
src/ConstraintCheck/Context/MainSnakContext.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 		EntityDocument $entity,
24 24
 		Statement $statement
25 25
 	) {
26
-		parent::__construct( $entity, $statement->getMainSnak() );
26
+		parent::__construct($entity, $statement->getMainSnak());
27 27
 		$this->statement = $statement;
28 28
 	}
29 29
 
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 		return $this->statement;
40 40
 	}
41 41
 
42
-	protected function &getMainArray( array &$container ) {
42
+	protected function &getMainArray(array &$container) {
43 43
 		$statementArray = &$this->getStatementArray(
44 44
 			$container,
45 45
 			$this->entity->getId()->getSerialization(),
@@ -47,8 +47,8 @@  discard block
 block discarded – undo
47 47
 			$this->statement->getGuid()
48 48
 		);
49 49
 
50
-		if ( !array_key_exists( 'mainsnak', $statementArray ) ) {
51
-			$statementArray['mainsnak'] = [ 'hash' => $this->statement->getMainSnak()->getHash() ];
50
+		if (!array_key_exists('mainsnak', $statementArray)) {
51
+			$statementArray['mainsnak'] = ['hash' => $this->statement->getMainSnak()->getHash()];
52 52
 		}
53 53
 		$mainsnakArray = &$statementArray['mainsnak'];
54 54
 
Please login to merge, or discard this patch.
src/ConstraintCheck/Context/QualifierContext.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 		Statement $statement,
22 22
 		Snak $snak
23 23
 	) {
24
-		parent::__construct( $entity, $snak );
24
+		parent::__construct($entity, $snak);
25 25
 		$this->statement = $statement;
26 26
 	}
27 27
 
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 		return self::TYPE_QUALIFIER;
30 30
 	}
31 31
 
32
-	protected function &getMainArray( array &$container ) {
32
+	protected function &getMainArray(array &$container) {
33 33
 		$statementArray = &$this->getStatementArray(
34 34
 			$container,
35 35
 			$this->entity->getId()->getSerialization(),
@@ -37,25 +37,25 @@  discard block
 block discarded – undo
37 37
 			$this->statement->getGuid()
38 38
 		);
39 39
 
40
-		if ( !array_key_exists( 'qualifiers', $statementArray ) ) {
40
+		if (!array_key_exists('qualifiers', $statementArray)) {
41 41
 			$statementArray['qualifiers'] = [];
42 42
 		}
43 43
 		$qualifiersArray = &$statementArray['qualifiers'];
44 44
 
45 45
 		$propertyId = $this->getSnak()->getPropertyId()->getSerialization();
46
-		if ( !array_key_exists( $propertyId, $qualifiersArray ) ) {
46
+		if (!array_key_exists($propertyId, $qualifiersArray)) {
47 47
 			$qualifiersArray[$propertyId] = [];
48 48
 		}
49 49
 		$propertyArray = &$qualifiersArray[$propertyId];
50 50
 
51
-		foreach ( $propertyArray as &$potentialQualifierArray ) {
52
-			if ( $potentialQualifierArray['hash'] === $this->getSnak()->getHash() ) {
51
+		foreach ($propertyArray as &$potentialQualifierArray) {
52
+			if ($potentialQualifierArray['hash'] === $this->getSnak()->getHash()) {
53 53
 				$qualifierArray = &$potentialQualifierArray;
54 54
 				break;
55 55
 			}
56 56
 		}
57
-		if ( !isset( $qualifierArray ) ) {
58
-			$qualifierArray = [ 'hash' => $this->getSnak()->getHash() ];
57
+		if (!isset($qualifierArray)) {
58
+			$qualifierArray = ['hash' => $this->getSnak()->getHash()];
59 59
 			$propertyArray[] = &$qualifierArray;
60 60
 		}
61 61
 
Please login to merge, or discard this patch.
src/ConstraintCheck/Context/ReferenceContext.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 		Reference $reference,
29 29
 		Snak $snak
30 30
 	) {
31
-		parent::__construct( $entity, $snak );
31
+		parent::__construct($entity, $snak);
32 32
 		$this->statement = $statement;
33 33
 		$this->reference = $reference;
34 34
 	}
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 		return self::TYPE_REFERENCE;
38 38
 	}
39 39
 
40
-	protected function &getMainArray( array &$container ) {
40
+	protected function &getMainArray(array &$container) {
41 41
 		$statementArray = &$this->getStatementArray(
42 42
 			$container,
43 43
 			$this->entity->getId()->getSerialization(),
@@ -45,38 +45,38 @@  discard block
 block discarded – undo
45 45
 			$this->statement->getGuid()
46 46
 		);
47 47
 
48
-		if ( !array_key_exists( 'references', $statementArray ) ) {
48
+		if (!array_key_exists('references', $statementArray)) {
49 49
 			$statementArray['references'] = [];
50 50
 		}
51 51
 		$referencesArray = &$statementArray['references'];
52 52
 
53
-		foreach ( $referencesArray as &$potentialReferenceArray ) {
54
-			if ( $potentialReferenceArray['hash'] === $this->reference->getHash() ) {
53
+		foreach ($referencesArray as &$potentialReferenceArray) {
54
+			if ($potentialReferenceArray['hash'] === $this->reference->getHash()) {
55 55
 				$referenceArray = &$potentialReferenceArray;
56 56
 				break;
57 57
 			}
58 58
 		}
59
-		if ( !isset( $referenceArray ) ) {
60
-			$referenceArray = [ 'hash' => $this->reference->getHash(), 'snaks' => [] ];
59
+		if (!isset($referenceArray)) {
60
+			$referenceArray = ['hash' => $this->reference->getHash(), 'snaks' => []];
61 61
 			$referencesArray[] = &$referenceArray;
62 62
 		}
63 63
 
64 64
 		$snaksArray = &$referenceArray['snaks'];
65 65
 
66 66
 		$propertyId = $this->getSnak()->getPropertyId()->getSerialization();
67
-		if ( !array_key_exists( $propertyId, $snaksArray ) ) {
67
+		if (!array_key_exists($propertyId, $snaksArray)) {
68 68
 			$snaksArray[$propertyId] = [];
69 69
 		}
70 70
 		$propertyArray = &$snaksArray[$propertyId];
71 71
 
72
-		foreach ( $propertyArray as &$potentialSnakArray ) {
73
-			if ( $potentialSnakArray['hash'] === $this->getSnak()->getHash() ) {
72
+		foreach ($propertyArray as &$potentialSnakArray) {
73
+			if ($potentialSnakArray['hash'] === $this->getSnak()->getHash()) {
74 74
 				$snakArray = &$potentialSnakArray;
75 75
 				break;
76 76
 			}
77 77
 		}
78
-		if ( !isset( $snakArray ) ) {
79
-			$snakArray = [ 'hash' => $this->getSnak()->getHash() ];
78
+		if (!isset($snakArray)) {
79
+			$snakArray = ['hash' => $this->getSnak()->getHash()];
80 80
 			$propertyArray[] = &$snakArray;
81 81
 		}
82 82
 
Please login to merge, or discard this patch.
src/ConstraintCheck/Result/NullResult.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -25,22 +25,22 @@
 block discarded – undo
25 25
 	 */
26 26
 	const NULL_PROPERTY_ID = 'P2147483647';
27 27
 
28
-	public function __construct( Context $context ) {
28
+	public function __construct(Context $context) {
29 29
 		$constraint = new Constraint(
30 30
 			'null',
31
-			new PropertyId( self::NULL_PROPERTY_ID ),
31
+			new PropertyId(self::NULL_PROPERTY_ID),
32 32
 			'none',
33 33
 			[]
34 34
 		);
35
-		parent::__construct( $context, $constraint );
35
+		parent::__construct($context, $constraint);
36 36
 	}
37 37
 
38 38
 	public function getConstraint() {
39
-		throw new DomainException( 'NullResult holds no constraint' );
39
+		throw new DomainException('NullResult holds no constraint');
40 40
 	}
41 41
 
42 42
 	public function getConstraintId() {
43
-		throw new DomainException( 'NullResult holds no constraint' );
43
+		throw new DomainException('NullResult holds no constraint');
44 44
 	}
45 45
 
46 46
 }
Please login to merge, or discard this patch.
src/ConstraintCheck/ConstraintChecker.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 	 * @throws ConstraintParameterException if the constraint parameters are invalid
23 23
 	 * @throws SparqlHelperException if the checker uses SPARQL and the query times out or some other error occurs
24 24
 	 */
25
-	public function checkConstraint( Context $context, Constraint $constraint );
25
+	public function checkConstraint(Context $context, Constraint $constraint);
26 26
 
27 27
 	/**
28 28
 	 * Check if the constraint parameters of $constraint are valid.
@@ -33,6 +33,6 @@  discard block
 block discarded – undo
33 33
 	 *
34 34
 	 * @return ConstraintParameterException[]
35 35
 	 */
36
-	public function checkConstraintParameters( Constraint $constraint );
36
+	public function checkConstraintParameters(Constraint $constraint);
37 37
 
38 38
 }
Please login to merge, or discard this patch.
src/ConstraintCheck/Checker/SingleValueChecker.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -35,33 +35,33 @@
 block discarded – undo
35 35
 	 *
36 36
 	 * @return CheckResult
37 37
 	 */
38
-	public function checkConstraint( Context $context, Constraint $constraint ) {
39
-		if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) {
40
-			return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED );
38
+	public function checkConstraint(Context $context, Constraint $constraint) {
39
+		if ($context->getSnakRank() === Statement::RANK_DEPRECATED) {
40
+			return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED);
41 41
 		}
42
-		if ( $context->getType() !== Context::TYPE_STATEMENT ) {
42
+		if ($context->getType() !== Context::TYPE_STATEMENT) {
43 43
 			// TODO T175566
44
-			return new CheckResult( $context, $constraint, [], CheckResult::STATUS_NOT_MAIN_SNAK );
44
+			return new CheckResult($context, $constraint, [], CheckResult::STATUS_NOT_MAIN_SNAK);
45 45
 		}
46 46
 
47 47
 		$propertyId = $context->getSnak()->getPropertyId();
48 48
 
49 49
 		$parameters = [];
50 50
 
51
-		$propertyCountArray = $this->valueCountCheckerHelper->getPropertyCount( $context->getEntity()->getStatements() );
51
+		$propertyCountArray = $this->valueCountCheckerHelper->getPropertyCount($context->getEntity()->getStatements());
52 52
 
53
-		if ( $propertyCountArray[$propertyId->getSerialization()] > 1 ) {
54
-			$message = wfMessage( "wbqc-violation-message-single-value" )->escaped();
53
+		if ($propertyCountArray[$propertyId->getSerialization()] > 1) {
54
+			$message = wfMessage("wbqc-violation-message-single-value")->escaped();
55 55
 			$status = CheckResult::STATUS_VIOLATION;
56 56
 		} else {
57 57
 			$message = '';
58 58
 			$status = CheckResult::STATUS_COMPLIANCE;
59 59
 		}
60 60
 
61
-		return new CheckResult( $context, $constraint, $parameters, $status, $message );
61
+		return new CheckResult($context, $constraint, $parameters, $status, $message);
62 62
 	}
63 63
 
64
-	public function checkConstraintParameters( Constraint $constraint ) {
64
+	public function checkConstraintParameters(Constraint $constraint) {
65 65
 		// no parameters
66 66
 		return [];
67 67
 	}
Please login to merge, or discard this patch.