JILS-26: Commit links are being converted to hyperlinks twice



Issue Information

Issue Type: Bug
 
Priority: Major
Status: In Progress

Reported By:
Ben Tasker
Assigned To:
Ben Tasker
Project: Jira Issue Listing Script (JILS)
Resolution: Unresolved
Affects Version: 0.1, 0.01b,
Target version: 0.1,

Created: 2014-11-12 01:20:32
Time Spent Working


Description
Similar behaviour to JILS-19, but this time caused by JILS-25.

The hyperlink used within the commit notifications is being autolinked, and the commit notification markup processing is then turning that into a link.

Needs a rounder solution, but the quick fix is to drop a negative lookback into the regex to ensure it's not converted if the link is prefixed with a pipe symbol (|)

Something like

(?<!\|)


Should do it, though will need testing


Issue Links

Toggle State Changes

Activity



Repo: Jira-Issue-Listing
Commit: 497efd9a0c4cd859266373d356b24d27a45a167b
Author: Ben Tasker <github@<Domain Hidden>>

Date: Wed Nov 12 01:44:23 2014 +0000
Commit Message: Updated autolinks to ignore commit links. See JILS-26



Modified (-)(+)
-------
utils.class.php




Webhook User-Agent

GitHub-Hookshot/0687198


View Commit

For the time being, have dropped a negative lookback into the regex

((?<!\|)(http|ftp|https))


The problem with trying to keep the regular expressions simple is that we're going to hit issues like this too regularly, but given the choice between unmaintainable regex and occasionally breaking links, I think I prefer the latter.
btasker changed status from 'Open' to 'In Progress'
btasker added 'Ben Tasker' to assignee
btasker added '0.1' to Fix Version
btasker added '0.1' to Version
btasker added '0.01b' to Version