TheAnachronism/docspell
1
0
Fork 0
mirror of https://github.com/TheAnachronism/docspell.git synced 2025-03-25 16:45:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

Improve logging when creating preview images

Browse Source
This commit is contained in:
Eike Kettner 2020-11-10 22:25:46 +01:00
parent ed26fe226b
commit 746e04c624
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/joex/src/main/scala/docspell/joex/preview/MakePreviewTask.scala
View File

@ -47,7 +47,7 @@ object MakePreviewTask {
_ <- ra _ <- ra
.map(AttachmentPreview.createPreview(ctx, preview, cfg.chunkSize)) .map(AttachmentPreview.createPreview(ctx, preview, cfg.chunkSize))
.getOrElse( .getOrElse(
ctx.logger.warn(s"No attachment found with id: ${ctx.args.attachment}") ctx.logger.error(s"No attachment found with id: ${ctx.args.attachment}")
) )
} yield () } yield ()

12
modules/joex/src/main/scala/docspell/joex/process/AttachmentPreview.scala
View File

@ -57,13 +57,17 @@ object AttachmentPreview {
case MimeType.PdfMatch(_) => case MimeType.PdfMatch(_) =>
preview.previewPNG(loadFile(ctx)(ra)).flatMap { preview.previewPNG(loadFile(ctx)(ra)).flatMap {
case Some(out) => case Some(out) =>
createRecord(ctx, out, ra, chunkSize).map(_.some) ctx.logger.debug("Preview generated, saving to database…") *>
createRecord(ctx, out, ra, chunkSize).map(_.some)
case None => case None =>
(None: Option[RAttachmentPreview]).pure[F] ctx.logger
.info(s"Preview could not be generated. Maybe the pdf has no pages?") *>
(None: Option[RAttachmentPreview]).pure[F]
} }
case _ => case mt =>
(None: Option[RAttachmentPreview]).pure[F] ctx.logger.warn(s"Not a pdf file, but ${mt.asString}, cannot get page count.") *>
(None: Option[RAttachmentPreview]).pure[F]
} }
private def createRecord[F[_]: Sync]( private def createRecord[F[_]: Sync](