CVS edit link for Ki

I have developed OG Hide Membership (og_hide_membership) module that works with Organic Group module. There has been a case at a Drupal community site built on OG module, where group membership itself was a sensitive information, for example, such as groups for gays and lesbians.

It allows group members to hide their membership from the group roster. While group roster is visible to members only, some groups allow membership join without moderation. This module will encourage users to join groups where they are hesitant to reveal their membership status.

Thank you.

CommentFileSizeAuthor
#3 og_hide_membership.zip1.16 KBki
#1 og_hide_membership.zip1.23 KBki

Comments

ki’s picture

ki commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new og_hide_membership.zip1.23 KB
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Needs work
Issue tags: +Drupal 6.x, +Module review 
    db_query("DELETE FROM {cache_page} WHERE cid = '%s'", "http://{$_SERVER[HTTP_HOST]}/user/{$user->uid}");

To generate the absolute URL, you should used url("user/{$user->uid}", array('absolute' => TRUE)); I am not then sure it is really required, because

'cache_page': This table stores generated pages for anonymous users. This is the only table affected by the page cache setting on the administrator pane

The rest is fine.

ki’s picture

ki commented
Status: Needs work » Needs review
StatusFileSize
new og_hide_membership.zip1.16 KB

On my site, I modified user profile to show all the groups that users belong to and it also shows to anonymous users with cached version. So when users hide their membership I wanted to clear the cache for the user profile to take immediate action. I think this may not be the case in general, so I have deleted the line. Attached is the modified version.

Thanks.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)
Issue tags: -Drupal 6.x, -Module review

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

Issue tags: +Drupal 6.x, +Module review

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Component: Miscellaneous » new project application
Assigned: Unassigned » avpaderno
Issue summary: View changes
Status: Closed (fixed) » Fixed
Issue tags: -Drupal 6.x

I am giving credits to the users who participated in this issue.

Status: Fixed » Closed (fixed)

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