vingo: fix settings endpoint

vingo/database/settings.go

@@ -2,11 +2,6 @@ package database
 
 import "log"
 
-func CreateSettings(user_id int) error {
-	_, err := db.Exec("INSERT INTO settings (user_id) VALUES ($1) ON CONFLICT DO NOTHING;", user_id)
-	return err
-}
-
 func GetSettings(user_id int) (*Settings, error) {
 	var user User
 	result := gorm_db.Preload("Settings").First(&user, user_id)
@@ -15,6 +10,17 @@ func GetSettings(user_id int) (*Settings, error) {
 }
 
 func UpdateSettings(user_id int, settings Settings) error {
-	_, err := db.Exec("UPDATE settings SET scan_in_out = $1, leaderboard = $2, public = $3 WHERE user_id = $4;", settings.ScanInOut, settings.Leaderboard, settings.Public, user_id)
-	return err
+	var user User
+	if err := gorm_db.Preload("Settings").First(&user, user_id).Error; err != nil {
+		return err
+	}
+
+	user.Settings.ScanInOut = settings.ScanInOut
+	user.Settings.Leaderboard = settings.Leaderboard
+	user.Settings.Public = settings.Public
+	if err := gorm_db.Save(&user.Settings).Error; err != nil {
+		return err
+	}
+
+	return nil
 }

vingo/handlers/settings.go

@@ -18,8 +18,13 @@ func (Settings) Update(c *fiber.Ctx) error {
 		return c.Status(400).SendString("Invalid payload")
 	}
 
+	err = database.UpdateSettings(user.Id, settings)
+	if err != nil {
+		logger.Println(err)
+		return c.Status(500).SendString("Error updating settings")
+	}
+
 	sess, _ := store.Get(c)
-	database.UpdateSettings(user.Id, settings)
 	user, _ = database.GetUser(user.Id)
 	sess.Set(STORE_USER, &user)
 	sess.Save()