bug fix and added icons the previous commit here, and forgot in the log.. heheheh

1 files changed, 141 insertions, 115 deletions

diff --git a/noncore/net/opieftp/opieftp.cpp b/noncore/net/opieftp/opieftp.cpp
index fb57193..7f5d71c 100644
--- a/noncore/net/opieftp/opieftp.cpp
+++ b/noncore/net/opieftp/opieftp.cpp
@@ -157,4 +157,6 @@ OpieFtp::OpieFtp( )
     Local_View->setAllColumnsShowFocus(TRUE);
-    Local_View->setMultiSelection( TRUE );
+
+    Local_View->setMultiSelection( TRUE);
     Local_View->setSelectionMode(QListView::Extended);
+    Local_View->setFocusPolicy(QWidget::ClickFocus);
 
@@ -187,4 +189,6 @@ OpieFtp::OpieFtp( )
     Remote_View->setAllColumnsShowFocus(TRUE);
-    Remote_View->setMultiSelection( TRUE );
+
+    Remote_View->setMultiSelection( FALSE);
     Remote_View->setSelectionMode(QListView::Extended);
+    Remote_View->setFocusPolicy(QWidget::ClickFocus);
 
@@ -380,3 +384,3 @@ void OpieFtp::connector()
 {
-//    QCopEnvelope ( "QPE/System", "busy()" );
+    QCopEnvelope ( "QPE/System", "busy()" );
 //    qApp->processEvents();
@@ -423,3 +427,3 @@ void OpieFtp::connector()
     connectServerBtn->setText( tr("Disconnect"));
-//    QCopEnvelope ( "QPE/System", "notBusy()" );
+    QCopEnvelope ( "QPE/System", "notBusy()" );
 }
@@ -471,9 +475,12 @@ void OpieFtp::localUpload()
             nullifyCallBack();
-        } //end currentSelected
         it.current()->setSelected(FALSE);
+        } //end currentSelected
+    }
+    for ( ; it.current(); ++it ) {
+        Local_View->clearSelection();
     }
+    Local_View->clearFocus();
     TabWidget->setCurrentPage(1);
     remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
-//    QCopEnvelope ( "QPE/System", "notBusy()" );
-
+    QCopEnvelope ( "QPE/System", "notBusy()" );
 }
@@ -492,3 +499,3 @@ void OpieFtp::remoteDownload()
     int fsz;
-//    QCopEnvelope ( "QPE/System", "busy()" );
+    QCopEnvelope ( "QPE/System", "busy()" );
 
@@ -526,9 +533,12 @@ void OpieFtp::remoteDownload()
             nullifyCallBack();
-        }
         it.current()->setSelected(FALSE);
+        }
+    }
+    for ( ; it.current(); ++it ) {
+        Remote_View->clearSelection();
     }
+    Remote_View->setFocus();    
     TabWidget->setCurrentPage(0);
     populateLocalView();
-//    QCopEnvelope ( "QPE/System", "notBusy()" );
-    
+    QCopEnvelope ( "QPE/System", "notBusy()" );
 }
@@ -543,3 +553,3 @@ bool OpieFtp::remoteDirList(const QString &dir)
 //    qDebug("Listing remote dir "+tmp);
