#36205 closed defect (bug) (invalid)
PHPDoc correction for wp-admin/includes/class-wp-upgrader.php
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | normal | Version: | |
| Component: | Upgrade/Install | Keywords: | has-patch |
| Focuses: | docs | Cc: |
Description
While looking for a hook, I noticed this correction in the docblocks. Simply changed to reflect the correct array key when using the Plugin_Upgrader child class.
Attachments (1)
Change History (5)
Note: See
TracTickets for help on using
tickets.
Hey there, thanks for taking your time to create this ticket!
It may be confusing, but
$packagesisn't always a list of plugins. Depending on the context (i.e. the$thisparameter), it might be a list of theme or core packages to update.Plugin_Upgraderisn't the only upgrader class, there's alsoTheme_UpgraderandCore_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.