fix logging
All checks were successful
ci/woodpecker/push/build Pipeline was successful
ci/woodpecker/push/dryrun Pipeline was successful
ci/woodpecker/push/publish-tag Pipeline was successful
ci/woodpecker/push/publish-latest Pipeline was successful

This commit is contained in:
Michael Thomson 2024-06-26 22:57:53 -04:00
parent c98fc938c9
commit 8b1468fc1f
2 changed files with 8 additions and 9 deletions

View File

@ -18,7 +18,7 @@ func (ts *PostgresTodoRepository) List() ([]models.Todo, error) {
rows, err := ts.db.Query(context.Background(), "SELECT id, name, done FROM todo") rows, err := ts.db.Query(context.Background(), "SELECT id, name, done FROM todo")
if err != nil { if err != nil {
log.Fatal(err) log.Printf("Database error: %v", err)
return todos, err return todos, err
} }
@ -26,7 +26,6 @@ func (ts *PostgresTodoRepository) List() ([]models.Todo, error) {
var r models.Todo var r models.Todo
err := rows.Scan(&r.Id, &r.Name, &r.Done) err := rows.Scan(&r.Id, &r.Name, &r.Done)
if err != nil { if err != nil {
log.Fatal(err)
return todos, err return todos, err
} }
todos = append(todos, r) todos = append(todos, r)
@ -40,7 +39,7 @@ func (ts *PostgresTodoRepository) Get(id uuid.UUID) (models.Todo, error) {
err := ts.db.QueryRow(context.Background(), "SELECT id, name, done FROM todo WHERE id=$1", id).Scan(&todo.Id, &todo.Name, &todo.Done) err := ts.db.QueryRow(context.Background(), "SELECT id, name, done FROM todo WHERE id=$1", id).Scan(&todo.Id, &todo.Name, &todo.Done)
if err != nil { if err != nil {
log.Fatal(err) log.Printf("Database error: %v", err)
return todo, err return todo, err
} }
@ -51,7 +50,7 @@ func (ts *PostgresTodoRepository) Update(id uuid.UUID, name string, done bool) e
_, err := ts.db.Exec(context.Background(), "UPDATE todo SET name=$1, done=$2 WHERE id=$3", name, done, id) _, err := ts.db.Exec(context.Background(), "UPDATE todo SET name=$1, done=$2 WHERE id=$3", name, done, id)
if err != nil { if err != nil {
log.Fatal(err) log.Printf("Database error: %v", err)
} }
return err return err
@ -61,7 +60,7 @@ func (ts *PostgresTodoRepository) Add(todo models.Todo) error {
_, err := ts.db.Exec(context.Background(), "INSERT INTO todo(id, name, done) values($1, $2, $3)", todo.Id, todo.Name, todo.Done) _, err := ts.db.Exec(context.Background(), "INSERT INTO todo(id, name, done) values($1, $2, $3)", todo.Id, todo.Name, todo.Done)
if err != nil { if err != nil {
log.Fatal(err) log.Printf("Database error: %v", err)
return err return err
} }
@ -72,7 +71,7 @@ func (ts *PostgresTodoRepository) Delete(id uuid.UUID) error {
_, err := ts.db.Exec(context.Background(), "DELETE FROM todo where id=$1", id) _, err := ts.db.Exec(context.Background(), "DELETE FROM todo where id=$1", id)
if err != nil { if err != nil {
log.Fatal(err) log.Printf("Database error: %v", err)
return err return err
} }

View File

@ -54,14 +54,14 @@ func (ts *TodoService) UpdateTodo(id uuid.UUID, name string, done bool) (models.
return ts.db.Get(id) return ts.db.Get(id)
} }
func (ts *TodoService) GetTodos() (todos []models.Todo, err error) { func (ts *TodoService) GetTodos() ([]models.Todo, error) {
return ts.db.List() return ts.db.List()
} }
func (ts *TodoService) GetTodoById(id uuid.UUID) (todo models.Todo, err error) { func (ts *TodoService) GetTodoById(id uuid.UUID) (models.Todo, error) {
return ts.db.Get(id) return ts.db.Get(id)
} }
func (ts *TodoService) DeleteTodoById(id uuid.UUID) (err error) { func (ts *TodoService) DeleteTodoById(id uuid.UUID) error {
return ts.db.Delete(id) return ts.db.Delete(id)
} }