-//    QCopEnvelope ( "QPE/System", "busy()" );
+    QCopEnvelope ( "QPE/System", "busy()" );
     if (!FtpDir( tmp.latin1(), dir.latin1(), conn) ) {
@@ -552,3 +562,3 @@ bool OpieFtp::remoteDirList(const QString &dir)
     populateRemoteView() ;
-//    QCopEnvelope ( "QPE/System", "notBusy()" );
+    QCopEnvelope ( "QPE/System", "notBusy()" );
     return true;
@@ -558,3 +568,3 @@ bool OpieFtp::remoteChDir(const QString &dir)
 {
-//    QCopEnvelope ( "QPE/System", "busy()" );
+    QCopEnvelope ( "QPE/System", "busy()" );
     if (!FtpChdir( dir.latin1(), conn )) {
@@ -565,6 +575,6 @@ bool OpieFtp::remoteChDir(const QString &dir)
 //        qDebug(msg);
-//        QCopEnvelope ( "QPE/System", "notBusy()" );
+        QCopEnvelope ( "QPE/System", "notBusy()" );
         return FALSE;
     }
-//    QCopEnvelope ( "QPE/System", "notBusy()" );
+    QCopEnvelope ( "QPE/System", "notBusy()" );
     return TRUE;
@@ -639,3 +649,2 @@ void OpieFtp::populateLocalView()
     fillCombo( (const QString &)currentDir);
-
 }
@@ -740,3 +749,3 @@ void OpieFtp::remoteListClicked(QListViewItem *selectedItem)
             } else {
-//                QCopEnvelope ( "QPE/System", "notBusy()" );
+                QCopEnvelope ( "QPE/System", "notBusy()" );
                 return;
@@ -792,3 +801,2 @@ void OpieFtp::localListClicked(QListViewItem *selectedItem)
     }
-    
 }
@@ -840,12 +848,14 @@ void OpieFtp::showRemoteMenu(QListViewItem * item)
 {
-    QPopupMenu  m;// = new QPopupMenu( Local_View );
+    QPopupMenu * m;// = new QPopupMenu( Local_View );
+    m = new QPopupMenu(this);
     if( /*item->text(0).right(1) == "/" ||*/ item->text(0).find("/",0,TRUE) != -1)
-        m.insertItem( tr( "Change Directory" ), this, SLOT( doRemoteCd() ));
+        m->insertItem( tr( "Change Directory" ), this, SLOT( doRemoteCd() ));
     else
-    m.insertItem( tr( "Download" ), this, SLOT( remoteDownload() ));
-    m.insertItem( tr( "Make Directory" ), this, SLOT( remoteMakDir() ));
-    m.insertItem( tr( "Rename" ), this, SLOT( remoteRename() ));
-    m.insertSeparator();
-    m.insertItem( tr( "Delete" ), this, SLOT( remoteDelete() ));
-    m.exec( QCursor::pos() );
+    m->insertItem( tr( "Download" ), this, SLOT( remoteDownload() ));
+    m->insertItem( tr( "Make Directory" ), this, SLOT( remoteMakDir() ));
+    m->insertItem( tr( "Rename" ), this, SLOT( remoteRename() ));
+    m->insertSeparator();
+    m->insertItem( tr( "Delete" ), this, SLOT( remoteDelete() ));
+    m->exec( QCursor::pos() );
+    delete m;
 }
@@ -855,20 +865,22 @@ void OpieFtp::showLocalMenu(QListViewItem * item)
 
- QPopupMenu  m;
-    m.insertItem(  tr( "Show Hidden Files" ), this,  SLOT( showHidden() ));
-    m.insertSeparator();
+ QPopupMenu  *m;
+ m = new QPopupMenu( this);
+    m->insertItem(  tr( "Show Hidden Files" ), this,  SLOT( showHidden() ));
+    m->insertSeparator();
     if( /*item->text(0).right(1) == "/" ||*/ item->text(0).find("/",0,TRUE) !=-1)
-    m.insertItem( tr( "Change Directory" ), this, SLOT( doLocalCd() ));
+    m->insertItem( tr( "Change Directory" ), this, SLOT( doLocalCd() ));
     else
-    m.insertItem( tr( "Upload" ), this, SLOT( localUpload() ));
-    m.insertItem( tr( "Make Directory" ), this, SLOT( localMakDir() ));
-    m.insertItem( tr( "Rename" ), this, SLOT( localRename() ));
-    m.insertSeparator();
-    m.insertItem( tr( "Delete" ), this, SLOT( localDelete() ));
-    m.setCheckable(TRUE);
+    m->insertItem( tr( "Upload" ), this, SLOT( localUpload() ));
+    m->insertItem( tr( "Make Directory" ), this, SLOT( localMakDir() ));
+    m->insertItem( tr( "Rename" ), this, SLOT( localRename() ));
+    m->insertSeparator();
+    m->insertItem( tr( "Delete" ), this, SLOT( localDelete() ));
+    m->setCheckable(TRUE);
     if (b)
