From 9560e23b6635f3358eafbf91881643c0e759f690 Mon Sep 17 00:00:00 2001
From: Magne Sjaastad <magne.sjaastad@ceetronsolutions.com>
Date: Wed, 9 May 2018 16:01:59 +0200
Subject: [PATCH] #2888 Auto Title : Crash when single summary case is nullptr

---
 .../Summary/RimSummaryPlotNameHelper.cpp      | 23 +++++++++++--------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/ApplicationCode/ProjectDataModel/Summary/RimSummaryPlotNameHelper.cpp b/ApplicationCode/ProjectDataModel/Summary/RimSummaryPlotNameHelper.cpp
index 5d97cc4d91..59789ed6fc 100644
--- a/ApplicationCode/ProjectDataModel/Summary/RimSummaryPlotNameHelper.cpp
+++ b/ApplicationCode/ProjectDataModel/Summary/RimSummaryPlotNameHelper.cpp
@@ -59,9 +59,9 @@ void RimSummaryPlotNameHelper::appendSummaryCases(const std::vector<RimSummaryCa
 {
     m_summaryCases.clear();
 
-    for (auto c : summaryCases)
+    for (auto summaryCase : summaryCases)
     {
-        m_summaryCases.insert(c);
+        if (summaryCase) m_summaryCases.insert(summaryCase);
     }
 
     extractPlotTitleSubStrings();
@@ -74,9 +74,9 @@ void RimSummaryPlotNameHelper::appendEnsembleCases(const std::vector<RimSummaryC
 {
     m_ensembleCases.clear();
 
-    for (auto c : ensembleCases)
+    for (auto ensembleCase : ensembleCases)
     {
-        if(c) m_ensembleCases.insert(c);
+        if (ensembleCase) m_ensembleCases.insert(ensembleCase);
     }
 
     extractPlotTitleSubStrings();
@@ -212,17 +212,20 @@ void RimSummaryPlotNameHelper::extractPlotTitleSubStrings()
         m_titleRegion = std::to_string(*(regions.begin()));
     }
 
+    if (m_summaryCases.size() == 1 && m_ensembleCases.empty())
     {
-        if (m_summaryCases.size() == 1 && m_ensembleCases.empty())
-        {
-            auto summaryCase = *(m_summaryCases.begin());
+        auto summaryCase = *(m_summaryCases.begin());
 
+        if (summaryCase)
+        {
             m_titleCaseName = summaryCase->caseName();
         }
-        else if (m_ensembleCases.size() == 1 && m_summaryCases.empty())
+    }
+    else if (m_ensembleCases.size() == 1 && m_summaryCases.empty())
+    {
+        auto ensembleCase = *(m_ensembleCases.begin());
+        if (ensembleCase)
         {
-            auto ensembleCase = *(m_ensembleCases.begin());
-
             m_titleCaseName = ensembleCase->name();
         }
     }