Qt4: allow adding node under playlist root item

diff --git a/modules/gui/qt4/components/playlist/playlist_model.cpp b/modules/gui/qt4/components/playlist/playlist_model.cpp
index d8ba78738d7a5ca68d5dcc0540e63156e965b1cf..be45ac84c27e42af5a215b19aa9ed433f6083c51 100644 (file)
--- a/modules/gui/qt4/components/playlist/playlist_model.cpp
+++ b/modules/gui/qt4/components/playlist/playlist_model.cpp
@@ -941,9 +941,11 @@ void PLModel::search( const QString& search_text )
 /*********** Popup *********/
 void PLModel::popup( QModelIndex & index, QPoint &point, QModelIndexList list )
 {
-    assert( index.isValid() );
     PL_LOCK;
-    playlist_item_t *p_item = playlist_ItemGetById( p_playlist, itemId( index ) );
+    int i_id;
+    if( index.isValid() ) i_id = itemId( index );
+    else i_id = rootItem->i_id;
+    playlist_item_t *p_item = playlist_ItemGetById( p_playlist, i_id );
     if( p_item )
     {
         i_popup_item = p_item->i_id;
@@ -964,22 +966,28 @@ void PLModel::popup( QModelIndex & index, QPoint &point, QModelIndexList list )
 
         current_selection = list;
         QMenu *menu = new QMenu;
-        menu->addAction( qtr(I_POP_PLAY), this, SLOT( popupPlay() ) );
-        menu->addAction( qtr(I_POP_DEL), this, SLOT( popupDel() ) );
-        menu->addSeparator();
-        menu->addAction( qtr(I_POP_STREAM), this, SLOT( popupStream() ) );
-        menu->addAction( qtr(I_POP_SAVE), this, SLOT( popupSave() ) );
-        menu->addSeparator();
-        menu->addAction( qtr(I_POP_INFO), this, SLOT( popupInfo() ) );
-        if( node )
+        if( index.isValid() )
         {
+            menu->addAction( qtr(I_POP_PLAY), this, SLOT( popupPlay() ) );
+            menu->addAction( qtr(I_POP_DEL), this, SLOT( popupDel() ) );
+            menu->addSeparator();
+            menu->addAction( qtr(I_POP_STREAM), this, SLOT( popupStream() ) );
+            menu->addAction( qtr(I_POP_SAVE), this, SLOT( popupSave() ) );
             menu->addSeparator();
+            menu->addAction( qtr(I_POP_INFO), this, SLOT( popupInfo() ) );
+        }
+        if( node )
+        {
+            if( index.isValid() ) menu->addSeparator();
             menu->addAction( qtr(I_POP_SORT), this, SLOT( popupSort() ) );
-            if( tree );
+            if( tree )
                 menu->addAction( qtr(I_POP_ADD), this, SLOT( popupAddNode() ) );
         }
-        menu->addSeparator();
-        menu->addAction( qtr( I_POP_EXPLORE ), this, SLOT( popupExplore() ) );
+        if( index.isValid() )
+        {
+            menu->addSeparator();
+            menu->addAction( qtr( I_POP_EXPLORE ), this, SLOT( popupExplore() ) );
+        }
         menu->popup( point );
     }
     else

diff --git a/modules/gui/qt4/components/playlist/standardpanel.cpp b/modules/gui/qt4/components/playlist/standardpanel.cpp
index 4ac1da2240845f635b0430e7267306f4ec1a71ac..8c1cef8d0d9478de7aae2b7b04c956514afbe354 100644 (file)
--- a/modules/gui/qt4/components/playlist/standardpanel.cpp
+++ b/modules/gui/qt4/components/playlist/standardpanel.cpp
@@ -336,7 +336,6 @@ void StandardPLPanel::search( const QString& searchText )
 
 void StandardPLPanel::doPopup( QModelIndex index, QPoint point )
 {
-    if( !index.isValid() ) return;
     QItemSelectionModel *selection = view->selectionModel();
     QModelIndexList list = selection->selectedIndexes();
     model->popup( index, point, list );