From 85d23524c8bf90231fef65139f5f6ef935dad666 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Tue, 10 Nov 2015 10:32:27 +0100 Subject: [PATCH 2/4] api_views: pep8ness (#7862) --- src/authentic2/api_views.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/authentic2/api_views.py b/src/authentic2/api_views.py index b404837..c374807 100644 --- a/src/authentic2/api_views.py +++ b/src/authentic2/api_views.py @@ -14,7 +14,6 @@ from rest_framework import serializers from rest_framework.generics import GenericAPIView from rest_framework.response import Response from rest_framework import authentication, permissions, status -from rest_framework.exceptions import PermissionDenied from . import utils, decorators @@ -59,21 +58,19 @@ class RegistrationSerializer(serializers.Serializer): 'this ou')) User = get_user_model() if data['ou'] and data['ou'].email_is_unique and \ - User.objects.filter(ou=data['ou'], - email__iexact=data['email']).exists(): + User.objects.filter(ou=data['ou'], email__iexact=data['email']).exists(): raise serializers.ValidationError( _('You already have an account')) - if data['ou'] and data['ou'].username_is_unique and not \ - 'username' in data: + if data['ou'] and data['ou'].username_is_unique and 'username' not in data: raise serializers.ValidationError( _('Username is required in this ou')) if data['ou'] and data['ou'].username_is_unique and \ - User.objects.filter(ou=data['ou'], - username=data['username']).exists(): + User.objects.filter(ou=data['ou'], username=data['username']).exists(): raise serializers.ValidationError( _('You already have an account')) return data + class RpcMixin(object): def post(self, request, format=None): serializer = self.get_serializer(data=request.data) @@ -156,6 +153,7 @@ class Register(BaseRpcView): register = Register.as_view() + class PasswordChangeSerializer(serializers.Serializer): '''Register RPC payload''' email = serializers.EmailField() -- 2.1.4