WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 13 years ago

#21046 closed defect (bug) (wontfix)

Dashboard "Right Now" widget hook should be right_now_end, not rightnow_end

Reported by: schreifels Owned by:
Priority: normal Milestone:
Component: Administration Version: 3.4
Severity: minor Keywords: has-patch commit
Cc: Focuses:

Description

In /wp-admin/includes/dashboard.php , the following callback is used:

do_action( 'rightnow_end' );

However, ​the documentation and logic (for consistency with right_now_content_table_end, right_now_table_end, etc) would tell us that the hook should actually be called right_now_end.

Attachments (1)

21046.patch​ (479 bytes ) - added by SergeyBiryukov 13 years ago.

Download all attachments as: .zip

Change History (5)

#1 @Ipstenu
13 years ago

  • Keywords needs-patch added

@SergeyBiryukov
13 years ago

#2 @SergeyBiryukov
13 years ago

  • Component Widgets → Administration
  • Keywords has-patch added; needs-patch removed
  • Milestone Awaiting Review → 3.7

21046.patch​ adds the new action for consistency and keeps the current one for backwards compatibility.

Last edited 13 years ago by SergeyBiryukov (previous) (diff)

#3 @SergeyBiryukov
13 years ago

  • Keywords commit added

#4 @nacin
13 years ago

  • Milestone 3.7
  • Resolution → wontfix
  • Status new → closed

I don't think this matters much. Inline documentation for hooks will handle nagging issues like whether hooks are discoverable. Also, if we're going to possibly do a redesigned Right Now at some point in the next few releases, this is little more than cruft.

Note: See TracTickets for help on using tickets.