From bbc36d361209929eb17055829c350936993a8074 Mon Sep 17 00:00:00 2001 From: Raymond Jessop Date: Fri, 18 Apr 2025 21:02:05 -0500 Subject: [PATCH] updated tests.py again --- django_aws_ses/views.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/django_aws_ses/views.py b/django_aws_ses/views.py index b25a0a9..84e6993 100644 --- a/django_aws_ses/views.py +++ b/django_aws_ses/views.py @@ -311,6 +311,7 @@ class HandleUnsubscribe(TemplateView): """Show confirmation page for unsubscribe or re-subscribe.""" uuid = self.kwargs['uuid'] token = self.kwargs['token'] + self.action = '' # Reset action to ensure confirmation page try: uuid = force_str(urlsafe_base64_decode(uuid)) @@ -329,7 +330,6 @@ class HandleUnsubscribe(TemplateView): return redirect(settings.HOME_URL) self.user_email = user.email - self.action = '' # Ensure GET renders confirmation page return super().get(request, *args, **kwargs) def post(self, request, *args, **kwargs): @@ -360,13 +360,13 @@ class HandleUnsubscribe(TemplateView): ses.save() logger.info(f"Unsubscribed user: {user.email}") self.action = 'unsubscribe' - return render(request, self.template_name, self.get_context_data()) elif action == 'resubscribe': ses.unsubscribe = False ses.save() logger.info(f"Re-subscribed user: {user.email}") self.action = 'resubscribe' - return render(request, self.template_name, self.get_context_data()) + else: + logger.warning(f"Invalid action for user: {user.email}") + return redirect(settings.HOME_URL) - logger.warning(f"Invalid action for user: {user.email}") - return redirect(settings.HOME_URL) \ No newline at end of file + return render(request, self.template_name, self.get_context_data()) \ No newline at end of file