Changeset 13992

Timestamp:

08/26/2024 02:12:20 AM (2 years ago)

Author:

dd32

Message:

Make: Trac Component overview: Generate a link that works for Meta trac (and any others).

Only core.trac has a rewrite handler for the /components/(.+) endpoint.

Props dd32, enrico.sorcinelli.
Fixes #7755.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-components.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-components.php

@@ -36 +36 @@
         function trac_url() {
                 return 'https://' . $this->trac . '.trac.wordpress.org';
+        }
+
+        function get_component_url( $component ) {
+                if ( 'core' === $this->trac ) {
+                        // Core has a rewrite for /component/(.+) to a report.
+                        return $this->trac_url() . '/component/' . rawurlencode( $component );
+                } elseif ( 'meta' === $this->trac ) {
+                        // Meta has the same report, but with no rewrite
+                        return $this->trac_url() . '/report/19?COMPONENT=' . rawurlencode( $component );
+                } else {
+                        // Otherwise, just query for open tickets.
+                        return $this->trac_url() . '/query?status=!closed&component=' . rawurlencode( $component );
+                }
         }
 
@@ -716 +729 @@
                         $open_tickets = array_sum( $this->breakdown_component_type[ $component ] );
                 }
-                echo '<td class="right"><a href="' . $this->trac_url() . '/component/' . esc_attr( rawurlencode( $component ) ) . '">' . $open_tickets . '</a></td>';
+                echo '<td class="right"><a href="' . esc_attr( $this->get_component_url( $component ) ) . '">' . $open_tickets . '</a></td>';
                 if ( $history['change'] ) {
                         $count = sprintf( "%+d", $history['change'] );