Changeset 7064 for sites/trunk/common/includes/slack/trac/comment-handler.php

Timestamp:

04/08/2018 11:43:27 AM (8 years ago)

Author:

ocean90

Message:

Slack/Trac: Add support for attachment notification mails.

See #3526.

File:

• sites/trunk/common/includes/slack/trac/comment-handler.php (modified) (2 diffs)

sites/trunk/common/includes/slack/trac/comment-handler.php

@@ -37 +37 @@
                         $line = array_shift( $lines );
                         if ( 0 === strpos( $line, 'X-Trac-Ticket-URL:' ) ) {
-                                // X-Trac-Ticket-URL: https://core.trac.wordpress.org/ticket/12345#comment:1
+                                $comment_id = 0;
                                 list( , $comment_url ) = explode( ': ', $line );
-                                list( $ticket_url, $comment_id ) = explode( '#comment:', $comment_url );
+
+                                // Get ticket URL, attachment notifcations have no specific URL.
+                                if ( false !== strpos( $line, '#comment:' ) ) {
+                                        // X-Trac-Ticket-URL: https://core.trac.wordpress.org/ticket/12345#comment:1
+                                        list( $ticket_url, $comment_id ) = explode( '#comment:', $comment_url );
+                                } else {
+                                        // X-Trac-Ticket-URL: https://core.trac.wordpress.org/ticket/12345
+                                        $ticket_url = $comment_url;
+                                }
+
                                 list( $trac_url, $ticket_id ) = explode( '/ticket/', $ticket_url );
 
@@ -167 +176 @@
                 if ( $has_changes ) {
                         while ( $lines && '' !== current( $lines ) ) {
-                                $changes[] = preg_replace( '~^ \* (.*?):  ~', '_*$1:*_ ', array_shift( $lines ) );
+                                $line = array_shift( $lines );
+
+                                if ( preg_match( '~Attachment "([^"]+)" (added|removed)\.$~', $line, $matches ) ) { // * Attachment "test.txt" added/removed.
+                                        $changes[] = "_*attachment:*_ `{$matches[1]}` {$matches[2]}";
+                                } else { // * status:  assigned => closed
+                                        $changes[] = preg_replace( '~^ \* (.*?):  ~', '_*$1:*_ ', $line );
+                                }
                         }
                 }