diff --git a/pandora/item/models.py b/pandora/item/models.py index 04ee7b68..d33ec204 100644 --- a/pandora/item/models.py +++ b/pandora/item/models.py @@ -209,10 +209,13 @@ class Item(models.Model): del data['id'] if 'groups' in data: groups = data.pop('groups') - self.groups.exclude(name__in=groups).delete() - for g in groups: - group, created = Group.objects.get_or_create(name=g) - self.groups.add(group) + if isinstance(groups, list): + groups = filter(lambda g: g.strip(), groups) + self.groups.exclude(name__in=groups).delete() + current_groups = [g.name for g in self.groups.all()] + for g in filter(lambda g: g not in current_groups, groups): + group, created = Group.objects.get_or_create(name=g) + self.groups.add(group) for key in data: self.data[key] = data[key] return self.save() diff --git a/pandora/item/views.py b/pandora/item/views.py index fca883cc..363e0174 100644 --- a/pandora/item/views.py +++ b/pandora/item/views.py @@ -404,6 +404,7 @@ def get(request): if not data['keys'] or 'notes' in data['keys'] \ and request.user.get_profile().capability('canEditMetadata'): info['notes'] = item.notes + info['groups'] = [g.name for g in item.groups.all()] response['data'] = info else: response = json_response(status=403, text='permission denied') diff --git a/pandora/user/models.py b/pandora/user/models.py index 11dcac9f..61fb0274 100644 --- a/pandora/user/models.py +++ b/pandora/user/models.py @@ -3,7 +3,7 @@ import copy from datetime import datetime -from django.contrib.auth.models import User +from django.contrib.auth.models import User, Group from django.db import models from django.db.models import Max from django.conf import settings @@ -149,6 +149,7 @@ class SessionData(models.Model): p = self.user.get_profile() j['disabled'] = not self.user.is_active j['email'] = self.user.email + j['groups'] = [g.name for g in self.user.groups.all()] j['level'] = p.get_level() j['newsletter'] = p.newsletter j['notes'] = p.notes diff --git a/pandora/user/views.py b/pandora/user/views.py index ec49f12b..e5125ee5 100644 --- a/pandora/user/views.py +++ b/pandora/user/views.py @@ -334,6 +334,15 @@ def editUser(request): profile.notes = data['notes'] if 'newsletter' in data: profile.newsletter = data['newsletter'] + if 'groups' in data: + groups = data['groups'] + if isinstance(groups, list): + groups = filter(lambda g: g.strip(), groups) + user.groups.exclude(name__in=groups).delete() + current_groups = [g.name for g in user.groups.all()] + for g in filter(lambda g: g not in current_groups, groups): + group, created = models.Group.objects.get_or_create(name=g) + user.groups.add(group) if 'username' in data: if models.User.objects.filter( username__iexact=data['username']).exclude(id=user.id).count()>0: diff --git a/static/js/pandora/infoView.padma.js b/static/js/pandora/infoView.padma.js index 9beeea5b..c0bc418d 100644 --- a/static/js/pandora/infoView.padma.js +++ b/static/js/pandora/infoView.padma.js @@ -264,7 +264,7 @@ pandora.ui.infoView = function(data) { ) .appendTo($text); - var list_keys = ['language', 'topic', 'director', 'cinematographer', 'features']; + var list_keys = ['language', 'topic', 'director', 'cinematographer', 'features', 'groups']; $('