1 files changed, 12 insertions, 23 deletions
diff --git a/src/plugins/General/library/librarymodel.cpp b/src/plugins/General/library/librarymodel.cpp
index d02e9ba27..a1c583149 100644
--- a/src/plugins/General/library/librarymodel.cpp
+++ b/src/plugins/General/library/librarymodel.cpp
@@ -91,15 +91,7 @@ QStringList LibraryModel::mimeTypes() const
 
 QMimeData *LibraryModel::mimeData(const QModelIndexList &indexes) const
 {
-    QList<PlayListTrack *> tracks;
-
-    for(const QModelIndex &index : indexes)
-    {
-        if(index.isValid())
-        {
-            tracks << getTracks(index);
-        }
-    }
+    QList<PlayListTrack *> tracks = getTracks(indexes);
 
     if(!tracks.isEmpty())
     {
@@ -308,20 +300,20 @@ void LibraryModel::refresh()
 
 void LibraryModel::add(const QModelIndexList &indexes)
 {
-    QList<PlayListTrack *> tracks;
+    PlayListManager::instance()->add(getTracks(indexes));
+}
 
-    for(const QModelIndex &index : indexes)
-    {
-        if(index.isValid())
-        {
-            tracks << getTracks(index);
-        }
-    }
+void LibraryModel::showInformation(const QModelIndexList &indexes, QWidget *parent)
+{
+    QList<PlayListTrack *> tracks = getTracks(indexes);
 
-    PlayListManager::instance()->add(tracks);
+    DetailsDialog *dialog = new DetailsDialog(tracks, parent);
+    dialog->setAttribute(Qt::WA_DeleteOnClose, true);
+    dialog->show();
+    connect(dialog, &QObject::destroyed, [=]() { qDeleteAll(tracks); });
 }
 
-void LibraryModel::showInformation(const QModelIndexList &indexes, QWidget *parent)
+QList<PlayListTrack *> LibraryModel::getTracks(const QModelIndexList &indexes) const
 {
     QList<PlayListTrack *> tracks;
 
@@ -333,10 +325,7 @@ void LibraryModel::showInformation(const QModelIndexList &indexes, QWidget *pare
         }
     }
 
-    DetailsDialog *dialog = new DetailsDialog(tracks, parent);
-    dialog->setAttribute(Qt::WA_DeleteOnClose, true);
-    dialog->show();
-    connect(dialog, &QObject::destroyed, [=]() { qDeleteAll(tracks); });
+    return tracks;
 }
 
 QList<PlayListTrack *> LibraryModel::getTracks(const QModelIndex &index) const