Refs #8547. Check group membership by name, not pointer.

Otherwise we end up with multiple workspaces with the same name in the group.
mantidproject · Feb 27, 2014 · 10ab90f · 10ab90f
1 parent 1e0b2c3
commit 10ab90f
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/Code/Mantid/MantidQt/CustomInterfaces/src/MuonAnalysis.cpp b/Code/Mantid/MantidQt/CustomInterfaces/src/MuonAnalysis.cpp
@@ -368,18 +368,18 @@ void MuonAnalysis::plotItem(ItemType itemType, int tableRow, PlotType plotType)
     MatrixWorkspace_sptr ws = createAnalysisWorkspace(itemType, tableRow, plotType);
     MatrixWorkspace_sptr wsRaw = createAnalysisWorkspace(itemType, tableRow, plotType, true);
 
+    // Find names for new workspaces
+    const std::string wsName = getNewAnalysisWSName(m_currentGroup->getName(), itemType, tableRow,
+                                                    plotType);
+    const std::string wsRawName = wsName + "_Raw"; 
+
     // Make sure they are in the current group
-    if ( ! m_currentGroup->contains(ws) )
+    if ( ! m_currentGroup->contains(wsName) )
     {
       m_currentGroup->addWorkspace(ws);
       m_currentGroup->addWorkspace(wsRaw);
     }
 
-    // Find names for new workspaces
-    const std::string wsName = getNewAnalysisWSName(m_currentGroup->getName(), itemType, tableRow,
-                                                    plotType);
-    const std::string wsRawName = wsName + "_Raw"; 
-
     // Make sure they end up in the ADS
     ads.addOrReplace(wsName, ws);
     ads.addOrReplace(wsRawName, wsRaw);