update member from one to several ids

api/eventHandler.go

@@ -25,8 +25,7 @@ func (a *APIHandler) StartNewEvent(c *gin.Context) {
 
 	if err := a.DbHandler.StartNewEvent(name); err != nil {
 		a.logger.Error("StartNewEvent", err)
-
-		c.JSON(http.StatusInternalServerError, nil)
+		c.JSON(http.StatusBadRequest, gin.H{"message": err.Error()})
 		return
 	}
 
@@ -115,7 +114,7 @@ func (a *APIHandler) AddNewAttendees(c *gin.Context) {
 
 	err = a.DbHandler.AddAttendeesToEvent(event)
 	if err != nil {
-		a.logger.Error("DeleteEvent", err)
+		a.logger.Error("AddNewAttendees", err)
 
 		c.JSON(http.StatusInternalServerError, gin.H{
 			"message": err.Error(),
@@ -147,7 +146,7 @@ func (a *APIHandler) DeleteAttendee(c *gin.Context) {
 
 	err = a.DbHandler.DeleteAttendeesFromEvent(event)
 	if err != nil {
-		a.logger.Error("DeleteAttendeesFromEvent", err)
+		a.logger.Error("DeleteAttendee", err)
 		c.JSON(http.StatusInternalServerError, gin.H{
 			"message": err.Error(),
 		})

api/membersHandler.go

@@ -93,50 +93,33 @@ func (a *APIHandler) GetMember(c *gin.Context) {
 	c.JSON(http.StatusOK, members)
 }
 
-func (a *APIHandler) EditMember(c *gin.Context) {
+func (a *APIHandler) UpdateMember(c *gin.Context) {
 	if !a.DBHandlerIsInitialized() {
 		a.logger.Error("EditMember", "database handler is not initialized")
 		c.JSON(http.StatusInternalServerError, nil)
 		return
 	}
 
-	var i int
-	var err error
-
-	id := c.Query("id")
-	if id != "" {
-		i, err = strconv.Atoi(id)
-		if err != nil {
-			a.logger.Error("EditMember", err)
-			c.JSON(http.StatusInternalServerError, nil)
-			return
-		}
-	} else {
-		a.logger.Error("EditMember", err)
-		c.JSON(http.StatusBadRequest, gin.H{
-			"message": "query parameter 'id' missing",
-		})
-		return
-	}
-
-	var member models.Member
-	err = c.BindJSON(&member)
+	var members []models.Member
+	err := c.BindJSON(&members)
 	if err != nil {
 		a.logger.Error("EditMember", err)
 		c.JSON(http.StatusInternalServerError, nil)
 		return
 	}
 
-	err = a.DbHandler.UpdateMember(i, member)
-	if err != nil {
-		a.logger.Error("EditMember", err)
-		c.JSON(http.StatusBadRequest, gin.H{
-			"message": err.Error(),
-		})
-		return
+	for _, member := range members {
+		err = a.DbHandler.UpdateMember(member.Id, member)
+		if err != nil {
+			a.logger.Error("EditMember", err)
+			c.JSON(http.StatusBadRequest, gin.H{
+				"message": err.Error(),
+			})
+			return
+		}
 	}
 	c.JSON(http.StatusOK, gin.H{
-		"message": "member updated",
+		"message": "member(s) updated",
 	})
 }
 

handlers/events.go

@@ -13,24 +13,28 @@ func (dh *DatabaseHandler) StartNewEvent(name string) error {
 		return errors.New("database not opened")
 	}
 
+	if nil == dh.database.Exists(&models.Event{}, "name", name, false) {
+		return errors.New("event with name: " + name + " exists already")
+	}
+
 	return dh.database.AddNewColum(&models.Event{
 		Name: name,
 		Date: time.Now().Format("2006-01-02 15:04:05"),
 	})
 }
 
-func (dh *DatabaseHandler) GetEvent(id uint) (attendees []models.Event, err error) {
+func (dh *DatabaseHandler) GetEvent(id uint) (event []models.Event, err error) {
 	if !dh.DatabaseOpened() {
-		return attendees, errors.New("database not opened")
+		return event, errors.New("database not opened")
 	}
 
-	err = dh.database.GetById(&attendees, id)
+	err = dh.database.GetById(&event, id)
 	if err != nil {
 		return
 	}
 
-	for i := range attendees {
-		attendees[i].Count = len(attendees[i].Attendees)
+	for i := range event {
+		event[i].Count = len(event[i].Attendees)
 	}
 	return
 }
@@ -53,7 +57,7 @@ func (dh *DatabaseHandler) AddAttendeesToEvent(newEvent models.Event) error {
 	}
 
 	var event models.Event
-	err := dh.database.GetById(&event, uint(event.Id))
+	err := dh.database.GetById(&event, uint(newEvent.Id))
 	if err != nil {
 		return err
 	}
@@ -70,7 +74,7 @@ next:
 
 	event.Count = len(event.Attendees)
 
-	return dh.database.UpdateValuesById(&event, uint(event.Id))
+	return dh.database.UpdateValuesById(&event, uint(newEvent.Id))
 }
 
 func (dh *DatabaseHandler) DeleteAttendeesFromEvent(newEvent models.Event) error {
@@ -79,7 +83,7 @@ func (dh *DatabaseHandler) DeleteAttendeesFromEvent(newEvent models.Event) error
 	}
 
 	var event models.Event
-	dh.database.GetById(&event, uint(event.Id))
+	dh.database.GetById(&event, uint(newEvent.Id))
 
 next:
 	for _, newAttendee := range newEvent.Attendees {
@@ -91,5 +95,5 @@ next:
 		}
 	}
 	event.Count = len(event.Attendees)
-	return dh.database.UpdateValuesById(&event, uint(event.Id))
+	return dh.database.UpdateValuesById(&event, uint(newEvent.Id))
 }

member_test.go

@@ -44,7 +44,7 @@ func TestErrors(t *testing.T) {
 
 	v1.POST("/database/open", apiHandler.OpenDatabase)
 	v1.POST("/members/add", apiHandler.AddNewMember)
-	v1.POST("/members/edit", apiHandler.EditMember)
+	v1.POST("/members/edit", apiHandler.UpdateMember)
 	v1.POST("/members/delete", apiHandler.DeleteMember)
 	v1.POST("/members/import/csv", apiHandler.ImportCSV)