https://confluence.atlassian.com/display/STASHKB/Integrating+with+custom+JIRA+issue+keystates that the default pattern for searching commit messages in Stash is
((?<!([A-Z]{1,10})-?)[A-Z]+-\d+)
Can someone please enlighten me why is lookbehind needed here?
Hi,
It's to avoid the pattern matching strings like CR-PROJ-1234 (which is a crucible review key, but Bamboo build plan IDs look similar)
Cheers,
Michael
Oh, that makes sense now. Thank you very much!
Blatant pilfering from Stackoverflow (http://stackoverflow.com/questions/19322669/regular-expression-for-a-jira-identifier/30518972#30518972 ):
Atlassian themselves have a couple webpages floating around that suggest a good (java) regex is this:
<span class="str">((?<!([A-Z]{1,10})-?)[A-Z]+-\d+)</span>
(Source:https://confluence.atlassian.com/display/STASHKB/Integrating+with+custom+JIRA+issue+key)
<span class="typ">Test</span><span class="pln"> </span><span class="typ">String</span><span class="pun">:</span><span class="pln"> </span><span class="str">"BF-18 abc-123 X-88 ABCDEFGHIJKL-999 abc XY-Z-333 abcDEF-33 ABC-1"<br></span><span class="typ">Matches</span><span class="pun">:</span><span class="pln"> BF</span><span class="pun">-</span><span class="lit">18</span><span class="pun">,</span><span class="pln"> X</span><span class="pun">-</span><span class="lit">88</span><span class="pun">,</span><span class="pln"> ABCDEFGHIJKL</span><span class="pun">-</span><span class="lit">999</span><span class="pun">,</span><span class="pln"> DEF</span><span class="pun">-</span><span class="lit">33</span><span class="pun">,</span><span class="pln"> ABC</span><span class="pun">-</span><span class="lit">1</span>
But, I don't really like it because it will match the "DEF-33" from "abcDEF-33", whereas I prefer to ignore "abcDEF-33" altogether. So in my own code I'm using:
<span class="str">((?<!([A-Za-z]{1,10})-?)[A-Z]+-\d+)</span>
Notice how "DEF-33" is no longer matched:
<span class="typ">Test</span><span class="pln"> </span><span class="typ">String</span><span class="pun">:</span><span class="pln"> </span><span class="str">"BF-18 abc-123 X-88 ABCDEFGHIJKL-999 abc XY-Z-333 abcDEF-33 ABC-1"<br></span><span class="typ">Matches</span><span class="pun">:</span><span class="pln"> BF</span><span class="pun">-</span><span class="lit">18</span><span class="pun">,</span><span class="pln"> X</span><span class="pun">-</span><span class="lit">88</span><span class="pun">,</span><span class="pln"> ABCDEFGHIJKL</span><span class="pun">-</span><span class="lit">999</span><span class="pun">,</span><span class="pln"> ABC</span><span class="pun">-</span><span class="lit">1</span>
I also needed this regex in JavaScript. Unfortunately, JavaScript does not support the LookBehind (?<!a)b, and so I had to port it to LookAhead a(?!b) and reverse everything:
(?<!a)b
a(?!b)
<span class="kwd">var</span><span class="pln"> jira_matcher </span><span class="pun">=</span><span class="pln"> </span><span class="str">/\d+-[A-Z]+(?!-?[a-zA-Z]{1,10})/</span><span class="pln">g</span>
This means the string to be matched needs to be reversed ahead of time, too:
<span class="kwd">var</span><span class="pln"> s </span><span class="pun">=</span><span class="pln"> </span><span class="str">"BF-18 abc-123 X-88 ABCDEFGHIJKL-999 abc XY-Z-333 abcDEF-33 ABC-1"<br></span><span class="pln">s </span><span class="pun">=</span><span class="pln"> reverse</span><span class="pun">(</span><span class="pln">s</span><span class="pun">)<br></span><span class="kwd">var</span><span class="pln"> m </span><span class="pun">=</span><span class="pln"> s</span><span class="pun">.</span><span class="pln">match</span><span class="pun">(</span><span class="pln">jira_matcher</span><span class="pun">);<br><br></span><span class="com">// Also need to reverse all the results!<br></span><span class="kwd">for</span><span class="pln"> </span><span class="pun">(</span><span class="kwd">var</span><span class="pln"> i </span><span class="pun">=</span><span class="pln"> </span><span class="lit">0</span><span class="pun">;</span><span class="pln"> i </span><span class="pun"><</span><span class="pln"> m</span><span class="pun">.</span><span class="pln">length</span><span class="pun">;</span><span class="pln"> i</span><span class="pun">++)</span><span class="pln"> </span><span class="pun">{<br></span><span class="pln"> m</span><span class="pun">[</span><span class="pln">i</span><span class="pun">]</span><span class="pln"> </span><span class="pun">=</span><span class="pln"> reverse</span><span class="pun">(</span><span class="pln">m</span><span class="pun">[</span><span class="pln">i</span><span class="pun">])<br></span><span class="pun">}<br></span><span class="pln">m</span><span class="pun">.</span><span class="pln">reverse</span><span class="pun">()<br></span><span class="pln">console</span><span class="pun">.</span><span class="pln">log</span><span class="pun">(</span><span class="pln">m</span><span class="pun">)<br><br></span><span class="com">// Output:</span><span class="pln"> </span><span class="pun">[</span><span class="pln"> </span><span class="str">'BF-18'</span><span class="pun">,</span><span class="pln"> </span><span class="str">'X-88'</span><span class="pun">,</span><span class="pln"> </span><span class="str">'ABCDEFGHIJKL-999'</span><span class="pun">,</span><span class="pln"> </span><span class="str">'ABC-1'</span><span class="pln"> </span><span class="pun">]</span>
var jiraMatcher = /((?!([A-Z0-9a-z]{1,10})-?$)[A-Z]{1}[A-Z0-9]+-\d+)/g;
For javascript, this works without having to do the crazy reversal stuff and accounts for the fact that numbers can be used in the Jira project key prefix.
this does not work. The answer provided by @G__Sylvie_Davies__bit-booster_com_ with the reversal does work.
Slightly adjusted @Shaun Dunning version to do not match "KEY-0":
/^((?!([A-Z0-9a-z]{1,10})-?$)[A-Z]{1}[A-Z0-9]+-[1-9][0-9]*)$/g
It looks like you're new here. Sign in or register to get started.
