Skip to:
Content

Opened 9 years ago

Closed 9 years ago

#1257 closed enhancement (fixed)

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

Reported by: madloki Owned by: 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 9 years ago.
V1 of the patch.
ticket-1257.jpg (12.6 KB) - added by Jason_JM 9 years ago.
Preview of patch.

Download all attachments as: .zip

Change History (8)

#1 @Jason_JM
9 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
9 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
9 years ago

Preview of patch.

#3 @Jason_JM
9 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
9 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
9 years ago

  • Keywords reporter-feedback added

#6 @johnjamesjacoby
9 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.