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

CommentFileSizeAuthor
og_db_rewrite_sql_user_lists.patch704 byteshefox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Grayside’s picture

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.

hefox’s picture

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.

Grayside’s picture

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.

Grayside’s picture

Status: Fixed » Closed (fixed)

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