Opened 9 years ago
Closed 9 years ago
#40703 closed defect (bug) (fixed)
Adjust the `add_filter()` calls for `wp_maybe_update_*_counts()` in `ms-default-filters.php`
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.8 | Priority: | normal |
| Severity: | normal | Version: | 4.8 |
| Component: | Networks and Sites | Keywords: | has-patch commit |
| Focuses: | multisite | Cc: |
Description
Now that wp_maybe_update_network_site_counts() and wp_maybe_update_network_user_counts() support a $network_id parameter, we should ensure that no parameter is passed to it when hooking it into a filter. For example, the make_*_blog hooks pass a site ID, which is currently passed to the hooked wp_maybe_update_network_site_counts() function, although its only parameter should be a network ID.
Attachments (2)
Change History (7)
#1
@
9 years ago
- Keywords has-patch added; needs-patch removed
- Owner set to flixos90
- Status changed from new to assigned
#3
@
9 years ago
40703.2.diff also fixes the same for the wp_update_network_counts() function on the update_network_counts cron hook.
This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.
9 years ago
Note: See
TracTickets for help on using
tickets.
40703.diff sets the
$accepted_argsparameter of theadd_filter()calls to 0, so that no parameters are passed to the functions.