diff options
| author |  kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> | 2023-07-18 09:43:17 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2023-07-18 09:43:17 +0100 |
| commit | f4319740ab02d680961781861335285f618f5f48 (patch) | |
| tree | 133595a10ec93cce9da269a4fa671c226bab7298 /internal/processing/list/delete.go | |
| parent | [bugfix] Add missing `continue` statement in `prepareXBetweenIDs` (#1996) (diff) | |
| download | gotosocial-f4319740ab02d680961781861335285f618f5f48.tar.xz | |
[bugfix] more robust list timeline invalidation (#1995)v0.10.0-rc3
Diffstat (limited to 'internal/processing/list/delete.go')
| -rw-r--r-- | internal/processing/list/delete.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/internal/processing/list/delete.go b/internal/processing/list/delete.go index 1c8ee5700..327ac9d16 100644 --- a/internal/processing/list/delete.go +++ b/internal/processing/list/delete.go @@ -27,7 +27,8 @@ import ( // Delete deletes one list for the given account. func (p *Processor) Delete(ctx context.Context, account *gtsmodel.Account, id string) gtserror.WithCode { - list, errWithCode := p.getList( + // Ensure list exists + is owned by requesting account. + _, errWithCode := p.getList( // Use barebones ctx; no embedded // structs necessary for this call. gtscontext.SetBarebones(ctx), @@ -38,7 +39,7 @@ func (p *Processor) Delete(ctx context.Context, account *gtsmodel.Account, id st return errWithCode } - if err := p.state.DB.DeleteListByID(ctx, list.ID); err != nil { + if err := p.state.DB.DeleteListByID(ctx, id); err != nil { return gtserror.NewErrorInternalError(err) } |