Changeset 30158 for trunk/src/wp-includes/post.php

Timestamp:

11/01/2014 08:36:23 PM (10 years ago)

Author:

wonderboymusic

Message:

All duplicate slugs across different post types.

Adds unit test.

Props mboynes, nacin.
Fixes #18962.

File:

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

trunk/src/wp-includes/post.php

@@ -3674 +3674 @@
         $feeds = array();
 
-    $hierarchical_post_types = get_post_types( array('hierarchical' => true) );
     if ( 'attachment' == $post_type ) {
         // Attachment slugs must be unique across all types.
@@ -3697 +3696 @@
             $slug = $alt_post_name;
         }
-    } elseif ( in_array( $post_type, $hierarchical_post_types ) ) {
+    } elseif ( is_post_type_hierarchical( $post_type ) ) {
         if ( 'nav_menu_item' == $post_type )
             return $slug;
@@ -3705 +3704 @@
          * namespace than posts so page slugs are allowed to overlap post slugs.
          */
-        $check_sql = "SELECT post_name FROM $wpdb->posts WHERE post_name = %s AND post_type IN ( '" . implode( "', '", esc_sql( $hierarchical_post_types ) ) . "' ) AND ID != %d AND post_parent = %d LIMIT 1";
-        $post_name_check = $wpdb->get_var( $wpdb->prepare( $check_sql, $slug, $post_ID, $post_parent ) );
+        $check_sql = "SELECT post_name FROM $wpdb->posts WHERE post_name = %s AND post_type = %s AND ID != %d AND post_parent = %d LIMIT 1";
+        $post_name_check = $wpdb->get_var( $wpdb->prepare( $check_sql, $slug, $post_type, $post_ID, $post_parent ) );
 
         /**