Jonas Kress (WMDE) has uploaded a new change for review. https://gerrit.wikimedia.org/r/243135
Change subject: Fix merging of entity usage aspects ...................................................................... Fix merging of entity usage aspects Fix merging of entity usage aspects by using array_merge instead of '+' in AffectedPagesFinder. Found while investigating bug T112649. Bug: T112649 Change-Id: I84f3b4dfa76731e4648a19913d0ad14e32ab02fb --- M client/includes/Changes/AffectedPagesFinder.php 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase refs/changes/35/243135/1 diff --git a/client/includes/Changes/AffectedPagesFinder.php b/client/includes/Changes/AffectedPagesFinder.php index b8d1376..69a5339 100644 --- a/client/includes/Changes/AffectedPagesFinder.php +++ b/client/includes/Changes/AffectedPagesFinder.php @@ -188,7 +188,7 @@ // @todo: more than one entity at once! array( $entityId ), // Look up pages that are marked as either using one of the changed or all aspects - $changedAspects + array( EntityUsage::ALL_USAGE ) + array_merge( $changedAspects, array( EntityUsage::ALL_USAGE ) ) ); // @todo: use iterators throughout! -- To view, visit https://gerrit.wikimedia.org/r/243135 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I84f3b4dfa76731e4648a19913d0ad14e32ab02fb Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Jonas Kress (WMDE) <jonas.kr...@wikimedia.de> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits