Changeset 3088

Timestamp:

05/05/2016 02:15:43 PM (8 years ago)

Author:

ocean90

Message:

Slack: Don't parse HTML entities or ticket numbers in backticks as tickets.

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( "/#(?<id>$digits)(?:\-($ticket_tracs)\b)?/", $text, $tickets, PREG_SET_ORDER );
-        preg_match_all( "/#($ticket_tracs)(?<id>$digits)\b/", $text, $tickets_alt, PREG_SET_ORDER );
+        preg_match_all( "/[^&`]#(?<id>$digits)(?:\-($ticket_tracs)\b)?/", $text, $tickets, PREG_SET_ORDER );
+        preg_match_all( "/[^&`]#($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 );
         foreach ( $tickets_url as &$ticket ) {