#3046 Fix crash when clicking on Eclipse case after import

This commit is contained in:
Gaute Lindkvist 2018-06-14 10:59:28 +02:00
parent a5f69a59d0
commit 3661827c62
2 changed files with 31 additions and 31 deletions

View File

@ -131,8 +131,6 @@ bool RimEclipseResultCase::importGridAndResultMetaData(bool showTimeStepFilter)
cvf::ref<RifReaderEclipseOutput> readerEclipseOutput = new RifReaderEclipseOutput;
readerEclipseOutput->setFilenamesWithFaults(this->filesContainingFaults());
if (showTimeStepFilter)
{
cvf::ref<RifEclipseRestartDataAccess> restartDataAccess = RifEclipseOutputFileTools::createDynamicResultAccess(caseFileName());
if (restartDataAccess.isNull())
{
@ -144,17 +142,17 @@ bool RimEclipseResultCase::importGridAndResultMetaData(bool showTimeStepFilter)
std::vector<double> daysSinceSimulationStart;
restartDataAccess->timeSteps(&timeSteps, &daysSinceSimulationStart);
m_timeStepFilter->setTimeStepsFromFile(timeSteps);
}
if (showTimeStepFilter)
{
// Restore cursor as the progress dialog is active
QApplication::restoreOverrideCursor();
// Show GUI to select time steps
m_timeStepFilter->setTimeStepsFromFile(timeSteps);
caf::PdmUiPropertyViewDialog propertyDialog(nullptr, m_timeStepFilter, "Time Step Filter", "", QDialogButtonBox::Ok | QDialogButtonBox::Cancel);
propertyDialog.resize(QSize(400, 400));
// Show GUI to select time steps
if (propertyDialog.exec() != QDialog::Accepted)
{
return false;
@ -165,7 +163,6 @@ bool RimEclipseResultCase::importGridAndResultMetaData(bool showTimeStepFilter)
QApplication::setOverrideCursor(QCursor(Qt::WaitCursor));
}
readerEclipseOutput->setFileDataAccess(restartDataAccess.p());
}
readerEclipseOutput->setTimeStepFilter(m_timeStepFilter->filteredTimeSteps());
cvf::ref<RigEclipseCaseData> eclipseCase = new RigEclipseCaseData(this);

View File

@ -235,10 +235,13 @@ QList<caf::PdmOptionItemInfo> RimTimeStepFilter::calculateValueOptions(const caf
{
std::vector<int> filteredTimeSteps = filteredTimeStepIndicesFromUi();
for (auto filteredIndex : filteredTimeSteps)
{
if (filteredIndex < static_cast<int>(m_timeStepNamesFromFile().size()))
{
optionItems.push_back(caf::PdmOptionItemInfo(m_timeStepNamesFromFile()[filteredIndex], static_cast<int>(filteredIndex)));
}
}
}
return optionItems;
}