wikimedia /
mediawiki-extensions-WikibaseQualityConstraints
@@ -43,52 +43,51 @@ |
||
| 43 | 43 | ) { |
| 44 | 44 | $results = []; |
| 45 | 45 | $metadatas = []; |
| 46 | - $statusesFlipped = array_flip( $statuses ); |
|
| 47 | - foreach ( $entityIds as $entityId ) { |
|
| 46 | + $statusesFlipped = array_flip($statuses); |
|
| 47 | + foreach ($entityIds as $entityId) { |
|
| 48 | 48 | $entityResults = $this->delegatingConstraintChecker->checkAgainstConstraintsOnEntityId( |
| 49 | 49 | $entityId, |
| 50 | 50 | $constraintIds, |
| 51 | - [ $this, 'defaultResultsPerContext' ], |
|
| 52 | - [ $this, 'defaultResultsPerEntity' ] |
|
| 51 | + [$this, 'defaultResultsPerContext'], |
|
| 52 | + [$this, 'defaultResultsPerEntity'] |
|
| 53 | 53 | ); |
| 54 | - foreach ( $entityResults as $result ) { |
|
| 54 | + foreach ($entityResults as $result) { |
|
| 55 | 55 | $metadatas[] = $result->getMetadata(); |
| 56 | - if ( $this->statusSelected( $statusesFlipped, $result ) ) { |
|
| 56 | + if ($this->statusSelected($statusesFlipped, $result)) { |
|
| 57 | 57 | $results[] = $result; |
| 58 | 58 | } |
| 59 | 59 | } |
| 60 | 60 | } |
| 61 | - foreach ( $claimIds as $claimId ) { |
|
| 61 | + foreach ($claimIds as $claimId) { |
|
| 62 | 62 | $claimResults = $this->delegatingConstraintChecker->checkAgainstConstraintsOnClaimId( |
| 63 | 63 | $claimId, |
| 64 | 64 | $constraintIds, |
| 65 | - [ $this, 'defaultResultsPerContext' ] |
|
| 65 | + [$this, 'defaultResultsPerContext'] |
|
| 66 | 66 | ); |
| 67 | - foreach ( $claimResults as $result ) { |
|
| 67 | + foreach ($claimResults as $result) { |
|
| 68 | 68 | $metadatas[] = $result->getMetadata(); |
| 69 | - if ( $this->statusSelected( $statusesFlipped, $result ) ) { |
|
| 69 | + if ($this->statusSelected($statusesFlipped, $result)) { |
|
| 70 | 70 | $results[] = $result; |
| 71 | 71 | } |
| 72 | 72 | } |
| 73 | 73 | } |
| 74 | 74 | return new CachedCheckResults( |
| 75 | 75 | $results, |
| 76 | - Metadata::merge( $metadatas ) |
|
| 76 | + Metadata::merge($metadatas) |
|
| 77 | 77 | ); |
| 78 | 78 | } |
| 79 | 79 | |
| 80 | - public function defaultResultsPerContext( Context $context ) { |
|
| 80 | + public function defaultResultsPerContext(Context $context) { |
|
| 81 | 81 | return $context->getType() === Context::TYPE_STATEMENT ? |
| 82 | - [ new NullResult( $context->getCursor() ) ] : |
|
| 83 | - []; |
|
| 82 | + [new NullResult($context->getCursor())] : []; |
|
| 84 | 83 | } |
| 85 | 84 | |
| 86 | - public function defaultResultsPerEntity( EntityId $entityId ) { |
|
| 87 | - return [ new NullResult( new EntityContextCursor( $entityId->getSerialization() ) ) ]; |
|
| 85 | + public function defaultResultsPerEntity(EntityId $entityId) { |
|
| 86 | + return [new NullResult(new EntityContextCursor($entityId->getSerialization()))]; |
|
| 88 | 87 | } |
| 89 | 88 | |
| 90 | - public function statusSelected( array $statusesFlipped, CheckResult $result ) { |
|
| 91 | - return array_key_exists( $result->getStatus(), $statusesFlipped ) || |
|
| 89 | + public function statusSelected(array $statusesFlipped, CheckResult $result) { |
|
| 90 | + return array_key_exists($result->getStatus(), $statusesFlipped) || |
|
| 92 | 91 | $result instanceof NullResult; |
| 93 | 92 | } |
| 94 | 93 | |
