From 1502563e592690104fafadb41b13affad8a1772b Mon Sep 17 00:00:00 2001
From: Jay Rainey <jawrainey@gmail.com>
Date: Thu, 13 Mar 2014 13:38:16 +0000
Subject: [PATCH] Remove unnecessary returns. Refs #8938.

---
 .../CustomDialogs/src/CatalogPublishDialog.cpp       | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/Code/Mantid/MantidQt/CustomDialogs/src/CatalogPublishDialog.cpp b/Code/Mantid/MantidQt/CustomDialogs/src/CatalogPublishDialog.cpp
index 18cd761d5e15..e9fd1946aaa8 100644
--- 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();
     }
   }
 }