Page MenuHomePhabricator
Create Task
Maniphest T697

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

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.

Related Objects

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 subscribed.
Byfserag triaged this task as Medium priority.Oct 19 2014, 7:02 AM
Byfserag subscribed.
Aklapper subscribed.Oct 19 2014, 12:28 PM
• chasemp lowered the priority of this task from Medium to Low.Oct 21 2014, 6:04 PM
adrianheine subscribed.Oct 29 2014, 9:25 AM
Qgil raised the priority of this task from Low to Medium.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 Medium to Low.Nov 7 2014, 10:49 AM
• chasemp mentioned this in Unknown Object (Diffusion Commit).Nov 13 2014, 3:51 PM
Qgil merged a task: T76922: Convert Bugzilla URLs in Tasks into Phabricator {links}.Dec 8 2014, 7:07 AM
Qgil added subscribers: Quiddity, scfc.
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
Aklapper added a comment.Dec 8 2014, 11:18 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).

Quiddity added a comment.Dec 9 2014, 12:36 AM

Works for me. :)

• chasemp mentioned this in rPHTOd93501eb407c: fixes migration concerns.Feb 11 2015, 7:47 PM
Ricordisamoa subscribed.Jul 15 2015, 11:03 PM
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptJul 2 2016, 7:35 PM
Danny_B subscribed.Jul 12 2016, 1:22 PM
Aklapper moved this task from Need discussion to Misc on the Phabricator board.Oct 7 2016, 5:03 PM
Liuxinyu970226 subscribed.Dec 17 2016, 4:06 PM
This comment was removed by Liuxinyu970226.
Quiddity added a comment.Dec 18 2016, 7:52 PM

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.

Aklapper moved this task from Misc to Needs code (in Phab or bot) on the Phabricator board.Aug 29 2018, 6:04 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL