#4018 Result management : Use ensureReservoirCaseIsOpen instead of open...

This commit is contained in:
Magne Sjaastad
2019-01-29 10:06:30 +01:00
parent 6935ea0688
commit fde60d1e34
7 changed files with 10 additions and 13 deletions

View File

@@ -71,13 +71,10 @@ void RicfExportProperty::execute()
return; return;
} }
if (!eclipseCase->eclipseCaseData()) if (!eclipseCase->ensureReservoirCaseIsOpen())
{ {
if (!eclipseCase->openReserviorCase()) RiaLogging::error(QString("exportProperty: Could not find eclipseCaseData with ID %1").arg(m_caseId()));
{ return;
RiaLogging::error(QString("exportProperty: Could not find eclipseCaseData with ID %1").arg(m_caseId()));
return;
}
} }
} }

View File

@@ -70,7 +70,6 @@ public:
void setFilesContainingFaults(const std::vector<QString>& val); void setFilesContainingFaults(const std::vector<QString>& val);
bool ensureReservoirCaseIsOpen(); bool ensureReservoirCaseIsOpen();
bool openReserviorCase();
virtual bool openEclipseGridFile() = 0; virtual bool openEclipseGridFile() = 0;
RigEclipseCaseData* eclipseCaseData(); RigEclipseCaseData* eclipseCaseData();
@@ -131,7 +130,8 @@ protected:
private: private:
void createTimeStepFormatString(); void createTimeStepFormatString();
std::vector<Rim3dView*> allSpecialViews() const override; std::vector<Rim3dView*> allSpecialViews() const override;
bool openReserviorCase();
protected: protected:
caf::PdmField<bool> m_flipXAxis; caf::PdmField<bool> m_flipXAxis;

View File

@@ -233,7 +233,7 @@ bool RimEclipseInputCase::openEclipseGridFile()
void RimEclipseInputCase::reloadEclipseGridFile() void RimEclipseInputCase::reloadEclipseGridFile()
{ {
setReservoirData(nullptr); setReservoirData(nullptr);
openReserviorCase(); ensureReservoirCaseIsOpen();
} }
#define for_all(stdVector, indexName) for (size_t indexName = 0; indexName < stdVector.size(); ++indexName) #define for_all(stdVector, indexName) for (size_t indexName = 0; indexName < stdVector.size(); ++indexName)

View File

@@ -243,7 +243,7 @@ void RimEclipseResultCase::reloadEclipseGridFile()
m_gridAndWellDataIsReadFromFile = false; m_gridAndWellDataIsReadFromFile = false;
m_activeCellInfoIsReadFromFile = false; m_activeCellInfoIsReadFromFile = false;
setReservoirData(nullptr); setReservoirData(nullptr);
openReserviorCase(); ensureReservoirCaseIsOpen();
} }
//-------------------------------------------------------------------------------------------------- //--------------------------------------------------------------------------------------------------

View File

@@ -176,7 +176,7 @@ bool RimEclipseStatisticsCase::openEclipseGridFile()
void RimEclipseStatisticsCase::reloadEclipseGridFile() void RimEclipseStatisticsCase::reloadEclipseGridFile()
{ {
setReservoirData(nullptr); setReservoirData(nullptr);
openReserviorCase(); ensureReservoirCaseIsOpen();
} }
//-------------------------------------------------------------------------------------------------- //--------------------------------------------------------------------------------------------------

View File

@@ -830,7 +830,7 @@ void RimEclipseView::onLoadDataAndUpdate()
if (m_eclipseCase) if (m_eclipseCase)
{ {
if (!m_eclipseCase->openReserviorCase()) if (!m_eclipseCase->ensureReservoirCaseIsOpen())
{ {
QMessageBox::warning(RiuMainWindow::instance(), QMessageBox::warning(RiuMainWindow::instance(),
"Error when opening project file", "Error when opening project file",

View File

@@ -171,7 +171,7 @@ void RimIdenticalGridCaseGroup::loadMainCaseAndActiveCellInfo()
// When reading active cell info, only the total cell count is tested for consistency // When reading active cell info, only the total cell count is tested for consistency
RimEclipseCase* mainCase = caseCollection()->reservoirs[0]; RimEclipseCase* mainCase = caseCollection()->reservoirs[0];
if (!mainCase->openReserviorCase()) if (!mainCase->ensureReservoirCaseIsOpen())
{ {
QMessageBox::warning(Riu3DMainWindowTools::mainWindowWidget(), QMessageBox::warning(Riu3DMainWindowTools::mainWindowWidget(),
"Error when opening project file", "Error when opening project file",