Changeset 7442 for trunk/bp-groups/bp-groups-screens.php

Timestamp:

10/17/2013 07:00:46 PM (11 years ago)

Author:

boonebgorges

Message:

Improve UX flow around overlapping group invites and requests

This changeset introduces a number of improvements to the flow surrounding
invitations and membership requests to private groups:

• When a user has previously requested membership in a group, add a note to that effect when the user is being invited to the group via the Send Invites interface.
• When a user has previously requested membership in a group, automatically accept that invitation and add the user to the group when an invitation is sent by an existing group member.
• When a user has previously been invited to a group, buttons in the group directory and in the group header should read Accept Invitation instead of Request Membership

The end result is a more logical user experience with regard to joining private
groups.

Fixes #5032

Props Mike_Cowobo

File:

• trunk/bp-groups/bp-groups-screens.php (modified) (1 diff)

trunk/bp-groups/bp-groups-screens.php

@@ -479 +479 @@
     if ( 'private' != $bp->groups->current_group->status )
         return false;
+
+    // If the user is already invited, accept invitation
+    if ( groups_check_user_has_invite( bp_loggedin_user_id(), $bp->groups->current_group->id ) ) {
+        if ( groups_accept_invite( bp_loggedin_user_id(), $bp->groups->current_group->id ) )
+            bp_core_add_message( __( 'Group invite accepted', 'buddypress' ) );
+        else
+            bp_core_add_message( __( 'There was an error accepting the group invitation, please try again.', 'buddypress' ), 'error' );
+        bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
+    }
 
     // If the user has submitted a request, send it.