owen/lms
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Minor cleanup

Browse Source
This commit is contained in:
emeric
2025-06-23 20:26:12 +02:00
parent 6bfc769cab
commit c1b73f9b76
2 changed files with 5 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/libs/services/scanner/impl/steps/ScanStepAssociateTrackImages.cpp
+3 -7
View File
@@ -126,11 +126,7 @@ namespace lms::scanner
// Fallback on the artwork already resolved for the release
if (const db::Release::pointer release{ db::Release::find(searchContext.session, releaseId) })
{
db::ArtworkId artworkId{ release->getPreferredArtworkId() };
if (artworkId.isValid())
res = artworkId;
}
res = release->getPreferredArtworkId();
return res;
}
@@ -203,7 +199,7 @@ namespace lms::scanner
artwork = db::Artwork::find(session, *artworkId);
// Using track.modify() is quite CPU intensive as the track class has too many fields
db::Track::updatePreferredArtwork(session, track->getId(), artwork->getId());
db::Track::updatePreferredArtwork(session, track->getId(), artwork ? artwork->getId() : db::ArtworkId{});
if (artwork)
LMS_LOG(DBUPDATER, DEBUG, "Updated preferred artwork in track " << track->getAbsoluteFilePath() << " with image in " << utils::toPath(session, artwork->getId()));
else
@@ -219,7 +215,7 @@ namespace lms::scanner
artwork = db::Artwork::find(session, *artworkId);
// Using track.modify() is quite CPU intensive as the track class has too many fields
db::Track::updatePreferredMediaArtwork(session, track->getId(), artwork->getId());
db::Track::updatePreferredMediaArtwork(session, track->getId(), artwork ? artwork->getId() : db::ArtworkId{});
if (artwork)
LMS_LOG(DBUPDATER, DEBUG, "Updated preferred media artwork in track '" << track->getAbsoluteFilePath() << "' with image in " << utils::toPath(session, artwork->getId()));
else
src/libs/services/scanner/impl/steps/ScanStepCheckForRemovedFiles.cpp
+2 -5
View File
@@ -36,11 +36,6 @@
namespace lms::scanner
{
namespace
{
constexpr std::size_t batchSize = 100;
}
bool ScanStepCheckForRemovedFiles::needProcess([[maybe_unused]] const ScanContext& context) const
{
// always check for removed files
@@ -90,6 +85,8 @@ namespace lms::scanner
objectsToRemove.clear();
{
constexpr std::size_t batchSize = 100;
auto transaction{ session.createReadTransaction() };
endReached = true;