WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 14 years ago

#18235 closed task (blessed) (fixed)

Clean up remnants from having negative Post_IDs

Reported by: azaozz's profile azaozz Owned by: azaozz's profile azaozz
Milestone: 3.3 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch needs-testing
Focuses: Cc:

Description

We switched to creating new post and having "real" post ID as soon as the user loads the Add New Post screen quite long time ago.

However there is still some code expecting negative post IDs and dealing with attachments added before a draft was saved. Seems it's time to remove it.

Attachments (3)

18235.patch​ (2.9 KB) - added by azaozz 15 years ago.
Marked the places with old and redundant code
18235.2.patch​ (4.0 KB) - added by ethitter 15 years ago.
Removed old code as indicated by azaozz, tested editor and QuickPress (JS and non-JS), fixed internal references to wp_write_post.
18235.diff​ (4.4 KB) - added by markjaquith 14 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (11)

@azaozz
15 years ago

Marked the places with old and redundant code

#1 @scribu
15 years ago

If the code is redundant it should be removed, rather than having comments around it.

#2 @ramoonus
15 years ago

  • Keywords has-patch added

#3 @ocean90
15 years ago

  • Keywords needs-patch added; has-patch removed

@ethitter
15 years ago

Removed old code as indicated by azaozz, tested editor and QuickPress (JS and non-JS), fixed internal references to wp_write_post.

#4 @ethitter
15 years ago

  • Keywords has-patch added; needs-patch removed

#5 @ethitter
15 years ago

  • Cc ehitter@… added
  • Keywords needs-testing added

@markjaquith
14 years ago

Refreshed patch

#6 @markjaquith
14 years ago

  • Owner set to azaozz
  • Status changed from new to reviewing

Refreshed the patch. Basic testing only. azaozz, can you take this one, since you were the one to mark up the areas in the first place?

#7 @markjaquith
14 years ago

  • Type changed from enhancement to task (blessed)

Not an enhancement.

#8 @azaozz
14 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In [18823]:

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

Note: See TracTickets for help on using tickets.