From fa8b40fb3fa8c60edaa624ac8e52e1438bdf5123 Mon Sep 17 00:00:00 2001 From: Allen Lee Date: Thu, 19 Mar 2015 11:53:24 -0700 Subject: [PATCH] minor hygiene --- vcweb/core/backends.py | 2 +- vcweb/core/models.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/vcweb/core/backends.py b/vcweb/core/backends.py index 28024710..467dc84a 100644 --- a/vcweb/core/backends.py +++ b/vcweb/core/backends.py @@ -15,7 +15,7 @@ class EmailAuthenticationBackend(ModelBackend): adapted from http://djangosnippets.org/snippets/74/ and http://scottbarnham.com/blog/2008/08/21/extending-the-django-user-model-with-inheritance/ - FIXME: should we check for and handle Participant experiment auth codes separately from actual login? + FIXME: check for and handle Participant experiment auth codes separately from actual login """ def authenticate(self, username=None, password=None, **kwargs): diff --git a/vcweb/core/models.py b/vcweb/core/models.py index 3e791116..dd0f4615 100644 --- a/vcweb/core/models.py +++ b/vcweb/core/models.py @@ -2246,7 +2246,7 @@ def group_ids(self): return self.group_relationship_set.select_related('group').values_list('group', flat=True) def get_groups(self): - return Group.objects.filter(pk__in=self.groups) + return Group.objects.filter(pk__in=self.group_ids) def add(self, group): return GroupRelationship.objects.create(cluster=self, group=group)