cmd/sklad/main.go

@@ -107,23 +107,14 @@ func handleContainerPost(r *http.Request) error {
 }
 
 func handleContainer(w http.ResponseWriter, r *http.Request) {
-	// When deleting, do not try to show the deleted entry but the context.
-	shownId := r.FormValue("context")
-	if shownId == "" {
-		shownId = r.FormValue("id")
-	}
-
 	var err error
 	if r.Method == http.MethodPost {
-		if err = handleContainerPost(r); err == nil {
-			redirect := "container"
-			if shownId != "" {
-				redirect += "?id=" + url.QueryEscape(shownId)
-			}
-			http.Redirect(w, r, redirect, http.StatusSeeOther)
-			return
-		}
-		// TODO: Use the last data as a prefill.
+		err = handleContainerPost(r)
+		// FIXME: This is rather ugly. When removing, we want to keep
+		// the context id, in addition to the id being changed.
+		// TODO: If there were no errors, redirect the user to GET,
+		// which is related to the previous comment.
+		// TODO: If there were errors, use the last data as a prefill.
 	} else if r.Method != http.MethodGet {
 		w.WriteHeader(http.StatusMethodNotAllowed)
 		return
@@ -137,7 +128,7 @@ func handleContainer(w http.ResponseWriter, r *http.Request) {
 	var container *Container
 	children := indexChildren[""]
 
-	if c, ok := indexContainer[ContainerId(shownId)]; ok {
+	if c, ok := indexContainer[ContainerId(r.FormValue("id"))]; ok {
 		children = c.Children()
 		container = c
 	}
@@ -197,10 +188,7 @@ func handleSeriesPost(r *http.Request) error {
 func handleSeries(w http.ResponseWriter, r *http.Request) {
 	var err error
 	if r.Method == http.MethodPost {
-		if err = handleSeriesPost(r); err == nil {
-			http.Redirect(w, r, "series", http.StatusSeeOther)
-			return
-		}
+		err = handleSeriesPost(r)
 		// XXX: This is rather ugly.
 		r.Form = url.Values{}
 	} else if r.Method != http.MethodGet {