Changeset 3519

Timestamp:

06/21/2016 10:06:58 AM (10 years ago)

Author:

tellyworth

Message:

Plugin directory: include all available translations in pseudo-meta for search.

Also add some defensive code, better testing and bugfixes.

See #1691, #1692

Location:

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

Files:

• class-plugin-directory.php (modified) (2 diffs)
• class-plugin-i18n.php (modified) (1 diff)

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

@@ -568 +568 @@
                 remove_filter( 'wporg_plugins_custom_meta_fields', array( $this, 'filter_post_meta_i18n' ) );
 
-                if ( $post_id <= 200 ) {
-
-                        // This should probably be a list of available translations for the plugin readme.
-                        $locales_to_sync = array(
+                if ( empty( $this->i18n_meta[ $post_id ] ) ) {
+
+                        $locales_to_sync = array( 
+                                // Default locales to translate, just in case we can't determine the available ones
                                 'fr_FR',
                                 'es_ES',
                         );
+                        $post = get_post( $post_id );
+                        if ( $post ) {
+                                $translations = Plugin_I18n::find_all_translations_for_plugin( $post->post_name, 'stable-readme', 10 ); // at least 10% translated
+                                if ( $translations )
+                                        $locales_to_sync = $translations;
+                        }
 
                         global $locale;
@@ -580 +586 @@
 
                         foreach ( $locales_to_sync as $locale ) {
-                                $this->i18n_meta[ $post_id ][ 'title_' . $locale ]   = $this->translate_post_title( get_the_title( $post_id ), $post_id );
-                                $this->i18n_meta[ $post_id ][ 'excerpt_' . $locale ] = $this->translate_post_excerpt( get_the_excerpt( $post_id ), $post_id );
+                                $the_title = $this->translate_post_title( get_the_title( $post_id ), $post_id );
+                                if ( $the_title && $the_title != get_the_title( $post_id ) ) {
+                                        $this->i18n_meta[ $post_id ][ 'title_' . $locale ] = $the_title;
+                                }
+
+                                $the_excerpt = $this->translate_post_excerpt( get_the_excerpt( $post_id ), $post_id );
+                                if ( $the_excerpt && $the_excerpt != get_the_excerpt( $post_id ) ) {
+                                        $this->i18n_meta[ $post_id ][ 'excerpt_' . $locale ] = $the_excerpt;
+                                }
 
                                 // Split up the content to translate it in sections.
-                                $content  = '';
+                                $the_content = array();
                                 $sections = $this->split_post_content_into_pages( get_the_content( $post_id ) );
                                 foreach ( $sections as $section => $section_content ) {
-                                        $content .= $this->translate_post_content( $section_content, $section, $post_id );
-                                }
-                                $this->i18n_meta[ $post_id ][ 'content_' . $locale ] = $content;
+                                        $translated_section = $this->translate_post_content( $section_content, $section, $post_id );
+                                        if ( $translated_section && $translated_section != $section_content ) {
+                                                $the_content[] = $translated_section;
+                                        }
+                                }
+                                if ( !empty( $the_content ) )
+                                        $this->i18n_meta[ $post_id ][ 'content_' . $locale ] = implode( $the_content );
                         }
 
                         $locale = $_locale;
 
+                }
+
+                if ( is_array( $this->i18n_meta[ $post_id ] ) )
                         $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 );

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

@@ -420 +420 @@
                 return $content;
         }
+
+        /**
+         * Returns a list of translation locales for a given plugin slug and branch.
+         *
+         * @param string $slug    Plugin slug.
+         * @param string $branch  Branch - 'stable-readme' for example.
+         * @param string $min_percent     Only return locales where percent_translated is >= this value.
+         * @return array
+         */
+        public function find_all_translations_for_plugin( $slug, $branch, $min_percent = 0 ) {
+
+                // This naively hits the API. It could probably be re-written to query the DB instead.
+                $api_url = esc_url_raw( 'https://translate.wordpress.org/api/projects/wp-plugins/' . $slug . '/' . $branch, array( 'https' ) );
+
+                $out = array();
+                if ( $json = file_get_contents( $api_url ) ) {
+                        if ( $data = json_decode( $json ) ) {
+                                if ( isset( $data->translation_sets ) ) {
+                                        foreach ( $data->translation_sets as $translation ) {
+                                                if ( $translation->percent_translated >= $min_percent )
+                                                        $out[] = $translation->wp_locale;
+                                        }
+                                }
+                        }
+                }
+
+                return $out;
+        }
+
 }