pkg/internal/services/programs.go

@@ -82,14 +82,15 @@ func ChargeForProgram(member models.ProgramMember) error {
 
 func PeriodicChargeProgramFee() {
 	var members []models.ProgramMember
-	// Every month paid once
+	if err := database.C.Preload("Program").Find(&members).Error; err != nil {
-	if err := database.C.Where("last_paid IS NULL OR last_paid < ?", time.Now().AddDate(0, 0, -30)).
-		Preload("Program").Find(&members).Error; err != nil {
 		return
 	}
 	for _, member := range members {
-		if err := ChargeForProgram(member); err == nil {
+		// every month paid once
-			database.C.Model(&member).Update("last_paid", time.Now())
+		if member.LastPaid == nil || time.Since(*member.LastPaid) < time.Hour*24*30 {
+			if err := ChargeForProgram(member); err == nil {
+				database.C.Model(&member).Update("last_paid", time.Now())
+			}
 		}
 	}
 }

pkg/internal/web/api/accounts_api.go

@@ -113,7 +113,7 @@ func getUserinfo(c *fiber.Ctx) error {
 	return c.JSON(resp)
 }
 
-func editUserinfo(c *fiber.Ctx) error {
+func updateUserinfo(c *fiber.Ctx) error {
 	if err := exts.EnsureAuthenticated(c); err != nil {
 		return err
 	}

pkg/internal/web/api/avatar_api.go

@@ -26,7 +26,9 @@ func setAvatar(c *fiber.Ctx) error {
 	}
 
 	og := user.Avatar
-	if err := database.C.Model(&user).Update("avatar", data.AttachmentID).Error; err != nil {
+	user.Avatar = &data.AttachmentID
+
+	if err := database.C.Save(&user).Error; err != nil {
 		return fiber.NewError(fiber.StatusInternalServerError, err.Error())
 	} else {
 		services.AddEvent(user.ID, "profile.edit.avatar", nil, c.IP(), c.Get(fiber.HeaderUserAgent))
@@ -62,7 +64,9 @@ func setBanner(c *fiber.Ctx) error {
 	}
 
 	og := user.Banner
-	if err := database.C.Model(&user).Update("banner", data.AttachmentID).Error; err != nil {
+	user.Banner = &data.AttachmentID
+
+	if err := database.C.Save(&user).Error; err != nil {
 		return fiber.NewError(fiber.StatusInternalServerError, err.Error())
 	} else {
 		services.AddEvent(user.ID, "profile.edit.banner", nil, c.IP(), c.Get(fiber.HeaderUserAgent))

pkg/internal/web/api/index.go

@@ -67,7 +67,7 @@ func MapControllers(app *fiber.App, baseURL string) {
 
 			me.Get("/", getUserinfo)
 			me.Get("/oidc", getUserinfoForOidc)
-			me.Put("/", editUserinfo)
+			me.Put("/", updateUserinfo)
 			me.Put("/language", updateAccountLanguage)
 			me.Get("/events", getEvents)
 			me.Get("/tickets", getTickets)