From 862611abce91fda4c5821bac68ca64df92e64bc3 Mon Sep 17 00:00:00 2001 From: DavidLazarescu Date: Wed, 13 Dec 2023 12:48:32 +0100 Subject: [PATCH] Added safety to prevent duplication error when getting file hash fails --- src/application/services/library_service.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/application/services/library_service.cpp b/src/application/services/library_service.cpp index e61ae6954..529ccb0b0 100644 --- a/src/application/services/library_service.cpp +++ b/src/application/services/library_service.cpp @@ -120,7 +120,9 @@ BookOperationStatus LibraryService::addBook(const QString& filePath, auto bookMetaData = m_bookMetadataHelper->getBookMetaData(); Book book(filePath, bookMetaData); - if(!allowDuplicates && bookWithFileHashAlreadyExists(book.getFileHash())) + auto hash = book.getFileHash(); + if(!allowDuplicates && !hash.isEmpty() && + bookWithFileHashAlreadyExists(hash)) { qWarning() << QString("Book with file hash: %1 already exists.") .arg(book.getFileHash());