Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#5077 closed enhancement (fixed)

Leaving "softly" an hidden group

Reported by: imath's profile imath Owned by: r-a-y's profile r-a-y
Milestone: 1.9 Priority: normal
Severity: minor Version: 1.8
Component: Groups Keywords: has-patch
Cc:

Description

1/ When you leave an hidden group from the link of the group header or from the siteurl/members/username/groups list of groups leads you to a 404 and the confirmation message is not showed.

2/ When you leave the hidden group from the groups directory 'my groups' scope, once the ajax request is done it leaves the container li, so if you click on the group link, then you have a 404.

I suggest the diff attached to this ticket to :
1/ redirect to siteurl/members/username/groups if the group status is hidden
2/ slide up the container li if the group status is hidden.

Attachments (1)

5077.diff (1.9 KB) - added by imath 11 years ago.

Download all attachments as: .zip

Change History (5)

@imath
11 years ago

#1 @boonebgorges
11 years ago

  • Milestone changed from Awaiting Review to 1.9

Good find, imath. Let's do this in 1.9.

#2 @r-a-y
11 years ago

  • Owner set to r-a-y
  • Resolution set to fixed
  • Status changed from new to closed

In 7384:

Fix redirect when a user leaves a hidden group from the group's page.

Usually when a user leaves a group from the group's page, it redirects
back to the group's homepage, however for hidden groups this creates
a 404.

This commit redirects the user back to their list of groups when
leaving a hidden group.

Commit also adds phpDoc and uses some API functions instead of the $bp
global.

Fixes #5077. Props imath.

#3 @r-a-y
11 years ago

In 7385:

Tweak JS when joining or leaving a group from the groups directory.

This commit does the following:

  • When leaving a hidden group, it removes the hidden group from the group loop.
  • Increases or decreases the "My Groups" count if you join or leave a group.

Props imath for the initial patch.

See #5077.

#4 @johnjamesjacoby
10 years ago

  • Version changed from 1.8-beta to 1.8
Note: See TracTickets for help on using tickets.