fix: 删除用户转移配方去重改为按成分内容比较,不只看名字
All checks were successful
PR Preview / teardown-preview (pull_request) Has been skipped
Test / unit-test (push) Successful in 5s
Test / build-check (push) Successful in 4s
PR Preview / test (pull_request) Successful in 5s
PR Preview / deploy-preview (pull_request) Successful in 10s
Test / e2e-test (push) Successful in 52s

相同成分组合(不管名字是否相同)视为重复,不转移。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-04-12 09:57:32 +00:00
parent cd08513985
commit b3c658b496

View File

@@ -1112,20 +1112,25 @@ def delete_user(user_id: int, user=Depends(require_role("admin"))):
diaries = conn.execute("SELECT id, name, ingredients FROM user_diary WHERE user_id = ?", (user_id,)).fetchall()
transferred = 0
if diaries:
# Load admin's diary and public recipes for dedup
admin_diaries = conn.execute("SELECT name, ingredients FROM user_diary WHERE user_id = ?", (user["id"],)).fetchall()
public_recipes = conn.execute(
"SELECT r.name, GROUP_CONCAT(ri.oil_name || ':' || ri.drops) as ings "
"FROM recipes r LEFT JOIN recipe_ingredients ri ON r.id = ri.recipe_id GROUP BY r.id"
).fetchall()
existing = {}
# Build set of ingredient fingerprints from admin diary + public recipes
def _ings_key(ings_json):
"""Normalize ingredients to a comparable key."""
try:
ings = json.loads(ings_json) if isinstance(ings_json, str) else []
return tuple(sorted((i.get("oil") or i.get("oil_name", ""), i.get("drops", 0)) for i in ings))
except Exception:
return ()
existing_keys = set()
admin_diaries = conn.execute("SELECT ingredients FROM user_diary WHERE user_id = ?", (user["id"],)).fetchall()
for row in admin_diaries:
existing[row["name"]] = row["ingredients"]
for row in public_recipes:
existing[row["name"]] = row["ings"] or ""
existing_keys.add(_ings_key(row["ingredients"]))
public_recipes = conn.execute("SELECT id FROM recipes").fetchall()
for pr in public_recipes:
pub_ings = conn.execute("SELECT oil_name, drops FROM recipe_ingredients WHERE recipe_id = ?", (pr["id"],)).fetchall()
existing_keys.add(tuple(sorted((r["oil_name"], r["drops"]) for r in pub_ings)))
for d in diaries:
# Check if same name+ingredients exists in admin diary or public
is_dup = d["name"] in existing
d_key = _ings_key(d["ingredients"])
is_dup = d_key in existing_keys and d_key != ()
if is_dup:
conn.execute("DELETE FROM user_diary WHERE id = ?", (d["id"],))
else: