Merge pull request #165 from ppolariss/main

fix: #163
OpenTreeHole · Nov 9, 2024 · 71615ea · 71615ea
2 parents 9970114 + 2f1daf8
commit 71615ea
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/apis/user/apis.go b/apis/user/apis.go
@@ -95,18 +95,23 @@ func ModifyUser(c *fiber.Ctx) error {
 		return err
 	}
 
-	var newUser User
-	err = DB.First(&newUser, userID).Error
-	if err != nil {
-		return err
+	// cannot get field "has_answered_questions" when admin changes other user's config
+	if user.ID != userID {
+		user = &User{
+			ID: userID,
+		}
+		err = DB.Take(user).Error
+		if err != nil {
+			return err
+		}
 	}
 
-	err = modifyUser(c, &newUser, body)
+	err = modifyUser(c, user, body)
 	if err != nil {
 		return err
 	}
 
-	return c.JSON(&newUser)
+	return c.JSON(user)
 }
 
 // ModifyCurrentUser