fix delete function

api/responsibleHandler.go

@@ -24,14 +24,13 @@ func (a *APIHandler) AddNewResponsible(c *gin.Context) {
 		return
 	}
 
-	for _, responsible := range responsibles {
+	err = a.DbHandler.AddNewResponsible(responsibles...)
-		err = a.DbHandler.AddNewResponsible(responsible)
+	if err != nil {
-		if err != nil {
+		c.JSON(http.StatusBadRequest, gin.H{
-			c.JSON(http.StatusBadRequest, gin.H{
+			"message": err.Error(),
-				"message": err.Error(),
+		})
-			})
+		return
-			return
+
-		}
 	}
 
 	c.JSON(http.StatusOK, gin.H{

go.mod

@@ -3,7 +3,7 @@ module gitea.tecamino.com/paadi/memberDB
 go 1.24.5
 
 require (
-	gitea.tecamino.com/paadi/dbHandler v1.0.4
+	gitea.tecamino.com/paadi/dbHandler v1.0.8
 	gitea.tecamino.com/paadi/tecamino-logger v0.2.1
 	github.com/gin-contrib/cors v1.7.6
 	github.com/gin-gonic/gin v1.11.0

go.sum

@@ -1,5 +1,5 @@
-gitea.tecamino.com/paadi/dbHandler v1.0.4 h1:ctnaec0GDdtw3gRQdUISVDYLJ9x+vt50VW41OemfhD4=
+gitea.tecamino.com/paadi/dbHandler v1.0.8 h1:ZWSBM/KFtLwTv2cBqwK1mOxWAxAfL0BcWEC3kJ9JALU=
-gitea.tecamino.com/paadi/dbHandler v1.0.4/go.mod h1:y/xn/POJg1DO++67uKvnO23lJQgh+XFQq7HZCS9Getw=
+gitea.tecamino.com/paadi/dbHandler v1.0.8/go.mod h1:y/xn/POJg1DO++67uKvnO23lJQgh+XFQq7HZCS9Getw=
 gitea.tecamino.com/paadi/tecamino-logger v0.2.1 h1:sQTBKYPdzn9mmWX2JXZBtGBvNQH7cuXIwsl4TD0aMgE=
 gitea.tecamino.com/paadi/tecamino-logger v0.2.1/go.mod h1:FkzRTldUBBOd/iy2upycArDftSZ5trbsX5Ira5OzJgM=
 github.com/bytedance/sonic v1.14.0 h1:/OfKt8HFw0kh2rj8N0F6C/qPGRESq0BbaNZgcNXXzQQ=

handlers/events.go

@@ -13,7 +13,7 @@ func (dh *DatabaseHandler) StartNewEvent(event *models.Event) error {
 		return errors.New("database not opened")
 	}
 
-	if nil == dh.database.Exists(&models.Event{}, "name", event.Name, false) {
+	if dh.database.Exists(&models.Event{}, "name", event.Name, false) {
 		return errors.New("event with name: " + event.Name + " exists already")
 	}
 

handlers/members.go

@@ -91,11 +91,6 @@ func (dh *DatabaseHandler) AddNewMember(members ...models.Member) error {
 			return err
 		}
 
-		encResponsiblePerson, err := crypto.Encrypt(member.ResponsiblePerson, dh.token)
-		if err != nil {
-			return err
-		}
-
 		member.FirstNameHash = dh.hashField(member.FirstName)
 		member.FirstName = encFirstName
 		member.LastNameHash = dh.hashField(member.LastName)
@@ -110,7 +105,6 @@ func (dh *DatabaseHandler) AddNewMember(members ...models.Member) error {
 		member.FirstVisit = encFirstVisit
 		member.LastVisit = encLastVisit
 		member.Group = encGroup
-		member.ResponsiblePerson = encResponsiblePerson
 
 		err = dh.database.AddNewColum(&member)
 		if err != nil {
@@ -198,11 +192,6 @@ func (dh *DatabaseHandler) GetMember(id uint) (members []models.Member, err erro
 		if err != nil {
 			return
 		}
-
-		members[i].ResponsiblePerson, err = crypto.Decrypt(members[i].ResponsiblePerson, dh.token)
-		if err != nil {
-			return
-		}
 	}
 	return
 }
@@ -298,13 +287,6 @@ func (dh *DatabaseHandler) UpdateMember(id int, member models.Member) (err error
 			return err
 		}
 	}
-
-	if member.ResponsiblePerson != "" {
-		member.ResponsiblePerson, err = crypto.Encrypt(member.ResponsiblePerson, dh.token)
-		if err != nil {
-			return err
-		}
-	}
 	return dh.database.UpdateValuesById(&member, uint(member.Id))
 }
 
@@ -315,8 +297,7 @@ func (dh *DatabaseHandler) memberExists(checkMember models.Member) (bool, error)
 	}
 
 	var member models.Member
-	err := dh.database.Exists(&member, "birthdayHash", dh.hashField(checkMember.Birthday), false)
+	if !dh.database.Exists(&member, "birthdayHash", dh.hashField(checkMember.Birthday), false) {
-	if err != nil {
 		return false, nil
 	}
 	return dh.hashField(checkMember.FirstName) == member.FirstNameHash && dh.hashField(checkMember.LastName) == member.LastNameHash && dh.hashField(checkMember.Birthday) == member.BirthdayHash, nil

handlers/responsible.go

@@ -10,11 +10,10 @@ func (dh *DatabaseHandler) AddNewResponsible(responsibles ...models.Person) erro
 	if !dh.DatabaseOpened() {
 		return errors.New("database not opened")
 	}
-	if !dh.DatabaseOpened() {
-		return errors.New("database not opened")
-	}
-	for _, responsible := range responsibles {
 
+	for _, responsible := range responsibles {
+		// set id to 0 so new uniq id is created
+		responsible.Id = 0
 		exists, err := dh.responsibleExists(responsible)
 		if err != nil {
 			return err
@@ -56,8 +55,7 @@ func (dh *DatabaseHandler) responsibleExists(CheckResponsible models.Person) (bo
 	}
 
 	var person models.Person
-	err := dh.database.Exists(&person, "lastName", CheckResponsible.LastName, false)
+	if !dh.database.Exists(&person, "lastName", CheckResponsible.LastName, false) {
-	if err != nil {
 		return false, nil
 	}
 	return person.FirstName == CheckResponsible.FirstName && person.LastName == CheckResponsible.LastName, nil

models/member.go

@@ -16,5 +16,5 @@ type Member struct {
 	FirstVisit        string `gorm:"column:firstVisit" json:"firstVisit,omitempty"`
 	LastVisit         string `gorm:"column:lastVisit" json:"lastVisit,omitempty"`
 	Group             string `gorm:"column:group" json:"group,omitempty"`
-	ResponsiblePerson string `gorm:"column:responsiblePerson" json:"responsiblePerson,omitempty"`
+	ResponsiblePerson Person `gorm:"type:json" json:"responsiblePerson"`
 }

models/person.go

@@ -8,12 +8,6 @@ import (
 
 type Persons []Person
 
-type Person struct {
-	Id        int    `gorm:"primaryKey" json:"id"`
-	FirstName string `gorm:"column:firstName" json:"firstName"`
-	LastName  string `gorm:"column:lastName" json:"lastName"`
-}
-
 // --- Implement driver.Valuer (for saving to DB)
 func (r Persons) Value() (driver.Value, error) {
 	return json.Marshal(r)
@@ -27,3 +21,23 @@ func (r *Persons) Scan(value any) error {
 	}
 	return json.Unmarshal(bytes, r)
 }
+
+type Person struct {
+	Id        int    `gorm:"primaryKey" json:"id"`
+	FirstName string `gorm:"column:firstName" json:"firstName"`
+	LastName  string `gorm:"column:lastName" json:"lastName"`
+}
+
+// --- Implement driver.Valuer (for saving to DB)
+func (r Person) Value() (driver.Value, error) {
+	return json.Marshal(r)
+}
+
+// --- Implement sql.Scanner (for reading from DB)
+func (r *Person) Scan(value any) error {
+	bytes, ok := value.([]byte)
+	if !ok {
+		return fmt.Errorf("failed to unmarshal Settings: %v", value)
+	}
+	return json.Unmarshal(bytes, r)
+}