Page MenuHomePhabricator

Convert wiki links in Bugzilla comments in links in Phabricator tasks
Closed, DeclinedPublic

Description

There are different ways to generate links to wiki pages in Bugzilla comments. Ideally those links should be kept in Phabricator comments.

See https://bugzilla.wikimedia.org/show_bug.cgi?id=72485

When finding something like the Perl regex in https://git.wikimedia.org/blob/wikimedia%2Fbugzilla%2Fmodifications.git/HEAD/extensions%2FWikimedia%2FExtension.pm#L36 we should take that markup and after html_quote/url_quote stuff pass it to the en.wp search like in https://git.wikimedia.org/blob/wikimedia%2Fbugzilla%2Fmodifications.git/HEAD/extensions%2FWikimedia%2FExtension.pm#L81

Event Timeline

Qgil raised the priority of this task from to Low.
Qgil updated the task description. (Show Details)
Qgil changed Security from none to None.
Qgil subscribed.
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, Bugzilla-Migration.

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

Is this how wiki links like [[mw:Phabricator]] (pointing to mediawiki.org) are constructed as well, or is this only good for en.wiki links?

In T875#952579, @Qgil wrote:

Is this how wiki links like [[mw:Phabricator]] (pointing to mediawiki.org) are constructed as well, or is this only good for en.wiki links?

In Bugzilla they were pointing to en.wiki, but they were used for other projects as well, through redirects like https://en.wikipedia.org/wiki/mw:Phabricator.
If Phabricator has access to the interwiki map, it could resolve link targets by itself instead of relying on en.wiki.

chasemp lowered the priority of this task from Low to Lowest.
Aklapper subscribed.

Realistically declining. We do not plan to fiddle with database content many years after migration.
Also see T76283#8999375 why only changing the rendering (but not touching database content) conditionally is not an option either.