Make sure attachments uploaded on unsaved drafts get reunited with their parent post. Props mdawaffe. fixes #3757

git-svn-id: http://svn.automattic.com/wordpress/trunk@4884 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
markjaquith 2007-02-15 01:03:06 +00:00
parent 629e270a8e
commit 6e9a8afe6e
1 changed files with 17 additions and 5 deletions

View File

@ -22,6 +22,7 @@ function wp_write_post() {
// Check for autosave collisions // Check for autosave collisions
$temp_id = false;
if ( isset($_POST['temp_ID']) ) { if ( isset($_POST['temp_ID']) ) {
$temp_id = (int) $_POST['temp_ID']; $temp_id = (int) $_POST['temp_ID'];
if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) ) if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) )
@ -33,7 +34,6 @@ function wp_write_post() {
if ( isset($draft_ids[$temp_id]) ) { // Edit, don't write if ( isset($draft_ids[$temp_id]) ) { // Edit, don't write
$_POST['post_ID'] = $draft_ids[$temp_id]; $_POST['post_ID'] = $draft_ids[$temp_id];
unset($_POST['temp_ID']); unset($_POST['temp_ID']);
relocate_children( $temp_id, $_POST['post_ID'] );
update_user_option( $user_ID, 'autosave_draft_ids', $draft_ids ); update_user_option( $user_ID, 'autosave_draft_ids', $draft_ids );
return edit_post(); return edit_post();
} }
@ -112,9 +112,15 @@ function wp_write_post() {
add_meta( $post_ID ); add_meta( $post_ID );
// Reunite any orphaned attachments with their parent // Reunite any orphaned attachments with their parent
if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) )
$draft_ids = array();
if ( $draft_temp_id = 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 // Update autosave collision detection
if ( $temp_id ) { if ( $temp_id ) {
relocate_children( $temp_id, $post_ID );
$draft_ids[$temp_id] = $post_ID; $draft_ids[$temp_id] = $post_ID;
update_user_option( $user_ID, 'autosave_draft_ids', $draft_ids ); update_user_option( $user_ID, 'autosave_draft_ids', $draft_ids );
} }
@ -139,14 +145,14 @@ function fix_attachment_links( $post_ID ) {
$post = & get_post( $post_ID, ARRAY_A ); $post = & get_post( $post_ID, ARRAY_A );
$search = "#<a[^>]+rel=('|\" )[^'\"]*attachment[^>]*>#ie"; $search = "#<a[^>]+rel=('|\")[^'\"]*attachment[^>]*>#ie";
// See if we have any rel="attachment" links // See if we have any rel="attachment" links
if ( 0 == preg_match_all( $search, $post['post_content'], $anchor_matches, PREG_PATTERN_ORDER ) ) if ( 0 == preg_match_all( $search, $post['post_content'], $anchor_matches, PREG_PATTERN_ORDER ) )
return; return;
$i = 0; $i = 0;
$search = "#[\s]+rel=(\"|' )(.*? )wp-att-(\d+ )\\1#i"; $search = "#[\s]+rel=(\"|')(.*?)wp-att-(\d+)\\1#i";
foreach ( $anchor_matches[0] as $anchor ) { foreach ( $anchor_matches[0] as $anchor ) {
if ( 0 == preg_match( $search, $anchor, $id_matches ) ) if ( 0 == preg_match( $search, $anchor, $id_matches ) )
continue; continue;
@ -279,7 +285,13 @@ function edit_post() {
add_meta( $post_ID ); add_meta( $post_ID );
wp_update_post( $_POST); wp_update_post( $_POST );
// Reunite any orphaned attachments with their parent
if ( !$draft_ids = get_user_option( 'autosave_draft_ids' ) )
$draft_ids = array();
if ( $draft_temp_id = 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 // Now that we have an ID we can fix any attachment anchor hrefs
fix_attachment_links( $post_ID ); fix_attachment_links( $post_ID );