Page MenuHomePhabricator

mw:Template:Archived (extension|skin): add new "task" field
Closed, ResolvedPublic

Description

In order to avoid having to rewrite by hand the default "reason" text to link to a Phabricator ticket, I propose that an optional "task" field be added to the templates as well.

{{archived extension| oldid = |reason = | task = }}

Thanks.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 17 2018, 10:26 AM
MarcoAurelio added a subscriber: Dinoguy1000.

I'm neutral; could also be manually added to the reason parameter, as a workaround?

We currently have to do this, which is using indeed the reason field: copying the default text and adding the Phab ticket. Being able to keep the default text and just adding |task=Txxxx would be more elegant IMHO. I'd do the change myself if the template syntax weren't so complicated to read.

This isn't a blocker, but if it's easy enough to do, the template should allow more than one task to be added in this way (whether via multiple parameters, or by allowing multiple values to be passed to the one). There has been at least one archival addressed in more than one ticket, though a quick search through my MW.org contribs fails to turn it up.

Pppery claimed this task.Fri, May 22, 4:04 PM
Restricted Application added a subscriber: Liuxinyu970226. · View Herald TranscriptFri, May 22, 4:04 PM
Pppery updated the task description. (Show Details)Fri, May 22, 4:04 PM

Coded in the sandbox of the templates in question, see https://www.mediawiki.org/w/index.php?title=Project:Sandbox&oldid=3871698

Looks good to me. Thanks for doing it. What do others think?

I'd still like support for multiple tasks, but I don't think that should block. As is, it looks fine.

I didn't code that, mostly because it seems to me that if the archival is complicated enough to merit more than one phabricator task, it's likely complicated enough to need a custom reason as well.

Pppery closed this task as Resolved.Sat, May 23, 7:15 PM

Template changes will need to be marked for translation

(which has now happened)