Changeset 4441

Timestamp:

11/28/2016 07:24:05 PM (8 years ago)

Author:

ocean90

Message:

WP i18n teams: Avoid notices if a locale doesn't exist.

Location:

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

Files:

• views/all-locales.php (modified) (1 diff)
• wp-i18n-teams.php (modified) (1 diff)

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

@@ -128 +128 @@
                     </td>
 
-                    <td>
+                    <td class="center">
                         <a href="<?php echo esc_url( add_query_arg( 'locale', $locale->wp_locale ) ); ?>">
                             <?php _e( 'View Team Page', 'wporg' ); ?>

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

@@ -84 +84 @@
             require_once GLOTPRESS_LOCALES_PATH;
             $locale = GP_Locales::by_field( 'wp_locale', $_GET['locale'] );
-            $locale_data = $this->get_extended_locale_data( $locale );
-            require( __DIR__ . '/views/locale-details.php' );
+            if ( $locale ) {
+                $locale_data = $this->get_extended_locale_data( $locale );
+                require( __DIR__ . '/views/locale-details.php' );
+            } else {
+                printf(
+                    '<div class="callout callout-warning"><p>%s</p><p><a href="%s">%s</a></p></div>',
+                    sprintf(
+                        __( 'Locale %s doesn&#8217;t exists.', 'wporg' ),
+                        '<code>' . esc_html( $_GET['locale'] ) . '</code>'
+                    ),
+                    esc_url( get_permalink() ),
+                    __( 'Return to All Locales', 'wporg' )
+                );
+            }
         }