Changeset 5413

Timestamp:

04/25/2017 04:09:12 PM (9 years ago)

Author:

ocean90

Message:

Plugin Directory: Bail early if no locale sites are found.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-developers.php (modified) (1 diff)

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

@@ -55 +55 @@
             ] );
 
-            $locales_list = implode( ', ', array_map( function( $site ) use ( $slug, $locale_names ) {
-                return sprintf( '<a href="%1$s">%2$s</a>', esc_url( "{$site->home}/plugins/{$slug}/" ), $locale_names[ $site->locale ] );
-            }, $sites ) );
+            if ( $sites ) {
+                $locales_list = implode( ', ', array_map( function( $site ) use ( $slug, $locale_names ) {
+                    return sprintf( '<a href="%1$s">%2$s</a>', esc_url( "{$site->home}/plugins/{$slug}/" ), $locale_names[ $site->locale ] );
+                }, $sites ) );
 
-            $locales_count = count( $sites );
+                $locales_count = count( $sites );
 
-            if ( 1 === $locales_count ) {
+                if ( 1 === $locales_count ) {
+                    $output .= sprintf(
+                        /* translators: 1: plugin name, 2: locale name */
+                        __( '&#8220;%1$s&#8221; has been translated into %2$s.' ),
+                        $title,
+                        $locales_list
+                    ) . ' ';
+                } else {
+                    $output .= sprintf(
+                        /* translators: 1: plugin name, 2: number of locales, 3: list of locales */
+                        _n(
+                            '&#8220;%1$s&#8221; has been translated into these %2$d locales: %3$s.',
+                            '&#8220;%1$s&#8221; has been translated into these %2$d locales: %3$s.',
+                            $locales_count
+                        ),
+                        $title,
+                        $locales_count,
+                        $locales_list
+                    ) . ' ';
+                }
+
                 $output .= sprintf(
-                    /* translators: 1: plugin name, 2: locale name */
-                    __( '&#8220;%1$s&#8221; has been translated into %2$s.' ),
-                    $title,
-                    $locales_list
-                ) . ' ';
-            } else {
-                $output .= sprintf(
-                    /* translators: 1: plugin name, 2: number of locales, 3: list of locales */
-                    _n(
-                        '&#8220;%1$s&#8221; has been translated into these %2$d locales: %3$s.',
-                        '&#8220;%1$s&#8221; has been translated into these %2$d locales: %3$s.',
-                        $locales_count
-                    ),
-                    $title,
-                    $locales_count,
-                    $locales_list
-                ) . ' ';
+                    /* 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 .= '</p>';
             }
-
-            $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 .= '</p>';
         }