Page MenuHomePhabricator

Converting "See Also" Bugzilla links to Phabricator tasks
Open, LowPublic

Description

bugzillapreview shows links to Bugzilla URLs in the "See also" information. Wouldn't it be nice if those links would point to the corresponding Phabricator tasks instead?

For instance, https://bugzillapreview.wmflabs.org/T4340 shows

See Also: https://bugzilla.wikimedia.org/show_bug.cgi?id=43345
https://bugzilla.wikimedia.org/show_bug.cgi?id=43449
https://bugzilla.wikimedia.org/show_bug.cgi?id=31680
https://bugzilla.wikimedia.org/show_bug.cgi?id=38879
https://bugzilla.wikimedia.org/show_bug.cgi?id=41113
https://bugzilla.wikimedia.org/show_bug.cgi?id=42963
https://bugzilla.wikimedia.org/show_bug.cgi?id=44391
https://bugzilla.wikimedia.org/show_bug.cgi?id=44269
https://bugzilla.wikimedia.org/show_bug.cgi?id=44428
https://bugzilla.wikimedia.org/show_bug.cgi?id=44872
https://bugzilla.wikimedia.org/show_bug.cgi?id=48927

Which isn't very elegant and forces you to click to each link in order to know what is the task about, and whether it is still open or closed.

Instead, we could have (using fake task numbers for the sake of showing the example):

See Also: T334 T344 T123 T342 T187 T321 T189 T296 T312 T174 T265

If such script / bot isn't simple to write, an option could be to leave it for later, without blocking the Bugzilla migration and the official launch.

Event Timeline

Qgil created this task.Oct 18 2014, 11:29 PM
Qgil raised the priority of this task from to Needs Triage.
Qgil updated the task description. (Show Details)
Qgil added a project: Bugzilla-Preview.
Qgil changed Security from none to None.
Qgil added a subscriber: Qgil.
Byfserag triaged this task as Normal priority.Oct 19 2014, 7:02 AM
Byfserag added a subscriber: Byfserag.
chasemp lowered the priority of this task from Normal to Low.Oct 21 2014, 6:04 PM
Qgil raised the priority of this task from Low to Normal.Oct 29 2014, 9:21 PM
Qgil edited projects, added Phabricator; removed Bugzilla-Preview.

This task is not essential, it has a risk of overcomplicating the migration process, and it can be done afterward either with direct changes to the database or a bot.

Qgil lowered the priority of this task from Normal to Low.Nov 7 2014, 10:49 AM
chasemp mentioned this in Unknown Object (Diffusion Commit).Nov 13 2014, 3:51 PM
Qgil added a comment.Dec 8 2014, 7:10 AM

Following up on T76922, we need to decide whether we want to translate full bugzilla.wikimedia.org URLs to T2345 or {T2345123}.

What about this:

  • "Bug 345" translates to T2345
  • "bugzilla.wikimedia.org/show_bug.cgi?id=345 translates to {T2345}.
Qgil moved this task from To Triage to Need discussion on the Phabricator board.Dec 8 2014, 7:11 AM

Sounds good to me - in both cases one is able to get more information without having to click (once it's shown directly, once when moving the mouse pointer over it).

Works for me. :)

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptJul 2 2016, 7:35 PM
Aklapper moved this task from Need discussion to Misc on the Phabricator board.Oct 7 2016, 5:03 PM
This comment was removed by Liuxinyu970226.

Is this a duplicate of T687: Convert Bugzilla's "Bug NNNNN" links to "TNNNNN" links in Phabricator ? (It seems to be, but perhaps I'm missing something?)

scfc added a comment.Dec 18 2016, 11:43 PM

The proposed format in T687 is slightly different ("See Also: http://bugzilla.wikimedia.org/show_bug.cgi?id=9123000 (T9125000)" instead of "See Also: T9125000" being suggested here). So if there was someone actively working on T687, I would merge these two, until then I think it's easier to keep them separate so that's more obvious what needs to be fixed.