From a5750deaaf824e99fec54848640c21d5395dd626 Mon Sep 17 00:00:00 2001
From: "Tomasz N." <tomasz.nowakowski@smartkyc.com>
Date: Wed, 22 Oct 2025 23:02:49 +0200
Subject: [PATCH] The key change: Move the Database.bookAuthorModel.create()
 block outside the if (!author) check, so it runs whether the author was just
 created OR already existed in the database.

This bug was visible when using "Match Books" for a library and the outcome was books had no author(s) assigned
despite the custom providers correctly providing those values.
---
 server/scanner/Scanner.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/server/scanner/Scanner.js b/server/scanner/Scanner.js
index 206068cc4..cf5c87b78 100644
--- a/server/scanner/Scanner.js
+++ b/server/scanner/Scanner.js
@@ -259,7 +259,7 @@ class Scanner {
             SocketAuthority.emitter('author_added', author.toOldJSON())
             // Update filter data
             Database.addAuthorToFilterData(libraryItem.libraryId, author.name, author.id)
-
+          }
             await Database.bookAuthorModel
               .create({
                 authorId: author.id,
@@ -270,7 +270,6 @@ class Scanner {
                 libraryItem.media.authors.push(author)
                 hasAuthorUpdates = true
               })
-          }
         }
         const authorsRemoved = libraryItem.media.authors.filter((a) => !matchData.author.find((ma) => ma.toLowerCase() === a.name.toLowerCase()))
         if (authorsRemoved.length) {