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

Timestamp:

09/15/2020 11:49:02 PM (6 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;
+} );