TheAnachronism/docspell
1
0
Fork 0
mirror of https://github.com/TheAnachronism/docspell.git synced 2025-06-22 02:18:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix personal/non-personal when updating bookmarks

Browse Source
This commit is contained in:
eikek
2022-01-16 00:31:59 +01:00
parent f71a5f2c14
commit 8a4471343c
2 changed files with 25 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
modules/store/src/main/scala/docspell/store/records/RQueryBookmark.scala
View File

@ -95,7 +95,8 @@ object RQueryBookmark {
DML.set(
T.name.setTo(r.name),
T.label.setTo(r.label),
T.query.setTo(r.query)
T.query.setTo(r.query),
T.userId.setTo(r.userId)
)
)
@ -119,9 +120,12 @@ object RQueryBookmark {
).build.query[Int].unique.map(_ > 0)
}
// impl note: store.add doesn't work, because it checks for duplicate
// after trying to insert the check is necessary because a name
// should be unique across personal *and* collective bookmarks
// impl note: store.add doesn't work, because it checks for
// duplicate after trying to insert. the check is necessary because
// a name should be unique across personal *and* collective
// bookmarks, which is not covered by db indexes. This is now
// checked before (and therefore subject to race conditions, but is
// neglected here)
def insertIfNotExists(
account: AccountId,
r: ConnectionIO[RQueryBookmark]