Check if user already exists and bug fix
This commit is contained in:
@ -44,7 +44,7 @@ func (db *DB) AddUser(user, pass, first, last string, writer, editor, admin bool
|
||||
INSERT INTO users
|
||||
(username, password, first_name, last_name, writer, editor, admin)
|
||||
VALUES
|
||||
(?, ?, ?, ?, ?, ?)
|
||||
(?, ?, ?, ?, ?, ?, ?)
|
||||
`
|
||||
_, err = db.Exec(query, user, string(hashedPass), first, last, writer, editor, admin)
|
||||
if err != nil {
|
||||
|
13
cmd/ui/ui.go
13
cmd/ui/ui.go
@ -59,9 +59,16 @@ func HandleAddUser(db *data.DB) http.HandlerFunc {
|
||||
|
||||
_, _, ok := checkUserStrings(user, first, last)
|
||||
if !ok {
|
||||
log.Println("checkUserStrings")
|
||||
template.Must(template.ParseFiles("web/templates/add-user.html")).Execute(w, nil)
|
||||
}
|
||||
id, _ := db.GetID(user)
|
||||
if id != 0 {
|
||||
log.Println("GetID")
|
||||
template.Must(template.ParseFiles("web/templates/add-user.html")).Execute(w, nil)
|
||||
}
|
||||
if pass != pass2 {
|
||||
log.Println("pass")
|
||||
template.Must(template.ParseFiles("web/templates/add-user.html")).Execute(w, nil)
|
||||
}
|
||||
switch role {
|
||||
@ -78,10 +85,14 @@ func HandleAddUser(db *data.DB) http.HandlerFunc {
|
||||
editor = false
|
||||
admin = true
|
||||
default:
|
||||
log.Println("switch")
|
||||
template.Must(template.ParseFiles("web/templates/add-user.html")).Execute(w, nil)
|
||||
}
|
||||
|
||||
db.AddUser(user, pass, first, last, writer, editor, admin)
|
||||
if err := db.AddUser(user, pass, first, last, writer, editor, admin); err != nil {
|
||||
log.Println(err)
|
||||
http.Error(w, err.Error(), http.StatusInternalServerError)
|
||||
}
|
||||
template.Must(template.ParseFiles("web/templates/editor.html")).Execute(w, nil)
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user