[wp-trac] [WordPress Trac] #36205: PHPDoc correction for wp-admin/includes/class-wp-upgrader.php

WordPress Trac noreply at wordpress.org
Fri Mar 11 08:13:57 UTC 2016


#36205: PHPDoc correction for wp-admin/includes/class-wp-upgrader.php
-----------------------------+----------------------
 Reporter:  JeffMatson       |       Owner:
     Type:  defect (bug)     |      Status:  closed
 Priority:  normal           |   Milestone:
Component:  Upgrade/Install  |     Version:
 Severity:  normal           |  Resolution:  invalid
 Keywords:  has-patch        |     Focuses:  docs
-----------------------------+----------------------
Changes (by swissspidy):

 * status:  new => closed
 * resolution:   => invalid
 * milestone:  Awaiting Review =>


Comment:

 Hey there, thanks for taking your time to create this ticket!

 It may be confusing, but `$packages` isn't always a list of plugins.
 Depending on the context (i.e. the `$this` parameter), it might be a list
 of theme or core packages to update. `Plugin_Upgrader` isn't the only
 upgrader class, there's also `Theme_Upgrader` and `Core_Upgrade`.

 I think the current DocBlock makes that clear, so I'm closing this ticket.

 If you feel the documentation should be improved to better highlight the
 different contexts, feel free to reopen this ticket.

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


More information about the wp-trac mailing list