From aadb30fd9cb9401d9f1ed9e82dbc02899851d11d Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Wed, 30 Sep 2009 08:59:50 +0000 Subject: Bug 511796: Allow groups to bless themselves Patch by Max Kanat-Alexander r=LpSolit, a=LpSolit --- editgroups.cgi | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'editgroups.cgi') diff --git a/editgroups.cgi b/editgroups.cgi index 0554638fa..e8d8cfe24 100755 --- a/editgroups.cgi +++ b/editgroups.cgi @@ -113,16 +113,19 @@ sub get_current_and_available { if !grep($_->id == $group_option->id, @visible_to_me_current); } - # The group itself should never show up in the bless or - # membership lists. + push(@bless_from_available, $group_option) + if !grep($_->id == $group_option->id, @bless_from_current); + + # The group itself should never show up in the membership lists, + # and should show up in only one of the bless lists (otherwise + # you can try to allow it to bless itself twice, leading to a + # database unique constraint error). next if $group_option->id == $group->id; push(@members_available, $group_option) if !grep($_->id == $group_option->id, @members_current); push(@member_of_available, $group_option) if !grep($_->id == $group_option->id, @member_of_current); - push(@bless_from_available, $group_option) - if !grep($_->id == $group_option->id, @bless_from_current); push(@bless_to_available, $group_option) if !grep($_->id == $group_option->id, @bless_to_current); } -- cgit v1.2.3-24-g4f1b