-        m.setItemChecked(m.idAt(0),TRUE);
+        m->setItemChecked(m->idAt(0),TRUE);
     else
-        m.setItemChecked(m.idAt(0),FALSE);
+        m->setItemChecked(m->idAt(0),FALSE);
 
-    m.exec( QCursor::pos() );
+    m->exec( QCursor::pos() );
+    delete m;
 }
@@ -889,33 +901,42 @@ void OpieFtp::localDelete()
 {
-    QString f = Local_View->currentItem()->text(0);
-    if(QDir(f).exists() ) {
-        switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f+
-                                      tr(" ?\nIt must be empty"),tr("Yes"),tr("No"),0,0,1) ) {
-          case 0: {
-              f=currentDir.canonicalPath()+"/"+f;
-              QString cmd="rmdir "+f;
-              system( cmd.latin1());
-              populateLocalView();
-          }
-              break;
-          case 1:
-                // exit
-              break;
-        };
+    QList<QListViewItem> * getSelectedItems( QListView * Local_View );
+    QListViewItemIterator it( Local_View );
+    for ( ; it.current(); ++it ) {
+        if ( it.current()->isSelected() ) {
+            QString f = it.current()->text(0);
+            it.current()->setSelected(FALSE);
+            
+//    QString f = Local_View->currentItem()->text(0);
+            if(QDir(f).exists() ) {
+                switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f+
+                                              tr(" ?\nIt must be empty"),tr("Yes"),tr("No"),0,0,1) ) {
+                  case 0: {
+                      f=currentDir.canonicalPath()+"/"+f;
+                      QString cmd="rmdir "+f;
+                      system( cmd.latin1());
+                  }
+                      break;
+                  case 1:
+                        // exit
+                      break;
+                };
 
-    } else {
-        switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f
-                                      +" ?",tr("Yes"),tr("No"),0,0,1) ) {
-          case 0: {
-              f=currentDir.canonicalPath()+"/"+f;
-              QString cmd="rm "+f;
-              system( cmd.latin1());
-              populateLocalView();
-          }
-              break;
-          case 1: 
-                // exit
-              break;
-        };
+            } else {
+                switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f
+                                              +" ?",tr("Yes"),tr("No"),0,0,1) ) {
+                  case 0: {
+                      f=currentDir.canonicalPath()+"/"+f;
+                      QString cmd="rm "+f;
+                      system( cmd.latin1());
+                  }
+                      break;
+                  case 1: 
+                        // exit
+                      break;
+                };
+            }
+        }
     }
+    populateLocalView();
+    
 }
@@ -930,3 +951,3 @@ void OpieFtp::remoteMakDir()
        QString tmp=currentRemoteDir+filename;
-//       QCopEnvelope ( "QPE/System", "busy()" );
+       QCopEnvelope ( "QPE/System", "busy()" );
        if(FtpMkdir( tmp.latin1(), conn) == 0) {
@@ -937,3 +958,3 @@ void OpieFtp::remoteMakDir()
        }
-//       QCopEnvelope ( "QPE/System", "notBusy()" );
+       QCopEnvelope ( "QPE/System", "notBusy()" );
     remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
