Aude has uploaded a new change for review. https://gerrit.wikimedia.org/r/287013
Change subject: Fix issue of missing languages in "other languages" term box ...................................................................... Fix issue of missing languages in "other languages" term box Concatenating the $allLanguages array in EntityParserOutputGenerator doesn't work correctly when the arrays have numeric keys. (any elements in the second array with duplicate keys in the first array won't get appended). Instead we should use array_merge here. @todo: split the code that generates the TermListView here into a separate class so it can be tested more properly. Bug: T134432 Change-Id: I309669d0149f3fd6185a50881b468012e7b30d63 (cherry picked from commit e03e437e7806818472bb49d3180e264e743ffdd8) --- M repo/includes/ParserOutput/EntityParserOutputGenerator.php 1 file changed, 6 insertions(+), 3 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase refs/changes/13/287013/1 diff --git a/repo/includes/ParserOutput/EntityParserOutputGenerator.php b/repo/includes/ParserOutput/EntityParserOutputGenerator.php index 22434e4..88d46aa 100644 --- a/repo/includes/ParserOutput/EntityParserOutputGenerator.php +++ b/repo/includes/ParserOutput/EntityParserOutputGenerator.php @@ -307,13 +307,16 @@ ); $allLanguages = []; if ( $entity instanceof AliasesProvider ) { - $allLanguages += array_keys( $entity->getAliasGroups()->toTextArray() ); + $aliasLanguages = array_keys( $entity->getAliasGroups()->toTextArray() ); + $allLanguages = array_merge( $allLanguages, $aliasLanguages ); } if ( $entity instanceof DescriptionsProvider ) { - $allLanguages += array_keys( $entity->getDescriptions()->toTextArray() ); + $descriptionLanguages = array_keys( $entity->getDescriptions()->toTextArray() ); + $allLanguages = array_merge( $allLanguages, $descriptionLanguages ); } if ( $entity instanceof LabelsProvider ) { - $allLanguages += array_keys( $entity->getLabels()->toTextArray() ); + $labelLanguages = array_keys( $entity->getLabels()->toTextArray() ); + $allLanguages = array_merge( $allLanguages, $labelLanguages ); } $allLanguages = array_unique( $allLanguages ); -- To view, visit https://gerrit.wikimedia.org/r/287013 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I309669d0149f3fd6185a50881b468012e7b30d63 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: wmf/1.27.0-wmf.23 Gerrit-Owner: Aude <aude.w...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits