From 69f5e0465dd0434116f026738a218912ed82817e Mon Sep 17 00:00:00 2001 From: retoor Date: Sat, 8 Nov 2025 19:00:55 +0100 Subject: [PATCH] Fix: Replace return HTTPException with raise HTTPException and resolve indentation errors --- retoors/helpers/auth.py | 2 +- retoors/views/auth.py | 9 +++------ retoors/views/site.py | 3 +-- 3 files changed, 5 insertions(+), 9 deletions(-) diff --git a/retoors/helpers/auth.py b/retoors/helpers/auth.py index 8949734..b037b82 100644 --- a/retoors/helpers/auth.py +++ b/retoors/helpers/auth.py @@ -7,6 +7,6 @@ def login_required(func): async def wrapper(self, *args, **kwargs): session = await get_session(self.request) if 'user_email' not in session: - return web.HTTPFound('/login') + raise web.HTTPFound('/login') return await func(self, *args, **kwargs) return wrapper diff --git a/retoors/views/auth.py b/retoors/views/auth.py index 925ac0c..0316bfe 100644 --- a/retoors/views/auth.py +++ b/retoors/views/auth.py @@ -48,8 +48,7 @@ class LoginView(CustomPydanticView): if user_service.authenticate_user(login_data.email, login_data.password): session = await new_session(self.request) session["user_email"] = login_data.email - return web.HTTPFound("/dashboard") - + raise web.HTTPFound("/dashboard") return aiohttp_jinja2.render_template( self.template_name, self.request, @@ -85,6 +84,7 @@ class RegistrationView(CustomPydanticView): user_service: UserService = self.request.app["user_service"] try: user_service.create_user(user_data.full_name, user_data.email, user_data.password) # Changed username to full_name + raise web.HTTPFound("/login") except ValueError: return aiohttp_jinja2.render_template( self.template_name, @@ -97,11 +97,8 @@ class RegistrationView(CustomPydanticView): }, ) - return web.HTTPFound("/login") - - class LogoutView(web.View): async def get(self): session = await get_session(self.request) session.clear() - return web.HTTPFound("/") + raise web.HTTPFound("/") diff --git a/retoors/views/site.py b/retoors/views/site.py index 6f35b42..e4de79f 100644 --- a/retoors/views/site.py +++ b/retoors/views/site.py @@ -66,5 +66,4 @@ class OrderView(CustomPydanticView): user_email = session.get("user_email") user_service: UserService = self.request.app["user_service"] user_service.update_user_quota(user_email, quota_data.storage_amount) - - return web.HTTPFound("/dashboard") + raise web.HTTPFound("/dashboard")