Fixes

osuAkatsuki · Aug 8, 2024 · 4843f30 · 4843f30
1 parent 36b0c0f
commit 4843f30
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/app/api/public/users.py b/app/api/public/users.py
@@ -71,7 +71,7 @@ async def update_username(
             ),
         )
 
-    response = await users.update_username(user_id, args.new_username)
+    response = await users.update_username(user_id, new_username=args.new_username)
     if isinstance(response, Error):
         return JSONResponse(
             content=response.model_dump(),
@@ -105,7 +105,9 @@ async def update_password(
         )
 
     response = await users.update_password(
-        user_id, args.current_password, args.new_password
+        user_id,
+        current_password=args.current_password,
+        new_password=args.new_password,
     )
     if isinstance(response, Error):
         return JSONResponse(
@@ -140,7 +142,9 @@ async def update_email_address(
         )
 
     response = await users.update_email_address(
-        user_id, args.current_password, args.new_email_address
+        user_id,
+        current_password=args.current_password,
+        new_email_address=args.new_email_address,
     )
     if isinstance(response, Error):
         return JSONResponse(

diff --git a/app/usecases/users.py b/app/usecases/users.py
@@ -110,7 +110,7 @@ async def fetch_one_by_user_id(user_id: int) -> User | Error:
     )
 
 
-async def update_username(user_id: int, new_username: str) -> None | Error:
+async def update_username(user_id: int, *, new_username: str) -> None | Error:
     user = await users.fetch_one_by_user_id(user_id)
     if user is None:
         return Error(
@@ -138,7 +138,10 @@ async def update_username(user_id: int, new_username: str) -> None | Error:
 
 
 async def update_password(
-    user_id: int, current_password: str, new_password: str
+    user_id: int,
+    *,
+    current_password: str,
+    new_password: str,
 ) -> None | Error:
     user = await users.fetch_one_by_user_id(user_id)
     if user is None:
@@ -158,7 +161,10 @@ async def update_password(
 
 
 async def update_email_address(
-    user_id: int, current_password: str, new_email_address: str
+    user_id: int,
+    *,
+    current_password: str,
+    new_email_address: str,
 ) -> None | Error:
     user = await users.fetch_one_by_user_id(user_id)
     if user is None: