Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#26124 closed defect (bug) (fixed)

Hook Docs (4): wp-admin/network/sites.php

Reported by: nikv's profile NikV Owned by: drewapicture's profile DrewAPicture
Milestone: 3.9 Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: has-patch commit
Focuses: docs, administration, multisite Cc:

Description

Attached patch include inline docs for wp-admin/network/sites.php.

Attachments (3)

sites.patch (1.2 KB) - added by NikV 11 years ago.
26124.patch (1.2 KB) - added by kpdesign 11 years ago.
Second pass
26124.2.patch (1.3 KB) - added by DrewAPicture 11 years ago.
Final pass.

Download all attachments as: .zip

Change History (9)

@NikV
11 years ago

#1 @DrewAPicture
11 years ago

  • Owner set to DrewAPicture
  • Status changed from new to reviewing
  • Summary changed from Hook Docs: wp-admin/network/sites.php to Hook Docs (4): wp-admin/network/sites.php

sites.patch looks like a pretty solid start. This is a pretty small file and we should be able to get this one wrapped up today during the sprint.

#2 @jeremyfelt
11 years ago

  • Component changed from Inline Docs to Network Admin
  • Focuses docs added

@kpdesign
11 years ago

Second pass

#3 @kpdesign
11 years ago

26124.patch contains changes to descriptions, and adds @since values for 2 hooks.

#4 @nacin
11 years ago

  • Component changed from Network Admin to Networks and Sites
  • Focuses administration multisite added

@DrewAPicture
11 years ago

Final pass.

#5 @DrewAPicture
11 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.9
  • Status changed from reviewing to accepted

26124.2.patch adds a long description for the network_sites_updated_message_$_GET['updated'] dynamic filter and corrects its @since version. Looks good to me.

#6 @DrewAPicture
11 years ago

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

In 27088:

Inline documentation for hooks in wp-admin/network/sites.php.

Props NikV, kpdesign.
Fixes #26124.

Note: See TracTickets for help on using tickets.