WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 11 years ago

#17891 closed enhancement (fixed)

Pass $post_type to 'restrict_manage_posts'

Reported by: scribu Owned by: DrewAPicture
Priority: normal Milestone: 4.4
Component: Posts, Post Types Version:
Severity: normal Keywords: good-first-bug
Cc: Focuses: administration

Description

It would be nice if the 'restrict_manage_posts' hook received the post type as an argument, rather than having to check the global.

Attachments (3)

17891.diff​ (548 bytes ) - added by scribu 15 years ago.
17891-2.diff​ (548 bytes ) - added by sunnyratilal 13 years ago.
17891-3.patch​ (517 bytes ) - added by pavelevap 11 years ago.

Download all attachments as: .zip

Change History (18)

@scribu
15 years ago

#1 @scribu
15 years ago

  • Severity normal → minor

#2 @Viper007Bond
14 years ago

  • Keywords needs-refresh added

Now that we're not passing around the $post_type global, we should instead pass the whole $this->screen object.

@sunnyratilal
13 years ago

#3 @sunnyratilal
13 years ago

  • Cc sunnyratilal5@… added
  • Keywords needs-refresh removed

Patch refreshed.

#4 @jeremyfelt
13 years ago

  • Component Administration → Posts, Post Types
  • Focuses admin added

#5 @wonderboymusic
11 years ago

  • Milestone Awaiting Review → 4.4

#6 @wonderboymusic
11 years ago

  • Owner set to wonderboymusic
  • Resolution → fixed
  • Status new → closed

In 33644:

The 'restrict_manage_posts' hook currently fires on the Post and Media list tables, but is passed zero arguments. Pass $post_type.

Props sunnyratilal, scribu.
Fixes #17891.

#7 @pavelevap
11 years ago

@since 4.4.0 is missing from hook changelog?

#8 @wonderboymusic
11 years ago

@pavelevap - wanna whip up a patch for some easy props?

#9 follow-up: @chriscct7
11 years ago

  • Keywords needs-docs added; has-patch removed
  • Resolution fixed
  • Severity minor → normal
  • Status closed → reopened

The hook actually needs a full docbloc, see ​https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#1-functions-and-class-methods. Shouldn't be more than a couple lines.

#10 @wonderboymusic
11 years ago

  • Keywords good-first-bug added

#11 in reply to: ↑ 9 @DrewAPicture
11 years ago

Replying to chriscct7:

The hook actually needs a full docbloc

Not sure what you're referring to. The 'restrict_manage_posts' hook is already fully documented in class-wp-posts-list-table. As @pavelevap pointed out, all we really need here is a changelog entry for the new parameter, unless I'm missing something.

#12 @chriscct7
11 years ago

Ah indeed you are correct.

#13 follow-up: @pavelevap
11 years ago

Sure, I will add patch in a while...

I found similar docs, but I am not sure if there is any standard for this sentence?

Added `$post_type` parameter.
The `$post_type` parameter was added.
Introduced the `$post_type` parameter.
...

@pavelevap
11 years ago

#14 in reply to: ↑ 13 @DrewAPicture
11 years ago

  • Keywords needs-docs removed
  • Owner changed from wonderboymusic to DrewAPicture
  • Status reopened → reviewing

Replying to pavelevap:

Sure, I will add patch in a while...

I found similar docs, but I am not sure if there is any standard for this sentence?

Added `$post_type` parameter.
The `$post_type` parameter was added.
Introduced the `$post_type` parameter.
...

Personally, I prefer the second one:

The `$post_type` parameter was added.

I'll fix it on commit. Thanks for the patch!

#15 @DrewAPicture
11 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 33691:

Docs: Add a changelog entry for the $post_type parameter added to the restrict_manage_posts hook in [33644].

Props pavelevap
Fixes #17891.

Note: See TracTickets for help on using tickets.