Last modified: 2013-07-25 07:07:39 UTC
https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/MobileFrontend.git;a=blob;f=MobileFormatter.php;h=e54b7c312985e3ebd569cac1fb4f34c597f4e8b8;hb=HEAD Bugzilla's auto-linker is mangling this Gerrit URL. It should look like the concatenation of: https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/ \ MobileFrontend.git;a=blob;f=MobileFormatter.php; \ h=e54b7c312985e3ebd \ 569cac1fb4f34c597f4e8b8;hb=HEAD
I've noticed this as well and it is most annoying! I wonder if the latest Bugzilla upgrade fixes this?
(In reply to comment #1) > I've noticed this as well and it is most annoying! I wonder if the latest > Bugzilla upgrade fixes this? I doubt it has anything to do with Bugzilla directly. Wikimedia has a Bugzilla extension that performs a few auto-linking regexen on Bugzilla comments. I imagine this particular regex just needs a tweak.
I guess that's stuff to solve in http://svn.wikimedia.org/viewvc/mediawiki/trunk/tools/bugzilla/bugzilla-4.0/extensions/Wikimedia/Extension.pm?view=markup , whoever wants to give it a try and knows regular expressions.
I noticed on bug 41745 comment 5 that the email I received had a correct link, so I suppose the damage is not permanent. Also: 2012-11-03 20.56 < jeremyb> why are spaces escaped there? 2012-11-03 20.56 < jeremyb> > match => qr{\brt\ ?\#?(\d+)}i
It only affects the rendering - the XML view at https://bugzilla.wikimedia.org/show_bug.cgi?ctype=xml&id=41745 also shows it correctly.
The issue seems to be that a word boundary \b also covers = while this is not intended by its author. Patch at https://gerrit.wikimedia.org/r/#/c/33557/ , review and testing welcome.
Gerrit annoys the hell out of me and wastes my time, so I unassign this though the patch works and is tested. See Gerrit.
Next try: https://gerrit.wikimedia.org/r/#/c/33747/
Created https://rt.wikimedia.org/Ticket/Display.html?id=4126 for deployment.
merged: https://gerrit.wikimedia.org/r/#/c/33747/3/bugzilla-4.0/extensions/Wikimedia/Extension.pm and deployed in production
and another change to move the Extension.pm from 4.0 to 4.2 directory https://gerrit.wikimedia.org/r/#/c/39065/
Thanks! Verifying, as it can be seen that these examples are not broken anymore: https://bugzilla.wikimedia.org/show_bug.cgi?id=18195#c14 https://bugzilla.wikimedia.org/show_bug.cgi?id=41745#c5
