Changeset 9156 for trunk/wp-includes/canonical.php

Timestamp:

10/14/2008 05:51:01 AM (17 years ago)

Author:

markjaquith

Message:

Canonical Feed URLs, first try. see #7182

File:

• trunk/wp-includes/canonical.php (modified) (4 diffs)

trunk/wp-includes/canonical.php

@@ -40 +40 @@
     global $wp_rewrite, $is_IIS, $wp_query, $wpdb;
 
-    if ( is_feed() || is_trackback() || is_search() || is_comments_popup() || is_admin() || $is_IIS || ( isset($_POST) && count($_POST) ) || is_preview() || is_robots() )
+    if ( is_trackback() || is_search() || is_comments_popup() || is_admin() || $is_IIS || ( isset($_POST) && count($_POST) ) || is_preview() || is_robots() )
         return;
 
@@ -118 +118 @@
         }
 
-    // paging
-        if ( $paged = get_query_var('paged') ) {
-            if ( $paged > 0 ) {
-                if ( !$redirect_url )
-                    $redirect_url = $requested_url;
-                $paged_redirect = @parse_url($redirect_url);
-                $paged_redirect['path'] = preg_replace('|/page/[0-9]+?(/+)?$|', '/', $paged_redirect['path']); // strip off any existing paging
-                $paged_redirect['path'] = preg_replace('|/index.php/?$|', '/', $paged_redirect['path']); // strip off trailing /index.php/
+    // paging and feeds
+        if ( get_query_var('paged') || is_feed() ) {
+            if ( !$redirect_url )
+                $redirect_url = $requested_url;
+            $paged_redirect = @parse_url($redirect_url);
+            while ( preg_match( '#page/[0-9]+?(/+)?$#', $paged_redirect['path'] ) || preg_match( '#/feed(/[a-z0-9-]*?(/+)?)?$#', $paged_redirect['path'] ) ) {
+                // Strip off paging and feed
+                $paged_redirect['path'] = preg_replace('#/page/[0-9]+?(/+)?$#', '/', $paged_redirect['path']); // strip off any existing paging
+                $paged_redirect['path'] = preg_replace('#/feed(/[a-z0-9-]*?(/+)?)?$#', '/', $paged_redirect['path']); // strip off any existing feed
+            }
+
+            $paged_redirect['path'] = preg_replace('|/index.php/?$|', '/', $paged_redirect['path']); // strip off trailing /index.php/
+            if ( get_query_var('paged') > 0 ) {
+                $paged = get_query_var('paged');
                 if ( $paged > 1 && !is_single() ) {
                     $paged_redirect['path'] = trailingslashit($paged_redirect['path']);
@@ -131 +137 @@
                         $paged_redirect['path'] .= 'index.php/';
                     $paged_redirect['path'] .= user_trailingslashit("page/$paged", 'paged');
-                } elseif ( !is_home() && !is_single() ){
+                } elseif ( !is_single() ) {
                     $paged_redirect['path'] = user_trailingslashit($paged_redirect['path'], 'paged');
                 }
-                $redirect_url = $paged_redirect['scheme'] . '://' . $paged_redirect['host'] . $paged_redirect['path'];
-                $redirect['path'] = $paged_redirect['path'];
-            }
-            $redirect['query'] = remove_query_arg('paged', $redirect['query']);
+            }
+            if ( is_feed() ) {
+                $paged_redirect['path'] = user_trailingslashit( trailingslashit( $paged_redirect['path'] ) . 'feed/' . ( ( 'rss2' ==  get_query_var('feed') || 'feed' == get_query_var('feed') ) ? '' : get_query_var('feed') ), 'feed' );
+            }
+            $redirect_url = $paged_redirect['scheme'] . '://' . $paged_redirect['host'] . $paged_redirect['path'];
+            $redirect['path'] = $paged_redirect['path'];
+            $redirect['query'] = remove_query_arg( 'paged', $redirect['query'] );
+            $redirect['query'] = remove_query_arg(  'feed', $redirect['query'] );
         }
     }
@@ -249 +259 @@
             exit();
         } else {
+            // Debug
+            // die("1: $redirect_url<br />2: " . redirect_canonical( $redirect_url, false ) );
             return false;
         }