Changeset 4328

Timestamp:

11/04/2016 11:53:08 AM (8 years ago)

Author:

ocean90

Message:

WP i18n teams: Use the term_link filter for linking #locale to the team page.

Avoids yet another content parsing which is already done by o2.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/wp-i18n-teams.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/wp-i18n-teams.php

@@ -11 +11 @@
 
 class WP_I18n_Teams {
+    const TEAM_PAGE = 'https://make.wordpress.org/polyglots/teams/?locale=%s';
+
     public function __construct() {
         add_action( 'plugins_loaded', array( $this, 'plugins_loaded' ) );
@@ -18 +20 @@
      * Attaches hooks and registers shortcodes once plugins are loasded.
      */
-    function plugins_loaded() {
+    public function plugins_loaded() {
         add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_assets' ) );
         add_shortcode( 'wp-locales',      array( $this, 'wp_locales' ) );
 
-        add_filter( 'the_content',  array( $this, 'link_locales' ), 0 );
-        add_filter( 'comment_text', array( $this, 'link_locales' ), 0 );
+        add_filter( 'term_link', array( $this, 'link_locales' ), 10, 3 );
     }
 
@@ -29 +30 @@
      * Links #locale to the teams page.
      *
-     * @param string $content Post or comment content.
-     * @return string Filtered post or comment content.
-     */
-    function link_locales( $content ) {
+     * @param string $termlink Term link URL.
+     * @param object $term     Term object.
+     * @param string $taxonomy Taxonomy slug.
+     * @return string URL to teams page of a locale.
+     */
+    public function link_locales( $termlink, $term, $taxonomy ) {
+        if ( 'post_tag' !== $taxonomy ) {
+            return $termlink;
+        }
+
         static $available_locales;
 
@@ -38 +45 @@
             $available_locales = self::get_locales();
             $available_locales = wp_list_pluck( $available_locales, 'wp_locale' );
-        }
-
-        $regex = '/\B#([\w]+)\b/';
-        if ( ! preg_match_all( $regex, $content, $matches ) ) {
-            return $content;
-        }
-
-        $possible_locales = $matches[1];
-        $possible_locales = array_unique( $possible_locales );
-        $locales = array_intersect( $available_locales, $possible_locales );
-
-        foreach ( $locales as $locale ) {
-            $url         = esc_url( 'https://make.wordpress.org/polyglots/teams/?locale=' . $locale );
-            $replacement = "<a href='$url'>#$locale</a>";
-            $content     = preg_replace( "/#$locale\b/", $replacement, $content );
-        }
-
-        return $content;
+            $available_locales = array_flip( $available_locales );
+        }
+
+        if ( isset( $available_locales[ $term->name ] ) || isset( $available_locales[ $term->slug ] ) ) {
+            return sprintf( self::TEAM_PAGE, $term->slug );
+        }
+
+        return $termlink;
     }