fix wrong database exists check

dbHandler.go

@@ -24,8 +24,9 @@ import (
 //   - db:     Active GORM database connection.
 //   - logger: Pointer to a custom logger instance for structured logging.
 type DBHandler struct {
-	db     *gorm.DB
-	logger *logging.Logger
+	db           *gorm.DB
+	logger       *logging.Logger
+	NewCreatedDB bool // indicater for new created database
 }
 
 // NewDBHandler
@@ -68,6 +69,11 @@ func NewDBHandler(name, path string, logger *logging.Logger) (dH *DBHandler, err
 	}
 
 	dbPath := filepath.Join(path, name)
+
+	//indicator if database exists
+	_, err = os.Stat(dbPath)
+	fmt.Println(1, err)
+	dH.NewCreatedDB = err != nil
 	logger.Debug("NewDBHandler", "open database "+dbPath)
 	dH.db, err = gorm.Open(sqlite.Open(dbPath), &gorm.Config{})
 	return
@@ -129,14 +135,21 @@ func (dH *DBHandler) AddNewColum(model any) error {
 //
 // Returns:
 //   - error: Any query error or “not found” message.
-func (dH *DBHandler) GetById(model any, id uint) error {
+func (dH *DBHandler) GetById(model any, id uint, relations ...string) (err error) {
 	dH.logger.Debug("getById", "find id "+fmt.Sprint(id))
 
-	if id == 0 {
-		return dH.db.Find(model).Error
+	query := dH.db
+
+	for _, relation := range relations {
+		query = query.Preload(relation)
 	}
 
-	err := dH.db.First(model, id).Error
+	if id == 0 {
+		return query.Find(model).Error
+	}
+
+	err = query.First(model, id).Error
+
 	if errors.Is(err, gorm.ErrRecordNotFound) {
 		return fmt.Errorf("no record found for id: %v", id)
 	} else if err != nil {
@@ -163,15 +176,20 @@ func (dH *DBHandler) GetById(model any, id uint) error {
 //
 // Returns:
 //   - error: Any database query error.
-func (dH *DBHandler) GetByKey(model any, key string, value any, LikeSearch bool) error {
+func (dH *DBHandler) GetByKey(model any, key string, value any, LikeSearch bool, relations ...string) error {
+	query := dH.db
+
+	for _, relation := range relations {
+		query = query.Preload(relation)
+	}
 	if LikeSearch {
 		value = strings.ReplaceAll(fmt.Sprint(value), "*", "%")
 		dH.logger.Debug("getByKey", "find like key "+key+" value "+fmt.Sprint(value))
-		return dH.db.Where(key+" LIKE ?", value).Find(model).Error
+		return query.Where(key+" LIKE ?", value).Find(model).Error
 	}
 
 	dH.logger.Debug("getByKey", "find equal key "+key+" value "+fmt.Sprint(value))
-	return dH.db.Find(model, key+" = ?", value).Error
+	return query.Find(model, key+" = ?", value).Error
 }
 
 // updateValuesById
@@ -191,18 +209,27 @@ func (dH *DBHandler) GetByKey(model any, key string, value any, LikeSearch bool)
 //
 // Returns:
 //   - error: If the model is invalid or query/update fails.
-func (dH *DBHandler) UpdateValuesById(model any, id uint) error {
-	dH.logger.Debug("updateValuesById", "model"+fmt.Sprint(model))
+func (dH *DBHandler) UpdateValuesById(model any, id uint, relations ...string) error {
+	dH.logger.Debug("updateValuesByKey", "model"+fmt.Sprint(model))
+
 	modelType := reflect.TypeOf(model)
 	if modelType.Kind() != reflect.Ptr {
 		return errors.New("model must be a pointer to struct")
 	}
 
 	lookUpModel := reflect.New(modelType.Elem()).Interface()
-	if err := dH.GetById(lookUpModel, id); err != nil {
+	if err := dH.GetById(lookUpModel, id, relations...); err != nil {
 		return err
 	}
-	return dH.db.Model(lookUpModel).Updates(model).Error
+
+	// Preload relations
+	query := dH.db
+	for _, relation := range relations {
+		query = query.Preload(relation)
+	}
+
+	// Update only provided keys
+	return query.Model(lookUpModel).Updates(model).Error
 }
 
 // updateValuesByKey
@@ -223,7 +250,7 @@ func (dH *DBHandler) UpdateValuesById(model any, id uint) error {
 //
 // Returns:
 //   - error: Any query or update error.
-func (dH *DBHandler) UpdateValuesByKey(model any, key string, value any) error {
+func (dH *DBHandler) UpdateValuesByKey(model any, key string, value any, relations ...string) error {
 	dH.logger.Debug("updateValuesByKey", "model"+fmt.Sprint(model))
 	modelType := reflect.TypeOf(model)
 	if modelType.Kind() != reflect.Ptr {
@@ -231,20 +258,26 @@ func (dH *DBHandler) UpdateValuesByKey(model any, key string, value any) error {
 	}
 
 	lookUpModel := reflect.New(modelType.Elem()).Interface()
-	if err := dH.GetByKey(lookUpModel, key, value, false); err != nil {
+	if err := dH.GetByKey(lookUpModel, key, value, false, relations...); err != nil {
 		return err
 	}
-	return dH.db.Model(lookUpModel).Updates(model).Error
+
+	query := dH.db
+
+	for _, relation := range relations {
+		query = query.Preload(relation)
+	}
+	return query.Model(lookUpModel).Updates(model).Error
 }
 
-func (dH *DBHandler) DeleteById(model any, ids ...uint) error {
+func (dH *DBHandler) DeleteById(model any, relation string, ids ...uint) error {
 	if len(ids) == 0 {
 		return fmt.Errorf("no IDs provided")
 	}
 
 	if len(ids) == 1 && ids[0] == 0 {
 		dH.logger.Debug("deleteById", "delete ALL records!")
-		return dH.db.Session(&gorm.Session{AllowGlobalUpdate: true}).
+		return dH.db.Session(&gorm.Session{AllowGlobalUpdate: true, FullSaveAssociations: true}).
 			Delete(model).Error
 	}
 
@@ -273,7 +306,7 @@ func (dH *DBHandler) DeleteById(model any, ids ...uint) error {
 //
 // Returns:
 //   - error: Any deletion error.
-func (dH *DBHandler) DeleteByKey(model any, key string, value any, LikeSearch bool) error {
+func (dH *DBHandler) DeleteByKey(model any, key string, value any, LikeSearch bool, relations ...string) error {
 	if LikeSearch {
 		value = strings.ReplaceAll(fmt.Sprint(value), "*", "%")
 		dH.logger.Debug("deleteByKey", "delete like key "+key+" value "+fmt.Sprint(value))
@@ -281,10 +314,17 @@ func (dH *DBHandler) DeleteByKey(model any, key string, value any, LikeSearch bo
 	}
 
 	dH.logger.Debug("deleteByKey", "delete equal key "+key+" value "+fmt.Sprint(value))
-	return dH.db.Where(key+" = ?", value).Delete(model).Error
+
+	query := dH.db
+
+	for _, relation := range relations {
+		query = query.Preload(relation)
+	}
+
+	return query.Where(key+" = ?", value).Delete(model).Error
 }
 
-func (dH *DBHandler) Exists(model any, key string, value any, likeSearch bool) (found bool) {
+func (dH *DBHandler) Exists(model any, key string, value any, likeSearch bool, relations ...string) (found bool) {
 	var query string
 	var args any
 
@@ -313,7 +353,12 @@ func (dH *DBHandler) Exists(model any, key string, value any, likeSearch bool) (
 
 	dH.logger.Debug("exists", "checking existence for key "+query+" value "+fmt.Sprint(args))
 
-	tx := dH.db.Where(query, args).Find(model)
+	datababase := dH.db
+	for _, relation := range relations {
+		datababase = datababase.Preload(relation)
+	}
+
+	tx := datababase.Where(query, args).Find(model)
 
 	if tx.Error != nil {
 		dH.logger.Error("Exists", fmt.Sprintf("query failed: %v", tx.Error))
@@ -322,23 +367,10 @@ func (dH *DBHandler) Exists(model any, key string, value any, likeSearch bool) (
 	return tx.RowsAffected > 0
 }
 
-func (dH *DBHandler) GetByIdWithRelation(model any, relation string, id uint) error {
-	if id == 0 {
-		return dH.db.Preload(relation).Find(model).Error
-	}
-
-	// Load member AND its events
-	err := dH.db.Preload(relation).First(model, id).Error
-
-	if errors.Is(err, gorm.ErrRecordNotFound) {
-		return fmt.Errorf("no record found for id: %v", id)
-	} else if err != nil {
-		return fmt.Errorf("query failed: %w", err)
-	}
-	return nil
-}
-
 func (dH *DBHandler) AddRelation(model, relation any, relationName string) error {
 	return dH.db.Model(model).Association(relationName).Append(relation)
-
+}
+
+func (dH *DBHandler) DeleteRelation(model, relation any, relationName string) error {
+	return dH.db.Model(model).Association(relationName).Delete(relation)
 }

db_test.go

@@ -12,10 +12,12 @@ type Event struct {
 }
 
 type Member struct {
-	Id        int      `gorm:"primaryKey" json:"id"`
-	FirstName string   `gorm:"column:firstName" json:"firstName,omitempty"`
-	LastName  string   `gorm:"column:lastName" json:"lastName,omitempty"`
-	Events    []*Event `gorm:"many2many:member_events;" json:"events"`
+	Id                  int      `gorm:"primaryKey" json:"id"`
+	FirstName           string   `gorm:"column:firstName" json:"firstName,omitempty"`
+	LastName            string   `gorm:"column:lastName" json:"lastName,omitempty"`
+	ResponsiblePersonID *int     `json:"responsiblePersonId"`
+	ResponsiblePerson   *Member  `gorm:"foreignKey:ResponsiblePersonID" json:"responsiblePerson"`
+	Events              []*Event `gorm:"many2many:member_events;" json:"events"`
 }
 
 func TestDbHandler(t *testing.T) {
@@ -33,13 +35,15 @@ func TestDbHandler(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	member := &Member{FirstName: "adrian", LastName: "zuercher"}
-	dbHandler.AddNewColum(member)
-	event := &Event{
-		Name: "testEvent",
+	membersIN := []*Member{{FirstName: "adrian", LastName: "zuercher"}, {FirstName: "adichild", LastName: "zuercher"}}
+	for _, member := range membersIN {
+		dbHandler.AddNewColum(member)
+		event := &Event{
+			Name: "testEvent",
+		}
+		dbHandler.AddNewColum(event)
+		dbHandler.AddRelation(event, member, "Attendees")
 	}
-	dbHandler.AddNewColum(event)
-	dbHandler.AddRelation(event, member, "Attendees")
 
 	var members []Member
 	if err := dbHandler.GetById(&members, 0); err != nil {
@@ -47,7 +51,7 @@ func TestDbHandler(t *testing.T) {
 	}
 	t.Log(members)
 
-	if err := dbHandler.GetByIdWithRelation(&members, "Events", 0); err != nil {
+	if err := dbHandler.GetById(&members, 0, "Events"); err != nil {
 		t.Fatal(err)
 	}
 	t.Log(members)
@@ -61,7 +65,7 @@ func TestDbHandler(t *testing.T) {
 
 	var events []Event
 
-	if err := dbHandler.GetByIdWithRelation(&events, "Attendees", 0); err != nil {
+	if err := dbHandler.GetById(&events, 0, "Attendees"); err != nil {
 		t.Fatal(err)
 	}
 
@@ -72,6 +76,27 @@ func TestDbHandler(t *testing.T) {
 		}
 	}
 
+	//add responsible
+	mem1 := &Member{}
+	if err := dbHandler.GetById(mem1, 2); err != nil {
+		t.Fatal(err)
+	}
+
+	mem2 := &Member{}
+	if err := dbHandler.GetById(mem2, 1); err != nil {
+		t.Fatal(err)
+	}
+
+	mem1.ResponsiblePerson = mem2
+	dbHandler.UpdateValuesById(mem1, uint(mem1.Id))
+
+	if err := dbHandler.GetById(mem1, 2); err != nil {
+		t.Fatal(err)
+	}
+
+	t.Log(mem1)
+	t.Log(mem1.ResponsiblePerson)
+
 	if err := dbHandler.Close(); err != nil {
 		t.Fatal(err)
 	}