From 9e6188786871db4cb8ab42bebe5a7a7a886e3bc7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Aur=C3=A9lien=20Delobelle?= <aurelien.delobelle@gmail.com>
Date: Mon, 11 Sep 2017 16:42:14 +0200
Subject: [PATCH] =?UTF-8?q?K-F=C3=AAt'=20groups=20edits=20don't=20remove?=
 =?UTF-8?q?=20non-kfet=20app=20permissions.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fixes #168.
---
 kfet/forms.py | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/kfet/forms.py b/kfet/forms.py
index 2d07581d..6ef3aefb 100644
--- a/kfet/forms.py
+++ b/kfet/forms.py
@@ -167,10 +167,22 @@ class GroupForm(forms.ModelForm):
         name = self.cleaned_data['name']
         return 'K-Fêt %s' % name
 
+    def clean_permissions(self):
+        kfet_perms = self.cleaned_data['permissions']
+        # TODO: With Django >=1.11, the QuerySet method 'difference' can be used.
+        # other_groups = self.instance.permissions.difference(
+        #     self.fields['permissions'].queryset
+        # )
+        other_perms = self.instance.permissions.exclude(
+            pk__in=[p.pk for p in self.fields['permissions'].queryset],
+        )
+        return list(kfet_perms) + list(other_perms)
+
     class Meta:
-        model  = Group
+        model = Group
         fields = ['name', 'permissions']
 
+
 class AccountNegativeForm(forms.ModelForm):
     class Meta:
         model  = AccountNegative