From 5c64dc9c4d8d0b25c75360bba9a090f0af8251cd Mon Sep 17 00:00:00 2001
From: Jonas Kvinge <jonas@jkvinge.net>
Date: Sun, 5 Jan 2025 03:50:51 +0100
Subject: [PATCH] MoodbarItemDelegate: Delete data if it fails to insert to
 cache

---
 src/moodbar/moodbaritemdelegate.cpp | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/moodbar/moodbaritemdelegate.cpp b/src/moodbar/moodbaritemdelegate.cpp
index 257943da2..b8052ffab 100644
--- a/src/moodbar/moodbaritemdelegate.cpp
+++ b/src/moodbar/moodbaritemdelegate.cpp
@@ -36,6 +36,7 @@
 #include <QPainter>
 #include <QRect>
 
+#include "core/logging.h"
 #include "core/settings.h"
 #include "playlist/playlist.h"
 #include "playlist/playlistview.h"
@@ -113,7 +114,11 @@ QPixmap MoodbarItemDelegate::PixmapForIndex(const QModelIndex &idx, const QSize
   }
   else {
     data = new Data;
-    if (!data_.insert(url, data)) return QPixmap();
+    if (!data_.insert(url, data)) {
+      qLog(Error) << "Could not insert moodbar data for URL" << url << "into cache";
+      delete data;
+      return QPixmap();
+    }
   }
 
   data->indexes_.insert(idx);