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

Fixed termination when a podcast updates its image url

Browse Source
This commit is contained in:
emeric
2026-05-14 15:23:55 +02:00
parent 7f7c0ee187
commit 24a74489aa
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/libs/services/podcast/impl/steps/RefreshPodcastsStep.cpp
+5 -5
View File
@@ -36,18 +36,18 @@ namespace lms::podcast
{
namespace
{
void removeArtwork(db::Session& session, const db::Artwork::pointer& artwork)
void removeArtwork(const db::Artwork::pointer& artwork)
{
const auto underlyingImageId{ artwork->getUnderlyingId() };
const auto* imageId{ std::get_if<db::ImageId>(&underlyingImageId) };
assert(imageId); // these artworks can only be an image
db::Image::pointer image{ artwork->getImage() };
assert(image); // these artworks can only be an image
std::error_code ec;
std::filesystem::remove(artwork->getAbsoluteFilePath(), ec);
if (ec)
LMS_LOG(PODCAST, WARNING, "Failed to remove old podcast artwork file '" << artwork->getAbsoluteFilePath() << "': " << ec.message());
session.destroy<db::Image>(*imageId);
image.remove();
}
void updatePodcast(db::Session& session, db::PodcastId podcastId, const Podcast& podcast)
@@ -83,7 +83,7 @@ namespace lms::podcast
{
LMS_LOG(PODCAST, INFO, "Podcast '" << podcast.title << "' : image url changed from '" << previousUrl << "' to '" << podcast.imageUrl << "'");
if (db::Artwork::pointer currentArtwork{ dbPodcast->getArtwork() })
removeArtwork(session, currentArtwork);
removeArtwork(currentArtwork);
dbPodcast.modify()->setImageUrl(podcast.imageUrl);
}