Qt4: correct the check whether item is node

( PLModel::childrenCount() > 0 ) is not equivalent to (
playlist_item_t*->i_children > -1 ). This patch swaps former for latter
which is a correct strict check wether an item is node.

Signed-off-by: Ilkka Ollakka <ileoo@videolan.org>

diff --git a/modules/gui/qt4/components/playlist/playlist_model.cpp b/modules/gui/qt4/components/playlist/playlist_model.cpp
index 355310344b3d1888adcde08a3e69f446fbcd2de0..08c6b563fba46ddebcd59e53350917357d5a73e4 100644 (file)
--- a/modules/gui/qt4/components/playlist/playlist_model.cpp
+++ b/modules/gui/qt4/components/playlist/playlist_model.cpp
@@ -158,10 +158,14 @@ Qt::ItemFlags PLModel::flags( const QModelIndex &index ) const
     }
     else
     {
-        if ( (item->childCount() > 0) &&
+        PL_LOCK;
+        playlist_item_t *plItem =
+            playlist_ItemGetById( p_playlist, item->i_id );
+        if ( plItem && ( plItem->i_children > -1 ) &&
             ( rootItem->p_input == pl_input ||
             rootItem->p_input == ml_input ) )
                 flags |= Qt::ItemIsDropEnabled;
+        PL_UNLOCK;
         flags |= Qt::ItemIsDragEnabled;
     }