WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25752 closed defect (bug) (fixed)

Hook Docs (25): wp-admin/includes/dashboard.php

Reported by: mauryaratan's profile mauryaratan Owned by: kpdesign's profile kpdesign
Milestone: 3.9 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch commit
Focuses: docs, multisite Cc:

Description

Attached patch has inline docs for:

  1. Actions:
    • wp_network_dashboard_setup
    • wp_user_dashboard_setup
    • wp_dashboard_setup
    • right_now_content_table_end
    • right_now_table_end
    • right_now_discussion_table_end
    • rightnow_end
    • activity_box_end
    • wpmuadminresult
    • mu_rightnow_end
    • mu_activity_box_end
    • media_buttons
  1. Filters:
    • dashboard_incoming_links_link
    • dashboard_incoming_links_feed
    • dashboard_primary_link
    • dashboard_primary_feed
    • dashboard_primary_title
    • dashboard_secondary_link
    • dashboard_secondary_feed
    • dashboard_secondary_title
    • wp_network_dashboard_widgets
    • wp_user_dashboard_widgets
    • wp_dashboard_widgets
    • privacy_on_link_title
    • privacy_on_link_text
    • comment_row_actions
    • browse-happy-notice

Attachments (3)

25752.diff​ (6.3 KB) - added by mauryaratan 11 years ago.
Updated the docs
25752.2.diff​ (8.9 KB) - added by DrewAPicture 11 years ago.
2nd pass
25752.3.diff​ (9.5 KB) - added by kpdesign 11 years ago.
Third pass

Download all attachments as: .zip

Change History (11)

#1 @DrewAPicture
11 years ago

  • Milestone changed from Awaiting Review to 3.8
  • Owner set to DrewAPicture
  • Status changed from new to reviewing

#2 @DrewAPicture
11 years ago

  • Milestone changed from 3.8 to Awaiting Review

#3 @DrewAPicture
11 years ago

  • Keywords needs-refresh added
  • Summary changed from Hook Docs: wp-admin/includes/dashboard.php to Hook Docs (25): wp-admin/includes/dashboard.php

Probably going to need a refresh here due to the DASH changes in 3.8. Also, two of these hooks are gone and there's at least one new one in 3.8.

@mauryaratan
11 years ago

Updated the docs

#4 @kpdesign
11 years ago

  • Keywords needs-refresh removed

#5 @jeremyfelt
11 years ago

  • Component changed from Inline Docs to Administration
  • Focuses docs added

@DrewAPicture
11 years ago

2nd pass

#6 @DrewAPicture
11 years ago

  • Focuses multisite added
  • Milestone changed from Awaiting Review to 3.9
  • Owner changed from DrewAPicture to kpdesign

25752.2.diff​ should do it. Needs a second look and recommendation.

@kpdesign
11 years ago

Third pass

#7 @kpdesign
11 years ago

  • Keywords commit added

25752.3.diff​ contains minor language and spacing changes.

This one's good to go in - recommend commit.

#8 @DrewAPicture
11 years ago

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

In 27669:

Inline documentation for hooks in wp-admin/includes/dashboard.php.

Props mauryaratan. Props kpdesign for some minor cleanup.
Fixes #25752.

Note: See TracTickets for help on using tickets.