summaryrefslogtreecommitdiff
authorllornkcor <llornkcor>2003-06-17 01:07:27 (UTC)
committer llornkcor <llornkcor>2003-06-17 01:07:27 (UTC)
commitf48df16919769a1abb73eab9d229659a792a6b8a (patch) (side-by-side diff)
treef1368b9f2d7c9656805d13d4f2c92bebf5f6b625
parentfc79272790448b77eeb7e656d80ce1b799c5cd38 (diff)
downloadopie-f48df16919769a1abb73eab9d229659a792a6b8a.zip
opie-f48df16919769a1abb73eab9d229659a792a6b8a.tar.gz
opie-f48df16919769a1abb73eab9d229659a792a6b8a.tar.bz2
fix crash on menu - no item selected
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/net/opieftp/opieftp.cpp23
1 files changed, 14 insertions, 9 deletions
diff --git a/noncore/net/opieftp/opieftp.cpp b/noncore/net/opieftp/opieftp.cpp
index 668a03d..269449e 100644
--- a/noncore/net/opieftp/opieftp.cpp
+++ b/noncore/net/opieftp/opieftp.cpp
@@ -763,5 +763,6 @@ bool OpieFtp::populateRemoteView( )
void OpieFtp::remoteListClicked(QListViewItem *selectedItem)
{
- if( selectedItem) {
+ if( selectedItem) {
+ // if(selectedItem!= NULL) {
// QCopEnvelope ( "QPE/System", "busy()" );
QString oldRemoteCurrentDir = currentRemoteDir;
@@ -909,8 +910,10 @@ void OpieFtp::showRemoteMenu(QListViewItem * item)
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() ));
- else
- m->insertItem( tr( "Download" ), this, SLOT( remoteDownload() ));
+ if(item != NULL ) {
+ if( item->text(0).find("/",0,TRUE) != -1)
+ 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("Rescan"), this, SLOT( populateLocalView() ));
@@ -929,8 +932,10 @@ void OpieFtp::showLocalMenu(QListViewItem * item)
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() ));
- else
- m->insertItem( tr( "Upload" ), this, SLOT( localUpload() ));
+ if(item != NULL ) {
+ if( item->text(0).find("/",0,TRUE) !=-1)
+ 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("Rescan"), this, SLOT( populateRemoteView() ));