Changeset 10313 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-locale-banner.php

Timestamp:

09/29/2020 06:28:26 AM (6 years ago)

Author:

dd32

Message:

Plugin Directory: Locale Banner: Don't suggest locales which we don't support.

Not all Locales on WordPress.org have valid WordPress locale codes but have a Rosetta site, such as la.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-locale-banner.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-locale-banner.php

@@ -42 +42 @@
                 // Retrieve all the WordPress locales.
                 $all_locales = wp_list_pluck( $locale_subdomain_assoc, 'locale' );
+
+                // Validate that all the locale subdomains have valid WordPress locale values (hint: They don't).
+                $all_locales = array_filter( $all_locales, function( $locale ) {
+                        return \GP_Locales::by_field( 'wp_locale', $locale );
+                } );
 
                 $is_plugin_request = ! empty( $plugin_slug );