listeneinstellungen funktionieren

wir sollten vielleicht nur in die Datenbank schreiben, wenn auch was geaendert ist.

2 files changed, 84 insertions, 4 deletions

diff --git a/modules/lists/lists.go b/modules/lists/lists.go
index 73a3fe7..2f37fb8 100644
--- a/modules/lists/lists.go
+++ b/modules/lists/lists.go
@@ -76,6 +76,7 @@ func (m *Module) Init(g *grilist.Grilist) {
         m.g.Router.POST("/list/:id", m.addGrilToList)
         m.g.Router.DELETE("/list/:id/order", m.removeGrilFromList)
         m.g.Router.GET("/list/:id/settings", m.viewListSettings)
+        m.g.Router.POST("/list/:id/settings", m.updateListSettings)
         m.g.Router.GET("/new/list", m.displayCreateList)
         m.g.Router.POST("/new/list", m.createList)
 
@@ -321,6 +322,84 @@ func (m *Module) viewListSettings(w http.ResponseWriter, r *http.Request, p http
         m.g.Renderer.RenderPage("list_settings", w, data)
 }
 
+func (m *Module) updateListSettings(w http.ResponseWriter, r *http.Request, p httprouter.Params) {
+        user, err := m.g.Charakterin.GetUserFromRequest(r)
+        if err != nil {
+                log.Println("invalid updateListSettings user")
+                http.Redirect(w, r, fmt.Sprintf("/list/%s", p.ByName("id")), 403)
+                return
+        }
+
+        id, err := strconv.Atoi(p.ByName("id"))
+        if err != nil {
+                log.Println("invalid updateListSettings ID")
+                http.Redirect(w, r, "/", 400)
+                return
+        }
+
+        list, err := m.FromID(id, false)
+        if err != nil {
+                log.Println("invalid updateListSettings ID (could not get list)")
+                http.Redirect(w, r, fmt.Sprintf("/list/%d", id), 500)
+                return
+        }
+
+        // checken, ob der user auch list owner ist
+        if user.ID != list.Owner.ID {
+                log.Println("invalid updateListSettings request: user not list owner")
+                http.Redirect(w, r, fmt.Sprintf("/list/%d", id), 403)
+                return
+        }
+
+        renderWithError := func(message string, nameError, descriptionError bool) {
+                data := m.g.Renderer.DefaultData()
+                data["user"] = user
+                data["list"] = list
+                data["error"] = message
+                data["error_name"] = nameError
+                data["error_description"] = descriptionError
+
+                m.g.Renderer.RenderPage("list_settings", w, data)
+        }
+
+        values, err := readBody(r)
+        if err != nil {
+                log.Println("invalid POST data in updateListSettings")
+                renderWithError("invalid post data", false, false)
+                return
+        }
+
+        name := values.Get("name")
+        if len(name) < 3 {
+                renderWithError("Name zu kurz", true, false)
+                return
+        } else if len(name) > 32 {
+                renderWithError("Name zu lang", true, false)
+                return
+        }
+
+        description := values.Get("description")
+        if len(description) < 3 {
+                renderWithError("Beschreibung zu kurz", false, true)
+                return
+        } else if len(description) > 100 {
+                renderWithError("Beschreibung zu lang", false, true)
+                return
+        }
+
+        list.Name = name
+        list.Description = description
+
+        _, err = m.g.DB.Query(`UPDATE grilist.lists SET name = $2, description = $3 WHERE id = $1`, id, name, description)
+        if err != nil {
+                log.Println("could not update list:", err)
+                renderWithError("interner fehler", false, false)
+                return
+        }
+
+        m.viewListSettings(w, r, p)
+}
+
 func (m *Module) addGrilToList(w http.ResponseWriter, r *http.Request, p httprouter.Params) {
         slistID := p.ByName("id")
 
diff --git a/views/pages/list_settings.html b/views/pages/list_settings.html
index 0ce3860..5612093 100644
--- a/views/pages/list_settings.html
+++ b/views/pages/list_settings.html
@@ -10,7 +10,7 @@
         <body>
                 {{ template "navbar" . }}
                 <div class="container">
-                        <h1>Listeneinstellungen</h1>
+                        <h1 class="row valign-wrapper"><a href="/list/{{ .list.ID}}" class="col s1 btn purple lighten-4" style="width: auto"><i class="material-icons">arrow_back</i></a><span class="col s11"> Listeneinstellungen</span></h1>
                         <div class="row">
                                 <div class="col s12 m6 offset-m3 l4 offset-l4">
                                         {{ if .error }}
@@ -23,14 +23,15 @@
                                                 </div>
                                         </div>
                                         {{ end }}
-                                        <form method="POST" name="list" action="#">
+                                        <form method="POST" name="list" action="">
                                                 <div class="input-field col s12">
                                                         <i class="material-icons prefix">assignment_ind</i>
-                                                        <input name="name" type="text" placeholder="Name" class="validate" value="{{ .list.Name }}"/>
+                                                        <input id="name" name="name" type="text" placeholder="Name" class="{{ if .error_name }}invalid{{ end }}" value="{{ .list.Name }}"/>
+                                                        <label for="name">Name</label>
                                                 </div>
                                                 <div class="input-field col s12">
                                                         <i class="material-icons prefix">mode_edit</i>
-                                                        <textarea id="description" name="description" class="materialize-textarea">{{ .list.Description }}</textarea>
+                                                        <textarea id="description" name="description" class="materialize-textarea {{ if .error_description }}invalid{{ end }}">{{ .list.Description }}</textarea>
                                                         <label for="description">Beschreibung</label>
                                                 </div>
                                                 <div class="col s12">