Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#6389 closed defect (bug) (fixed)

Attachments added before first manual Save are lost from the post

Reported by: markjaquith Owned by: andy
Priority: high Milestone: 2.5
Component: Administration Version: 2.5
Severity: blocker Keywords:
Cc: Focuses:

Description

  1. New Post
  2. Add Media
  3. Add content to post
  4. Manually save post
  5. Note that media is no longer attached to the post

OR

  1. New Post
  2. Add content to post
  3. Allow autosave to fire, giving the post a "positive" ID
  4. Add media
  5. Manually Save Post
  6. Note that media is no longer attached to the post

We need to have autosave:

  1. reparent attachments uploaded on a temporary ID
  2. dynamically replace the temporary ID with the new "positive" ID in the Add Media hrefs

Attachments (1)

6389-autosave.diff​ (1.1 KB ) - added by andy 19 years ago.

Download all attachments as: .zip

Change History (4)

@andy
19 years ago

#1 @markjaquith
19 years ago

  • Resolution → fixed
  • Status new → closed

(In [7532]) Autosave fixes for attachments from andy. fixes #6389

#2 @markjaquith
19 years ago

  • Resolution fixed
  • Status closed → reopened

Found a case where it doesn't work.

  1. New Post (don't provide content)
  2. Add Media
  3. Close the box, and wait about a minute
  4. I think autosave tries to fire, but doesn't (no title/content)
  5. The post_id in the Add Media links turns to zero.

#3 @markjaquith
19 years ago

  • Resolution → fixed
  • Status reopened → closed

(In [7533]) Don't replace temp_id for "add media" hrefs if we don't yet have a positive post_id

Note: See TracTickets for help on using tickets.