Wordpress 3.5
[autoinstalls/wordpress.git] / wp-includes / post.php
index dee484e0b1af02494c4128ae9797e220f8b30542..54f17f41596e69fa7d0ee8948eec1fab6e9b5d7f 100644 (file)
@@ -52,21 +52,27 @@ function create_initial_post_types() {
 
        register_post_type( 'attachment', array(
                'labels' => array(
-                       'name' => __( 'Media' ),
-                       'edit_item' => __( 'Edit Media' ),
+                       'name' => _x('Media', 'post type general name'),
+                       'name_admin_bar' => _x( 'Media', 'add new from admin bar' ),
+                       'add_new' => _x( 'Add New', 'add new media' ),
+                       'edit_item' => __( 'Edit Media' ),
+                       'view_item' => __( 'View Attachment Page' ),
                ),
                'public' => true,
-               'show_ui' => false,
+               'show_ui' => true,
                '_builtin' => true, /* internal use only. don't use this when registering your own post type. */
-               '_edit_link' => 'media.php?attachment_id=%d', /* internal use only. don't use this when registering your own post type. */
+               '_edit_link' => 'post.php?post=%d', /* internal use only. don't use this when registering your own post type. */
                'capability_type' => 'post',
+               'capabilities' => array(
+                       'create_posts' => 'upload_files',
+               ),
                'map_meta_cap' => true,
                'hierarchical' => false,
                'rewrite' => false,
                'query_var' => false,
                'show_in_nav_menus' => false,
                'delete_with_user' => true,
-               'supports' => array( 'comments', 'author' ),
+               'supports' => array( 'title', 'author', 'comments' ),
        ) );
 
        register_post_type( 'revision', array(
@@ -204,9 +210,10 @@ function update_attached_file( $attachment_id, $file ) {
                return false;
 
        $file = apply_filters( 'update_attached_file', $file, $attachment_id );
-       $file = _wp_relative_upload_path($file);
-
-       return update_post_meta( $attachment_id, '_wp_attached_file', $file );
+       if ( $file = _wp_relative_upload_path( $file ) )
+               return update_post_meta( $attachment_id, '_wp_attached_file', $file );
+       else
+               return delete_post_meta( $attachment_id, '_wp_attached_file' );
 }
 
 /**
@@ -223,11 +230,10 @@ function update_attached_file( $attachment_id, $file ) {
 function _wp_relative_upload_path( $path ) {
        $new_path = $path;
 
-       if ( ($uploads = wp_upload_dir()) && false === $uploads['error'] ) {
-               if ( 0 === strpos($new_path, $uploads['basedir']) ) {
-                               $new_path = str_replace($uploads['basedir'], '', $new_path);
-                               $new_path = ltrim($new_path, '/');
-               }
+       $uploads = wp_upload_dir();
+       if ( 0 === strpos( $new_path, $uploads['basedir'] ) ) {
+                       $new_path = str_replace( $uploads['basedir'], '', $new_path );
+                       $new_path = ltrim( $new_path, '/' );
        }
 
        return apply_filters( '_wp_relative_upload_path', $new_path, $path );
@@ -372,56 +378,297 @@ function get_extended($post) {
  * @uses $wpdb
  * @link http://codex.wordpress.org/Function_Reference/get_post
  *
- * @param int|object $post Post ID or post object.
+ * @param int|object $post Post ID or post object. Optional, default is the current post from the loop.
  * @param string $output Optional, default is Object. Either OBJECT, ARRAY_A, or ARRAY_N.
  * @param string $filter Optional, default is raw.
- * @return mixed Post data
+ * @return WP_Post|null WP_Post on success or null on failure
  */
-function &get_post(&$post, $output = OBJECT, $filter = 'raw') {
-       global $wpdb;
-       $null = null;
+function get_post( $post = null, $output = OBJECT, $filter = 'raw' ) {
+       if ( empty( $post ) && isset( $GLOBALS['post'] ) )
+               $post = $GLOBALS['post'];
 
-       if ( empty($post) ) {
-               if ( isset($GLOBALS['post']) )
-                       $_post = & $GLOBALS['post'];
-               else
-                       return $null;
-       } elseif ( is_object($post) && empty($post->filter) ) {
-               _get_post_ancestors($post);
-               $_post = sanitize_post($post, 'raw');
-               wp_cache_add($post->ID, $_post, 'posts');
-       } elseif ( is_object($post) && 'raw' == $post->filter ) {
+       if ( is_a( $post, 'WP_Post' ) ) {
                $_post = $post;
+       } elseif ( is_object( $post ) ) {
+               if ( empty( $post->filter ) ) {
+                       $_post = sanitize_post( $post, 'raw' );
+                       $_post = new WP_Post( $_post );
+               } elseif ( 'raw' == $post->filter ) {
+                       $_post = new WP_Post( $post );
+               } else {
+                       $_post = WP_Post::get_instance( $post->ID );
+               }
        } else {
-               if ( is_object($post) )
-                       $post_id = $post->ID;
-               else
-                       $post_id = $post;
+               $_post = WP_Post::get_instance( $post );
+       }
+
+       if ( ! $_post )
+               return null;
+
+       $_post = $_post->filter( $filter );
+
+       if ( $output == ARRAY_A )
+               return $_post->to_array();
+       elseif ( $output == ARRAY_N )
+               return array_values( $_post->to_array() );
+
+       return $_post;
+}
+
+/**
+ * WordPress Post class.
+ *
+ * @since 3.5.0
+ *
+ */
+final class WP_Post {
+
+       /**
+        *
+        * @var int
+        */
+       public $ID;
+
+       /**
+        *
+        * @var int
+        */
+       public $post_author = 0;
+
+       /**
+        *
+        * @var string
+        */
+       public $post_date = '0000-00-00 00:00:00';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_date_gmt = '0000-00-00 00:00:00';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_content = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_title = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_excerpt = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_status = 'publish';
+
+       /**
+        *
+        * @var string
+        */
+       public $comment_status = 'open';
+
+       /**
+        *
+        * @var string
+        */
+       public $ping_status = 'open';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_password = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_name = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $to_ping = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $pinged = '';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_modified = '0000-00-00 00:00:00';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_modified_gmt = '0000-00-00 00:00:00';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_content_filtered = '';
+
+       /**
+        *
+        * @var int
+        */
+       public $post_parent = 0;
+
+       /**
+        *
+        * @var string
+        */
+       public $guid = '';
+
+       /**
+        *
+        * @var int
+        */
+       public $menu_order = 0;
+
+       /**
+        *
+        * @var string
+        */
+       public $post_type = 'post';
+
+       /**
+        *
+        * @var string
+        */
+       public $post_mime_type = '';
+
+       /**
+        *
+        * @var int
+        */
+       public $comment_count = 0;
+
+       /**
+        *
+        * @var string
+        */
+       public $filter;
+
+       public static function get_instance( $post_id ) {
+               global $wpdb;
 
                $post_id = (int) $post_id;
-               if ( ! $_post = wp_cache_get($post_id, 'posts') ) {
-                       $_post = $wpdb->get_row($wpdb->prepare("SELECT * FROM $wpdb->posts WHERE ID = %d LIMIT 1", $post_id));
+               if ( ! $post_id )
+                       return false;
+
+               $_post = wp_cache_get( $post_id, 'posts' );
+
+               if ( ! $_post ) {
+                       $_post = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->posts WHERE ID = %d LIMIT 1", $post_id ) );
+
                        if ( ! $_post )
-                               return $null;
-                       _get_post_ancestors($_post);
-                       $_post = sanitize_post($_post, 'raw');
-                       wp_cache_add($_post->ID, $_post, 'posts');
+                               return false;
+
+                       $_post = sanitize_post( $_post, 'raw' );
+                       wp_cache_add( $_post->ID, $_post, 'posts' );
+               } elseif ( empty( $_post->filter ) ) {
+                       $_post = sanitize_post( $_post, 'raw' );
                }
+
+               return new WP_Post( $_post );
        }
 
-       if ($filter != 'raw')
-               $_post = sanitize_post($_post, $filter);
+       public function __construct( $post ) {
+               foreach ( get_object_vars( $post ) as $key => $value )
+                       $this->$key = $value;
+       }
 
-       if ( $output == OBJECT ) {
-               return $_post;
-       } elseif ( $output == ARRAY_A ) {
-               $__post = get_object_vars($_post);
-               return $__post;
-       } elseif ( $output == ARRAY_N ) {
-               $__post = array_values(get_object_vars($_post));
-               return $__post;
-       } else {
-               return $_post;
+       public function __isset( $key ) {
+               if ( 'ancestors' == $key )
+                       return true;
+
+               if ( 'page_template' == $key )
+                       return ( 'page' == $this->post_type );
+
+               if ( 'post_category' == $key )
+                  return true;
+
+               if ( 'tags_input' == $key )
+                  return true;
+
+               return metadata_exists( 'post', $this->ID, $key );
+       }
+
+       public function __get( $key ) {
+               if ( 'page_template' == $key && $this->__isset( $key ) ) {
+                       return get_post_meta( $this->ID, '_wp_page_template', true );
+               }
+
+               if ( 'post_category' == $key ) {
+                       if ( is_object_in_taxonomy( $this->post_type, 'category' ) )
+                               $terms = get_the_terms( $this, 'category' );
+
+                       if ( empty( $terms ) )
+                               return array();
+
+                       return wp_list_pluck( $terms, 'term_id' );
+               }
+
+               if ( 'tags_input' == $key ) {
+                       if ( is_object_in_taxonomy( $this->post_type, 'post_tag' ) )
+                               $terms = get_the_terms( $this, 'post_tag' );
+
+                       if ( empty( $terms ) )
+                               return array();
+
+                       return wp_list_pluck( $terms, 'name' );
+               }
+
+               // Rest of the values need filtering
+
+               if ( 'ancestors' == $key )
+                       $value = get_post_ancestors( $this );
+               else
+                       $value = get_post_meta( $this->ID, $key, true );
+
+               if ( $this->filter )
+                       $value = sanitize_post_field( $key, $value, $this->ID, $this->filter );
+
+               return $value;
+       }
+
+       public function filter( $filter ) {
+               if ( $this->filter == $filter )
+                       return $this;
+
+               if ( $filter == 'raw' )
+                       return self::get_instance( $this->ID );
+
+               return sanitize_post( $this, $filter );
+       }
+
+       public function to_array() {
+               $post = get_object_vars( $this );
+
+               foreach ( array( 'ancestors', 'page_template', 'post_category', 'tags_input' ) as $key ) {
+                       if ( $this->__isset( $key ) )
+                               $post[ $key ] = $this->__get( $key );
+               }
+
+               return $post;
        }
 }
 
@@ -433,16 +680,28 @@ function &get_post(&$post, $output = OBJECT, $filter = 'raw') {
  * @param int|object $post Post ID or post object
  * @return array Ancestor IDs or empty array if none are found.
  */
-function get_post_ancestors($post) {
-       $post = get_post($post);
+function get_post_ancestors( $post ) {
+       if ( ! $post )
+               return false;
 
-       if ( ! isset( $post->ancestors ) )
-               _get_post_ancestors( $post );
+       $post = get_post( $post );
 
-       if ( ! empty( $post->ancestors ) )
-               return $post->ancestors;
+       if ( empty( $post->post_parent ) || $post->post_parent == $post->ID )
+               return array();
 
-       return array();
+       $ancestors = array();
+
+       $id = $ancestors[] = $post->post_parent;
+
+       while ( $ancestor = get_post( $id ) ) {
+               // Loop detection: If the ancestor has been seen before, break.
+               if ( empty( $ancestor->post_parent ) || ( $ancestor->post_parent == $post->ID ) || in_array( $ancestor->post_parent, $ancestors ) )
+                       break;
+
+               $id = $ancestors[] = $ancestor->post_parent;
+       }
+
+       return $ancestors;
 }
 
 /**
@@ -460,16 +719,12 @@ function get_post_ancestors($post) {
  * @param string $field Post field name
  * @param id $post Post ID
  * @param string $context Optional. How to filter the field. Default is display.
- * @return WP_Error|string Value in post field or WP_Error on failure
+ * @return bool|string False on failure or returns the value in post field
  */
 function get_post_field( $field, $post, $context = 'display' ) {
-       $post = (int) $post;
        $post = get_post( $post );
 
-       if ( is_wp_error($post) )
-               return $post;
-
-       if ( !is_object($post) )
+       if ( !$post )
                return '';
 
        if ( !isset($post->$field) )
@@ -490,7 +745,7 @@ function get_post_field( $field, $post, $context = 'display' ) {
  * @return bool|string False on failure or returns the mime type
  */
 function get_post_mime_type($ID = '') {
-       $post = get_post($ID);
+       $post = get_post($ID);
 
        if ( is_object($post) )
                return $post->post_mime_type;
@@ -671,7 +926,19 @@ function register_post_status($post_status, $args = array()) {
                $wp_post_statuses = array();
 
        // Args prefixed with an underscore are reserved for internal use.
-       $defaults = array('label' => false, 'label_count' => false, 'exclude_from_search' => null, '_builtin' => false, '_edit_link' => 'post.php?post=%d', 'capability_type' => 'post', 'hierarchical' => false, 'public' => null, 'internal' => null, 'protected' => null, 'private' => null, 'show_in_admin_all' => null, 'publicly_queryable' => null, 'show_in_admin_status_list' => null, 'show_in_admin_all_list' => null, 'single_view_cap' => null);
+       $defaults = array(
+               'label' => false,
+               'label_count' => false,
+               'exclude_from_search' => null,
+               '_builtin' => false,
+               'public' => null,
+               'internal' => null,
+               'protected' => null,
+               'private' => null,
+               'publicly_queryable' => null,
+               'show_in_admin_status_list' => null,
+               'show_in_admin_all_list' => null,
+       );
        $args = wp_parse_args($args, $defaults);
        $args = (object) $args;
 
@@ -703,10 +970,7 @@ function register_post_status($post_status, $args = array()) {
                $args->show_in_admin_all_list = !$args->internal;
 
        if ( null === $args->show_in_admin_status_list )
-                       $args->show_in_admin_status_list = !$args->internal;
-
-       if ( null === $args->single_view_cap )
-               $args->single_view_cap = $args->public ? '' : 'edit';
+               $args->show_in_admin_status_list = !$args->internal;
 
        if ( false === $args->label )
                $args->label = $post_status;
@@ -804,19 +1068,12 @@ function post_type_exists( $post_type ) {
  *
  * @uses $post The Loop current post global
  *
- * @param mixed $the_post Optional. Post object or post ID.
+ * @param mixed $post Optional. Post object or post ID.
  * @return bool|string post type or false on failure.
  */
-function get_post_type( $the_post = false ) {
-       global $post;
-
-       if ( false === $the_post )
-               $the_post = $post;
-       elseif ( is_numeric($the_post) )
-               $the_post = get_post($the_post);
-
-       if ( is_object($the_post) )
-               return $the_post->post_type;
+function get_post_type( $post = null ) {
+       if ( $post = get_post( $post ) )
+               return $post->post_type;
 
        return false;
 }
@@ -1020,7 +1277,7 @@ function register_post_type( $post_type, $args = array() ) {
        if ( ! empty($args->supports) ) {
                add_post_type_support($post_type, $args->supports);
                unset($args->supports);
-       } else {
+       } elseif ( false !== $args->supports ) {
                // Add default features
                add_post_type_support($post_type, array('title', 'editor'));
        }
@@ -1028,7 +1285,8 @@ function register_post_type( $post_type, $args = array() ) {
        if ( false !== $args->query_var && !empty($wp) ) {
                if ( true === $args->query_var )
                        $args->query_var = $post_type;
-               $args->query_var = sanitize_title_with_dashes($args->query_var);
+               else
+                       $args->query_var = sanitize_title_with_dashes($args->query_var);
                $wp->add_query_var($args->query_var);
        }
 
@@ -1182,6 +1440,10 @@ function get_post_type_capabilities( $args ) {
 
        $capabilities = array_merge( $default_capabilities, $args->capabilities );
 
+       // Post creation capability simply maps to edit_posts by default:
+       if ( ! isset( $capabilities['create_posts'] ) )
+               $capabilities['create_posts'] = $capabilities['edit_posts'];
+
        // Remember meta capabilities for future reference.
        if ( $args->map_meta_cap )
                _post_type_meta_capabilities( $capabilities );
@@ -1245,7 +1507,11 @@ function get_post_type_labels( $post_type_object ) {
                'all_items' => array( __( 'All Posts' ), __( 'All Pages' ) )
        );
        $nohier_vs_hier_defaults['menu_name'] = $nohier_vs_hier_defaults['name'];
-       return _get_custom_object_labels( $post_type_object, $nohier_vs_hier_defaults );
+
+       $labels = _get_custom_object_labels( $post_type_object, $nohier_vs_hier_defaults );
+
+       $post_type = $post_type_object->name;
+       return apply_filters( "post_type_labels_{$post_type}", $labels );
 }
 
 /**
@@ -1668,9 +1934,9 @@ function is_sticky( $post_id = 0 ) {
  * @since 2.3.0
  * @uses sanitize_post_field() Used to sanitize the fields.
  *
- * @param object|array $post The Post Object or Array
+ * @param object|WP_Post|array $post The Post Object or Array
  * @param string $context Optional, default is 'display'. How to sanitize post fields.
- * @return object|array The now sanitized Post Object or Array (will be the same type as $post)
+ * @return object|WP_Post|array The now sanitized Post Object or Array (will be the same type as $post)
  */
 function sanitize_post($post, $context = 'display') {
        if ( is_object($post) ) {
@@ -1916,6 +2182,23 @@ function wp_count_attachments( $mime_type = '' ) {
        return (object) $stats;
 }
 
+/**
+ * Get default post mime types
+ *
+ * @since 2.9.0
+ *
+ * @return array
+ */
+function get_post_mime_types() {
+       $post_mime_types = array(       //      array( adj, noun )
+               'image' => array(__('Images'), __('Manage Images'), _n_noop('Image <span class="count">(%s)</span>', 'Images <span class="count">(%s)</span>')),
+               'audio' => array(__('Audio'), __('Manage Audio'), _n_noop('Audio <span class="count">(%s)</span>', 'Audio <span class="count">(%s)</span>')),
+               'video' => array(__('Video'), __('Manage Video'), _n_noop('Video <span class="count">(%s)</span>', 'Video <span class="count">(%s)</span>')),
+       );
+
+       return apply_filters('post_mime_types', $post_mime_types);
+}
+
 /**
  * Check a MIME-Type against a list.
  *
@@ -2086,8 +2369,8 @@ function wp_delete_post( $postid = 0, $force_delete = false ) {
 
        clean_post_cache( $post );
 
-       if ( is_post_type_hierarchical( $post->post_type ) ) {
-               foreach ( (array) $children as $child )
+       if ( is_post_type_hierarchical( $post->post_type ) && $children ) {
+               foreach ( $children as $child )
                        clean_post_cache( $child );
        }
 
@@ -2115,7 +2398,7 @@ function wp_trash_post($post_id = 0) {
        if ( !EMPTY_TRASH_DAYS )
                return wp_delete_post($post_id, true);
 
-       if ( !$post = wp_get_single_post($post_id, ARRAY_A) )
+       if ( !$post = get_post($post_id, ARRAY_A) )
                return $post;
 
        if ( $post['post_status'] == 'trash' )
@@ -2147,7 +2430,7 @@ function wp_trash_post($post_id = 0) {
  * @return mixed False on failure
  */
 function wp_untrash_post($post_id = 0) {
-       if ( !$post = wp_get_single_post($post_id, ARRAY_A) )
+       if ( !$post = get_post($post_id, ARRAY_A) )
                return $post;
 
        if ( $post['post_status'] != 'trash' )
@@ -2377,49 +2660,6 @@ function wp_get_recent_posts( $args = array(), $output = ARRAY_A ) {
 
 }
 
-/**
- * Retrieve a single post, based on post ID.
- *
- * Has categories in 'post_category' property or key. Has tags in 'tags_input'
- * property or key.
- *
- * @since 1.0.0
- *
- * @param int $postid Post ID.
- * @param string $mode How to return result, either OBJECT, ARRAY_N, or ARRAY_A.
- * @return object|array Post object or array holding post contents and information
- */
-function wp_get_single_post($postid = 0, $mode = OBJECT) {
-       $postid = (int) $postid;
-
-       $post = get_post($postid, $mode);
-
-       if (
-               ( OBJECT == $mode && empty( $post->ID ) ) ||
-               ( OBJECT != $mode && empty( $post['ID'] ) )
-       )
-               return ( OBJECT == $mode ? null : array() );
-
-       // Set categories and tags
-       if ( $mode == OBJECT ) {
-               $post->post_category = array();
-               if ( is_object_in_taxonomy($post->post_type, 'category') )
-                       $post->post_category = wp_get_post_categories($postid);
-               $post->tags_input = array();
-               if ( is_object_in_taxonomy($post->post_type, 'post_tag') )
-                       $post->tags_input = wp_get_post_tags($postid, array('fields' => 'names'));
-       } else {
-               $post['post_category'] = array();
-               if ( is_object_in_taxonomy($post['post_type'], 'category') )
-                       $post['post_category'] = wp_get_post_categories($postid);
-               $post['tags_input'] = array();
-               if ( is_object_in_taxonomy($post['post_type'], 'post_tag') )
-                       $post['tags_input'] = wp_get_post_tags($postid, array('fields' => 'names'));
-       }
-
-       return $post;
-}
-
 /**
  * Insert a post.
  *
@@ -2547,6 +2787,18 @@ function wp_insert_post($postarr, $wp_error = false) {
        if ( empty($post_date) || '0000-00-00 00:00:00' == $post_date )
                $post_date = current_time('mysql');
 
+               // validate the date
+               $mm = substr( $post_date, 5, 2 );
+               $jj = substr( $post_date, 8, 2 );
+               $aa = substr( $post_date, 0, 4 );
+               $valid_date = wp_checkdate( $mm, $jj, $aa, $post_date );
+               if ( !$valid_date ) {
+                       if ( $wp_error )
+                               return new WP_Error( 'invalid_date', __( 'Whoops, the provided date is invalid.' ) );
+                       else
+                               return 0;
+               }
+
        if ( empty($post_date_gmt) || '0000-00-00 00:00:00' == $post_date_gmt ) {
                if ( !in_array( $post_status, array( 'draft', 'pending', 'auto-draft' ) ) )
                        $post_date_gmt = get_gmt_from_date($post_date);
@@ -2667,18 +2919,18 @@ function wp_insert_post($postarr, $wp_error = false) {
 
        $current_guid = get_post_field( 'guid', $post_ID );
 
-       clean_post_cache( $post_ID );
-
        // Set GUID
        if ( !$update && '' == $current_guid )
                $wpdb->update( $wpdb->posts, array( 'guid' => get_permalink( $post_ID ) ), $where );
 
+       clean_post_cache( $post_ID );
+
        $post = get_post($post_ID);
 
        if ( !empty($page_template) && 'page' == $data['post_type'] ) {
                $post->page_template = $page_template;
-               $page_templates = get_page_templates();
-               if ( 'default' != $page_template && !in_array($page_template, $page_templates) ) {
+               $page_templates = wp_get_theme()->get_page_templates();
+               if ( 'default' != $page_template && ! isset( $page_templates[ $page_template ] ) ) {
                        if ( $wp_error )
                                return new WP_Error('invalid_page_template', __('The page template is invalid.'));
                        else
@@ -2710,9 +2962,10 @@ function wp_insert_post($postarr, $wp_error = false) {
  * @since 1.0.0
  *
  * @param array|object $postarr Post data. Arrays are expected to be escaped, objects are not.
- * @return int 0 on failure, Post ID on success.
+ * @param bool $wp_error Optional. Allow return of WP_Error on failure.
+ * @return int|WP_Error The value 0 or WP_Error on failure. The post ID on success.
  */
-function wp_update_post($postarr = array()) {
+function wp_update_post( $postarr = array(), $wp_error = false ) {
        if ( is_object($postarr) ) {
                // non-escaped post was passed
                $postarr = get_object_vars($postarr);
@@ -2720,7 +2973,7 @@ function wp_update_post($postarr = array()) {
        }
 
        // First, get all of the original fields
-       $post = wp_get_single_post($postarr['ID'], ARRAY_A);
+       $post = get_post($postarr['ID'], ARRAY_A);
 
        // Escape data pulled from DB.
        $post = add_magic_quotes($post);
@@ -2750,39 +3003,25 @@ function wp_update_post($postarr = array()) {
        if ($postarr['post_type'] == 'attachment')
                return wp_insert_attachment($postarr);
 
-       return wp_insert_post($postarr);
+       return wp_insert_post( $postarr, $wp_error );
 }
 
 /**
  * Publish a post by transitioning the post status.
  *
  * @since 2.1.0
- * @uses $wpdb
- * @uses do_action() Calls 'edit_post', 'save_post', and 'wp_insert_post' on post_id and post data.
+ * @uses wp_update_post()
  *
- * @param int $post_id Post ID.
- * @return null
+ * @param mixed $post Post ID or object.
  */
-function wp_publish_post($post_id) {
-       global $wpdb;
-
-       $post = get_post($post_id);
-
-       if ( empty($post) )
+function wp_publish_post( $post ) {
+       if ( ! $post = get_post( $post ) )
                return;
-
        if ( 'publish' == $post->post_status )
                return;
 
-       $wpdb->update( $wpdb->posts, array( 'post_status' => 'publish' ), array( 'ID' => $post_id ) );
-
-       $old_status = $post->post_status;
        $post->post_status = 'publish';
-       wp_transition_post_status('publish', $old_status, $post);
-
-       do_action('edit_post', $post_id, $post);
-       do_action('save_post', $post_id, $post);
-       do_action('wp_insert_post', $post_id, $post);
+       wp_update_post( $post );
 }
 
 /**
@@ -2837,6 +3076,8 @@ function wp_unique_post_slug( $slug, $post_ID, $post_status, $post_type, $post_p
 
        global $wpdb, $wp_rewrite;
 
+       $original_slug = $slug;
+
        $feeds = $wp_rewrite->feeds;
        if ( ! is_array( $feeds ) )
                $feeds = array();
@@ -2857,6 +3098,8 @@ function wp_unique_post_slug( $slug, $post_ID, $post_status, $post_type, $post_p
                        $slug = $alt_post_name;
                }
        } elseif ( in_array( $post_type, $hierarchical_post_types ) ) {
+               if ( 'nav_menu_item' == $post_type )
+                       return $slug;
                // Page slugs must be unique within their own trees. Pages are in a separate
                // 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";
@@ -2887,7 +3130,7 @@ function wp_unique_post_slug( $slug, $post_ID, $post_status, $post_type, $post_p
                }
        }
 
-       return apply_filters( 'wp_unique_post_slug', $slug, $post_ID, $post_status, $post_type, $post_parent );
+       return apply_filters( 'wp_unique_post_slug', $slug, $post_ID, $post_status, $post_type, $post_parent, $original_slug );
 }
 
 /**
@@ -2953,11 +3196,10 @@ function wp_set_post_terms( $post_id = 0, $tags = '', $taxonomy = 'post_tag', $a
        // Hierarchical taxonomies must always pass IDs rather than names so that children with the same
        // names but different parents aren't confused.
        if ( is_taxonomy_hierarchical( $taxonomy ) ) {
-               $tags = array_map( 'intval', $tags );
-               $tags = array_unique( $tags );
+               $tags = array_unique( array_map( 'intval', $tags ) );
        }
 
-       return wp_set_object_terms($post_id, $tags, $taxonomy, $append);
+       return wp_set_object_terms( $post_id, $tags, $taxonomy, $append );
 }
 
 /**
@@ -2986,12 +3228,7 @@ function wp_set_post_categories($post_ID = 0, $post_categories = array()) {
                return true;
        }
 
-       if ( !empty($post_categories) ) {
-               $post_categories = array_map('intval', $post_categories);
-               $post_categories = array_unique($post_categories);
-       }
-
-       return wp_set_object_terms($post_ID, $post_categories, 'category');
+       return wp_set_post_terms($post_ID, $post_categories, 'category');
 }
 
 /**
@@ -3127,7 +3364,7 @@ function get_to_ping($post_id) {
 function trackback_url_list($tb_list, $post_id) {
        if ( ! empty( $tb_list ) ) {
                // get post data
-               $postdata = wp_get_single_post($post_id, ARRAY_A);
+               $postdata = get_post($post_id, ARRAY_A);
 
                // import postdata as variables
                extract($postdata, EXTR_SKIP);
@@ -3174,16 +3411,18 @@ function get_all_page_ids() {
 /**
  * Retrieves page data given a page ID or page object.
  *
+ * Use get_post() instead of get_page().
+ *
  * @since 1.5.1
+ * @deprecated 3.5.0
  *
  * @param mixed $page Page object or page ID. Passed by reference.
  * @param string $output What to output. OBJECT, ARRAY_A, or ARRAY_N.
  * @param string $filter How the return value should be filtered.
- * @return mixed Page data.
+ * @return WP_Post|null WP_Post on success or null on failure
  */
-function &get_page(&$page, $output = OBJECT, $filter = 'raw') {
-       $p = get_post($page, $output, $filter);
-       return $p;
+function get_page( $page, $output = OBJECT, $filter = 'raw') {
+       return get_post( $page, $output, $filter );
 }
 
 /**
@@ -3195,7 +3434,7 @@ function &get_page(&$page, $output = OBJECT, $filter = 'raw') {
  * @param string $page_path Page path
  * @param string $output Optional. Output type. OBJECT, ARRAY_N, or ARRAY_A. Default OBJECT.
  * @param string $post_type Optional. Post type. Default page.
- * @return mixed Null when complete.
+ * @return WP_Post|null WP_Post on success or null on failure
  */
 function get_page_by_path($page_path, $output = OBJECT, $post_type = 'page') {
        global $wpdb;
@@ -3210,7 +3449,7 @@ function get_page_by_path($page_path, $output = OBJECT, $post_type = 'page') {
        $in_string = "'". implode( "','", $parts ) . "'";
        $post_type_sql = $post_type;
        $wpdb->escape_by_ref( $post_type_sql );
-       $pages = $wpdb->get_results( "SELECT ID, post_name, post_parent FROM $wpdb->posts WHERE post_name IN ($in_string) AND (post_type = '$post_type_sql' OR post_type = 'attachment')", OBJECT_K );
+       $pages = $wpdb->get_results( "SELECT ID, post_name, post_parent, post_type FROM $wpdb->posts WHERE post_name IN ($in_string) AND (post_type = '$post_type_sql' OR post_type = 'attachment')", OBJECT_K );
 
        $revparts = array_reverse( $parts );
 
@@ -3229,13 +3468,14 @@ function get_page_by_path($page_path, $output = OBJECT, $post_type = 'page') {
 
                        if ( $p->post_parent == 0 && $count+1 == count( $revparts ) && $p->post_name == $revparts[ $count ] ) {
                                $foundid = $page->ID;
-                               break;
+                               if ( $page->post_type == $post_type )
+                                       break;
                        }
                }
        }
 
        if ( $foundid )
-               return get_page( $foundid, $output );
+               return get_post( $foundid, $output );
 
        return null;
 }
@@ -3249,13 +3489,13 @@ function get_page_by_path($page_path, $output = OBJECT, $post_type = 'page') {
  * @param string $page_title Page title
  * @param string $output Optional. Output type. OBJECT, ARRAY_N, or ARRAY_A. Default OBJECT.
  * @param string $post_type Optional. Post type. Default page.
- * @return mixed
+ * @return WP_Post|null WP_Post on success or null on failure
  */
 function get_page_by_title($page_title, $output = OBJECT, $post_type = 'page' ) {
        global $wpdb;
        $page = $wpdb->get_var( $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE post_title = %s AND post_type= %s", $page_title, $post_type ) );
        if ( $page )
-               return get_page($page, $output);
+               return get_post( $page, $output );
 
        return null;
 }
@@ -3273,7 +3513,7 @@ function get_page_by_title($page_title, $output = OBJECT, $post_type = 'page' )
  * @param array $pages List of pages' objects.
  * @return array
  */
-function &get_page_children($page_id, $pages) {
+function get_page_children($page_id, $pages) {
        $page_list = array();
        foreach ( (array) $pages as $page ) {
                if ( $page->post_parent == $page_id ) {
@@ -3297,7 +3537,7 @@ function &get_page_children($page_id, $pages) {
  * @param int $page_id Parent page ID.
  * @return array A list arranged by hierarchy. Children immediately follow their parents.
  */
-function &get_page_hierarchy( &$pages, $page_id = 0 ) {
+function get_page_hierarchy( &$pages, $page_id = 0 ) {
        if ( empty( $pages ) ) {
                $result = array();
                return $result;
@@ -3342,16 +3582,11 @@ function _page_traverse_name( $page_id, &$children, &$result ){
  */
 function get_page_uri($page) {
        if ( ! is_object($page) )
-               $page = get_page($page);
+               $page = get_post( $page );
        $uri = $page->post_name;
 
-       // A page cannot be it's own parent.
-       if ( $page->post_parent == $page->ID )
-               return $uri;
-
-       while ($page->post_parent != 0) {
-               $page = get_page($page->post_parent);
-               $uri = $page->post_name . "/" . $uri;
+       foreach ( $page->ancestors as $parent ) {
+               $uri = get_post( $parent )->post_name . "/" . $uri;
        }
 
        return $uri;
@@ -3370,9 +3605,11 @@ function get_page_uri($page) {
  * @param mixed $args Optional. Array or string of options that overrides defaults.
  * @return array List of pages matching defaults or $args
  */
-function &get_pages($args = '') {
+function get_pages($args = '') {
        global $wpdb;
 
+       $pages = false;
+
        $defaults = array(
                'child_of' => 0, 'sort_order' => 'ASC',
                'sort_column' => 'post_title', 'hierarchical' => 1,
@@ -3391,19 +3628,21 @@ function &get_pages($args = '') {
        // Make sure the post type is hierarchical
        $hierarchical_post_types = get_post_types( array( 'hierarchical' => true ) );
        if ( !in_array( $post_type, $hierarchical_post_types ) )
-               return false;
+               return $pages;
 
        // Make sure we have a valid post status
        if ( !is_array( $post_status ) )
                $post_status = explode( ',', $post_status );
        if ( array_diff( $post_status, get_post_stati() ) )
-               return false;
+               return $pages;
 
        $cache = array();
        $key = md5( serialize( compact(array_keys($defaults)) ) );
        if ( $cache = wp_cache_get( 'get_pages', 'posts' ) ) {
-               if ( is_array($cache) && isset( $cache[ $key ] ) ) {
-                       $pages = apply_filters('get_pages', $cache[ $key ], $r );
+               if ( is_array($cache) && isset( $cache[ $key ] ) && is_array( $cache[ $key ] ) ) {
+                       // Convert to WP_Post instances
+                       $pages = array_map( 'get_post', $cache[ $key ] );
+                       $pages = apply_filters( 'get_pages', $pages, $r );
                        return $pages;
                }
        }
@@ -3559,7 +3798,7 @@ function &get_pages($args = '') {
        update_post_cache( $pages );
 
        if ( $child_of || $hierarchical )
-               $pages = get_page_children($child_of, $pages);
+               $pages = get_page_children($child_of, $pages);
 
        if ( !empty($exclude_tree) ) {
                $exclude = (int) $exclude_tree;
@@ -3578,6 +3817,9 @@ function &get_pages($args = '') {
        $cache[ $key ] = $pages;
        wp_cache_set( 'get_pages', $cache, 'posts' );
 
+       // Convert to WP_Post instances
+       $pages = array_map( 'get_post', $pages );
+
        $pages = apply_filters('get_pages', $pages, $r);
 
        return $pages;
@@ -3601,7 +3843,7 @@ function is_local_attachment($url) {
        if (strpos($url, home_url('/?attachment_id=')) !== false)
                return true;
        if ( $id = url_to_postid($url) ) {
-               $post = get_post($id);
+               $post = get_post($id);
                if ( 'attachment' == $post->post_type )
                        return true;
        }
@@ -3676,13 +3918,12 @@ function wp_insert_attachment($object, $file = false, $parent = 0) {
        if ( ! in_array( $post_status, array( 'inherit', 'private' ) ) )
                $post_status = 'inherit';
 
+       if ( !empty($post_category) )
+               $post_category = array_filter($post_category); // Filter out empty terms
+
        // Make sure we set a valid category.
-       if ( !isset($post_category) || 0 == count($post_category) || !is_array($post_category) ) {
-               // 'post' requires at least one category.
-               if ( 'post' == $post_type )
-                       $post_category = array( get_option('default_category') );
-               else
-                       $post_category = array();
+       if ( empty($post_category) || 0 == count($post_category) || !is_array($post_category) ) {
+               $post_category = array();
        }
 
        // Are we updating or creating?
@@ -3767,7 +4008,22 @@ function wp_insert_attachment($object, $file = false, $parent = 0) {
                $wpdb->update( $wpdb->posts, compact("post_name"), array( 'ID' => $post_ID ) );
        }
 
-       wp_set_post_categories($post_ID, $post_category);
+       if ( is_object_in_taxonomy($post_type, 'category') )
+               wp_set_post_categories( $post_ID, $post_category );
+
+       if ( isset( $tags_input ) && is_object_in_taxonomy($post_type, 'post_tag') )
+               wp_set_post_tags( $post_ID, $tags_input );
+
+       // support for all custom taxonomies
+       if ( !empty($tax_input) ) {
+               foreach ( $tax_input as $taxonomy => $tags ) {
+                       $taxonomy_obj = get_taxonomy($taxonomy);
+                       if ( is_array($tags) ) // array = hierarchical, string = non-hierarchical.
+                               $tags = array_filter($tags);
+                       if ( current_user_can($taxonomy_obj->cap->assign_terms) )
+                               wp_set_post_terms( $post_ID, $tags, $taxonomy );
+               }
+       }
 
        if ( $file )
                update_attached_file( $post_ID, $file );
@@ -3897,7 +4153,7 @@ function wp_delete_attachment( $post_id, $force_delete = false ) {
  */
 function wp_get_attachment_metadata( $post_id = 0, $unfiltered = false ) {
        $post_id = (int) $post_id;
-       if ( !$post =& get_post( $post_id ) )
+       if ( !$post = get_post( $post_id ) )
                return false;
 
        $data = get_post_meta( $post->ID, '_wp_attachment_metadata', true );
@@ -3919,12 +4175,13 @@ function wp_get_attachment_metadata( $post_id = 0, $unfiltered = false ) {
  */
 function wp_update_attachment_metadata( $post_id, $data ) {
        $post_id = (int) $post_id;
-       if ( !$post =& get_post( $post_id ) )
+       if ( !$post = get_post( $post_id ) )
                return false;
 
-       $data = apply_filters( 'wp_update_attachment_metadata', $data, $post->ID );
-
-       return update_post_meta( $post->ID, '_wp_attachment_metadata', $data);
+       if ( $data = apply_filters( 'wp_update_attachment_metadata', $data, $post->ID ) )
+               return update_post_meta( $post->ID, '_wp_attachment_metadata', $data );
+       else
+               return delete_post_meta( $post->ID, '_wp_attachment_metadata' );
 }
 
 /**
@@ -3937,7 +4194,7 @@ function wp_update_attachment_metadata( $post_id, $data ) {
  */
 function wp_get_attachment_url( $post_id = 0 ) {
        $post_id = (int) $post_id;
-       if ( !$post =& get_post( $post_id ) )
+       if ( !$post = get_post( $post_id ) )
                return false;
 
        if ( 'attachment' != $post->post_type )
@@ -3976,7 +4233,7 @@ function wp_get_attachment_url( $post_id = 0 ) {
  */
 function wp_get_attachment_thumb_file( $post_id = 0 ) {
        $post_id = (int) $post_id;
-       if ( !$post =& get_post( $post_id ) )
+       if ( !$post = get_post( $post_id ) )
                return false;
        if ( !is_array( $imagedata = wp_get_attachment_metadata( $post->ID ) ) )
                return false;
@@ -3998,7 +4255,7 @@ function wp_get_attachment_thumb_file( $post_id = 0 ) {
  */
 function wp_get_attachment_thumb_url( $post_id = 0 ) {
        $post_id = (int) $post_id;
-       if ( !$post =& get_post( $post_id ) )
+       if ( !$post = get_post( $post_id ) )
                return false;
        if ( !$url = wp_get_attachment_url( $post->ID ) )
                return false;
@@ -4025,7 +4282,7 @@ function wp_get_attachment_thumb_url( $post_id = 0 ) {
  */
 function wp_attachment_is_image( $post_id = 0 ) {
        $post_id = (int) $post_id;
-       if ( !$post =& get_post( $post_id ) )
+       if ( !$post = get_post( $post_id ) )
                return false;
 
        if ( !$file = get_attached_file( $post->ID ) )
@@ -4033,7 +4290,7 @@ function wp_attachment_is_image( $post_id = 0 ) {
 
        $ext = preg_match('/\.([^.]+)$/', $file, $matches) ? strtolower($matches[1]) : false;
 
-       $image_exts = array('jpg', 'jpeg', 'gif', 'png');
+       $image_exts = array( 'jpg', 'jpeg', 'jpe', 'gif', 'png' );
 
        if ( 'image/' == substr($post->post_mime_type, 0, 6) || $ext && 'import' == $post->post_mime_type && in_array($ext, $image_exts) )
                return true;
@@ -4051,12 +4308,13 @@ function wp_attachment_is_image( $post_id = 0 ) {
 function wp_mime_type_icon( $mime = 0 ) {
        if ( !is_numeric($mime) )
                $icon = wp_cache_get("mime_type_icon_$mime");
+
+       $post_id = 0;
        if ( empty($icon) ) {
-               $post_id = 0;
                $post_mimes = array();
                if ( is_numeric($mime) ) {
                        $mime = (int) $mime;
-                       if ( $post =& get_post( $mime ) ) {
+                       if ( $post = get_post( $mime ) ) {
                                $post_id = (int) $post->ID;
                                $ext = preg_replace('/^.+?\.([^.]+)$/', '$1', $post->guid);
                                if ( !empty($ext) ) {
@@ -4098,7 +4356,7 @@ function wp_mime_type_icon( $mime = 0 ) {
                                        closedir($dh);
                                }
                        }
-                       wp_cache_set('icon_files', $icon_files, 600);
+                       wp_cache_add( 'icon_files', $icon_files, 'default', 600 );
                }
 
                // Icon basename - extension = MIME wildcard
@@ -4118,7 +4376,7 @@ function wp_mime_type_icon( $mime = 0 ) {
                        if ( isset($types[$wilds[0]])) {
                                $icon = $types[$wilds[0]];
                                if ( !is_numeric($mime) )
-                                       wp_cache_set("mime_type_icon_$mime", $icon);
+                                       wp_cache_add("mime_type_icon_$mime", $icon);
                                break;
                        }
                }
@@ -4195,9 +4453,10 @@ function get_private_posts_cap_sql( $post_type ) {
  * @param string $post_type Post type.
  * @param bool $full Optional. Returns a full WHERE statement instead of just an 'andalso' term.
  * @param int $post_author Optional. Query posts having a single author ID.
+ * @param bool $public_only Optional. Only return public posts. Skips cap checks for $current_user.  Default is false.
  * @return string SQL WHERE code that can be added to a query.
  */
-function get_posts_by_author_sql( $post_type, $full = true, $post_author = null ) {
+function get_posts_by_author_sql( $post_type, $full = true, $post_author = null, $public_only = false ) {
        global $user_ID, $wpdb;
 
        // Private posts
@@ -4221,18 +4480,21 @@ function get_posts_by_author_sql( $post_type, $full = true, $post_author = null
 
        $sql .= "(post_status = 'publish'";
 
-       if ( current_user_can( $cap ) ) {
-               // Does the user have the capability to view private posts? Guess so.
-               $sql .= " OR post_status = 'private'";
-       } elseif ( is_user_logged_in() ) {
-               // Users can view their own private posts.
-               $id = (int) $user_ID;
-               if ( null === $post_author || ! $full ) {
-                       $sql .= " OR post_status = 'private' AND post_author = $id";
-               } elseif ( $id == (int) $post_author ) {
+       // Only need to check the cap if $public_only is false
+       if ( false === $public_only ) {
+               if ( current_user_can( $cap ) ) {
+                       // Does the user have the capability to view private posts? Guess so.
                        $sql .= " OR post_status = 'private'";
+               } elseif ( is_user_logged_in() ) {
+                       // Users can view their own private posts.
+                       $id = (int) $user_ID;
+                       if ( null === $post_author || ! $full ) {
+                               $sql .= " OR post_status = 'private' AND post_author = $id";
+                       } elseif ( $id == (int) $post_author ) {
+                               $sql .= " OR post_status = 'private'";
+                       } // else none
                } // else none
-       } // else none
+       }
 
        $sql .= ')';
 
@@ -4351,8 +4613,6 @@ function update_post_cache( &$posts ) {
  * Cleaning means delete from the cache of the post. Will call to clean the term
  * object cache associated with the post ID.
  *
- * clean_post_cache() will call itself recursively for each child post.
- *
  * This function not run if $_wp_suspend_cache_invalidation is not empty. See
  * wp_suspend_cache_invalidation().
  *
@@ -4383,23 +4643,13 @@ function clean_post_cache( $post ) {
 
        do_action( 'clean_post_cache', $post->ID, $post );
 
+       if ( is_post_type_hierarchical( $post->post_type ) )
+               wp_cache_delete( 'get_pages', 'posts' );
+
        if ( 'page' == $post->post_type ) {
                wp_cache_delete( 'all_page_ids', 'posts' );
-               wp_cache_delete( 'get_pages', 'posts' );
                do_action( 'clean_page_cache', $post->ID );
        }
-
-       if ( $children = $wpdb->get_results( $wpdb->prepare("SELECT ID, post_type FROM $wpdb->posts WHERE post_parent = %d", $post->ID) ) ) {
-               foreach ( $children as $child ) {
-                       // Loop detection
-                       if ( $child->ID == $post->ID )
-                               continue;
-                       clean_post_cache( $child );
-               }
-       }
-
-       if ( is_multisite() )
-               wp_cache_delete( $wpdb->blogid . '-' . $post->ID, 'global-posts' );
 }
 
 /**
@@ -4430,8 +4680,8 @@ function update_post_caches(&$posts, $post_type = 'post', $update_term_cache = t
        foreach ( $posts as $post )
                $post_ids[] = $post->ID;
 
-       if ( empty($post_type) )
-               $post_type = 'post';
+       if ( ! $post_type )
+               $post_type = 'any';
 
        if ( $update_term_cache ) {
                if ( is_array($post_type) ) {
@@ -4571,9 +4821,8 @@ function _future_post_hook( $deprecated = '', $post ) {
  * @since 2.3.0
  * @access private
  * @uses $wpdb
- * @uses XMLRPC_REQUEST and APP_REQUEST constants.
+ * @uses XMLRPC_REQUEST constant.
  * @uses do_action() Calls 'xmlprc_publish_post' on post ID if XMLRPC_REQUEST is defined.
- * @uses do_action() Calls 'app_publish_post' on post ID if APP_REQUEST is defined.
  *
  * @param int $post_id The ID in the database table of the post being published
  */
@@ -4582,8 +4831,6 @@ function _publish_post_hook($post_id) {
 
        if ( defined('XMLRPC_REQUEST') )
                do_action('xmlrpc_publish_post', $post_id);
-       if ( defined('APP_REQUEST') )
-               do_action('app_publish_post', $post_id);
 
        if ( defined('WP_IMPORTING') )
                return;
@@ -4595,58 +4842,6 @@ function _publish_post_hook($post_id) {
        wp_schedule_single_event(time(), 'do_pings');
 }
 
-/**
- * Hook used to prevent page/post cache from staying dirty when a post is saved.
- *
- * @since 2.3.0
- * @access private
- *
- * @param int $post_id The ID in the database table for the $post
- * @param object $post Object type containing the post information
- */
-function _save_post_hook( $post_id, $post ) {
-       clean_post_cache( $post );
-}
-
-/**
- * Retrieve post ancestors and append to post ancestors property.
- *
- * Will only retrieve ancestors once, if property is already set, then nothing
- * will be done. If there is not a parent post, or post ID and post parent ID
- * are the same then nothing will be done.
- *
- * The parameter is passed by reference, so nothing needs to be returned. The
- * property will be updated and can be referenced after the function is
- * complete. The post parent will be an ancestor and the parent of the post
- * parent will be an ancestor. There will only be two ancestors at the most.
- *
- * @since 2.5.0
- * @access private
- * @uses $wpdb
- *
- * @param object $_post Post data.
- * @return null When nothing needs to be done.
- */
-function _get_post_ancestors(&$_post) {
-       global $wpdb;
-
-       if ( isset($_post->ancestors) )
-               return;
-
-       $_post->ancestors = array();
-
-       if ( empty($_post->post_parent) || $_post->ID == $_post->post_parent )
-               return;
-
-       $id = $_post->ancestors[] = (int) $_post->post_parent;
-       while ( $ancestor = $wpdb->get_var( $wpdb->prepare("SELECT `post_parent` FROM $wpdb->posts WHERE ID = %d LIMIT 1", $id) ) ) {
-               // Loop detection: If the ancestor has been seen before, break.
-               if ( ( $ancestor == $_post->ID ) || in_array($ancestor,  $_post->ancestors) )
-                       break;
-               $id = $_post->ancestors[] = (int) $ancestor;
-       }
-}
-
 /**
  * Determines which fields of posts are to be saved in revisions.
  *
@@ -4726,6 +4921,9 @@ function wp_save_post_revision( $post_id ) {
        if ( !$post = get_post( $post_id, ARRAY_A ) )
                return;
 
+       if ( 'auto-draft' == $post['post_status'] )
+               return;
+
        if ( !post_type_supports($post['post_type'], 'revisions') )
                return;
 
@@ -4890,7 +5088,7 @@ function _wp_put_post_revision( $post = null, $autosave = false ) {
  * @param string $filter Optional sanitation filter. @see sanitize_post()
  * @return mixed Null if error or post object if success
  */
-function &wp_get_post_revision(&$post, $output = OBJECT, $filter = 'raw') {
+function wp_get_post_revision(&$post, $output = OBJECT, $filter = 'raw') {
        $null = null;
        if ( !$revision = get_post( $post, OBJECT, $filter ) )
                return $revision;
@@ -5166,10 +5364,10 @@ function set_post_thumbnail( $post, $thumbnail_id ) {
        $post = get_post( $post );
        $thumbnail_id = absint( $thumbnail_id );
        if ( $post && $thumbnail_id && get_post( $thumbnail_id ) ) {
-               $thumbnail_html = wp_get_attachment_image( $thumbnail_id, 'thumbnail' );
-               if ( ! empty( $thumbnail_html ) ) {
+               if ( $thumbnail_html = wp_get_attachment_image( $thumbnail_id, 'thumbnail' ) )
                        return update_post_meta( $post->ID, '_thumbnail_id', $thumbnail_id );
-               }
+               else
+                       return delete_post_meta( $post->ID, '_thumbnail_id' );
        }
        return false;
 }