Changeset 3417 for sites/trunk/common/includes/slack/trac/bot.php

Timestamp:

06/17/2016 03:37:51 PM (10 years ago)

Author:

ocean90

Message:

Slack: In Bot::parse_tickets() move the word boundary out of the trac matching group to prevent matching #3575D9 as a ticket.

Props voldemortensen.

File:

• sites/trunk/common/includes/slack/trac/bot.php (modified) (1 diff)

sites/trunk/common/includes/slack/trac/bot.php

@@ -43 +43 @@
                 $digits = Trac::get_digit_capture();
                 $ticket_tracs = '?<trac>' . Trac::get_regex();
-                preg_match_all( "/(?:\s|^|\()#(?<id>$digits)(?:\-($ticket_tracs)\b)?/", $text, $tickets, PREG_SET_ORDER );
+                preg_match_all( "/(?:\s|^|\()#(?<id>$digits)(?:\-($ticket_tracs))?\b/", $text, $tickets, PREG_SET_ORDER );
                 preg_match_all( "/(?:\s|^|\()#($ticket_tracs)(?<id>$digits)\b/", $text, $tickets_alt, PREG_SET_ORDER );
                 preg_match_all( "~https?://($ticket_tracs).trac.wordpress.org/ticket/(?<id>$digits)~", $text, $tickets_url, PREG_SET_ORDER );