Skip to content
Snippets Groups Projects
Commit cc648630 authored by Raphael Das Gupta's avatar Raphael Das Gupta
Browse files

refactoring: omit default argument for <dict-like>.get(...)

parent 780e5f1b
No related branches found
No related tags found
1 merge request!854Feature/#780 return user to original page after email confirmation
......@@ -34,7 +34,7 @@ class ProfileView(LoginRequiredMixin, generic.UpdateView):
)
if self.is_new_user():
self._move_email_from_user_to_profile(user, profile)
profile.request_email_address_confirmation(self.request, redirection_target=self.request.GET.get('next', None))
profile.request_email_address_confirmation(self.request, redirection_target=self.request.GET.get('next'))
else:
self._ensure_profile_has_email(profile, user)
user.refresh_from_db()
......@@ -51,7 +51,7 @@ class ProfileView(LoginRequiredMixin, generic.UpdateView):
if isinstance(response, HttpResponseRedirect): # successful form validation
profile = Profile.objects.get(associated_user=self.request.user)
if not profile.has_validated_email():
profile.request_email_address_confirmation(self.request, redirection_target=self.request.POST.get('next', None))
profile.request_email_address_confirmation(self.request, redirection_target=self.request.POST.get('next'))
return response
def _ensure_profile_has_email(self, profile, user):
......@@ -103,7 +103,7 @@ class ResendVerificationEmail(LoginRequiredMixin, generic.RedirectView):
def get(self, request, *args, **kwargs):
profile = Profile.objects.get(associated_user=self.request.user)
profile.request_email_address_confirmation(self.request, redirection_target=self.request.GET.get('next', None))
profile.request_email_address_confirmation(self.request, redirection_target=self.request.GET.get('next'))
return super().get(request, *args, **kwargs)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment