diff options
Diffstat (limited to 'internal/handlers/settings.go')
| -rw-r--r-- | internal/handlers/settings.go | 22 |
1 files changed, 4 insertions, 18 deletions
diff --git a/internal/handlers/settings.go b/internal/handlers/settings.go index c0ba654..a780170 100644 --- a/internal/handlers/settings.go +++ b/internal/handlers/settings.go @@ -196,19 +196,12 @@ func (h *Handler) HandleGetSourceOptions(w http.ResponseWriter, r *http.Request) // HandleToggleFeature toggles a feature flag func (h *Handler) HandleToggleFeature(w http.ResponseWriter, r *http.Request) { - if err := r.ParseForm(); err != nil { - JSONError(w, http.StatusBadRequest, "Failed to parse form", err) + name, ok := requireFormValue(w, r, "name") + if !ok { return } - - name := r.FormValue("name") enabled := r.FormValue("enabled") == "true" - if name == "" { - JSONError(w, http.StatusBadRequest, "Feature name required", nil) - return - } - if err := h.store.SetFeatureEnabled(name, enabled); err != nil { JSONError(w, http.StatusInternalServerError, "Failed to update feature", err) return @@ -220,19 +213,12 @@ func (h *Handler) HandleToggleFeature(w http.ResponseWriter, r *http.Request) { // HandleCreateFeature creates a new feature toggle func (h *Handler) HandleCreateFeature(w http.ResponseWriter, r *http.Request) { - if err := r.ParseForm(); err != nil { - JSONError(w, http.StatusBadRequest, "Failed to parse form", err) + name, ok := requireFormValue(w, r, "name") + if !ok { return } - - name := r.FormValue("name") description := r.FormValue("description") - if name == "" { - JSONError(w, http.StatusBadRequest, "Feature name required", nil) - return - } - if err := h.store.CreateFeatureToggle(name, description, false); err != nil { JSONError(w, http.StatusInternalServerError, "Failed to create feature", err) return |