Replace is_super_admin() check with current_user_can( 'manage_network_users' ) in wp-admin/includes/ms.php

[bhargavbhandari90]

This is part of the #37616 task. There are 1 is_super_admin() checks in wp-admin/includes/ms.php that should be replaced with current_user_can( 'manage_network_users' ).

[flixos90]

Thanks for the patch @bhargavbhandari90! It looks good so far, there are only two small adjustments that should be made, which are both related to coding standards:

• insert a space between the ! and current_user_can( 'manage_network_users' )
• add curly braces around the clause (this is an acceptable change since we are adjusting that code anyway)

[bhargavbhandari90]

Replying to flixos90:

Thanks for the patch @bhargavbhandari90! It looks good so far, there are only two small adjustments that should be made, which are both related to coding standards:

• insert a space between the ! and current_user_can( 'manage_network_users' )
• add curly braces around the clause (this is an acceptable change since we are adjusting that code anyway)

Thanks for the update. I have created another patch.

[flixos90]

Great, thanks a lot @bhargavbhandari90 - looks good! :)

[bhargavbhandari90]

Replying to flixos90:

Great, thanks a lot @bhargavbhandari90 - looks good! :)

Thanks. :-)

[flixos90]

In 39945:

Multisite: Replace is_super_admin() with manage_network_users when trying to import new users.

Props bhargavbhandari90.
Fixes #39212. See #37616.