[wp-trac] [WordPress Trac] #25607: Hook Docs: /wp-admin/includes/class-wp-ms-sites-list-table.php

[WordPress Trac]

#25607: Hook Docs: /wp-admin/includes/class-wp-ms-sites-list-table.php
--------------------------+------------------------
 Reporter:  ShinichiN     |       Owner:  kpdesign
     Type:  defect (bug)  |      Status:  reviewing
 Priority:  normal        |   Milestone:  3.8
Component:  Inline Docs   |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |
--------------------------+------------------------
Changes (by DrewAPicture):

 * keywords:  has-patch => needs-patch


Comment:

 `wpmu_blogs_columns` filter
 * This is specific to the Multisite Network Admin, correct?

 `manage_sites_action_links` filter
 * We should use a different doc-specific variable for `$blog['blog_id']`.
 I suggest `$blog_id`
 * For the `$blogname` parameter, we don't need to dance around the types
 of installations ... there's only two. So maybe, "Site path, depending on
 whether it's a subdomain or subdirectory install"

 `wpmublogsaction` action:
 * "Fires in the plugin column to display plugin links and information." --
 remove the second "plugin"
 * Use `$blog_id` instead of `$blog['blog_id']` for the parameter variable

 `manage_sites_custom_column` action:
 * "Fires for each registered custom column in the sites list table."
 * Use `$blog_id` instead of `$blog['blog_id']` as the parameter variable

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25607#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software