Patch attached. I believe listing of users (and counting) need a db_rewrite_sql so can alter what is shown, etc.

Files: 
CommentFileSizeAuthor
og_db_rewrite_sql_user_lists.patch704 byteshefox
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch og_db_rewrite_sql_user_lists.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Comments

Is there a best practice around using aliases as table names? Also, in the optional $args element, would it not be good to specific this is for OG? Or is the idea that in a general sense, any adjustment to a listing of users would have the same kinds of tweaks?

Haven't done much with db_sql_rewrite().

Status:Needs review» Needs work

The last submitted patch, og_db_rewrite_sql_user_lists.patch, failed testing.

http://api.drupal.org/api/drupal/modules!node!node.module/function/node_... actually ignores table name so don't think there's any problems for generic use as most things are likely keying in on key.

Haven't seen args used as you mentioned, so don't think it's needed.

Status:Needs work» Needs review

og_db_rewrite_sql_user_lists.patch queued for re-testing.

Status:Needs review» Needs work

The last submitted patch, og_db_rewrite_sql_user_lists.patch, failed testing.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.