Page MenuHomePhabricator

[Regex] Wrapped Gerrit links aren't recognized
Closed, DeclinedPublic

Description

From bug 35144 comment 20, gerrit changeset
12345 should be autolinked, but isn't. The regular expression should treat all whitespace equal.


Version: unspecified
Severity: trivial

Details

Reference
bz41851

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 22 2014, 12:47 AM
bzimport set Reference to bz41851.
bzimport added a subscriber: Unknown Object (MLST).

I think this bug usually strikes most for wiki-links, which tend to be very long. Sometimes, for them, putting them on their own line or adding a colon at the beginning or leaving underscores instead of spaces can help.

I'm not sure if wrapped wikilinks should be recognized as they aren't by MediaWiki neither. (BTW, what I deeply miss in Bugzilla is a preview function - it would be *very* nice if you could check a comment for autolinkification and layout in general *before* you submit it.)

Patches welcome (in case this link works):
https://gerrit.wikimedia.org/r/gitweb?p=wikimedia/bugzilla/modifications.git;a=blob;f=bugzilla-4.0/extensions/Wikimedia/Extension.pm

(In reply to comment #2)
As you are only expected to enter plain text and as you have no influence on its formatting, I don't see how a preview makes any sense. The formatting should either be fixed or disabled. Autolinkification in upstream code only supports "bug 123456" stuff & there are no plans to provide a preview function.

Wikimedia has migrated from Bugzilla to Phabricator. Learn more about it here: https://www.mediawiki.org/wiki/Phabricator/versus_Bugzilla - This task does not make sense anymore in the concept of Phabricator, hence closing as declined.