p/sklad
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

sklad: redirect to GET on successful DB changes

Browse Source
This commit is contained in:
Přemysl Eric Janouch 2019-04-22 10:10:12 +02:00
parent 7d2ca09d1b
commit 04e66d7888
Signed by: p
GPG Key ID: A0420B94F92B9493
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
cmd/sklad/main.go
View File

@ -115,12 +115,15 @@ func handleContainer(w http.ResponseWriter, r *http.Request) {
var err error
if r.Method == http.MethodPost {
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.
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.
} else if r.Method != http.MethodGet {
w.WriteHeader(http.StatusMethodNotAllowed)
return
@ -194,7 +197,10 @@ func handleSeriesPost(r *http.Request) error {
func handleSeries(w http.ResponseWriter, r *http.Request) {
var err error
if r.Method == http.MethodPost {
err = handleSeriesPost(r)
if err = handleSeriesPost(r); err == nil {
http.Redirect(w, r, "series", http.StatusSeeOther)
return
}
// XXX: This is rather ugly.
r.Form = url.Values{}
} else if r.Method != http.MethodGet {