Changeset 18823 for trunk/wp-admin/includes/post.php

Timestamp:

09/29/2011 09:47:52 PM (15 years ago)

Author:

azaozz

Message:

Clean up remnants from having negative Post_IDs, props markjaquith, fixes #18235

File:

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

trunk/wp-admin/includes/post.php

@@ -229 +229 @@
 
     wp_update_post( $post_data );
-
-    // Reunite any orphaned attachments with their parent
-    if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) )
-        $draft_ids = array();
-    if ( $draft_temp_id = (int) array_search( $post_ID, $draft_ids ) )
-        _relocate_children( $draft_temp_id, $post_ID );
 
     // Now that we have an ID we can fix any attachment anchor hrefs
@@ -539 +533 @@
     global $user_ID;
 
-
     if ( isset($_POST['post_type']) )
         $ptype = get_post_type_object($_POST['post_type']);
@@ -557 +550 @@
     unset( $_POST['filter'] );
 
-    // Check for autosave collisions
-    // Does this need to be updated? ~ Mark
-    $temp_id = false;
-    if ( isset($_POST['temp_ID']) ) {
-        $temp_id = (int) $_POST['temp_ID'];
-        if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) )
-            $draft_ids = array();
-        foreach ( $draft_ids as $temp => $real )
-            if ( time() + $temp > 86400 ) // 1 day: $temp is equal to -1 * time( then )
-                unset($draft_ids[$temp]);
-
-        if ( isset($draft_ids[$temp_id]) ) { // Edit, don't write
-            $_POST['post_ID'] = $draft_ids[$temp_id];
-            unset($_POST['temp_ID']);
-            update_user_option( $user_ID, 'autosave_draft_ids', $draft_ids );
-            return edit_post();
-        }
-    }
-
     // Edit don't write if we have a post id.
-    if ( isset( $_POST['ID'] ) ) {
-        $_POST['post_ID'] = $_POST['ID'];
-        unset ( $_POST['ID'] );
-    }
-    if ( isset( $_POST['post_ID'] ) ) {
+    if ( isset( $_POST['post_ID'] ) )
         return edit_post();
-    }
 
     $translated = _wp_translate_postdata( false );
@@ -617 +586 @@
     add_post_meta( $post_ID, '_edit_last', $GLOBALS['current_user']->ID );
 
-    // Reunite any orphaned attachments with their parent
-    // Does this need to be updated? ~ Mark
-    if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) )
-        $draft_ids = array();
-    if ( $draft_temp_id = (int) array_search( $post_ID, $draft_ids ) )
-        _relocate_children( $draft_temp_id, $post_ID );
-    if ( $temp_id && $temp_id != $draft_temp_id )
-        _relocate_children( $temp_id, $post_ID );
-
-    // Update autosave collision detection
-    if ( $temp_id ) {
-        $draft_ids[$temp_id] = $post_ID;
-        update_user_option( $user_ID, 'autosave_draft_ids', $draft_ids );
-    }
-
     // Now that we have an ID we can fix any attachment anchor hrefs
     _fix_attachment_links( $post_ID );
@@ -644 +598 @@
  *
  * @since 2.0.0
- *
+ 
+ * @uses wp_write_post()
+ * @uses is_wp_error()
+ * @uses wp_die()
  * @return unknown
  */
@@ -1323 +1280 @@
  * @since 2.7.0
  *
- * @uses wp_write_post()
+ * @uses get_post_status()
  * @uses edit_post()
  * @uses get_post()
  * @uses current_user_can()
+ * @uses wp_die()
  * @uses wp_create_post_autosave()
+ * @uses add_query_arg()
+ * @uses wp_create_nonce()
  *
  * @return str URL to redirect to show the preview