Re #18056 Sep now specified with string arg rather than actual char

Framework/DataHandling/src/AsciiPointBase.cpp

@@ -34,12 +34,12 @@ void AsciiPointBase::init() {
                   "The filename of the output file.");
 
   std::vector<std::string> propOptions;
-  propOptions.push_back(",");
-  propOptions.push_back(" ");
-  propOptions.push_back("\t");
-  declareProperty("Separator", "\t",
+  propOptions.push_back("comma");
+  propOptions.push_back("space");
+  propOptions.push_back("tab");
+  declareProperty("Separator", "tab",
       boost::make_shared<StringListValidator>(propOptions),
-      "The separator used for splitting data columns. Comma, space or tab.");
+      "The separator used for splitting data columns. Allowed values: ['comma', 'space', 'tab']");
   extraProps();
 }
 
@@ -56,8 +56,14 @@ void AsciiPointBase::exec() {
   }
   m_ws = getProperty("InputWorkspace");
   g_log.information("FILENAME: " + filename);
-  std::string sepString = getProperty("Separator");
-  sep = boost::lexical_cast<char>(sepString);
+  std::string sepOption = getProperty("Separator");
+  if (sepOption == "comma") {
+    sep = ',';
+  } else if (sepOption == "space") {
+    sep = ' ';
+  } else {
+    sep = '\t';
+  }
   std::vector<double> XData = header(file);
   extraHeaders(file);
   data(file, XData);

Framework/DataHandling/test/SaveANSTOAsciiTest.h

@@ -167,7 +167,7 @@ public:
       Mantid::API::AlgorithmManager::Instance().create("SaveANSTOAscii");
     alg->setPropertyValue("InputWorkspace", m_name);
     alg->setPropertyValue("Filename", m_filename);
-    alg->setPropertyValue("Separator", ",");
+    alg->setPropertyValue("Separator", "comma");
     TS_ASSERT_THROWS_NOTHING(alg->execute());
 
     if (!alg->isExecuted()) {

Framework/DataHandling/test/SaveILLCosmosAsciiTest.h

@@ -184,7 +184,7 @@ public:
     alg->setPropertyValue("Filename", m_filename);
     alg->setPropertyValue("UserContact", "John Smith");
     alg->setPropertyValue("Title", "Testing this algorithm");
-    alg->setPropertyValue("Separator", ",");
+    alg->setPropertyValue("Separator", "comma");
     TS_ASSERT_THROWS_NOTHING(alg->execute());
 
     if (!alg->isExecuted()) {

Framework/DataHandling/test/SaveReflCustomAsciiTest.h

@@ -179,7 +179,7 @@ public:
     alg->setPropertyValue("InputWorkspace", m_name);
     alg->setPropertyValue("Filename", m_filename);
     alg->setPropertyValue("Title", "Testing this algorithm");
-    alg->setPropertyValue("Separator", ",");
+    alg->setPropertyValue("Separator", "comma");
     TS_ASSERT_THROWS_NOTHING(alg->execute());
 
     if (!alg->isExecuted()) {

Framework/DataHandling/test/SaveReflThreeColumnAsciiTest.h

@@ -172,7 +172,7 @@ public:
         "SaveReflThreeColumnAscii");
     alg->setPropertyValue("InputWorkspace", m_name);
     alg->setPropertyValue("Filename", m_filename);
-    alg->setPropertyValue("Separator", ",");
+    alg->setPropertyValue("Separator", "comma");
     TS_ASSERT_THROWS_NOTHING(alg->execute());
 
     if (!alg->isExecuted()) {