From 8196aac3debb2be6ff8a0dd84fad613be809e65b Mon Sep 17 00:00:00 2001
From: LittleSheep <littlesheep.code@hotmail.com>
Date: Mon, 29 Jul 2024 21:11:42 +0800
Subject: [PATCH] :sparkles: Change update attachment policy to prevent
 conflict with analyzer

---
 pkg/internal/server/api/attachments_api.go | 8 +++-----
 pkg/internal/services/attachments.go       | 2 +-
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/pkg/internal/server/api/attachments_api.go b/pkg/internal/server/api/attachments_api.go
index 988e977..659b444 100644
--- a/pkg/internal/server/api/attachments_api.go
+++ b/pkg/internal/server/api/attachments_api.go
@@ -140,10 +140,9 @@ func updateAttachmentMeta(c *fiber.Ctx) error {
 	user := c.Locals("user").(models.Account)
 
 	var data struct {
-		Alternative string         `json:"alt"`
-		Usage       string         `json:"usage"`
-		Metadata    map[string]any `json:"metadata"`
-		IsMature    bool           `json:"is_mature"`
+		Alternative string `json:"alt"`
+		Usage       string `json:"usage"`
+		IsMature    bool   `json:"is_mature"`
 	}
 
 	if err := exts.BindAndValidate(c, &data); err != nil {
@@ -157,7 +156,6 @@ func updateAttachmentMeta(c *fiber.Ctx) error {
 
 	attachment.Alternative = data.Alternative
 	attachment.Usage = data.Usage
-	attachment.Metadata = data.Metadata
 	attachment.IsMature = data.IsMature
 
 	if attachment, err := services.UpdateAttachment(attachment); err != nil {
diff --git a/pkg/internal/services/attachments.go b/pkg/internal/services/attachments.go
index e74e621..bc0d26d 100644
--- a/pkg/internal/services/attachments.go
+++ b/pkg/internal/services/attachments.go
@@ -113,7 +113,7 @@ func TryLinkAttachment(tx *gorm.DB, og models.Attachment, hash string) (bool, er
 }
 
 func UpdateAttachment(item models.Attachment) (models.Attachment, error) {
-	if err := database.C.Save(&item).Error; err != nil {
+	if err := database.C.Updates(&item).Error; err != nil {
 		return item, err
 	} else {
 		MaintainAttachmentCache()