fix playlist adding when two file share the same basename but different paths. made it faster adding files to playlist

1 files changed, 36 insertions, 67 deletions

diff --git a/noncore/multimedia/opieplayer2/playlistwidget.cpp b/noncore/multimedia/opieplayer2/playlistwidget.cpp
index 8e88e9b..a6a9eca 100644
--- a/noncore/multimedia/opieplayer2/playlistwidget.cpp
+++ b/noncore/multimedia/opieplayer2/playlistwidget.cpp
@@ -210,6 +210,7 @@ void PlayListWidget::addToSelection( const DocLnk& lnk ) {
     if ( mediaPlayerState->playlist() ) {
         if( QFileInfo( lnk.file() ).exists() ||
-            lnk.file().left(4) == "http" )
+            lnk.file().left(4) == "http" ) {
             d->selectedFiles->addToSelection( lnk );
+        }
 //          writeCurrentM3u();          
     }
@@ -266,5 +267,8 @@ void PlayListWidget::addAllToList() {
         }
     }
+    tabWidget->setCurrentPage(0);
+    
     writeCurrentM3u();          
+    d->selectedFiles->first();
 }
 
@@ -279,5 +283,7 @@ void PlayListWidget::addAllMusicToList() {
         }
     }
+    tabWidget->setCurrentPage(0);
     writeCurrentM3u();          
+    d->selectedFiles->first();
 }
 
@@ -292,5 +298,7 @@ void PlayListWidget::addAllVideoToList() {
         }
     }
+    tabWidget->setCurrentPage(0);
     writeCurrentM3u();          
+    d->selectedFiles->first();
 }
 
@@ -424,9 +432,9 @@ bool PlayListWidget::last() {
 void PlayListWidget::loadList( const DocLnk & lnk) {
     QString name = lnk.name();
-  qDebug("<<<<<<<<<<<<<<<<<<<<<<<<currentList is "+name);
+    //  qDebug("<<<<<<<<<<<<<<<<<<<<<<<<currentList is "+name);
 
     if( name.length()>0) {
         setCaption("OpiePlayer: "+name);
-       qDebug("<<<<<<<<<<<<load list "+ lnk.file());
+//       qDebug("<<<<<<<<<<<<load list "+ lnk.file());
         clearList();
         readm3u(lnk.file());
@@ -447,50 +455,29 @@ void PlayListWidget::setPlaylist( bool shown ) {
 void PlayListWidget::addSelected() {
     qDebug("addSelected");
+  DocLnk lnk;
+  QString filename;
     switch (whichList()) {
+
       case 0: //playlist
+    return;
           break;
       case 1: { //audio
-          QListViewItemIterator it(  audioView  );
-            // iterate through all items of the listview
-          for ( ; it.current(); ++it ) {
-              if ( it.current()->isSelected() ) {
-                  QListIterator<DocLnk> dit( files.children() );
-                  for ( ; dit.current(); ++dit ) {
-                      if( dit.current()->name() == it.current()->text(0) ) {
-                        if( QFileInfo( dit.current()->file()).exists()) {
-                              d->selectedFiles->addToSelection(  **dit );
-                              audioView->setSelected( it.current(),FALSE);
-                        }
-                      }
-                  }
-              }
-          }
-//          tabWidget->setCurrentPage(0);
-          writeCurrentM3u();          
-    d->selectedFiles->next();
+    filename=audioView->currentItem()->text(3);
+    //    d->selectedFiles->next();
       }
           break;
+    
       case 2: { // video
-          QListViewItemIterator it(  videoView  );
-            // iterate through all items of the listview
-          for ( ; it.current(); ++it ) {
-              if ( it.current()->isSelected() ) {
-                  QListIterator<DocLnk> dit( vFiles.children() );
-                  for ( ; dit.current(); ++dit ) {
-                      if( dit.current()->name() == it.current()->text(0) ) {
-                        if(QFileInfo( dit.current()->file()).exists()) {
-                              d->selectedFiles->addToSelection(  **dit );
-                              videoView->setSelected( it.current(),FALSE);
-                        }
-                      }
-                  }
-              }
-          }
+    filename=videoView->currentItem()->text(3);
 //          tabWidget->setCurrentPage(0);
-          writeCurrentM3u();          
     
       }
           break;
     };
+  lnk.setName( QFileInfo(filename).baseName() ); //sets name
+  lnk.setFile( filename ); //sets file name
+  d->selectedFiles->addToSelection(  lnk);
+  tabWidget->setCurrentPage(0);
+  writeCurrentM3u();          
 }
 
@@ -515,36 +502,18 @@ void PlayListWidget::addToSelection( QListViewItem *it) {
     if(it) {
         switch ( whichList()) {
-          case 1: {
-              QListIterator<DocLnk> dit( files.children() );
-              for ( ; dit.current(); ++dit ) {
-                  if( dit.current()->name() == it->text(0)) {
-                      if(QFileInfo( dit.current()->file()).exists()) {
-                          d->selectedFiles->addToSelection(  **dit );
-//                          qDebug("blah "+ dit.current()->name());
-//                          d->selectedFiles->setSelectedItem( dit.current()->name());
-                      }
-                  }
-              }
-              writeCurrentM3u();          
-          }
-              break;
-          case 2: {
-              QListIterator<DocLnk> dit( vFiles.children() );
-              for ( ; dit.current(); ++dit ) {
-                  if( dit.current()->name() == it->text(0)) {
-                      if( QFileInfo( dit.current()->file()).exists() ) {
-                          d->selectedFiles->addToSelection(  **dit );
-//                          qDebug("blah "+ dit.current()->name());
-//                          d->selectedFiles->setSelectedItem( dit.current()->name());
-                      }
-                  }
-              }
-              writeCurrentM3u();          
-            
-          }
-              break;
-          case 0:
+    case 0: //playlist
+      return;
               break;
         };
+    //      case 1: {
+    DocLnk lnk;
+    QString filename;
+
+    filename=it->text(3);
+    lnk.setName( QFileInfo(filename).baseName() ); //sets name
+    lnk.setFile( filename ); //sets file name
+    d->selectedFiles->addToSelection(  lnk);
+ 
+    writeCurrentM3u();          
         tabWidget->setCurrentPage(0);