@@ -16,7 +16,7 @@ discard block |
||
16 | 16 | /** |
17 | 17 | * @param bool $serializeConstraintParameters Whether to serialize constraint parameters or not. |
18 | 18 | */ |
19 | - public function __construct( $serializeConstraintParameters = true ) { |
|
19 | + public function __construct($serializeConstraintParameters = true) { |
|
20 | 20 | $this->serializeConstraintParameters = $serializeConstraintParameters; |
21 | 21 | } |
22 | 22 | |
@@ -24,13 +24,13 @@ discard block |
||
24 | 24 | * @param Constraint $constraint |
25 | 25 | * @return array |
26 | 26 | */ |
27 | - public function serialize( Constraint $constraint ) { |
|
27 | + public function serialize(Constraint $constraint) { |
|
28 | 28 | $serialization = [ |
29 | 29 | 'id' => $constraint->getConstraintId(), |
30 | 30 | 'pid' => $constraint->getPropertyId()->getSerialization(), |
31 | 31 | 'qid' => $constraint->getConstraintTypeItemId(), |
32 | 32 | ]; |
33 | - if ( $this->serializeConstraintParameters ) { |
|
33 | + if ($this->serializeConstraintParameters) { |
|
34 | 34 | $serialization['params'] = $constraint->getConstraintParameters(); |
35 | 35 | } |
36 | 36 | return $serialization; |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | * @param EntityId $entityId An entity ID from which the value was derived. |
38 | 38 | * @return self Indication that a value was derived from the entity with the given ID. |
39 | 39 | */ |
40 | - public static function ofEntityId( EntityId $entityId ) { |
|
40 | + public static function ofEntityId(EntityId $entityId) { |
|
41 | 41 | $ret = new self; |
42 | 42 | $ret->entityIds[] = $entityId; |
43 | 43 | return $ret; |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | * @return self Indication that a value will only remain valid |
50 | 50 | * as long as the given time value is in the future, not in the past. |
51 | 51 | */ |
52 | - public static function ofFutureTime( TimeValue $timeValue ) { |
|
52 | + public static function ofFutureTime(TimeValue $timeValue) { |
|
53 | 53 | $ret = new self; |
54 | 54 | $ret->timeValue = $timeValue; |
55 | 55 | return $ret; |
@@ -59,17 +59,17 @@ discard block |
||
59 | 59 | * @param self[] $metadatas |
60 | 60 | * @return self |
61 | 61 | */ |
62 | - public static function merge( array $metadatas ) { |
|
63 | - Assert::parameterElementType( self::class, $metadatas, '$metadatas' ); |
|
62 | + public static function merge(array $metadatas) { |
|
63 | + Assert::parameterElementType(self::class, $metadatas, '$metadatas'); |
|
64 | 64 | $ret = new self; |
65 | 65 | $entityIds = []; |
66 | - foreach ( $metadatas as $metadata ) { |
|
67 | - foreach ( $metadata->entityIds as $entityId ) { |
|
66 | + foreach ($metadatas as $metadata) { |
|
67 | + foreach ($metadata->entityIds as $entityId) { |
|
68 | 68 | $entityIds[$entityId->getSerialization()] = $entityId; |
69 | 69 | } |
70 | - $ret->timeValue = self::minTimeValue( $ret->timeValue, $metadata->timeValue ); |
|
70 | + $ret->timeValue = self::minTimeValue($ret->timeValue, $metadata->timeValue); |
|
71 | 71 | } |
72 | - $ret->entityIds = array_values( $entityIds ); |
|
72 | + $ret->entityIds = array_values($entityIds); |
|
73 | 73 | return $ret; |
74 | 74 | } |
75 | 75 | |
@@ -78,14 +78,14 @@ discard block |
||
78 | 78 | * @param TimeValue|null $t2 |
79 | 79 | * @return TimeValue|null |
80 | 80 | */ |
81 | - private static function minTimeValue( TimeValue $t1 = null, TimeValue $t2 = null ) { |
|
82 | - if ( $t1 === null ) { |
|
81 | + private static function minTimeValue(TimeValue $t1 = null, TimeValue $t2 = null) { |
|
82 | + if ($t1 === null) { |
|
83 | 83 | return $t2; |
84 | 84 | } |
85 | - if ( $t2 === null ) { |
|
85 | + if ($t2 === null) { |
|
86 | 86 | return $t1; |
87 | 87 | } |
88 | - return ( new TimeValueComparer() )->getMinimum( $t1, $t2 ); |
|
88 | + return (new TimeValueComparer())->getMinimum($t1, $t2); |
|
89 | 89 | } |
90 | 90 | |
91 | 91 | /** |
@@ -92,43 +92,43 @@ |
||
92 | 92 | return $this->referenceHash; |
93 | 93 | } |
94 | 94 | |
95 | - protected function &getMainArray( array &$container ) { |
|
96 | - $statementArray = &$this->getStatementArray( $container ); |
|
95 | + protected function &getMainArray(array &$container) { |
|
96 | + $statementArray = &$this->getStatementArray($container); |
|
97 | 97 | |
98 | - if ( !array_key_exists( 'references', $statementArray ) ) { |
|
98 | + if (!array_key_exists('references', $statementArray)) { |
|
99 | 99 | $statementArray['references'] = []; |
100 | 100 | } |
101 | 101 | $referencesArray = &$statementArray['references']; |
102 | 102 | |
103 | 103 | $referenceHash = $this->getReferenceHash(); |
104 | - foreach ( $referencesArray as &$potentialReferenceArray ) { |
|
105 | - if ( $potentialReferenceArray['hash'] === $referenceHash ) { |
|
104 | + foreach ($referencesArray as &$potentialReferenceArray) { |
|
105 | + if ($potentialReferenceArray['hash'] === $referenceHash) { |
|
106 | 106 | $referenceArray = &$potentialReferenceArray; |
107 | 107 | break; |
108 | 108 | } |
109 | 109 | } |
110 | - if ( !isset( $referenceArray ) ) { |
|
111 | - $referenceArray = [ 'hash' => $referenceHash, 'snaks' => [] ]; |
|
110 | + if (!isset($referenceArray)) { |
|
111 | + $referenceArray = ['hash' => $referenceHash, 'snaks' => []]; |
|
112 | 112 | $referencesArray[] = &$referenceArray; |
113 | 113 | } |
114 | 114 | |
115 | 115 | $snaksArray = &$referenceArray['snaks']; |
116 | 116 | |
117 | 117 | $snakPropertyId = $this->getSnakPropertyId(); |
118 | - if ( !array_key_exists( $snakPropertyId, $snaksArray ) ) { |
|
118 | + if (!array_key_exists($snakPropertyId, $snaksArray)) { |
|
119 | 119 | $snaksArray[$snakPropertyId] = []; |
120 | 120 | } |
121 | 121 | $propertyArray = &$snaksArray[$snakPropertyId]; |
122 | 122 | |
123 | 123 | $snakHash = $this->getSnakHash(); |
124 | - foreach ( $propertyArray as &$potentialSnakArray ) { |
|
125 | - if ( $potentialSnakArray['hash'] === $snakHash ) { |
|
124 | + foreach ($propertyArray as &$potentialSnakArray) { |
|
125 | + if ($potentialSnakArray['hash'] === $snakHash) { |
|
126 | 126 | $snakArray = &$potentialSnakArray; |
127 | 127 | break; |
128 | 128 | } |
129 | 129 | } |
130 | - if ( !isset( $snakArray ) ) { |
|
131 | - $snakArray = [ 'hash' => $snakHash ]; |
|
130 | + if (!isset($snakArray)) { |
|
131 | + $snakArray = ['hash' => $snakHash]; |
|
132 | 132 | $propertyArray[] = &$snakArray; |
133 | 133 | } |
134 | 134 |
@@ -80,29 +80,29 @@ |
||
80 | 80 | return $this->snakHash; |
81 | 81 | } |
82 | 82 | |
83 | - protected function &getMainArray( array &$container ) { |
|
84 | - $statementArray = &$this->getStatementArray( $container ); |
|
83 | + protected function &getMainArray(array &$container) { |
|
84 | + $statementArray = &$this->getStatementArray($container); |
|
85 | 85 | |
86 | - if ( !array_key_exists( 'qualifiers', $statementArray ) ) { |
|
86 | + if (!array_key_exists('qualifiers', $statementArray)) { |
|
87 | 87 | $statementArray['qualifiers'] = []; |
88 | 88 | } |
89 | 89 | $qualifiersArray = &$statementArray['qualifiers']; |
90 | 90 | |
91 | 91 | $snakPropertyId = $this->getSnakPropertyId(); |
92 | - if ( !array_key_exists( $snakPropertyId, $qualifiersArray ) ) { |
|
92 | + if (!array_key_exists($snakPropertyId, $qualifiersArray)) { |
|
93 | 93 | $qualifiersArray[$snakPropertyId] = []; |
94 | 94 | } |
95 | 95 | $propertyArray = &$qualifiersArray[$snakPropertyId]; |
96 | 96 | |
97 | 97 | $snakHash = $this->getSnakHash(); |
98 | - foreach ( $propertyArray as &$potentialQualifierArray ) { |
|
99 | - if ( $potentialQualifierArray['hash'] === $snakHash ) { |
|
98 | + foreach ($propertyArray as &$potentialQualifierArray) { |
|
99 | + if ($potentialQualifierArray['hash'] === $snakHash) { |
|
100 | 100 | $qualifierArray = &$potentialQualifierArray; |
101 | 101 | break; |
102 | 102 | } |
103 | 103 | } |
104 | - if ( !isset( $qualifierArray ) ) { |
|
105 | - $qualifierArray = [ 'hash' => $snakHash ]; |
|
104 | + if (!isset($qualifierArray)) { |
|
105 | + $qualifierArray = ['hash' => $snakHash]; |
|
106 | 106 | $propertyArray[] = &$qualifierArray; |
107 | 107 | } |
108 | 108 |
@@ -72,12 +72,12 @@ |
||
72 | 72 | return $this->snakHash; |
73 | 73 | } |
74 | 74 | |
75 | - protected function &getMainArray( array &$container ) { |
|
76 | - $statementArray = &$this->getStatementArray( $container ); |
|
75 | + protected function &getMainArray(array &$container) { |
|
76 | + $statementArray = &$this->getStatementArray($container); |
|
77 | 77 | |
78 | - if ( !array_key_exists( 'mainsnak', $statementArray ) ) { |
|
78 | + if (!array_key_exists('mainsnak', $statementArray)) { |
|
79 | 79 | $snakHash = $this->getSnakHash(); |
80 | - $statementArray['mainsnak'] = [ 'hash' => $snakHash ]; |
|
80 | + $statementArray['mainsnak'] = ['hash' => $snakHash]; |
|
81 | 81 | } |
82 | 82 | $mainsnakArray = &$statementArray['mainsnak']; |
83 | 83 |
@@ -11,8 +11,8 @@ discard block |
||
11 | 11 | */ |
12 | 12 | class ContextCursorDeserializer { |
13 | 13 | |
14 | - public function deserialize( array $serialization ) { |
|
15 | - switch ( $serialization['t'] ) { |
|
14 | + public function deserialize(array $serialization) { |
|
15 | + switch ($serialization['t']) { |
|
16 | 16 | case Context::TYPE_STATEMENT: |
17 | 17 | return new MainSnakContextCursor( |
18 | 18 | $serialization['i'], |
@@ -39,7 +39,7 @@ discard block |
||
39 | 39 | ); |
40 | 40 | default: |
41 | 41 | throw new InvalidArgumentException( |
42 | - 'Unknown serialization type ' . $serialization['t'] |
|
42 | + 'Unknown serialization type '.$serialization['t'] |
|
43 | 43 | ); |
44 | 44 | } |
45 | 45 | } |
@@ -21,9 +21,9 @@ |
||
21 | 21 | /** |
22 | 22 | * @param ViolationMessage $violationMessage |
23 | 23 | */ |
24 | - public function __construct( ViolationMessage $violationMessage ) { |
|
25 | - $message = '⧼' . $violationMessage->getMessageKey() . '⧽'; |
|
26 | - parent::__construct( $message ); |
|
24 | + public function __construct(ViolationMessage $violationMessage) { |
|
25 | + $message = '⧼'.$violationMessage->getMessageKey().'⧽'; |
|
26 | + parent::__construct($message); |
|
27 | 27 | |
28 | 28 | $this->violationMessage = $violationMessage; |
29 | 29 | } |
@@ -75,7 +75,7 @@ discard block |
||
75 | 75 | * @throws ConstraintParameterException |
76 | 76 | * @return CheckResult |
77 | 77 | */ |
78 | - public function checkConstraint( Context $context, Constraint $constraint ) { |
|
78 | + public function checkConstraint(Context $context, Constraint $constraint) { |
|
79 | 79 | $parameters = []; |
80 | 80 | $constraintParameters = $constraint->getConstraintParameters(); |
81 | 81 | $unitsParameter = $this->constraintParameterParser |
@@ -85,50 +85,49 @@ discard block |
||
85 | 85 | ); |
86 | 86 | |
87 | 87 | $snak = $context->getSnak(); |
88 | - if ( !$snak instanceof PropertyValueSnak ) { |
|
88 | + if (!$snak instanceof PropertyValueSnak) { |
|
89 | 89 | // nothing to check |
90 | - return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE ); |
|
90 | + return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_COMPLIANCE); |
|
91 | 91 | } |
92 | 92 | |
93 | 93 | $dataValue = $snak->getDataValue(); |
94 | - if ( !$dataValue instanceof UnboundedQuantityValue ) { |
|
95 | - $message = ( new ViolationMessage( 'wbqc-violation-message-value-needed-of-type' ) ) |
|
96 | - ->withEntityId( new ItemId( $constraint->getConstraintTypeItemId() ), Role::CONSTRAINT_TYPE_ITEM ) |
|
97 | - ->withDataValueType( 'quantity' ); |
|
98 | - return new CheckResult( $context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message ); |
|
94 | + if (!$dataValue instanceof UnboundedQuantityValue) { |
|
95 | + $message = (new ViolationMessage('wbqc-violation-message-value-needed-of-type')) |
|
96 | + ->withEntityId(new ItemId($constraint->getConstraintTypeItemId()), Role::CONSTRAINT_TYPE_ITEM) |
|
97 | + ->withDataValueType('quantity'); |
|
98 | + return new CheckResult($context, $constraint, $parameters, CheckResult::STATUS_VIOLATION, $message); |
|
99 | 99 | } |
100 | 100 | |
101 | - if ( $dataValue->getUnit() === '1' ) { |
|
102 | - return $this->checkUnitless( $context, $constraint, $unitsParameter, $snak ); |
|
101 | + if ($dataValue->getUnit() === '1') { |
|
102 | + return $this->checkUnitless($context, $constraint, $unitsParameter, $snak); |
|
103 | 103 | } |
104 | 104 | |
105 | 105 | $status = CheckResult::STATUS_VIOLATION; |
106 | - $actualUnit = $this->standardize( $dataValue )->getUnit(); |
|
107 | - foreach ( $unitsParameter->getUnitQuantities() as $unitQuantity ) { |
|
108 | - $allowedUnit = $this->standardize( $unitQuantity )->getUnit(); |
|
109 | - if ( $actualUnit === $allowedUnit ) { |
|
106 | + $actualUnit = $this->standardize($dataValue)->getUnit(); |
|
107 | + foreach ($unitsParameter->getUnitQuantities() as $unitQuantity) { |
|
108 | + $allowedUnit = $this->standardize($unitQuantity)->getUnit(); |
|
109 | + if ($actualUnit === $allowedUnit) { |
|
110 | 110 | $status = CheckResult::STATUS_COMPLIANCE; |
111 | 111 | break; |
112 | 112 | } |
113 | 113 | } |
114 | 114 | |
115 | - if ( $status === CheckResult::STATUS_VIOLATION ) { |
|
116 | - if ( $unitsParameter->getUnitItemIds() === [] ) { |
|
117 | - $message = ( new ViolationMessage( 'wbqc-violation-message-units-none' ) ) |
|
118 | - ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY ); |
|
115 | + if ($status === CheckResult::STATUS_VIOLATION) { |
|
116 | + if ($unitsParameter->getUnitItemIds() === []) { |
|
117 | + $message = (new ViolationMessage('wbqc-violation-message-units-none')) |
|
118 | + ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY); |
|
119 | 119 | } else { |
120 | 120 | $messageKey = $unitsParameter->getUnitlessAllowed() ? |
121 | - 'wbqc-violation-message-units-or-none' : |
|
122 | - 'wbqc-violation-message-units'; |
|
123 | - $message = ( new ViolationMessage( $messageKey ) ) |
|
124 | - ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY ) |
|
125 | - ->withEntityIdList( $unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE ); |
|
121 | + 'wbqc-violation-message-units-or-none' : 'wbqc-violation-message-units'; |
|
122 | + $message = (new ViolationMessage($messageKey)) |
|
123 | + ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY) |
|
124 | + ->withEntityIdList($unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE); |
|
126 | 125 | } |
127 | 126 | } else { |
128 | 127 | $message = null; |
129 | 128 | } |
130 | 129 | |
131 | - return new CheckResult( $context, $constraint, $parameters, $status, $message ); |
|
130 | + return new CheckResult($context, $constraint, $parameters, $status, $message); |
|
132 | 131 | } |
133 | 132 | |
134 | 133 | /** |
@@ -144,17 +143,17 @@ discard block |
||
144 | 143 | UnitsParameter $unitsParameter, |
145 | 144 | PropertyValueSnak $snak |
146 | 145 | ) { |
147 | - if ( $unitsParameter->getUnitlessAllowed() ) { |
|
146 | + if ($unitsParameter->getUnitlessAllowed()) { |
|
148 | 147 | $message = null; |
149 | 148 | $status = CheckResult::STATUS_COMPLIANCE; |
150 | 149 | } else { |
151 | - $message = ( new ViolationMessage( 'wbqc-violation-message-units' ) ) |
|
152 | - ->withEntityId( $snak->getPropertyId(), Role::CONSTRAINT_PROPERTY ) |
|
153 | - ->withEntityIdList( $unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE ); |
|
150 | + $message = (new ViolationMessage('wbqc-violation-message-units')) |
|
151 | + ->withEntityId($snak->getPropertyId(), Role::CONSTRAINT_PROPERTY) |
|
152 | + ->withEntityIdList($unitsParameter->getUnitItemIds(), Role::CONSTRAINT_PARAMETER_VALUE); |
|
154 | 153 | $status = CheckResult::STATUS_VIOLATION; |
155 | 154 | } |
156 | 155 | |
157 | - return new CheckResult( $context, $constraint, [], $status, $message ); |
|
156 | + return new CheckResult($context, $constraint, [], $status, $message); |
|
158 | 157 | } |
159 | 158 | |
160 | 159 | /** |
@@ -163,20 +162,20 @@ discard block |
||
163 | 162 | * @param UnboundedQuantityValue $value |
164 | 163 | * @return UnboundedQuantityValue |
165 | 164 | */ |
166 | - private function standardize( UnboundedQuantityValue $value ) { |
|
167 | - if ( $this->unitConverter === null ) { |
|
165 | + private function standardize(UnboundedQuantityValue $value) { |
|
166 | + if ($this->unitConverter === null) { |
|
168 | 167 | return $value; |
169 | 168 | } |
170 | 169 | |
171 | - $standard = $this->unitConverter->toStandardUnits( $value ); |
|
172 | - if ( $standard !== null ) { |
|
170 | + $standard = $this->unitConverter->toStandardUnits($value); |
|
171 | + if ($standard !== null) { |
|
173 | 172 | return $standard; |
174 | 173 | } else { |
175 | 174 | return $value; |
176 | 175 | } |
177 | 176 | } |
178 | 177 | |
179 | - public function checkConstraintParameters( Constraint $constraint ) { |
|
178 | + public function checkConstraintParameters(Constraint $constraint) { |
|
180 | 179 | $constraintParameters = $constraint->getConstraintParameters(); |
181 | 180 | $exceptions = []; |
182 | 181 | try { |
@@ -185,7 +184,7 @@ discard block |
||
185 | 184 | $constraint->getConstraintTypeItemId(), |
186 | 185 | true |
187 | 186 | ); |
188 | - } catch ( ConstraintParameterException $e ) { |
|
187 | + } catch (ConstraintParameterException $e) { |
|
189 | 188 | $exceptions[] = $e; |
190 | 189 | } |
191 | 190 | return $exceptions; |
@@ -66,9 +66,9 @@ discard block |
||
66 | 66 | * |
67 | 67 | * @return CheckResult |
68 | 68 | */ |
69 | - public function checkConstraint( Context $context, Constraint $constraint ) { |
|
70 | - if ( $context->getSnakRank() === Statement::RANK_DEPRECATED ) { |
|
71 | - return new CheckResult( $context, $constraint, [], CheckResult::STATUS_DEPRECATED ); |
|
69 | + public function checkConstraint(Context $context, Constraint $constraint) { |
|
70 | + if ($context->getSnakRank() === Statement::RANK_DEPRECATED) { |
|
71 | + return new CheckResult($context, $constraint, [], CheckResult::STATUS_DEPRECATED); |
|
72 | 72 | } |
73 | 73 | |
74 | 74 | $parameters = []; |
@@ -80,13 +80,13 @@ discard block |
||
80 | 80 | |
81 | 81 | $propertyId = $context->getSnak()->getPropertyId(); |
82 | 82 | $bestRankCount = $this->valueCountCheckerHelper->getPropertyCount( |
83 | - $context->getSnakGroup( Context::GROUP_BEST_RANK, $separators ), |
|
83 | + $context->getSnakGroup(Context::GROUP_BEST_RANK, $separators), |
|
84 | 84 | $propertyId |
85 | 85 | ); |
86 | 86 | |
87 | - if ( $bestRankCount > 1 ) { |
|
87 | + if ($bestRankCount > 1) { |
|
88 | 88 | $nonDeprecatedCount = $this->valueCountCheckerHelper->getPropertyCount( |
89 | - $context->getSnakGroup( Context::GROUP_NON_DEPRECATED ), |
|
89 | + $context->getSnakGroup(Context::GROUP_NON_DEPRECATED), |
|
90 | 90 | $propertyId |
91 | 91 | ); |
92 | 92 | $message = $this->getViolationMessage( |
@@ -101,15 +101,15 @@ discard block |
||
101 | 101 | $status = CheckResult::STATUS_COMPLIANCE; |
102 | 102 | } |
103 | 103 | |
104 | - return new CheckResult( $context, $constraint, $parameters, $status, $message ); |
|
104 | + return new CheckResult($context, $constraint, $parameters, $status, $message); |
|
105 | 105 | } |
106 | 106 | |
107 | - public function checkConstraintParameters( Constraint $constraint ) { |
|
107 | + public function checkConstraintParameters(Constraint $constraint) { |
|
108 | 108 | $constraintParameters = $constraint->getConstraintParameters(); |
109 | 109 | $exceptions = []; |
110 | 110 | try { |
111 | - $this->constraintParameterParser->parseSeparatorsParameter( $constraintParameters ); |
|
112 | - } catch ( ConstraintParameterException $e ) { |
|
111 | + $this->constraintParameterParser->parseSeparatorsParameter($constraintParameters); |
|
112 | + } catch (ConstraintParameterException $e) { |
|
113 | 113 | $exceptions[] = $e; |
114 | 114 | } |
115 | 115 | return $exceptions; |
@@ -128,23 +128,23 @@ discard block |
||
128 | 128 | $separators, |
129 | 129 | $propertyId |
130 | 130 | ) { |
131 | - if ( $bestRankCount === $nonDeprecatedCount ) { |
|
132 | - if ( $separators === [] ) { |
|
131 | + if ($bestRankCount === $nonDeprecatedCount) { |
|
132 | + if ($separators === []) { |
|
133 | 133 | $messageKey = 'wbqc-violation-message-single-best-value-no-preferred'; |
134 | 134 | } else { |
135 | 135 | $messageKey = 'wbqc-violation-message-single-best-value-no-preferred-separators'; |
136 | 136 | } |
137 | 137 | } else { |
138 | - if ( $separators === [] ) { |
|
138 | + if ($separators === []) { |
|
139 | 139 | $messageKey = 'wbqc-violation-message-single-best-value-multi-preferred'; |
140 | 140 | } else { |
141 | 141 | $messageKey = 'wbqc-violation-message-single-best-value-multi-preferred-separators'; |
142 | 142 | } |
143 | 143 | } |
144 | 144 | |
145 | - return ( new ViolationMessage( $messageKey ) ) |
|
146 | - ->withEntityId( $propertyId ) |
|
147 | - ->withEntityIdList( $separators ); |
|
145 | + return (new ViolationMessage($messageKey)) |
|
146 | + ->withEntityId($propertyId) |
|
147 | + ->withEntityIdList($separators); |
|
148 | 148 | } |
149 | 149 | |
150 | 150 | } |