diff --git a/src/postorius/auth/utils.py b/src/postorius/auth/utils.py index fc23c18..fc43e8f 100644 --- a/src/postorius/auth/utils.py +++ b/src/postorius/auth/utils.py @@ -20,6 +20,10 @@ Authentication and authorization-related utilities. """ +from django.utils import six +from postorius.utils import set_other_emails +from postorius.models import List + def user_is_in_list_roster(user, mailing_list, roster): if not user.is_authenticated(): return False @@ -32,7 +36,7 @@ def set_user_access_props(user, mlist): - if not isinstance(mlist, List): + if isinstance(mlist, six.string_types): mlist = List.objects.get_or_404(mlist) if not hasattr(user, 'is_list_owner'): user.is_list_owner = user_is_in_list_roster( diff --git a/src/postorius/views/generic.py b/src/postorius/views/generic.py index 3b18d66..e38bfe6 100644 --- a/src/postorius/views/generic.py +++ b/src/postorius/views/generic.py @@ -52,7 +52,7 @@ self.mailing_list = self._get_list(kwargs['list_id'], int(kwargs.get('page', 1))) utils.set_other_emails(request.user) - set_user_access_props(request.user) + set_user_access_props(request.user, self.mailing_list) # set the template if 'template' in kwargs: self.template = kwargs['template']