Opened 2 years ago

Closed 2 years ago

#15599 closed defect (bug) (fixed)

Update check on multisite should use get_user_count()

Reported by: nacin Owned by:
Priority: normal Milestone: 3.1
Component: Multisite Version:
Severity: normal Keywords: has-patch
Cc:

Description

count_users() is intensive, and it's also not the right query, because it only counts the current blog. We should leverage get_user_count() when running multisite in wp_version_check().

Attachments (1)

15599.diff​ (713 bytes) - added by PeteMall 2 years ago.

Download all attachments as: .zip

Change History (4)

PeteMall — 2 years ago

comment:1   PeteMall — 2 years ago

  • Keywords has-patch added

comment:2   ryan — 2 years ago

Looks good.

comment:3   nacin — 2 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [16604]) Use get_user_count() if multisite in wp_version_check. props PeteMall, fixes #15599.

Note: See TracTickets for help on using tickets.