IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

MM-56774: Delete file along with bookmark (#27495)

Browse Source
This commit is contained in:
Caleb Roseland 2024-07-03 09:31:39 -04:00 committed by GitHub
parent 762ff9b96c
commit 90534b13cf
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
server/channels/store/sqlstore/channel_bookmark_store.go
View File

@ -317,7 +317,6 @@ func (s *SqlChannelBookmarkStore) Delete(bookmarkId string, deleteFile bool) err
From("ChannelBookmarks").
Where(sq.And{
sq.Eq{"Id": bookmarkId},
sq.Eq{"DeleteAt": 0},
})
fileQuery, fileArgs, fileErr := s.getQueryBuilder().

5
server/channels/store/storetest/channel_bookmark.go
View File

@ -434,6 +434,11 @@ func testDeleteChannelBookmark(t *testing.T, rctx request.CTX, ss store.Store) {
err = ss.ChannelBookmark().Delete(bookmark2.Id, true)
assert.NoError(t, err)
_, err = ss.FileInfo().Get(file.Id)
assert.Error(t, err)
var nfErr *store.ErrNotFound
assert.ErrorAs(t, err, &nfErr)
bookmarks, err := ss.ChannelBookmark().GetBookmarksForChannelSince(channelId, now)
assert.NoError(t, err)
assert.Len(t, bookmarks, 2) // we have two as the deleted record also gets returned for sync'ing purposes