Make WordPress Core

Opened 12 years ago

Closed 20 months ago

Last modified 20 months ago

#19954 closed defect (bug) (fixed)

wp_insert_post overwrites categories when updating existing post

Reported by: markoheijnen's profile markoheijnen Owned by: johnbillion's profile johnbillion
Milestone: 6.1 Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: has-patch has-unit-tests commit
Focuses: Cc:

Description

When creating an post object with get_default_post_to_edit(), wp_insert_post() will ignored the set category.
It doesn't look if the post has already a good category.

Attachments (2)

19954.patch (1.0 KB) - added by markoheijnen 12 years ago.
Does fix the issue. Maybe need better formating
19954.diff (3.7 KB) - added by leewillis77 8 years ago.
Patch

Download all attachments as: .zip

Change History (18)

@markoheijnen
12 years ago

Does fix the issue. Maybe need better formating

#1 @nacin
12 years ago

  • Milestone changed from Awaiting Review to 3.4

#2 @hughwillfayle
12 years ago

  • Cc t.herzog@… added

Why should this be? WordPress sets category to the default one if there hasn't been setted one.

#3 @jkudish
12 years ago

  • Keywords has-patch added

#4 @nacin
12 years ago

Could we have some example code that triggers this?

Is this an issue to fix/benefit XML-RPC changes in 3.4 at this point?

#5 @markoheijnen
12 years ago

I don't believe it is xml-rpc related. What I did was adding the category with the metod wp_set_object_terms() on the post ID what you get from get_default_post_to_edit(). I know thisis a rare condition.

#6 @ryan
12 years ago

  • Milestone changed from 3.4 to Future Release

#7 @markoheijnen
11 years ago

#22220 was marked as a duplicate.

#8 @chriscct7
9 years ago

  • Keywords needs-refresh added

@leewillis77
8 years ago

Patch

#9 @leewillis77
8 years ago

  • Keywords has-unit-tests added; needs-refresh removed

Hi;

I just hit this bug in a project. The issue is that if you call wp_insert_post() with an existing post ID, and do not supply post_category then any categories assigned to the post will be removed and replaced with the default category.

Patch with unit tests attached.

#10 @leewillis77
8 years ago

  • Summary changed from wp_insert_post should look if default category is nessesary to wp_insert_post overwrites categories when updating existing post

#11 @joehoyle
8 years ago

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

This ticket was mentioned in PR #3057 on WordPress/wordpress-develop by desrosj.


20 months ago
#12

Trac ticket:

#13 @desrosj
20 months ago

  • Keywords needs-testing added
  • Milestone set to 6.1
  • Owner joehoyle deleted

Looks like this still exists today.

Since it has not had any activity in 6+ years, I'm resetting the Owned by field. If anyone that has participated in the past would like to reclaim it, please feel free.

I've refreshed 19954.diff to apply to the current trunk in a pull request and adjusted the approach slightly. The coverage of this area in wp_insert_post() is actually quite good. With the added tests, I think this is a pretty safe change to make, but would still like some additional testing and sanity checking.

Last edited 20 months ago by desrosj (previous) (diff)

#14 @johnbillion
20 months ago

  • Keywords commit added; needs-testing removed

#15 @johnbillion
20 months ago

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

In 53883:

Posts, Post Types: Prevent categories from being overwritten when updating a post using wp_insert_post().

This prevents existing category relationships being overridden with the default category when none is provided in the post data.

Props markoheijnen, leewillis77, desrosj

Fixes #19954

Note: See TracTickets for help on using tickets.