[wp-trac] [WordPress Trac] #32907: We should pass the site ID to the 'wpmu_update_blog_options' action

WordPress Trac noreply at wordpress.org
Fri Aug 21 16:18:44 UTC 2015


#32907: We should pass the site ID to the 'wpmu_update_blog_options' action
-------------------------------------------------+-------------------------
 Reporter:  DrewAPicture                         |       Owner:  swissspidy
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  Future
Component:  Options, Meta APIs                   |  Release
 Severity:  normal                               |     Version:  3.0
 Keywords:  good-first-bug has-patch commit      |  Resolution:
  4.4-early                                      |     Focuses:  docs,
                                                 |  multisite
-------------------------------------------------+-------------------------
Changes (by swissspidy):

 * focuses:  multisite => docs, multisite


Comment:

 Replying to [comment:8 jeremyfelt]:
 > I don't have strong feelings, though it would be nice to find another
 example of a hook in core where an ID was passed that was readily
 available through something like `get_current_blog_id()`.

 In the same file `$id` is passed to the `wpmueditblogaction` action. Since
 `$id` is already available, I don't really see a benefit of using
 `get_current_blog_id()` instead. But there's a patch for either scenario.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32907#comment:10>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list