@@ -27,24 +27,24 @@ discard block |
||
27 | 27 | use WikibaseQuality\ConstraintReport\ConstraintCheck\Result\CheckResultSerializer; |
28 | 28 | |
29 | 29 | return [ |
30 | - ConstraintsServices::LOGGING_HELPER => function( MediaWikiServices $services ) { |
|
30 | + ConstraintsServices::LOGGING_HELPER => function(MediaWikiServices $services) { |
|
31 | 31 | return new LoggingHelper( |
32 | 32 | $services->getStatsdDataFactory(), |
33 | - LoggerFactory::getInstance( 'WikibaseQualityConstraints' ), |
|
33 | + LoggerFactory::getInstance('WikibaseQualityConstraints'), |
|
34 | 34 | $services->getMainConfig() |
35 | 35 | ); |
36 | 36 | }, |
37 | 37 | |
38 | - ConstraintsServices::CONSTRAINT_REPOSITORY => function( MediaWikiServices $services ) { |
|
38 | + ConstraintsServices::CONSTRAINT_REPOSITORY => function(MediaWikiServices $services) { |
|
39 | 39 | return new ConstraintRepository(); |
40 | 40 | }, |
41 | 41 | |
42 | - ConstraintsServices::CONSTRAINT_LOOKUP => function( MediaWikiServices $services ) { |
|
43 | - $constraintRepository = ConstraintsServices::getConstraintRepository( $services ); |
|
44 | - return new CachingConstraintLookup( $constraintRepository ); |
|
42 | + ConstraintsServices::CONSTRAINT_LOOKUP => function(MediaWikiServices $services) { |
|
43 | + $constraintRepository = ConstraintsServices::getConstraintRepository($services); |
|
44 | + return new CachingConstraintLookup($constraintRepository); |
|
45 | 45 | }, |
46 | 46 | |
47 | - ConstraintsServices::CHECK_RESULT_SERIALIZER => function( MediaWikiServices $services ) { |
|
47 | + ConstraintsServices::CHECK_RESULT_SERIALIZER => function(MediaWikiServices $services) { |
|
48 | 48 | return new CheckResultSerializer( |
49 | 49 | new ConstraintSerializer( |
50 | 50 | false // constraint parameters are not exposed |
@@ -55,7 +55,7 @@ discard block |
||
55 | 55 | ); |
56 | 56 | }, |
57 | 57 | |
58 | - ConstraintsServices::CHECK_RESULT_DESERIALIZER => function( MediaWikiServices $services ) { |
|
58 | + ConstraintsServices::CHECK_RESULT_DESERIALIZER => function(MediaWikiServices $services) { |
|
59 | 59 | // TODO in the future, get EntityIdParser and DataValueFactory from $services? |
60 | 60 | $repo = WikibaseRepo::getDefaultInstance(); |
61 | 61 | $entityIdParser = $repo->getEntityIdParser(); |
@@ -72,11 +72,11 @@ discard block |
||
72 | 72 | ); |
73 | 73 | }, |
74 | 74 | |
75 | - ConstraintsServices::VIOLATION_MESSAGE_SERIALIZER => function( MediaWikiServices $services ) { |
|
75 | + ConstraintsServices::VIOLATION_MESSAGE_SERIALIZER => function(MediaWikiServices $services) { |
|
76 | 76 | return new ViolationMessageSerializer(); |
77 | 77 | }, |
78 | 78 | |
79 | - ConstraintsServices::VIOLATION_MESSAGE_DESERIALIZER => function( MediaWikiServices $services ) { |
|
79 | + ConstraintsServices::VIOLATION_MESSAGE_DESERIALIZER => function(MediaWikiServices $services) { |
|
80 | 80 | // TODO in the future, get EntityIdParser and DataValueFactory from $services? |
81 | 81 | $repo = WikibaseRepo::getDefaultInstance(); |
82 | 82 | $entityIdParser = $repo->getEntityIdParser(); |
@@ -88,7 +88,7 @@ discard block |
||
88 | 88 | ); |
89 | 89 | }, |
90 | 90 | |
91 | - ConstraintsServices::CONSTRAINT_PARAMETER_PARSER => function( MediaWikiServices $services ) { |
|
91 | + ConstraintsServices::CONSTRAINT_PARAMETER_PARSER => function(MediaWikiServices $services) { |
|
92 | 92 | // TODO in the future, get DeserializerFactory and concept base URIs from $services? |
93 | 93 | $repo = WikibaseRepo::getDefaultInstance(); |
94 | 94 | $deserializerFactory = $repo->getBaseDataModelDeserializerFactory(); |
@@ -101,11 +101,11 @@ discard block |
||
101 | 101 | ); |
102 | 102 | }, |
103 | 103 | |
104 | - ConstraintsServices::CONNECTION_CHECKER_HELPER => function( MediaWikiServices $services ) { |
|
104 | + ConstraintsServices::CONNECTION_CHECKER_HELPER => function(MediaWikiServices $services) { |
|
105 | 105 | return new ConnectionCheckerHelper(); |
106 | 106 | }, |
107 | 107 | |
108 | - ConstraintsServices::RANGE_CHECKER_HELPER => function( MediaWikiServices $services ) { |
|
108 | + ConstraintsServices::RANGE_CHECKER_HELPER => function(MediaWikiServices $services) { |
|
109 | 109 | // TODO in the future, get UnitConverter from $services? |
110 | 110 | $repo = WikibaseRepo::getDefaultInstance(); |
111 | 111 | $unitConverter = $repo->getUnitConverter(); |
@@ -116,9 +116,9 @@ discard block |
||
116 | 116 | ); |
117 | 117 | }, |
118 | 118 | |
119 | - ConstraintsServices::SPARQL_HELPER => function( MediaWikiServices $services ) { |
|
120 | - $endpoint = $services->getMainConfig()->get( 'WBQualityConstraintsSparqlEndpoint' ); |
|
121 | - if ( $endpoint === '' ) { |
|
119 | + ConstraintsServices::SPARQL_HELPER => function(MediaWikiServices $services) { |
|
120 | + $endpoint = $services->getMainConfig()->get('WBQualityConstraintsSparqlEndpoint'); |
|
121 | + if ($endpoint === '') { |
|
122 | 122 | return new DummySparqlHelper(); |
123 | 123 | } |
124 | 124 | |
@@ -134,114 +134,114 @@ discard block |
||
134 | 134 | $entityIdParser, |
135 | 135 | $propertyDataTypeLookup, |
136 | 136 | $services->getMainWANObjectCache(), |
137 | - ConstraintsServices::getViolationMessageSerializer( $services ), |
|
138 | - ConstraintsServices::getViolationMessageDeserializer( $services ), |
|
137 | + ConstraintsServices::getViolationMessageSerializer($services), |
|
138 | + ConstraintsServices::getViolationMessageDeserializer($services), |
|
139 | 139 | $services->getStatsdDataFactory(), |
140 | - wfWikiID() . ' WikibaseQualityConstraints ' . Http::userAgent() |
|
140 | + wfWikiID().' WikibaseQualityConstraints '.Http::userAgent() |
|
141 | 141 | ); |
142 | 142 | }, |
143 | 143 | |
144 | - ConstraintsServices::TYPE_CHECKER_HELPER => function( MediaWikiServices $services ) { |
|
144 | + ConstraintsServices::TYPE_CHECKER_HELPER => function(MediaWikiServices $services) { |
|
145 | 145 | return new TypeCheckerHelper( |
146 | - WikibaseServices::getEntityLookup( $services ), |
|
146 | + WikibaseServices::getEntityLookup($services), |
|
147 | 147 | $services->getMainConfig(), |
148 | - ConstraintsServices::getSparqlHelper( $services ), |
|
148 | + ConstraintsServices::getSparqlHelper($services), |
|
149 | 149 | $services->getStatsdDataFactory() |
150 | 150 | ); |
151 | 151 | }, |
152 | 152 | |
153 | - ConstraintsServices::DELEGATING_CONSTRAINT_CHECKER => function( MediaWikiServices $services ) { |
|
153 | + ConstraintsServices::DELEGATING_CONSTRAINT_CHECKER => function(MediaWikiServices $services) { |
|
154 | 154 | // TODO in the future, get StatementGuidParser from $services? |
155 | 155 | $repo = WikibaseRepo::getDefaultInstance(); |
156 | 156 | $statementGuidParser = $repo->getStatementGuidParser(); |
157 | 157 | |
158 | 158 | $config = $services->getMainConfig(); |
159 | 159 | $checkerMap = [ |
160 | - $config->get( 'WBQualityConstraintsConflictsWithConstraintId' ) |
|
161 | - => ConstraintCheckerServices::getConflictsWithChecker( $services ), |
|
162 | - $config->get( 'WBQualityConstraintsItemRequiresClaimConstraintId' ) |
|
163 | - => ConstraintCheckerServices::getItemChecker( $services ), |
|
164 | - $config->get( 'WBQualityConstraintsValueRequiresClaimConstraintId' ) |
|
165 | - => ConstraintCheckerServices::getTargetRequiredClaimChecker( $services ), |
|
166 | - $config->get( 'WBQualityConstraintsSymmetricConstraintId' ) |
|
167 | - => ConstraintCheckerServices::getSymmetricChecker( $services ), |
|
168 | - $config->get( 'WBQualityConstraintsInverseConstraintId' ) |
|
169 | - => ConstraintCheckerServices::getInverseChecker( $services ), |
|
170 | - $config->get( 'WBQualityConstraintsUsedAsQualifierConstraintId' ) |
|
171 | - => ConstraintCheckerServices::getQualifierChecker( $services ), |
|
172 | - $config->get( 'WBQualityConstraintsAllowedQualifiersConstraintId' ) |
|
173 | - => ConstraintCheckerServices::getQualifiersChecker( $services ), |
|
174 | - $config->get( 'WBQualityConstraintsMandatoryQualifierConstraintId' ) |
|
175 | - => ConstraintCheckerServices::getMandatoryQualifiersChecker( $services ), |
|
176 | - $config->get( 'WBQualityConstraintsRangeConstraintId' ) |
|
177 | - => ConstraintCheckerServices::getRangeChecker( $services ), |
|
178 | - $config->get( 'WBQualityConstraintsDifferenceWithinRangeConstraintId' ) |
|
179 | - => ConstraintCheckerServices::getDiffWithinRangeChecker( $services ), |
|
180 | - $config->get( 'WBQualityConstraintsTypeConstraintId' ) |
|
181 | - => ConstraintCheckerServices::getTypeChecker( $services ), |
|
182 | - $config->get( 'WBQualityConstraintsValueTypeConstraintId' ) |
|
183 | - => ConstraintCheckerServices::getValueTypeChecker( $services ), |
|
184 | - $config->get( 'WBQualityConstraintsSingleValueConstraintId' ) |
|
185 | - => ConstraintCheckerServices::getSingleValueChecker( $services ), |
|
186 | - $config->get( 'WBQualityConstraintsMultiValueConstraintId' ) |
|
187 | - => ConstraintCheckerServices::getMultiValueChecker( $services ), |
|
188 | - $config->get( 'WBQualityConstraintsDistinctValuesConstraintId' ) |
|
189 | - => ConstraintCheckerServices::getUniqueValueChecker( $services ), |
|
190 | - $config->get( 'WBQualityConstraintsFormatConstraintId' ) |
|
191 | - => ConstraintCheckerServices::getFormatChecker( $services ), |
|
192 | - $config->get( 'WBQualityConstraintsCommonsLinkConstraintId' ) |
|
193 | - => ConstraintCheckerServices::getCommonsLinkChecker( $services ), |
|
194 | - $config->get( 'WBQualityConstraintsOneOfConstraintId' ) |
|
195 | - => ConstraintCheckerServices::getOneOfChecker( $services ), |
|
196 | - $config->get( 'WBQualityConstraintsUsedForValuesOnlyConstraintId' ) |
|
197 | - => ConstraintCheckerServices::getValueOnlyChecker( $services ), |
|
198 | - $config->get( 'WBQualityConstraintsUsedAsReferenceConstraintId' ) |
|
199 | - => ConstraintCheckerServices::getReferenceChecker( $services ), |
|
200 | - $config->get( 'WBQualityConstraintsNoBoundsConstraintId' ) |
|
201 | - => ConstraintCheckerServices::getNoBoundsChecker( $services ), |
|
202 | - $config->get( 'WBQualityConstraintsAllowedUnitsConstraintId' ) |
|
203 | - => ConstraintCheckerServices::getAllowedUnitsChecker( $services ), |
|
204 | - $config->get( 'WBQualityConstraintsSingleBestValueConstraintId' ) |
|
205 | - => ConstraintCheckerServices::getSingleBestValueChecker( $services ), |
|
206 | - $config->get( 'WBQualityConstraintsAllowedEntityTypesConstraintId' ) |
|
207 | - => ConstraintCheckerServices::getEntityTypeChecker( $services ), |
|
208 | - $config->get( 'WBQualityConstraintsNoneOfConstraintId' ) |
|
209 | - => ConstraintCheckerServices::getNoneOfChecker( $services ), |
|
210 | - $config->get( 'WBQualityConstraintsIntegerConstraintId' ) |
|
211 | - => ConstraintCheckerServices::getIntegerChecker( $services ), |
|
212 | - $config->get( 'WBQualityConstraintsCitationNeededConstraintId' ) |
|
213 | - => ConstraintCheckerServices::getCitationNeededChecker( $services ), |
|
214 | - $config->get( 'WBQualityConstraintsPropertyScopeConstraintId' ) |
|
215 | - => ConstraintCheckerServices::getPropertyScopeChecker( $services ), |
|
216 | - $config->get( 'WBQualityConstraintsContemporaryConstraintId' ) |
|
217 | - => ConstraintCheckerServices::getContemporaryChecker( $services ), |
|
160 | + $config->get('WBQualityConstraintsConflictsWithConstraintId') |
|
161 | + => ConstraintCheckerServices::getConflictsWithChecker($services), |
|
162 | + $config->get('WBQualityConstraintsItemRequiresClaimConstraintId') |
|
163 | + => ConstraintCheckerServices::getItemChecker($services), |
|
164 | + $config->get('WBQualityConstraintsValueRequiresClaimConstraintId') |
|
165 | + => ConstraintCheckerServices::getTargetRequiredClaimChecker($services), |
|
166 | + $config->get('WBQualityConstraintsSymmetricConstraintId') |
|
167 | + => ConstraintCheckerServices::getSymmetricChecker($services), |
|
168 | + $config->get('WBQualityConstraintsInverseConstraintId') |
|
169 | + => ConstraintCheckerServices::getInverseChecker($services), |
|
170 | + $config->get('WBQualityConstraintsUsedAsQualifierConstraintId') |
|
171 | + => ConstraintCheckerServices::getQualifierChecker($services), |
|
172 | + $config->get('WBQualityConstraintsAllowedQualifiersConstraintId') |
|
173 | + => ConstraintCheckerServices::getQualifiersChecker($services), |
|
174 | + $config->get('WBQualityConstraintsMandatoryQualifierConstraintId') |
|
175 | + => ConstraintCheckerServices::getMandatoryQualifiersChecker($services), |
|
176 | + $config->get('WBQualityConstraintsRangeConstraintId') |
|
177 | + => ConstraintCheckerServices::getRangeChecker($services), |
|
178 | + $config->get('WBQualityConstraintsDifferenceWithinRangeConstraintId') |
|
179 | + => ConstraintCheckerServices::getDiffWithinRangeChecker($services), |
|
180 | + $config->get('WBQualityConstraintsTypeConstraintId') |
|
181 | + => ConstraintCheckerServices::getTypeChecker($services), |
|
182 | + $config->get('WBQualityConstraintsValueTypeConstraintId') |
|
183 | + => ConstraintCheckerServices::getValueTypeChecker($services), |
|
184 | + $config->get('WBQualityConstraintsSingleValueConstraintId') |
|
185 | + => ConstraintCheckerServices::getSingleValueChecker($services), |
|
186 | + $config->get('WBQualityConstraintsMultiValueConstraintId') |
|
187 | + => ConstraintCheckerServices::getMultiValueChecker($services), |
|
188 | + $config->get('WBQualityConstraintsDistinctValuesConstraintId') |
|
189 | + => ConstraintCheckerServices::getUniqueValueChecker($services), |
|
190 | + $config->get('WBQualityConstraintsFormatConstraintId') |
|
191 | + => ConstraintCheckerServices::getFormatChecker($services), |
|
192 | + $config->get('WBQualityConstraintsCommonsLinkConstraintId') |
|
193 | + => ConstraintCheckerServices::getCommonsLinkChecker($services), |
|
194 | + $config->get('WBQualityConstraintsOneOfConstraintId') |
|
195 | + => ConstraintCheckerServices::getOneOfChecker($services), |
|
196 | + $config->get('WBQualityConstraintsUsedForValuesOnlyConstraintId') |
|
197 | + => ConstraintCheckerServices::getValueOnlyChecker($services), |
|
198 | + $config->get('WBQualityConstraintsUsedAsReferenceConstraintId') |
|
199 | + => ConstraintCheckerServices::getReferenceChecker($services), |
|
200 | + $config->get('WBQualityConstraintsNoBoundsConstraintId') |
|
201 | + => ConstraintCheckerServices::getNoBoundsChecker($services), |
|
202 | + $config->get('WBQualityConstraintsAllowedUnitsConstraintId') |
|
203 | + => ConstraintCheckerServices::getAllowedUnitsChecker($services), |
|
204 | + $config->get('WBQualityConstraintsSingleBestValueConstraintId') |
|
205 | + => ConstraintCheckerServices::getSingleBestValueChecker($services), |
|
206 | + $config->get('WBQualityConstraintsAllowedEntityTypesConstraintId') |
|
207 | + => ConstraintCheckerServices::getEntityTypeChecker($services), |
|
208 | + $config->get('WBQualityConstraintsNoneOfConstraintId') |
|
209 | + => ConstraintCheckerServices::getNoneOfChecker($services), |
|
210 | + $config->get('WBQualityConstraintsIntegerConstraintId') |
|
211 | + => ConstraintCheckerServices::getIntegerChecker($services), |
|
212 | + $config->get('WBQualityConstraintsCitationNeededConstraintId') |
|
213 | + => ConstraintCheckerServices::getCitationNeededChecker($services), |
|
214 | + $config->get('WBQualityConstraintsPropertyScopeConstraintId') |
|
215 | + => ConstraintCheckerServices::getPropertyScopeChecker($services), |
|
216 | + $config->get('WBQualityConstraintsContemporaryConstraintId') |
|
217 | + => ConstraintCheckerServices::getContemporaryChecker($services), |
|
218 | 218 | ]; |
219 | 219 | |
220 | 220 | return new DelegatingConstraintChecker( |
221 | - WikibaseServices::getEntityLookup( $services ), |
|
221 | + WikibaseServices::getEntityLookup($services), |
|
222 | 222 | $checkerMap, |
223 | - ConstraintsServices::getConstraintLookup( $services ), |
|
224 | - ConstraintsServices::getConstraintParameterParser( $services ), |
|
223 | + ConstraintsServices::getConstraintLookup($services), |
|
224 | + ConstraintsServices::getConstraintParameterParser($services), |
|
225 | 225 | $statementGuidParser, |
226 | - ConstraintsServices::getLoggingHelper( $services ), |
|
227 | - $config->get( 'WBQualityConstraintsCheckQualifiers' ), |
|
228 | - $config->get( 'WBQualityConstraintsCheckReferences' ), |
|
229 | - $config->get( 'WBQualityConstraintsPropertiesWithViolatingQualifiers' ) |
|
226 | + ConstraintsServices::getLoggingHelper($services), |
|
227 | + $config->get('WBQualityConstraintsCheckQualifiers'), |
|
228 | + $config->get('WBQualityConstraintsCheckReferences'), |
|
229 | + $config->get('WBQualityConstraintsPropertiesWithViolatingQualifiers') |
|
230 | 230 | ); |
231 | 231 | }, |
232 | 232 | |
233 | - ConstraintsServices::RESULTS_SOURCE => function( MediaWikiServices $services ) { |
|
233 | + ConstraintsServices::RESULTS_SOURCE => function(MediaWikiServices $services) { |
|
234 | 234 | $config = $services->getMainConfig(); |
235 | 235 | $resultsSource = new CheckingResultsSource( |
236 | - ConstraintsServices::getDelegatingConstraintChecker( $services ) |
|
236 | + ConstraintsServices::getDelegatingConstraintChecker($services) |
|
237 | 237 | ); |
238 | 238 | |
239 | - if ( $config->get( 'WBQualityConstraintsCacheCheckConstraintsResults' ) ) { |
|
239 | + if ($config->get('WBQualityConstraintsCacheCheckConstraintsResults')) { |
|
240 | 240 | $possiblyStaleConstraintTypes = [ |
241 | - $config->get( 'WBQualityConstraintsCommonsLinkConstraintId' ), |
|
242 | - $config->get( 'WBQualityConstraintsTypeConstraintId' ), |
|
243 | - $config->get( 'WBQualityConstraintsValueTypeConstraintId' ), |
|
244 | - $config->get( 'WBQualityConstraintsDistinctValuesConstraintId' ), |
|
241 | + $config->get('WBQualityConstraintsCommonsLinkConstraintId'), |
|
242 | + $config->get('WBQualityConstraintsTypeConstraintId'), |
|
243 | + $config->get('WBQualityConstraintsValueTypeConstraintId'), |
|
244 | + $config->get('WBQualityConstraintsDistinctValuesConstraintId'), |
|
245 | 245 | ]; |
246 | 246 | // TODO in the future, get EntityIdParser and WikiPageEntityMetaDataAccessor from $services? |
247 | 247 | $repo = WikibaseRepo::getDefaultInstance(); |
@@ -253,21 +253,21 @@ discard block |
||
253 | 253 | $entityNamespaceLookup, |
254 | 254 | $services->getSlotRoleStore() |
255 | 255 | ), |
256 | - $repo->getSettings()->getSetting( 'changesDatabase' ), |
|
256 | + $repo->getSettings()->getSetting('changesDatabase'), |
|
257 | 257 | '' // Empty string here means this only works for the local repo |
258 | 258 | ); |
259 | 259 | |
260 | 260 | $resultsSource = new CachingResultsSource( |
261 | 261 | $resultsSource, |
262 | 262 | ResultsCache::getDefaultInstance(), |
263 | - ConstraintsServices::getCheckResultSerializer( $services ), |
|
264 | - ConstraintsServices::getCheckResultDeserializer( $services ), |
|
263 | + ConstraintsServices::getCheckResultSerializer($services), |
|
264 | + ConstraintsServices::getCheckResultDeserializer($services), |
|
265 | 265 | $wikiPageEntityMetaDataAccessor, |
266 | 266 | $entityIdParser, |
267 | - $config->get( 'WBQualityConstraintsCacheCheckConstraintsTTLSeconds' ), |
|
267 | + $config->get('WBQualityConstraintsCacheCheckConstraintsTTLSeconds'), |
|
268 | 268 | $possiblyStaleConstraintTypes, |
269 | - $config->get( 'WBQualityConstraintsCacheCheckConstraintsMaximumRevisionIds' ), |
|
270 | - ConstraintsServices::getLoggingHelper( $services ) |
|
269 | + $config->get('WBQualityConstraintsCacheCheckConstraintsMaximumRevisionIds'), |
|
270 | + ConstraintsServices::getLoggingHelper($services) |
|
271 | 271 | ); |
272 | 272 | } |
273 | 273 |