Changeset 19444 for trunk/wp-includes/post.php

Timestamp:

11/24/2011 12:20:21 AM (14 years ago)

Author:

dd32

Message:

Switch to sanitize_title_for_query() for Query sanitization (allows for pre-3.3 page slugs to be viewable), Don't update page slugs to new slug-types when the slug is not being changed, Don't issue a XHR if the page slug hasn't changed. Group effort props xknown, markjaquith, nacin. See #19292

File:

• trunk/wp-includes/post.php (modified) (2 diffs)

trunk/wp-includes/post.php

@@ -2491 +2491 @@
             $post_name = '';
     } else {
-        $post_name = sanitize_title($post_name);
+        // On updates, we need to check to see if it's using the old, fixed sanitization context.
+        $check_name = sanitize_title( $post_name, '', 'old-save' );
+        if ( $update && strtolower( urlencode( $post_name ) ) == $check_name && get_post_field( 'post_name', $ID ) == $check_name )
+            $post_name = $check_name;
+        else // new post, or slug has changed.
+            $post_name = sanitize_title($post_name);
     }
 
@@ -3154 +3159 @@
     $parts = explode( '/', trim( $page_path, '/' ) );
     $parts = array_map( 'esc_sql', $parts );
-    $parts = array_map( 'sanitize_title', $parts );
+    $parts = array_map( 'sanitize_title_for_query', $parts );
 
     $in_string = "'". implode( "','", $parts ) . "'";