Groups has voting on it now, which begins some more qualitative metrics that the site can collect and that can be exposed on user profiles to get a sense of a user's interaction with groups.

What can and should we expose on user profiles on Groups?

Comments

coltrane’s picture

Status: Active » Needs review
StatusFileSize
new2.43 KB

Here's a patch which exposes even information and some of that voting data.

greggles’s picture

Status: Needs review » Fixed

Works great for me.

Thanks, Ben!

http://drupal.org/node/1063992

avpaderno’s picture

+    $groups = array();
+    if (!empty($groups)) {
+      $account->content['summary']['group_manager'] = array(
+        '#type' => 'item',
+        '#title' => t('Manager of groups'),
+        '#value' => 'sdfsdfd',
+      );
+    }

The $groups variable is set to an empty array; the successive if control statement will never be executed, then.

christefano’s picture

That's pretty cool.

It would be interesting to see "number of upvotes" and "number of downvotes" rather than "number of times voted", which counts both upvotes and downvotes.

coltrane’s picture

Status: Fixed » Needs work

@kiamlaluno definitely, I meant to remove that or finish before it get committed. Putting this back to work.

greggles’s picture

@christefano - I mostly like that idea, but I'm hesitant. I'm currently 80% of the way through Building Web Reputation Systems and before we expose too much I want to see what it has to say. We need to be careful about how much detail we expose and number of votes is just surface enough that it feels OK to me for now :)

coltrane’s picture

Status: Needs work » Needs review
StatusFileSize
new1.24 KB

And here is a count of the groups the user is an admin of. That's the same as a manager, right?

christefano’s picture

Status: Needs review » Needs work

My understanding is that groups.drupal.org no longer has "managers" or "admins" and lists "group organizers" instead.

coltrane’s picture

Status: Needs work » Needs review
StatusFileSize
new1.84 KB

I was calling db_rewrite_sql() on the voting table query which I think is unnecessary.

coltrane’s picture

Status: Needs review » Needs work

oh, cross-post, good point christefano! But is it right that the internal is_admin defines organizers?

greggles’s picture

Coltrane, yes that is the correct place to query.

One was the node author, the other is the "admin" from og's perspective. We now only expose "admin from og's perspective" to make the group of managers appear more equal.

coltrane’s picture

Status: Needs work » Needs review
StatusFileSize
new1.82 KB

Great, here that is fixed.

greggles’s picture

Status: Needs review » Needs work

Thanks coltrane - now committed - http://drupal.org/cvs?commit=501364

Marked "needs work" for any more work, otherwise...send this to fixed.

greggles’s picture

Status: Needs work » Fixed

Good for now - thanks again.

Status: Fixed » Closed (fixed)

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