For non-super-admins, use get_blogs_of_user() to determine authroized sites.

[peterwilsoncc]

Description of the Change

Modifies the query used to determine a user's authorized sites to use get_blogs_of_user().

As super admins have access to all sites without being a member, the existing get_sites() query is used for super admins.

Closes #1176

How to test the Change

1. Install Distributor on a Multisite Install in sub-directory mode
2. Make it big export XDEBUG_MODE=off; for i in {1..1050}; do wp site create --slug=$i --title="MS $i" --url=https://ms-distributor.local; done; (change the URL if needs be)
3. Create a non super-admin user
4. Add the new user as an administrator on sites 2, 3 and 1050
5. Log in/switch to the new user
6. Visit the pull screen of either site 2 or 3.
7. The last site should be displayed (1050)

Creating the sites may take some time so you should leave the command going and switch tasks for a while.

Changelog Entry

Fixed - Improved database query for getting a users authorized sites on very large networks.

Credits

Props @jeffpaul, @boonebgorges, @peterwilsoncc.

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.

[kirtangajjar]

@peterwilsoncc Tested and ensured that the PR is working for me.

[kirtangajjar]

@peterwilsoncc It seems that the 1000 site limit problem will still be there for super admins, so why not query for number of sites like this and store it in transients instead of passing a static number:

SELECT COUNT(blog_id) FROM {$wpdb->blogs} WHERE archived = '0' AND spam = '0' AND deleted = '0'

[peterwilsoncc]

Thanks for looking at this @kirtangajjar.

The limit of 1000 sites was added in 3077cf9. Based on the commit message it was a coding standards fix to avoid a warning for an unlimited query.

I'd be surprised if many users were hitting the 1000 site limit but do you think it's worth adding a filter to allow developers to increase the size of the dropdown?

[jeffpaul]

Fwiw filtering for that seems like a good handling to me.

[vikrampm1]

@Sidsector9 checking if you can help with the review here?

[dkotter]

Since the original issue mentions a site that has more than 1000 sites on the network, I do think we should add a filter around this so people can increase this limit if needed