From ef074752d0944fcfca9abe60795a1b0d448489c9 Mon Sep 17 00:00:00 2001 From: kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> Date: Sat, 4 Mar 2023 14:53:21 +0000 Subject: [PATCH] use updateattachment when updating to ensure cache is invalidated (#1587) Signed-off-by: kim --- internal/processing/media/update.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/processing/media/update.go b/internal/processing/media/update.go index cf49168f0..ba5497728 100644 --- a/internal/processing/media/update.go +++ b/internal/processing/media/update.go @@ -45,7 +45,7 @@ func (p *Processor) Update(ctx context.Context, account *gtsmodel.Account, media return nil, gtserror.NewErrorNotFound(errors.New("attachment not owned by requesting account")) } - updatingColumns := []string{} + var updatingColumns []string if form.Description != nil { attachment.Description = text.SanitizePlaintext(*form.Description) @@ -62,7 +62,7 @@ func (p *Processor) Update(ctx context.Context, account *gtsmodel.Account, media updatingColumns = append(updatingColumns, "focus_x", "focus_y") } - if err := p.state.DB.UpdateByID(ctx, attachment, attachment.ID, updatingColumns...); err != nil { + if err := p.state.DB.UpdateAttachment(ctx, attachment, updatingColumns...); err != nil { return nil, gtserror.NewErrorInternalError(fmt.Errorf("database error updating media: %s", err)) }