diff options
-rw-r--r-- | modules/lists/lists.go | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/modules/lists/lists.go b/modules/lists/lists.go index c699271..455c39c 100644 --- a/modules/lists/lists.go +++ b/modules/lists/lists.go | |||
@@ -695,7 +695,7 @@ func (m *Module) removeGrilFromList(w http.ResponseWriter, r *http.Request, p ht | |||
695 | return | 695 | return |
696 | } | 696 | } |
697 | 697 | ||
698 | err = m.g.DB.QueryRow(`DELETE FROM grilist.lists_grils WHERE list_id=$1 AND gril_id=$2`, listID, grilID).Scan(&oldOrder) | 698 | err = m.g.DB.QueryRow(`DELETE FROM grilist.lists_grils WHERE list_id=$1 AND gril_id=$2 RETURNING "order"`, listID, grilID).Scan(&oldOrder) |
699 | if err != nil { | 699 | if err != nil { |
700 | log.Println("error removing gril:", err) | 700 | log.Println("error removing gril:", err) |
701 | http.Error(w, "could not remove gril", 500) | 701 | http.Error(w, "could not remove gril", 500) |
@@ -708,6 +708,12 @@ func (m *Module) removeGrilFromList(w http.ResponseWriter, r *http.Request, p ht | |||
708 | http.Error(w, "could not remove gril", 500) | 708 | http.Error(w, "could not remove gril", 500) |
709 | return | 709 | return |
710 | } | 710 | } |
711 | |||
712 | if r, err := res.RowsAffected(); err != nil || r == 0 { | ||
713 | log.Println("invalid rows affected in removing gril") | ||
714 | http.Error(w, "gril not in list / other error", 500) | ||
715 | return | ||
716 | } | ||
711 | 717 | ||
712 | list.Grils = list.Grils[:0] | 718 | list.Grils = list.Grils[:0] |
713 | 719 | ||