Mercurial Mercurial > platform / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | zip | gz | bz2 | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
correct group test in admin user creation form V01.59.04
authorymh <ymh.work@gmail.com>
Wed, 16 Sep 2015 22:53:40 +0200
changeset 1438 d42d5ceacea5
parent 1437 cfdab0c64e7c
child 1439 e0d8998e24be
correct group test in admin user creation form
src/ldt/ldt/__init__.py file | annotate | diff | comparison | revisions
src/ldt/ldt/user/forms.py file | annotate | diff | comparison | revisions 
--- a/src/ldt/ldt/__init__.py	Wed Sep 16 22:46:15 2015 +0200
+++ b/src/ldt/ldt/__init__.py	Wed Sep 16 22:53:40 2015 +0200
@@ -1,6 +1,6 @@
 __all__ = ["VERSION", "get_version", "__version__", "default_app_config"]
 
-VERSION = (1, 59, 3, "final", 0)
+VERSION = (1, 59, 4, "final", 0)
 
 
 def get_version():
--- a/src/ldt/ldt/user/forms.py	Wed Sep 16 22:46:15 2015 +0200
+++ b/src/ldt/ldt/user/forms.py	Wed Sep 16 22:53:40 2015 +0200
@@ -72,10 +72,11 @@
         
         public_group_name = getattr(settings,'PUBLIC_GROUP_NAME', None)
         
-        if public_group_name and public_group_name not in [g.name for g in user.groups.all()]:
+        if public_group_name:
             def save_m2m():
-                everyone, _ = Group.objects.get_or_create(name=public_group_name)
-                user.groups.add(everyone)
+                if public_group_name not in [g.name for g in user.groups.all()]:
+                    everyone, _ = Group.objects.get_or_create(name=public_group_name)
+                    user.groups.add(everyone)
             if commit:
                 user.save()
                 save_m2m()
platform