Changeset 3449

Timestamp:

06/19/2016 08:46:20 AM (2 years ago)

Author:

tellyworth

Message:

Inject translated content, title, and excerpt into pseudo-meta values for search indexing.

This is intentionally limited for testing purposes to fr and es locales, and to post IDs below 200.

See #1692.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php

@@ -10 +10 @@
  */
 class Plugin_Directory {
+
+    /**
+     * Local cache for translated content injected into meta
+     */
+    private $i18n_meta = array();
 
     /**
@@ -238 +243 @@
             \Jetpack_Search::instance();
         }
+
+        // When Jetpack syncs, we want to add filters to inject additional metadata for Jetpack, so it syncs for ElasticSearch indexing.
+        add_action( 'shutdown', array( $this, 'append_meta_for_jetpack' ), 8 );
+
     }
 
@@ -516 +525 @@
 
     /**
+     * Shutdown action that will add a filter to inject additional postmeta containing translated content if Jetpack is syncing.
+     *
+     */
+    public function append_meta_for_jetpack() {
+        // TEMP: only do this for low numbered plugin IDs, till we're sure it works.
+        if ( get_post()->ID > 200 )
+            return;
+
+        // Guess if a Jetpack sync is scheduled to run. It runs during shutdown at a lower priority than this action, so we can get in first.
+        // Fetching the extra meta to inject is expensive, so we only want to do this if a sync is likely.
+        if ( class_exists( 'Jetpack' ) && !empty(\Jetpack::init()->sync->sync) ) {
+            add_filter( 'wporg_plugins_custom_meta_fields', array( $this, 'filter_post_meta_i18n' ), 10, 2 );
+        }
+
+    }
+
+    /**
+     * Filter for wporg_plugins_custom_meta_fields to inject translated content for ES.
+     *
+     * @param array $meta
+     * @param int $post_id
+     * @return array
+     */
+    public function filter_post_meta_i18n( $meta, $post_id ) {
+        // Prevent recursion and repeat runs
+        remove_filter( 'wporg_plugins_custom_meta_fields', array( $this, 'filter_post_meta_i18n' ) );
+
+        if ( get_post()->ID == $post_id ) {
+            $locales_to_sync = array( 'fr_fr', 'es_es' ); // This should probably be a list of available translations for the plugin readme.
+
+            global $locale;
+            $_locale = $locale;
+            foreach ( $locales_to_sync as $locale ) {
+                $this->i18n_meta[$post_id]['title_'.$locale] = $this->translate_post_title( get_the_title(), $post_id );
+                $this->i18n_meta[$post_id]['excerpt_'.$locale] = $this->translate_post_excerpt( get_the_excerpt() );
+
+                // Split up the content to translate it in sections
+                $content = '';
+                $sections = $this->split_post_content_into_pages( get_the_content() );
+                foreach ( $sections as $section => $section_content )
+                    $content .= $this->translate_post_content( $section_content, $section );
+                $this->i18n_meta[$post_id]['content_'.$locale] = $content;
+
+            }
+
+            $locale = $_locale;
+
+            $meta = array_merge( $meta, array_keys( $this->i18n_meta[$post_id] ) );
+        }
+
+        add_filter( 'wporg_plugins_custom_meta_fields', array( $this, 'filter_post_meta_i18n'), 10, 2 );
+        return $meta;
+    }
+
+
+    /**
      * Filter for rest_api_allowed_post_types to enable JP syncing of the CPT
      *
@@ -662 +727 @@
                 }
 
+                break;
+            default:
+                if ( isset( $this->i18n_meta[ $object_id ][ $meta_key ] ) )
+                    return array( $this->i18n_meta[ $object_id ][ $meta_key ] );
                 break;
         }