Opened 8 years ago
Closed 8 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
@
8 years ago
- Keywords has-patch added; needs-patch removed
- Owner set to flixos90
- Status changed from new to assigned
#3
@
8 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.
8 years ago
Note: See
TracTickets for help on using
tickets.
40703.diff sets the
$accepted_args
parameter of theadd_filter()
calls to 0, so that no parameters are passed to the functions.