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