Should be allowed to disable "Switch user" list in configuration

That list comes from Devel module. It's the same list as in Devel module's Switch user block.

I can't see where devel_switch_user_list() is called from admin_menu. The only reference to this function is within devel.module itself, for the Switch User block. I'd like to disable this, as the query takes one second on our site, and we have many users with switch user permission.

I'd be happy to submit a patch if you can tell me who calls devel_switch_user_list().

Search for "switch_user_list" only in order to find it.

However, admin_menu 1.x won't get any further, non-critical features and improvements, since 3.x has superseded a long time ago already.

I'm reclassifying this issue as a bug on the 3.x branch. A one second delay on every page load is unacceptable in my book (against 200,000 users, but lots of site have this many users or more), so the solution is either to remove the list, allow disablement in config, or improve query performance.

I'd be happy to provide a patch -- let me know your thoughts on best approach.

Thanks for taking the time to report this issue.

However, marking as duplicate of #293768: Allow to enable/disable menu additions. You can follow up on that issue to track its status instead. If any information from this issue is missing in the other issue, please make sure you provide it over there.

A couple years late probably, but I've successfully reduced the >1 second delay by simply changing the LEFT JOIN to an INNER JOIN on the query that generates the list of users one can switch to.

#2624370: Increase query performance of devel_switch_user_list