Add standard ID value (for user id's) to output of get_users_of_blog() to be compatible
|Reported by:||jeremyclarke||Owned by:||jeremyclarke|
Currently the get_users_of_blog() function (which fetches all users of a given blog for WPMU but is also the only API way to get all the users at once on a normal wp install) returns an array of objects containing the most generally relevant information about users, but not the full contents of their row in the users and user_meta tables (it gets id, login, display name, email and permissions).
Unfortunately and a bit insanely, when fetching the user id it gets the user_id field from the user meta table rather than the plain ID from the users table. Pretty much everywhere in wp the ID label is used for checking IDs, and so the output of get_users_of_blog is pretty much useless except if you are writing custom functions (probably what it was designed for).
This patch does just one little thing, in the SQL inside get_users_of_blog() it fetches both the user_id from user meta table AND the ID from the users table. This way the output can be used with other user API functions when appropriate and it is backwards compatible with whatever people were already doign with the user_id field.
Admittedly it returns redundant information but IMHO its so worth it because it makes the function much more usefull and less confusing/insane.
[NOTE: This function should be replaced with a more general get_users() function that accepts get_posts/get_terms style arguments like blog=1 or role=admin. But this will make it more useful in the meantime]
I can't think of a reason not to commit this. Thanks,