+jgroup/member won't remove someone who no longer has access to the group or the system

[GoogleCodeExporter]

aJobs 6.1, MUX 2.9.1.7 #6 [2010-JUL-11]

+jgroup/member

Adding a staffer to a jgroup then removing the staffbit gives an error when 
trying to remove them from the jgroup.  (e.g., "That player does not have +jobs 
access.")

I've put in a temporary fix by putting the check for removing before other 
validation checks (*1, <remove code>):

CMD_JGROUP/MEMBER: $+jgroup/member *=*:@switch/first 
[u(%va/FN_WIZONLY,%#)][setq(0,switch(trim(%0),me,%#,pmatch(trim(%0))))][setq(1,u
(%va/FN_FIND-JGROUP,%1))][isdbref(%q0)][isdbref(%q1)][u(%va/HAS_ACCESS,%q0)][str
match(get(%q1/ISMEMBER),*MEMBERLIST*)][u(%q1/ISMEMBER,%q0)]=0*,{@pemit 
%#=Permission denied.},10*,{@pemit %#=There is no player by that 
name.},110*,{@pemit %#=There is no jgroup by that name.},*1, {@pemit %#=You 
have removed [name(%q0)] from membership in the [name(%q1)] jgroup.;&memberlist 
%q1=remove(get(%q1/memberlist),%q0);}, 1110*,{@pemit %#=That player does not 
have +jobs access.},11110*,{@pemit %#=Membership in that jgroup is unaffected 
by this command.},111110*,{@pemit %#=You have added [name(%q0)] to membership 
in the [name(%q1)] jgroup.;&memberlist 
%q1=setunion(get(%q1/memberlist),%q0);},{@pemit %#=Unknown +jgroup/member 
error.}

Original issue reported on code.google.com by thenomain@gmail.com on 23 Sep 2011 at 8:47

[GoogleCodeExporter]

Fixed in r409.  

Original comment by widdis@gmail.com on 25 Sep 2011 at 2:38

• Changed state: Fixed
• Added labels: Priority-Medium, Milestone-Release6.5, Plat-All, Usability

[GoogleCodeExporter]

Issue 161 has been merged into this issue.

Original comment by widdis@gmail.com on 22 Nov 2011 at 6:44