Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

#1257 closed enhancement (fixed)

Group admin can set his status to user + leave group bug [has patch]

Reported by: madloki's profile madloki Owned by: jason_jm's profile Jason_JM
Milestone: 1.2 Priority: minor
Severity: Version:
Component: Keywords: Group, Leave, Error Message, has-patch, tested, reporter-feedback
Cc: Jason_JM

Description

When the group has only 1 admin he can set his status to "user". An group without admin. Than he can leave the group, what is when the group has only one member...?

Attachments (2)

(JASON_JM)(BP)ajax.js_ticket-1257(v1).patch (863 bytes) - added by Jason_JM 13 years ago.
V1 of the patch.
ticket-1257.jpg (12.6 KB) - added by Jason_JM 13 years ago.
Preview of patch.

Download all attachments as: .zip

Change History (8)

#1 @Jason_JM
13 years ago

  • Cc Jason_JM added
  • Owner set to Jason_JM
  • Status changed from new to assigned

Scenario 1: Pass

  1. When the user is the admin
  2. Is the only member of that group
  3. Tries to leave the group

User will get an error while trying to leave the group.

Albeit the error is not properly formated :-/ I am patching this now.

The message may occur in multiple places:
1.) Group Directory. Here the message may not be properly formated, but indeed the message will display alerting the user the app prevented the admin/only member to leave.

2.) Group Pages. Here a proper error message will appear in the template message area. "As the only group administrator, you cannot leave this group.".

#2 @Jason_JM
13 years ago

  • Keywords Group Leave Error Message added

To Re-iterate:

I see no errors, as the app prevents the only group admin from leaving.

However, I've gone and added proper formating to the error messages where applicable.

Patch and image will be added.

@Jason_JM
13 years ago

Preview of patch.

#3 @Jason_JM
13 years ago

  • Keywords has-patch tested added
  • Summary changed from Group admin can set his status to user + leave group bug to Group admin can set his status to user + leave group bug [has patch]

#4 @Jason_JM
13 years ago

  • Priority changed from critical to minor
  • Type changed from defect to enhancement

Flagging as minor unless the user can supply additional information about the scenario(s) he/she is experiencing in detail.

#5 @DJPaul
13 years ago

  • Keywords reporter-feedback added

#6 @johnjamesjacoby
13 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [2067]) Fixes #1257 (two new strings added, translations will be needed)

Note: See TracTickets for help on using tickets.