Changeset 13992

Timestamp:

08/26/2024 02:12:20 AM (6 months 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'] );