Changeset 5218

Timestamp:

03/30/2017 09:44:49 PM (10 years ago)

Author:

obenland

Message:

Plugin Directory: Update developers section.

Replaces lists with full sentences and adds locale details.

See #1824.
Fixes #2629.

Location:

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

Files:

• class-plugin-i18n.php (modified) (1 diff)
• shortcodes/class-developers.php (modified) (3 diffs)

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

@@ -431 +431 @@
          */
         public function find_all_translations_for_plugin( $slug, $branch, $min_percent = 0 ) {
+                $post = Plugin_Directory::get_plugin_post( $slug );
+
+                return wp_filter_object_list( $this->get_locales( $post, $branch, $min_percent ), null, null, 'wp_locale' );
+        }
+
+        /**
+         * Returns a list of locale objects for a given plugin slug and branch.
+         *
+         * @param int|\WP_Post|null $post        Optional. Post ID or post object. Defaults to global $post.
+         * @param string            $branch      Optional. Branch - 'stable-readme' for example. Default: 'stable'.
+         * @param int               $min_percent Optional. Only return locales where percent_translated is >= this value.
+         *                                       Default: 95.
+         * @return array
+         */
+        public function get_locales( $post = null, $branch = 'stable', $min_percent = 95 ) {
+                $post = get_post( $post );
 
                 // 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();
-                $http   = new \WP_Http();
-                $result = $http->request( $api_url );
+                $api_url = esc_url_raw( 'https://translate.wordpress.org/api/projects/wp-plugins/' . $post->post_name . '/' . $branch, array( 'https' ) );
+
+                $locales = array();
+                $http    = new \WP_Http();
+                $result  = $http->request( $api_url );
 
                 if ( ! is_wp_error( $result ) ) {
                         $data = json_decode( $result['body'] );
 
-                        if ( $data && isset( $data->translation_sets ) ) {
-                                foreach ( $data->translation_sets as $translation ) {
-                                        if ( $translation->percent_translated >= $min_percent ) {
-                                                $out[] = $translation->wp_locale;
-                                        }
-                                }
+                        if ( ! empty( $data->translation_sets ) ) {
+                                $locales = array_filter( $data->translation_sets, function( $locale ) use ( $min_percent ) {
+                                        return $locale->percent_translated >= $min_percent;
+                                } );
                         }
                 }
 
-                return $out;
+                return $locales;
         }
 }

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

@@ -1 +1 @@
 <?php
 namespace WordPressdotorg\Plugin_Directory\Shortcodes;
-use WordPressdotorg\Plugin_Directory\Template;
+use WordPressdotorg\Plugin_Directory\Plugin_I18n;
 use WordPressdotorg\Plugin_Directory\Tools;
 
