Changeset 7898 for sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/theme-directory.php

Timestamp:

11/26/2018 04:51:14 AM (6 years ago)

Author:

dd32

Message:

WordPress.org: hreflang: Add a default x-default hreflang tag and use the canonical URL for singular pages when possible.

Fixes #3947, #3948.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/theme-directory.php (modified) (4 diffs)

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

@@ -1090 +1090 @@
  */
 function wporg_themes_add_hreflang_link_attributes() {
+    if ( is_404() ) {
+        return;
+    }
+
     wp_cache_add_global_groups( array( 'locale-associations' ) );
 
@@ -1136 +1140 @@
         );
 
+        // Add x-default to the list of sites.
+        $sites['x-default'] = (object) array(
+            'locale'    => 'x-default',
+            'hreflang'  => 'x-default',
+            'subdomain' => '',
+        );
+
         uasort( $sites, function( $a, $b ) {
             return strcasecmp( $a->hreflang, $b->hreflang );
@@ -1141 +1152 @@
 
         wp_cache_set( 'local-sites', $sites, 'locale-associations' );
+    }
+
+    if ( is_singular() ) {
+        $path = parse_url( get_permalink(), PHP_URL_PATH );
+    } else {
+        // WordPress doesn't have a good way to get the canonical version of non-singular urls.
+        $path = $_SERVER['REQUEST_URI']; // phpcs:ignore
     }
 
@@ -1147 +1165 @@
             'https://%swordpress.org%s',
             $site->subdomain ? "{$site->subdomain}." : '',
-            $_SERVER[ 'REQUEST_URI' ]
+            $path
         );