Remove implementation from methods. Refs #9026.

- Update add to be const correct.
mantidproject · Feb 24, 2014 · b6a48e1 · b6a48e1
1 parent ec10422
commit b6a48e1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 12 deletions.
diff --git a/Code/Mantid/Framework/ICat/inc/MantidICat/CompositeCatalog.h b/Code/Mantid/Framework/ICat/inc/MantidICat/CompositeCatalog.h
@@ -40,7 +40,7 @@ namespace Mantid
         /// Destructor
         ~CompositeCatalog();
         /// Adds a catalog to the list of catalogs (m_catalogs)
-        void add(API::ICatalog_sptr catalog);
+        void add(const API::ICatalog_sptr catalog);
 
         /// Log the user into the catalog system.
         virtual void login(const std::string& username,const std::string& password,const std::string& endpoint);

diff --git a/Code/Mantid/Framework/ICat/src/CompositeCatalog.cpp b/Code/Mantid/Framework/ICat/src/CompositeCatalog.cpp
@@ -18,7 +18,7 @@ namespace Mantid
      * Add a catalog to the catalog container.
      * @param catalog :: The catalog to add to the container.
      */
-    void CompositeCatalog::add(API::ICatalog_sptr catalog)
+    void CompositeCatalog::add(const API::ICatalog_sptr catalog)
     {
       m_catalogs.push_back(catalog);
     }
@@ -160,10 +160,6 @@ namespace Mantid
      */
     void CompositeCatalog::getDownloadURL(const long long& fileID,std::string& url)
     {
-      for(auto catalog = m_catalogs.begin(); catalog != m_catalogs.end(); ++catalog)
-      {
-        (*catalog)->getDownloadURL(fileID, url);
-      }
     }
 
     /**
@@ -176,12 +172,6 @@ namespace Mantid
     const std::string CompositeCatalog::getUploadURL(const std::string &investigationID,
         const std::string &createFileName, const std::string &dataFileDescription)
     {
-      std::string uploadURL;
-      for(auto catalog = m_catalogs.begin(); catalog != m_catalogs.end(); ++catalog)
-      {
-        uploadURL = (*catalog)->getUploadURL(investigationID, createFileName, dataFileDescription);
-      }
-      return uploadURL;
     }
 
     /**