Page MenuHomePhabricator

advancedsearch-field-preview-sort uses "lego" concatenation
Closed, ResolvedPublic



It appears that advancedsearch-field-preview-sort is a message of the kind that is called "lego" in the MediaWiki jargon: another message is concatenated to it in run time using $.append() instead of being added as a $1 parameter.

This is strongly discouraged, because it causes major difficulties in translation:'patchwork'_messages

For example, this causes this message to appear in the Russian Wikipedia at "Сортировать по релевантность", which is grammatically incorrect. The translators have no way to translate this correctly.

Also, I'm not sure why is the <bdi> tag needed there.

This message must stop using run-time concatenation, and document in qqq what words can after it.


Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 530231 had a related patch set uploaded (by Tonina Zhelyazkova; owner: Tonina Zhelyazkova):
[mediawiki/extensions/AdvancedSearch@master] Fix lego message for sort preview label

Change 530231 merged by jenkins-bot:
[mediawiki/extensions/AdvancedSearch@master] Fix lego message for sort preview label

Pikne assigned this task to Tonina_Zhelyazkova_WMDE.

Looks good to me on beta wiki.