add proper case handling

1 files changed, 16 insertions, 13 deletions

diff --git a/noncore/apps/advancedfm/advancedfmMenu.cpp b/noncore/apps/advancedfm/advancedfmMenu.cpp
index 8b98898..e770304 100644
--- a/noncore/apps/advancedfm/advancedfmMenu.cpp
+++ b/noncore/apps/advancedfm/advancedfmMenu.cpp
@@ -187,36 +187,38 @@ void AdvancedFm::doDelete() {
 								startProcess( (const QString)cmd.latin1() );
 								populateView();
 							}
 							break;
             case 1:
 							// exit
 							break;
 						default:
 							break;
             };
 
          } else {
-            if(doMsg) {
-							switch ( QMessageBox::warning(this,tr("Delete"),
-																						tr("<p>Really delete %1?</p>").arg( myFile ),
-																						tr("Yes"), tr("No"), 0, 0, 1) ) {
-							case 1:
-								return;
-								break;
-							default:
-                return;
-								break;
-							};
-            }
+					 if(doMsg) {
+						 switch ( QMessageBox::warning(this,tr("Delete"),
+																					 tr("<p>Really delete %1?</p>").arg( myFile ),
+																					 tr("Yes"), tr("No"), 0, 0, 1) ) {
+							 case 0;
+							 break;
+						 case 1:
+							 return;
+							 break;
+						 default:
+							 return;
+							 break;
+						 };
+					 }
 
             QString cmd="rm "+f;
             QFile file(f);
             QFileInfo fi(myFile);
             if( fi.fileName().find("../",0,TRUE)==-1) {
 //               odebug << "remove link files "+myFile << oendl;
 
 //               DocLnk lnk(f);
                  DocLnk *lnk;
                  lnk = new DocLnk(f);
 //                 odebug << "Deleting doclnk " + lnk->linkFile() << oendl;
                  if(lnk->isValid())
@@ -313,24 +315,26 @@ void AdvancedFm::copy() {
 						curFile = thisDir->canonicalPath()+"/"+ item;
 						destFile = thatDir->canonicalPath()+"/"+ item;
 
 //          odebug << "Destination file is "+destFile << oendl;
 //          odebug << "CurrentFile file is " + curFile << oendl;
 
 						QFile f(destFile);
 						if( f.exists()) {
 								if(doMsg) {
 									switch ( QMessageBox::warning(this,tr("File Exists!"),
 																								tr("<p>%1 already  exists. Ok to overwrite?</P>").arg(item),
 																								tr("Yes"),tr("No"),0,0,1)) {
+										case 0;
+										break;
 									case 1:
 										return;
 										break;
 									default:
 										return;
 										break;
 									};
 								}
 								f.remove();
 						}
 
 						if( !copyFile( curFile, destFile) )  {
@@ -421,25 +425,24 @@ void AdvancedFm::copySameDir() {
 
       if( fileDlg->result() == 1 ) {
 
           QString filename = fileDlg->LineEdit1->text();
           destFile = thisDir->canonicalPath()+"/"+filename;
 
           QFile f(destFile);
           if( f.exists()) {
 						switch (QMessageBox::warning(this,tr("Delete"),
 																				 tr("<p> %1 already exists. Do you really want to delete it?</P>").arg(destFile),
 																				 tr("Yes"),tr("No"),0,0,1) ) {
 						case 0:
-
 							f.remove();
 							break;
 						case 1:
 							return;
 							break;
 						default:
 							return;
 							break;
 						};
             }
           if(!copyFile( curFile,destFile) )  {
               QMessageBox::message("AdvancedFm",tr("<P>Could not copy %1 to %2</P>").arg(curFile).arg(destFile));