@@ -944,37 +965,44 @@ void OpieFtp::remoteDelete()
 {
-    QString f = Remote_View->currentItem()->text(0);
-//    QCopEnvelope ( "QPE/System", "busy()" );
-    if( f.right(1) =="/") {
-        QString path= currentRemoteDir+f;
-        switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f+"?"
-                                      ,tr("Yes"),tr("No"),0,0,1) ) {
-          case 0: {
-              f=currentDir.canonicalPath()+"/"+f;
-              if(FtpRmdir( path.latin1(), conn) ==0) {
-                  QString msg;
-                  msg.sprintf(tr("Unable to remove directory\n")+"%s",FtpLastResponse(conn));
-                  msg.replace(QRegExp(":"),"\n");
-                  QMessageBox::message(tr("Note"),msg);
-              }
-              remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
-          }
-          break;
-        };
-    } else {
-        switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f+"?"
-                                      ,tr("Yes"),tr("No"),0,0,1) ) {
-          case 0: {
-              QString path= currentRemoteDir+f;
-              if(FtpDelete( path.latin1(), conn)==0) {
-                  QString msg;
-                  msg.sprintf(tr("Unable to delete file\n")+"%s",FtpLastResponse(conn));
-                  msg.replace(QRegExp(":"),"\n");
-                  QMessageBox::message(tr("Note"),msg);
-              }
-              remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
-          }
-          break;
-        };
+    QList<QListViewItem> * getSelectedItems( QListView * Remote_View );
+    QListViewItemIterator it( Remote_View );
+    for ( ; it.current(); ++it ) {
+        if ( it.current()->isSelected() ) {
+            QString f = it.current()->text(0);
+//    QString f = Remote_View->currentItem()->text(0);
+            QCopEnvelope ( "QPE/System", "busy()" );
+            if( f.right(1) =="/") {
+                QString path= currentRemoteDir+f;
+                switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f+"?"
+                                              ,tr("Yes"),tr("No"),0,0,1) ) {
+                  case 0: {
+                      f=currentDir.canonicalPath()+"/"+f;
+                      if(FtpRmdir( path.latin1(), conn) ==0) {
+                          QString msg;
+                          msg.sprintf(tr("Unable to remove directory\n")+"%s",FtpLastResponse(conn));
+                          msg.replace(QRegExp(":"),"\n");
+                          QMessageBox::message(tr("Note"),msg);
+                      }
+                      remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
+                  }
+                      break;
+                };
+            } else {
+                switch ( QMessageBox::warning(this,tr("Delete"),tr("Do you really want to delete\n")+f+"?"
+                                              ,tr("Yes"),tr("No"),0,0,1) ) {
+                  case 0: {
+                      QString path= currentRemoteDir+f;
+                      if(FtpDelete( path.latin1(), conn)==0) {
+                          QString msg;
+                          msg.sprintf(tr("Unable to delete file\n")+"%s",FtpLastResponse(conn));
+                          msg.replace(QRegExp(":"),"\n");
+                          QMessageBox::message(tr("Note"),msg);
+                      }
+                      remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
+                  }
+                      break;
+                };
+            }
+        }
     }
-//    QCopEnvelope ( "QPE/System", "notBusy()" );
+    QCopEnvelope ( "QPE/System", "notBusy()" );
 }
@@ -991,3 +1019,3 @@ void OpieFtp::remoteRename()
         QString newName = currentRemoteDir  +"/"+ fileDlg->LineEdit1->text();//+".playlist";
-//        QCopEnvelope ( "QPE/System", "busy()" );
+        QCopEnvelope ( "QPE/System", "busy()" );
         if(FtpRename( oldName.latin1(), newName.latin1(),conn) == 0) {
@@ -998,3 +1026,3 @@ void OpieFtp::remoteRename()
         }
-//        QCopEnvelope ( "QPE/System", "notBusy()" );
+        QCopEnvelope ( "QPE/System", "notBusy()" );
     remoteDirList( (const QString &)currentRemoteDir); //this also calls populate
@@ -1267,3 +1295,3 @@ void OpieFtp::upDir()
         fillRemoteCombo( (const QString &)currentRemoteDir);
-        populateRemoteView( );        
+
     }
@@ -1281,9 +1309,7 @@ void OpieFtp::docButtonPushed() {
 void OpieFtp::homeButtonPushed() {
-    if (TabWidget->currentPageIndex() == 0) {
-        QString current = QDir::homeDirPath();
-        chdir( current.latin1() );
-        currentDir.cd(  current, TRUE);
-        populateLocalView();
-        update();
-    }
+    QString current = QDir::homeDirPath();
+    chdir( current.latin1() );
+    currentDir.cd(  current, TRUE);
+     populateLocalView();
+    update();
 }