fix: 编辑保存失败时显示错误提示；修复只修改标签时返回False的问题

xian_favor/api.py

@@ -656,7 +656,6 @@ INDEX_TEMPLATE = '''
         .is-starred { border-left: 4px solid #ffc107; background: #fffbe6; }
         .is-starred:hover { background: #fff9e0; }
         .star-btn { font-size: 11px; }
-        .star-btn.active { color: #ffc107; }
         .status-pending { color: #ffc107; }
         .status-in_progress { color: #17a2b8; }
         .status-completed { color: #28a745; text-decoration: line-through; }
@@ -1371,7 +1370,7 @@ function renderItems(items) {
                     </div>
                     <div class="d-flex align-items-center gap-1 flex-wrap ms-2" onclick="event.stopPropagation();">
                         ${item.tags.slice(0, 2).map(t => `<span class="badge bg-secondary" style="font-size:10px;">${t}</span>`).join('')}
-                        <button class="btn btn-sm btn-outline-warning py-0 px-1 star-btn ${item.is_starred ? 'active' : ''}" onclick="toggleStar(${item.id})" title="${item.is_starred ? '取消重点关注' : '设为重点关注'}">
+                        <button class="btn btn-sm btn-outline-warning py-0 px-1 star-btn" onclick="toggleStar(${item.id})" title="${item.is_starred ? '取消重点关注' : '设为重点关注'}">
                             <i class="bi bi-star${item.is_starred ? '-fill' : ''}" style="font-size:11px; ${item.is_starred ? 'color:#ffc107;' : ''}"></i>
                         </button>
                         ${item.type !== 'todo' ? `<button class="btn btn-sm btn-outline-secondary py-0 px-1" onclick="showConvertModal(${item.id})" title="转为待办"><i class="bi bi-arrow-repeat" style="font-size:11px;"></i></button>` : ''}
@@ -1782,15 +1781,23 @@ async function saveEdit() {
         is_starred: document.getElementById('editStarred').checked ? 1 : 0
     };
     
-    const res = await fetch(`${API_BASE}/items/${id}`, {
-        method: 'PUT',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify(data)
-    });
-    
-    if (res.ok) {
-        bootstrap.Modal.getInstance(document.getElementById('editModal')).hide();
-        refreshData();
+    try {
+        const res = await fetch(`${API_BASE}/items/${id}`, {
+            method: 'PUT',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify(data)
+        });
+        
+        const result = await res.json();
+        
+        if (res.ok && result.success) {
+            bootstrap.Modal.getInstance(document.getElementById('editModal')).hide();
+            refreshData();
+        } else {
+            alert('保存失败: ' + (result.error || '未知错误'));
+        }
+    } catch (e) {
+        alert('保存失败: ' + e.message);
     }
 }
 

xian_favor/db.py

@@ -263,6 +263,7 @@ class Database:
         allowed_fields = ['type', 'title', 'content', 'url', 'source', 'status', 'priority', 'due_date', 'note', 'is_starred']
         update_fields = {k: v for k, v in kwargs.items() if k in allowed_fields}
         
+        # 只有 tags 变化也算有效更新
         if not update_fields and 'tags' not in kwargs:
             return False
         
@@ -271,6 +272,11 @@ class Database:
         with self.get_conn() as conn:
             cursor = conn.cursor()
             
+            # 检查条目是否存在
+            cursor.execute("SELECT id FROM items WHERE id = ?", (item_id,))
+            if not cursor.fetchone():
+                return False
+            
             if update_fields:
                 set_clause = ", ".join(f"{k} = ?" for k in update_fields.keys())
                 set_clause += ", updated_at = ?"
@@ -285,7 +291,7 @@ class Database:
                     self._add_tags_to_item(conn, item_id, kwargs['tags'])
             
             conn.commit()
-            return cursor.rowcount > 0
+            return True
     
     def delete_item(self, item_id: int) -> bool:
         """删除条目"""