HomePhabricator

Gerrit: Avoid breaking full phabricator URLs

Description

Gerrit: Avoid breaking full phabricator URLs

What this fix does is it fixes this:

T12345" target="_blank">https://phabricator.wikimedia.org/T12345

so that it will look like:

https://phabricator.wikimedia.org/T12345

Also this dosent affect correct linking of plain text e.g. T1.

This regex is from @Thiemo Mättig (WMDE)

However, one thing found in testing is that it will highlight
https://phabricator.wikimedia.org/ and T12345 seperately while
both still link correctly to the correct webpage.
It should not be noticable unless someone hovers over it.

This seems to be the best fix we can do for this without having to break
other links in the process.

Thanks @Thiemo Mättig (WMDE) for your suggestion here
https://gerrit.wikimedia.org/r/#/c/242237/2/modules/gerrit/templates/gerrit.config.erb

Bug: T75997
Change-Id: I69be5bf547bf0017902db67b56cea9234d37aabb

Details

Provenance
PaladoxAuthored on Jul 31 2016, 6:06 PM
DzahnCommitted on Aug 2 2016, 9:03 PM
Parents
rOPUPa696cf558d44: move management of pagetitles and mediatitles dirs to datasets manifest
Branches
Unknown
Tags
Unknown
ChangeId
I69be5bf547bf0017902db67b56cea9234d37aabb