Page MenuHomePhabricator

Adding multiple maintenance tags to page with one maintenance tag and no {{Multiple issues}} should place old tag inside {{Multiple issues}}
Open, Needs TriagePublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Log out
  • Create a page with wikitext {{Disputed}} (one maintenance tag)
  • Log in
  • Visit page
  • Page curation toolbar -> maintenance tags -> tag it with two random tags such as {{Advert}} and {{All plot}}`

What happens?:

image.png (518×387 px, 17 KB)

What should have happened instead?:

image.png (515×412 px, 18 KB)

Software version (skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Event Timeline

Twinkle's maintenance tag code is pretty convoluted. I wrote a patch for it once and I think I ended up abandoning it.

AFC Helper script has a ticket like this open for combining WikiProject banners. If I come up with a good approach for that, maybe I'll swing by and apply it here too.

The fact that Wikicode allows nested templates really complicates things. RegEx would be great for just one layer of templates, but starts falling apart once you nest templates.

A somewhat related use case: there is already (only) one maint tag in the article, and a reviewer is adding only one more, then both tags should be put inside a {{Multiple issues}} template.

I'm going to claim all the wikicode tagging bugs since I recently "rewrote" (stole Novem's code) the toolbar tagging logic. I will hopefully be done with all of them in at most a week.

Change #1338368 had a related patch set uploaded (by Apehitkey; author: Apehitkey):

[mediawiki/extensions/PageTriage@master] toolbar: include preexisting tags in the new {{multiple issues}} when adding new tags

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

Change #1341411 had a related patch set uploaded (by Apehitkey; author: Apehitkey):

[mediawiki/extensions/PageTriage@master] toolbar: include previous tags in the new wrapper when adding new tags

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

Change #1338368 abandoned by Apehitkey:

[mediawiki/extensions/PageTriage@master] toolbar: include previous tags in the new wrapper when adding new tags

Reason:

Replaced by https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageTriage/+/1341411

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