Type checking error resolved

Author: Saiyashwanth7

src/app/api/v1/users.py

@@ -72,6 +72,8 @@ async def read_user(request: Request, username: str, db: Annotated[AsyncSession,
     return cast(UserRead, db_user)
 
 
+# In src/app/api/v1/users.py, replace the patch_user function with this:
+
 @router.patch("/user/{username}")
 async def patch_user(
     request: Request,
@@ -80,25 +82,32 @@ async def patch_user(
     current_user: Annotated[dict, Depends(get_current_user)],
     db: Annotated[AsyncSession, Depends(async_get_db)],
 ) -> dict[str, str]:
-    # Remove schema_to_select to get a dictionary
     db_user = await crud_users.get(db=db, username=username)
     if db_user is None:
         raise NotFoundException("User not found")
 
-    # Now both are dictionaries, use dictionary access
-    if db_user["username"] != current_user["username"]:
-        raise ForbiddenException()
+    # Handle both dict and UserRead object types
+    if isinstance(db_user, dict):
+        db_username = db_user["username"]
+        db_email = db_user["email"]
+    else:
+        db_username = db_user.username
+        db_email = db_user.email
 
-    if values.username != db_user["username"]:
-        existing_username = await crud_users.exists(db=db, username=values.username)
-        if existing_username:
-            raise DuplicateValueException("Username not available")
+    if db_username != current_user["username"]:
+        raise ForbiddenException()
 
-    if values.email != db_user["email"]:
-        existing_email = await crud_users.exists(db=db, email=values.email)
-        if existing_email:
+    # Check for email conflicts if email is being updated
+    if values.email is not None and values.email != db_email:
+        if await crud_users.exists(db=db, email=values.email):
             raise DuplicateValueException("Email is already registered")
 
+    # Check for username conflicts if username is being updated  
+    if values.username is not None and values.username != db_username:
+        if await crud_users.exists(db=db, username=values.username):
+            raise DuplicateValueException("Username not available")
+
+    # Update the user
     await crud_users.update(db=db, object=values, username=username)
     return {"message": "User updated"}