feat: 更多改进 #21
@@ -1094,6 +1094,9 @@ def delete_user(user_id: int, user=Depends(require_role("admin"))):
|
||||
@app.put("/api/users/{user_id}")
|
||||
def update_user(user_id: int, body: UserUpdate, user=Depends(require_role("admin"))):
|
||||
conn = get_db()
|
||||
target = conn.execute("SELECT role, display_name, username FROM users WHERE id = ?", (user_id,)).fetchone()
|
||||
old_role = target["role"] if target else "unknown"
|
||||
target_name = (target["display_name"] or target["username"]) if target else "unknown"
|
||||
if body.role is not None:
|
||||
if body.role == "admin":
|
||||
conn.close()
|
||||
@@ -1101,8 +1104,15 @@ def update_user(user_id: int, body: UserUpdate, user=Depends(require_role("admin
|
||||
conn.execute("UPDATE users SET role = ?, role_changed_at = datetime('now') WHERE id = ?", (body.role, user_id))
|
||||
if body.display_name is not None:
|
||||
conn.execute("UPDATE users SET display_name = ? WHERE id = ?", (body.display_name, user_id))
|
||||
log_audit(conn, user["id"], "update_user", "user", user_id, None,
|
||||
json.dumps({"role": body.role, "display_name": body.display_name}))
|
||||
role_labels = {"admin": "管理员", "senior_editor": "高级编辑", "editor": "编辑", "viewer": "查看者"}
|
||||
detail = {}
|
||||
if body.role is not None and body.role != old_role:
|
||||
detail["from_role"] = role_labels.get(old_role, old_role)
|
||||
detail["to_role"] = role_labels.get(body.role, body.role)
|
||||
if body.display_name is not None:
|
||||
detail["display_name"] = body.display_name
|
||||
log_audit(conn, user["id"], "update_user", "user", user_id, target_name,
|
||||
json.dumps(detail, ensure_ascii=False))
|
||||
conn.commit()
|
||||
conn.close()
|
||||
return {"ok": True}
|
||||
|
||||
@@ -153,9 +153,10 @@ function parsedDetail(log) {
|
||||
try {
|
||||
const d = JSON.parse(log.detail)
|
||||
const parts = []
|
||||
if (d.from_role && d.to_role) parts.push(`${d.from_role} → ${d.to_role}`)
|
||||
if (d.from_user) parts.push(`来自: ${d.from_user}`)
|
||||
if (d.reason) parts.push(`原因: ${d.reason}`)
|
||||
if (d.role) parts.push(`角色: ${d.role}`)
|
||||
if (d.business_name) parts.push(`商户: ${d.business_name}`)
|
||||
if (d.display_name) parts.push(`显示名: ${d.display_name}`)
|
||||
if (d.original_log_id) parts.push(`恢复自 #${d.original_log_id}`)
|
||||
if (parts.length) return parts.join(' · ')
|
||||
|
||||
Reference in New Issue
Block a user