@@ -16 +16 @@
         static function display() {
                 $post   = get_post();
-                $output = '<p>' . sprintf( __( '%s is open source software. The following people have contributed to this plugin.', 'wporg-plugins' ), get_the_title( $post ) ) . '</p>';
+                $slug   = $post->post_name;
+                $title  = get_the_title( $post );
+                $output = '<p>' . sprintf( __( '%s is open source software. The following people have contributed to this plugin.', 'wporg-plugins' ), $title ) . '</p>';
 
                 ob_start();
@@ -28 +30 @@
                 $output .= ob_get_clean();
 
+                $locales = Plugin_I18n::instance()->get_locales();
+                $output .= '<p>';
+
+                if ( ! empty( $locales ) ) {
+                        $locales_list = implode( ', ', array_map( function( $locale ) use ( $slug ) {
+                                return sprintf( '<a href="%1$s">%2$s</a>', esc_url( "{$locale->locale}.wordpress.org/plugins/{$slug}/" ), $locale->name );
+                        }, $locales ) );
+                        /* Translators: 1: Plugin name; 2: Number of locales; 3: List of locales; */
+                        $output .= sprintf( '%1$s has been translated into these %2$d locales: %3$s.', $title, count( $locales ), $locales_list ) . ' ';
+                        $output .= sprintf(
+                                /* Translators: URL to translator view; */
+                                __( 'Thank you to <a href="%s">the translators</a> for their contributions.', 'wporg-plugins' ),
+                                esc_url( "https://translate.wordpress.org/projects/wp-plugins/{$slug}/contributors" )
+                        );
+                        $output .= '<br />';
+                }
+
+                /* Translators: 1: GlotPress URL; 2: Plugin name; */
+                $output .= sprintf( __( '<a href="%1$s">Translate %2$s into your language.</a>', 'wporg-plugins' ), esc_url( 'https://translate.wordpress.org/projects/wp-plugins/' . $slug ), $title );
+                $output .= '</p>';
+
+                $output   .= '<h3>' . __( 'Interested in development?', 'wporg-plugins' ) . '</h3>';
+                /* Translators: 1: Trac URL; 2: Development log URL; 3: RSS URL; */
+                $format    = __( '<a href="%1$s">Browse the code</a> or subscribe to the <a href="%2$s">development log</a> by <a href="%3$s">RSS</a>.' );
+                $email_url = '';
+
                 if ( is_user_logged_in() ) {
+                        /* Translators: 1: Trac URL; 2: Development log URL; 3: RSS URL; 4: Email subscription URL; */
+                        $format     = __( '<a href="%1$s">Browse the code</a> or subscribe to the <a href="%2$s">development log</a> by <a href="%4$s">email</a> or <a href="%3$s">RSS</a>.' );
                         $subscribed = Tools::subscribed_to_plugin_commits( $post, get_current_user_id() );
-
-                        $subscribe_change_url = esc_url( add_query_arg( array(
+                        $email_url  = esc_url( add_query_arg( array(
                                 '_wpnonce' => wp_create_nonce( 'wp_rest' ),
                                 ( $subscribed ? 'unsubscribe' : 'subscribe' ) => '1',
-                        ), home_url( 'wp-json/plugins/v1/plugin/' . $post->post_name . '/commit-subscription' ) ) );
+                        ), home_url( "wp-json/plugins/v1/plugin/{$slug}/commit-subscription" ) ) );
                 }
 
-                return $output .
-                        '<h3>' . __( 'Browse the code', 'wporg-plugins' ) . '</h3>' .
-                        '<ul>' .
-                                '<li>' .
-                                        '<a href="' . esc_url( "https://plugins.trac.wordpress.org/log/{$post->post_name}/" ) . '" rel="nofollow">' . __( 'Development Log', 'wporg-plugins' ) . '</a>' . "\n" .
-                                        '<a href="' . esc_url( "https://plugins.trac.wordpress.org/log/{$post->post_name}/?limit=100&mode=stop_on_copy&format=rss" ) . '" rel="nofollow"><img src="https://s.w.org/style/images/feedicon.png" /></a>' .
-                                '</li>' .
-                                '<li><a href="' . esc_url( "https://plugins.svn.wordpress.org/{$post->post_name}/" ) . '" rel="nofollow">' . __( 'Subversion Repository', 'wporg-plugins' ) . '</a></li>' .
-                                '<li><a href="' . esc_url( "https://plugins.trac.wordpress.org/browser/{$post->post_name}/" ) . '" rel="nofollow">' . __( 'Browse in Trac', 'wporg-plugins' ) . '</a></li>' .
-                                '<li><a href="' . esc_url( "https://translate.wordpress.org/projects/wp-plugins/{$post->post_name}/" ) . '" rel="nofollow">' . __( 'Translation Contributors', 'wporg-plugins' ) . '</a></li>' .
-                                 ( !is_user_logged_in() ? '' : (
-                                        $subscribed ?
-                                        "<li><a href='$subscribe_change_url'>" . __( 'Unsubscribe from plugin commits', 'wporg-plugins' ) . '</a></li>' :
-                                        "<li><a href='$subscribe_change_url'>" . __( 'Subscribe to plugin commits via email', 'wporg-plugins' ) . '</a></li>'
-                                 ) ) .
-                        '</ul>';
+                $output .= '<p>' . sprintf(
+                        $format,
+                        esc_url( "https://plugins.trac.wordpress.org/browser/{$slug}/" ),
+                        esc_url( "https://plugins.trac.wordpress.org/log/{$slug}/" ),
+                        esc_url( "https://plugins.trac.wordpress.org/log/{$slug}/?limit=100&mode=stop_on_copy&format=rss" ),
+                        $email_url
+                ) . '</p>';
+
+                return $output;
         }
 }