Make WordPress Core

Opened 10 years ago

Last modified 4 years ago

#23168 new enhancement

Introduce remove_post_status

Reported by: kovshenin's profile kovshenin Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: needs-patch
Focuses: Cc:

Description

Plugins and themes should be able to remove the default post statuses defined by core.

Change History (19)

#1 @kovshenin
10 years ago

  • Type changed from defect (bug) to enhancement

#2 follow-up: @sirzooro
10 years ago

IMHO this should go to #12706.

#3 in reply to: ↑ 2 ; follow-up: @kovshenin
10 years ago

Replying to sirzooro: Yeah, just trying to break that ticket down into smaller pieces, that's all ;)

#4 @sabreuse
10 years ago

  • Cc sabreuse added

#5 in reply to: ↑ 3 @sirzooro
10 years ago

  • Cc sirzooro added

Replying to kovshenin:

Replying to sirzooro: Yeah, just trying to break that ticket down into smaller pieces, that's all ;)

OK, I see :)

#6 @danielbachhuber
10 years ago

There are probably some core statuses (e.g. 'Trash') we'll need to protect from removal to avoid breaking all the things.

#7 @SergeyBiryukov
10 years ago

Perhaps unregister_post_status() would be a better name, for consistency with unregister_taxonomy() (suggested in #11058).

#8 @nacin
10 years ago

Removing core post statuses, for core post types, is likely going to be very prone to breakage, which is why we have yet to introduce an unregistration for taxonomies or post types.

A better way to approach this is to segment post statuses against individual post types. Really, you probably shouldn't be able to remove a post status at all, only remove it for individual post types. Should post statuses even be shared against a post type? That can cause problems when you might want certain behavior for "active" on one type but different behavior for another type. Worse when two plugins both want an "active" status for their own post types — as they should be allowed to do.

#9 @kovshenin
10 years ago

I also think you shouldn't be able to completely remove post statuses, types and taxonomies, but rather unregister them from object types, which I think would cover most of the use cases. So perhaps we should add unregister_post_status_for_object_type instead.

unregister_taxonomy_for_object_type was also discussed in #11058.

#10 @greenshady
10 years ago

  • Cc justin@… added

#12 @adamsilverstein
10 years ago

  • Cc ADAMSILVERSTEIN@… added

#13 @sc0ttkclark
10 years ago

  • Cc lol@… added

#14 @aniketpant
10 years ago

  • Cc me@… added

#15 @m_uysl
10 years ago

  • Cc m_uysl@… added

#16 @mordauk
10 years ago

  • Cc pippin@… added

I'll throw my two cents in and agree with @kovshenin and @nacin that statuses should be removed on a per-object basis.

#17 @SergeyBiryukov
10 years ago

  • Milestone changed from 3.6 to Future Release

#18 @nacin
9 years ago

  • Component changed from General to Post Types

#19 @chriscct7
8 years ago

  • Keywords editorial-flow removed
Note: See TracTickets for help on using tickets.