TheAnachronism/docspell
1
0
Fork 0
mirror of https://github.com/TheAnachronism/docspell.git synced 2025-04-04 10:29:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix querying shares without a name

Browse Source 
Fixes: #1840
This commit is contained in:
eikek 2022-11-14 11:07:14 +01:00
parent e8204208cb
commit 05ed96b2cc
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
modules/common/src/main/scala/docspell/common/syntax/StringSyntax.scala
View File

@ -17,6 +17,11 @@ trait StringSyntax {
def parseJsonAs[A](implicit d: Decoder[A]): Either[Throwable, A] =
parser.decode[A](s)
}
implicit class OptionStringOpts(s: Option[String]) {
def asNonBlank: Option[String] =
s.filter(_.trim.nonEmpty)
}
}
object StringSyntax extends StringSyntax

3
modules/restserver/src/main/scala/docspell/restserver/routes/ShareRoutes.scala
View File

@ -14,6 +14,7 @@ import docspell.backend.BackendApp
import docspell.backend.auth.AuthToken
import docspell.backend.ops.OShare
import docspell.backend.ops.OShare.{SendResult, ShareMail, VerifyResult}
import docspell.common.syntax.string._
import docspell.common.{Ident, Timestamp}
import docspell.restapi.model._
import docspell.restserver.Config
@ -37,7 +38,7 @@ object ShareRoutes {
case GET -> Root :? QP.Query(q) :? QP.OwningFlag(owning) =>
val login = if (owning) Some(user.account.login) else None
for {
all <- backend.share.findAll(user.account.collectiveId, login, q)
all <- backend.share.findAll(user.account.collectiveId, login, q.asNonBlank)
now <- Timestamp.current[F]
res <- Ok(ShareList(all.map(mkShareDetail(now))))
} yield res