Block for quick backswitches use user_load in cycle just tot get uid and name.
It's bad practice and have bad perfomance impact.

I propose use single SQL query to fetch this data at once.

Files: 
CommentFileSizeAuthor
#1 user-load-in-cycle-1214410-1.patch1.88 KBIRuslan

Comments

StatusFileSize
new1.88 KB

While working on another module, I noticed that Masquerade does this a lot. Is there a reason? If not, I'll happily provide a patch to do this module-wide (or as much as possible).

This is a really annoying but required to allow modules a-la realname to work - override a username

Status:Active» Needs review

Suppose we needs more reviews