Changeset 10258

Timestamp:

09/15/2020 11:49:02 PM (3 years ago)

Author:

dd32

Message:

Robots: remove some hard-coded sitemap links.

Additionally, temporarily disable the news sitemap when no published posts are present until an upstream determination is made.

See #5438.

File:

• sites/trunk/wordpress.org/public_html/wp-content/mu-plugins/pub/wporg-robots.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/mu-plugins/pub/wporg-robots.php

@@ -40 +40 @@
     if ( 'wordpress.org' === $blog_details->domain ) {
         $robots = "Sitemap: https://wordpress.org/news/sitemap.xml\n" .
-                  "Sitemap: https://wordpress.org/news/news-sitemap.xml\n" .
                   "Sitemap: https://wordpress.org/showcase/sitemap.xml\n" .
-                  "Sitemap: https://wordpress.org/showcase/news-sitemap.xml\n" .
                   $robots;
     }
@@ -49 +47 @@
      * Add the Plugins and Theme directory Sitemaps
      * Currently disabled for Rosetta as Jetpack sitemaps aren't working there.
-     * Themes are currently disabled as there's a redirect in place breaking direct access.
      */
     if (
@@ -62 +59 @@
 }
 add_filter( 'robots_txt', 'wporg_robots_prefix_sitemaps', 1 );
+
+// Remove the Jetpack News sitemap when there's no news on the site.
+// Remove once the upstream Jetpack issue is closed or merged https://meta.trac.wordpress.org/ticket/5438
+add_filter( 'jetpack_news_sitemap_include_in_robotstxt', function( $include ) {
+    if ( $include && empty( wp_count_posts()->publish ) ) {
+        $include = false;
+    }
+
+    return $include;
+} );