sklad: create, update, remove

This commit is contained in:
Přemysl Eric Janouch 2019-04-15 01:10:48 +02:00
parent 608ba10eec
commit 37b7edb2ef
Signed by: p
GPG Key ID: A0420B94F92B9493
4 changed files with 264 additions and 27 deletions

View File

@ -2,6 +2,22 @@
*/}}{{ if .Container }}{{ .Container.Id }}{{ else }}Obaly{{ end }}{{ end }} */}}{{ if .Container }}{{ .Container.Id }}{{ else }}Obaly{{ end }}{{ end }}
{{ define "Content" }} {{ define "Content" }}
{{ if .ErrorNoSuchSeries }}
<p>Chyba: Řada neexistuje.
{{ else if .ErrorContainerAlreadyExists }}
<p>Chyba: Obal s tímto ID už existuje.
{{ else if .ErrorNoSuchContainer }}
<p>Chyba: Obal neexistuje.
{{ else if .ErrorCannotChangeSeriesNotEmpty }}
<p>Chyba: Řadu u neprázdných obalů nelze měnit.
{{ else if .ErrorCannotChangeNumber }}
<p>Chyba: Číslo obalu v řadě nelze měnit.
{{ else if .ErrorContainerInUse }}
<p>Chyba: Obal se používá.
{{ else if .Error }}
<p>Chyba: {{ .Error }}
{{ end }}
{{ if .Container }} {{ if .Container }}
<section> <section>

View File

