Page MenuHomePhabricator

Remove <DynamicPageList> tags with less-than signs from intersection/DynamicPageList extension messages
Closed, ResolvedPublicBUG REPORT

Description

The L10n bot change https://gerrit.wikimedia.org/r/c/mediawiki/extensions/intersection/+/995166 had to be manually merged because the message translation included a <graph> tag, which is unrecognized by Jenkins job configuration:

[mediawiki-i18n-check-docker] $ /bin/bash /tmp/jenkins1401340557113915177.sh
+ cd src
++ pwd
+ git config --global --add safe.directory /srv/jenkins/workspace/mediawiki-i18n-check-docker/src
++ mktemp
+ additions=/tmp/tmp.IzZMsfg0h8
+ git show FETCH_HEAD -U0
+ grep '^+'
+ contains_disallowed_html /tmp/tmp.IzZMsfg0h8
+ cat /tmp/tmp.IzZMsfg0h8
+ sed -E -e 's/<\/? ?(abbr|span|br|strong|em|pre|nowiki|charinsert|b|del|code|kbd|p|tt|div|i|big|sup|sub|samp|var|small|h1|h2|h3|h4|u|ol|ul|li|mark|user|page|hr|templatestyles|NDL|math|syntaxhighlight|templatedata|pagelist|bdi|ref|dd|dl)( ((class|title|lang|xml:lang|dir|xmlns|display|alttext)=\\?["'\''][^=<>"'\'']*\\?["'\'']))* ?\/?>//g' -e 's/<!--//g' -e 's/<https?:\/\/[a-zA-Z0-9./-]*>//g'
+ grep '<'
+	"intersection-category-desc": "Síðan innifelur <DynamicPageList>-merki"
+ echo 'HTML detected. Manual review required'
HTML detected. Manual review required

The easy solution appears to be to replace <code><nowiki><DynamicPageList></nowiki></code> with <code>&lt;DynamicPageList&gt;</code>, similarly to T356549.

Event Timeline

Amire80 renamed this task from Remove <graph> tags with less-than signs from Graph extension messages to Remove <DynamicPageList> tags with less-than signs from intersection/DynamicPageList extension messages.Feb 3 2024, 3:56 PM

Change 995552 had a related patch set uploaded (by Amire80; author: Amire80):

[mediawiki/extensions/intersection@master] Replace < with &lt; in a message

https://gerrit.wikimedia.org/r/995552

There are many messages (mainly extension descriptions) with the same issue. It would be good to change them all to be consistent, or go a proposed step further: T326722: Parser function to render a bare wikitext element.

Change 995552 merged by jenkins-bot:

[mediawiki/extensions/intersection@master] Replace < with &lt; in a message

https://gerrit.wikimedia.org/r/995552