Remove unnecessary returns. Refs #8938.

mantidproject · Mar 13, 2014 · 1502563 · 1502563
1 parent e114fac
commit 1502563
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/Code/Mantid/MantidQt/CustomDialogs/src/CatalogPublishDialog.cpp b/Code/Mantid/MantidQt/CustomDialogs/src/CatalogPublishDialog.cpp
@@ -120,12 +120,16 @@ namespace MantidQt
         if (m_uiForm.dataSelector->getFullFilePath().isEmpty())
         {
           QMessageBox::critical(this,"Error in catalog publishing.","No file specified.");
-          return;
         }
-        QMessageBox::critical(this,"Error in catalog publishing.",m_uiForm.dataSelector->getProblem());
-        return;
+        else
+        {
+          QMessageBox::critical(this,"Error in catalog publishing.",m_uiForm.dataSelector->getProblem());
+        }
+      }
+      else
+      {
+        AlgorithmDialog::accept();
       }
-      AlgorithmDialog::accept();
     }
   }
 }