@ -14,6 +14,7 @@ import (
type Series struct { type Series struct {
Prefix string // PK: prefix Prefix string // PK: prefix
Description string // what kind of containers this is for Description string // what kind of containers this is for
Counter uint // last used container number
} }
func (s *Series) Containers() []*Container { func (s *Series) Containers() []*Container {
@ -71,9 +72,6 @@ var (
labelFont *bdf.Font labelFont *bdf.Font
) )
// TODO: Some functions to add, remove and change things in the database.
// Indexes must be kept valid, just like any invariants.
func dbSearchSeries(query string) (result []*Series) { func dbSearchSeries(query string) (result []*Series) {
query = strings.ToLower(query) query = strings.ToLower(query)
added := map[string]bool{} added := map[string]bool{}
@ -110,6 +108,134 @@ func dbSearchContainers(query string) (result []*Container) {
return return
} }
var errInvalidPrefix = errors.New("invalid prefix")
var errSeriesAlreadyExists = errors.New("series already exists")
var errCannotChangePrefix = errors.New("cannot change the prefix")
var errNoSuchSeries = errors.New("no such series")
var errSeriesInUse = errors.New("series is in use")
// Find and filter out the series in O(n).
func filterSeries(slice []*Series, s *Series) (filtered []*Series) {
for _, series := range slice {
if s != series {
filtered = append(filtered, series)
}
}
return
}
func dbSeriesCreate(s *Series) error {
if s.Prefix == "" {
return errInvalidPrefix
}
if _, ok := indexSeries[s.Prefix]; ok {
return errSeriesAlreadyExists
}
db.Series = append(db.Series, s)
indexSeries[s.Prefix] = s
return dbCommit()
}
func dbSeriesUpdate(s *Series, updated Series) error {
// It might be easily possible with no members, though this
// is not reachable from the UI and can be solved by deletion.
if updated.Prefix != s.Prefix {
return errCannotChangePrefix
}
*s = updated
return dbCommit()
}
func dbSeriesRemove(s *Series) error {
if len(s.Containers()) > 0 {
return errSeriesInUse
}
db.Series = filterSeries(db.Series, s)
delete(indexSeries, s.Prefix)
delete(indexMembers, s.Prefix)
return dbCommit()
}
var errContainerAlreadyExists = errors.New("container already exists")
var errNoSuchContainer = errors.New("no such container")
var errCannotChangeSeriesNotEmpty = errors.New(
"cannot change the series of a non-empty container")
var errCannotChangeNumber = errors.New("cannot change the number")
var errContainerInUse = errors.New("container is in use")
// Find and filter out the container in O(n).
func filterContainer(slice []*Container, c *Container) (filtered []*Container) {
for _, container := range slice {
if c != container {
filtered = append(filtered, container)
}
}
return
}
func dbContainerCreate(c *Container) error {
if series, ok := indexSeries[c.Series]; !ok {
return errNoSuchSeries
} else if c.Number == 0 {
c.Number = series.Counter
for {
c.Number++
if _, ok := indexContainer[c.Id()]; !ok {
break
}
}
series.Counter = c.Number
}
if _, ok := indexContainer[c.Id()]; ok {
return errContainerAlreadyExists
}
if c.Parent != "" && indexContainer[c.Parent] == nil {
return errNoSuchContainer
}
db.Containers = append(db.Containers, c)
indexMembers[c.Series] = append(indexMembers[c.Series], c)
indexChildren[c.Parent] = append(indexChildren[c.Parent], c)
indexContainer[c.Id()] = c
return dbCommit()
}
func dbContainerUpdate(c *Container, updated Container) error {
newId := updated.Id()
if updated.Series != c.Series && len(c.Children()) > 0 {
return errCannotChangeSeriesNotEmpty
}
if updated.Number != c.Number {
return errCannotChangeNumber
}
if _, ok := indexContainer[newId]; ok && newId != c.Id() {
return errContainerAlreadyExists
}
if updated.Parent != c.Parent {
indexChildren[c.Parent] = filterContainer(indexChildren[c.Parent], c)
indexChildren[newId] = append(indexChildren[newId], c)
}
*c = updated
return dbCommit()
}
func dbContainerRemove(c *Container) error {
if len(indexChildren[c.Id()]) > 0 {
return errContainerInUse
}
db.Containers = filterContainer(db.Containers, c)
indexMembers[c.Series] = filterContainer(indexMembers[c.Series], c)
indexChildren[c.Parent] = filterContainer(indexChildren[c.Parent], c)
delete(indexContainer, c.Id())
delete(indexChildren, c.Id())
return dbCommit()
}
func dbCommit() error { func dbCommit() error {
// Write a timestamp. // Write a timestamp.
e := json.NewEncoder(dbLog) e := json.NewEncoder(dbLog)

View File

@ -7,6 +7,7 @@ import (
"log" "log"
"math/rand" "math/rand"
"net/http" "net/http"
"net/url"
"os" "os"
"path/filepath" "path/filepath"
"time" "time"
@ -83,11 +84,40 @@ func handleLogout(w http.ResponseWriter, r *http.Request) {
http.Redirect(w, r, "/", http.StatusSeeOther) http.Redirect(w, r, "/", http.StatusSeeOther)
} }
func handleContainer(w http.ResponseWriter, r *http.Request) { func handleContainerPost(r *http.Request) error {
if r.Method == http.MethodPost { id := ContainerId(r.FormValue("id"))
// TODO description := r.FormValue("description")
series := r.FormValue("series")
parent := ContainerId(r.FormValue("parent"))
_, remove := r.Form["remove"]
if container, ok := indexContainer[id]; ok {
if remove {
return dbContainerRemove(container)
} else {
c := *container
c.Description = description
c.Series = series
return dbContainerUpdate(container, c)
}
} else if remove {
return errNoSuchContainer
} else {
return dbContainerCreate(&Container{
Series: series,
Parent: parent,
Description: description,
})
} }
if r.Method != http.MethodGet { }
func handleContainer(w http.ResponseWriter, r *http.Request) {
var err error
if r.Method == http.MethodPost {
err = handleContainerPost(r)
// XXX: This is rather ugly. When removing, we want to keep
// the context id, in addition to the id being changed.
} else if r.Method != http.MethodGet {
w.WriteHeader(http.StatusMethodNotAllowed) w.WriteHeader(http.StatusMethodNotAllowed)
return return
} }
@ -98,33 +128,70 @@ func handleContainer(w http.ResponseWriter, r *http.Request) {
} }
var container *Container var container *Container
children := []*Container{} children := indexChildren[""]
if id := ContainerId(r.FormValue("id")); id == "" { if c, ok := indexContainer[ContainerId(r.FormValue("id"))]; ok {
children = indexChildren[""]
} else if c, ok := indexContainer[id]; ok {
children = c.Children() children = c.Children()
container = c container = c
} }
params := struct { params := struct {
Container *Container Error error
Children []*Container ErrorNoSuchSeries bool
AllSeries map[string]string ErrorContainerAlreadyExists bool
ErrorNoSuchContainer bool
ErrorCannotChangeSeriesNotEmpty bool
ErrorCannotChangeNumber bool
ErrorContainerInUse bool
Container *Container
Children []*Container
AllSeries map[string]string
}{ }{
Container: container, Error: err,
Children: children, ErrorNoSuchSeries: err == errNoSuchSeries,
AllSeries: allSeries, ErrorContainerAlreadyExists: err == errContainerAlreadyExists,
ErrorNoSuchContainer: err == errNoSuchContainer,
ErrorCannotChangeSeriesNotEmpty: err == errCannotChangeSeriesNotEmpty,
ErrorCannotChangeNumber: err == errCannotChangeNumber,
ErrorContainerInUse: err == errContainerInUse,
Container: container,
Children: children,
AllSeries: allSeries,
} }
executeTemplate("container.tmpl", w, &params) executeTemplate("container.tmpl", w, &params)
} }
func handleSeries(w http.ResponseWriter, r *http.Request) { func handleSeriesPost(r *http.Request) error {
if r.Method == http.MethodPost { prefix := r.FormValue("prefix")
// TODO description := r.FormValue("description")
_, remove := r.Form["remove"]
if series, ok := indexSeries[prefix]; ok {
if remove {
return dbSeriesRemove(series)
} else {
s := *series
s.Description = description
return dbSeriesUpdate(series, s)
}
} else if remove {
return errNoSuchSeries
} else {
return dbSeriesCreate(&Series{
Prefix: prefix,
Description: description,
})
} }
if r.Method != http.MethodGet { }
func handleSeries(w http.ResponseWriter, r *http.Request) {
var err error
if r.Method == http.MethodPost {
err = handleSeriesPost(r)
// XXX: This is rather ugly.
r.Form = url.Values{}
} else if r.Method != http.MethodGet {
w.WriteHeader(http.StatusMethodNotAllowed) w.WriteHeader(http.StatusMethodNotAllowed)
return return
} }
@ -140,16 +207,30 @@ func handleSeries(w http.ResponseWriter, r *http.Request) {
if prefix == "" { if prefix == "" {
} else if series, ok := indexSeries[prefix]; ok { } else if series, ok := indexSeries[prefix]; ok {
description = series.Description description = series.Description
} else {
err = errNoSuchSeries
} }
params := struct { params := struct {
Prefix string Error error
Description string ErrorInvalidPrefix bool
AllSeries map[string]*Series ErrorSeriesAlreadyExists bool
ErrorCannotChangePrefix bool
ErrorNoSuchSeries bool
ErrorSeriesInUse bool
Prefix string
Description string
AllSeries map[string]*Series
}{ }{
Prefix: prefix, Error: err,
Description: description, ErrorInvalidPrefix: err == errInvalidPrefix,
AllSeries: allSeries, ErrorSeriesAlreadyExists: err == errSeriesAlreadyExists,
ErrorCannotChangePrefix: err == errCannotChangePrefix,
ErrorNoSuchSeries: err == errNoSuchSeries,
ErrorSeriesInUse: err == errSeriesInUse,
Prefix: prefix,
Description: description,
AllSeries: allSeries,
} }
executeTemplate("series.tmpl", w, &params) executeTemplate("series.tmpl", w, &params)

View File

@ -1,6 +1,20 @@
{{ define "Title" }}{{ or .Prefix "Řady" }}{{ end }} {{ define "Title" }}{{ or .Prefix "Řady" }}{{ end }}
{{ define "Content" }} {{ define "Content" }}
{{ if .ErrorInvalidPrefix }}
<p>Chyba: Neplatný prefix.
{{ else if .ErrorSeriesAlreadyExists }}
<p>Chyba: Řada s tímto prefixem už existuje.
{{ else if .ErrorCannotChangePrefix }}
<p>Chyba: Prefix nelze měnit.
{{ else if .ErrorNoSuchSeries }}
<p>Chyba: Řada neexistuje.
{{ else if .ErrorSeriesInUse }}
<p>Chyba: Řada se používá.
{{ else if .Error }}
<p>Chyba: {{ .Error }}
{{ end }}
{{ if .Prefix }} {{ if .Prefix }}
<h2>{{ .Prefix }}</h2> <h2>{{ .Prefix }}</h2>