fix: Get, GetAll handler

Author: KunalKumar-1

server/cmd/api/events.go

@@ -1,6 +1,7 @@
 package main
 
 import (
+	"fmt"
 	"net/http"
 	"strconv"
 
@@ -14,14 +15,18 @@ func (app *application) createEvent(c *gin.Context) {
 	var event database.Event
 
 	if err := c.ShouldBindJSON(&event); err != nil {
+		fmt.Println("Bind error:", err)
 		c.JSON(http.StatusBadRequest, gin.H{
 			"error": err.Error(),
 		})
 		return
 	}
 
+	// Insert into db
 	err := app.models.Events.Insert(&event)
+
 	if err != nil {
+		fmt.Println("DB insert error:", err)
 		c.JSON(http.StatusInternalServerError, gin.H{
 			"error": "Failed to create event",
 		})
@@ -82,7 +87,7 @@ func (app *application) updateEvent(c *gin.Context) {
 	existingEvent, err := app.models.Events.Get(id)
 	if err != nil {
 		c.JSON(http.StatusInternalServerError, gin.H{
-			"error": "Failed to retireve event",
+			"error": "Failed to retireve event " + err.Error(),
 		})
 		return
 	}
@@ -96,10 +101,11 @@ func (app *application) updateEvent(c *gin.Context) {
 
 	updatedEvent := &database.Event{}
 
-	if err := c.ShouldBind(updatedEvent); err != nil {
+	if err := c.ShouldBindJSON(updatedEvent); err != nil {
 		c.JSON(http.StatusBadGateway, gin.H{
 			"error": err.Error(),
 		})
+		return
 	}
 
 	updatedEvent.Id = id
@@ -110,6 +116,9 @@ func (app *application) updateEvent(c *gin.Context) {
 		})
 		return
 	}
+
+	// Return the updated event
+	c.JSON(http.StatusOK, updatedEvent)
 }
 
 func (app *application) deleteEvent(c *gin.Context) {

server/cmd/api/routes.go

@@ -17,7 +17,7 @@ func (app *application) routes() http.Handler {
 		v1.PUT("/events/:id", app.updateEvent)    // update event by id
 		v1.DELETE("/events/:id", app.deleteEvent) //delete the event by id
 
-		v1.POST("/register", app.registerUser) // register user
+		v1.POST("/auth/register", app.registerUser) // register user
 	}
 
 	return r

server/data.db

@@ -3,6 +3,7 @@ package database
 import (
 	"context"
 	"database/sql"
+	"fmt"
 	"time"
 )
 
@@ -11,21 +12,26 @@ type EventModel struct {
 }
 
 type Event struct {
-	Id          int       `json:"id"`
-	OwnerId     int       `json:"ownerId" binding:"required"`
-	Name        string    `json:"name" binding:"required, min=3, max=50"`
-	Description string    `json:"description" binding:"required, min=3, max=500"`
-	Date        time.Time `json:"date" binding:"required" datetime:"2003-02-12"`
-	Location    string    `json:"location" binding:"required, min=3"`
+	Id          int    `json:"id"`
+	OwnerId     int    `json:"ownerId" binding:"required"`
+	Name        string `json:"name" binding:"required,min=3,max=50"`
+	Description string `json:"description" binding:"required,min=3,max=500"`
+	Date        string `json:"date" binding:"required"`
+	Location    string `json:"location" binding:"required,min=3"`
 }
 
 func (e *EventModel) Insert(event *Event) error {
 	ctx, cancel := context.WithTimeout(context.Background(), time.Second*3)
 	defer cancel()
 
-	query := `INSERT INTO events(owner_id, name, description, date, location) VALUES($1, $2, $3, $4, $5)`
+	parsedDate, err := time.Parse("2006-01-02", event.Date)
+	if err != nil {
+		return fmt.Errorf("invalid date format: %w", err)
+	}
+
+	query := `INSERT INTO events(owner_id, name, description, date, location) VALUES($1, $2, $3, $4, $5) RETURNING id`
 
-	return e.Db.QueryRowContext(ctx, query, event.OwnerId, event.Name, event.Description, event.Date, event.Location).Scan(&event.Id)
+	return e.Db.QueryRowContext(ctx, query, event.OwnerId, event.Name, event.Description, parsedDate, event.Location).Scan(&event.Id)
 }
 
 func (e *EventModel) GetAll() ([]*Event, error) {
@@ -66,7 +72,7 @@ func (e *EventModel) Get(id int) (*Event, error) {
 	query := `SELECT * FROM events WHERE id = $1`
 	var event Event
 
-	err := e.Db.QueryRowContext(ctx, query, id).Scan(&event, id, &event.OwnerId, &event.Name, &event.Description, &event.Date, &event.Location)
+	err := e.Db.QueryRowContext(ctx, query, id).Scan(&event.Id, &event.OwnerId, &event.Name, &event.Description, &event.Date, &event.Location)
 	if err != nil {
 		if err == sql.ErrNoRows {
 			return nil, nil
@@ -80,8 +86,13 @@ func (e *EventModel) Update(event *Event) error {
 	ctx, cancel := context.WithTimeout(context.Background(), time.Second*3)
 	defer cancel()
 
+	parsedDate, err := time.Parse("2006-01-02", event.Date)
+	if err != nil {
+		return fmt.Errorf("invalid date format: %w", err)
+	}
+
 	query := `UPDATE events SET name = $2, description = $3, date = $4, location = $5 WHERE id = $1`
-	_, err := e.Db.ExecContext(ctx, query, event.Id, event.Name, event.Description, event.Date, event.Location)
+	_, err = e.Db.ExecContext(ctx, query, event.Id, event.Name, event.Description, parsedDate, event.Location)
 	if err != nil {
 		return err
 	}

server/internals/database/event.go

@@ -1 +1 @@
-exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1
+exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1exit status 1