diff --git a/src/EnergyPlus/EconomicTariff.cc b/src/EnergyPlus/EconomicTariff.cc index 0fa54fdaff5..a60c844ae5a 100644 --- a/src/EnergyPlus/EconomicTariff.cc +++ b/src/EnergyPlus/EconomicTariff.cc @@ -2716,7 +2716,7 @@ void ComputeTariff(EnergyPlusData &state) Real64 annualAggregate; - if (!(state.files.outputControl.tabular || state.files.outputControl.sqlite)) { + if (!state.files.outputControl.writeTabular(state)) { state.dataOutRptTab->WriteTabularFiles = false; return; } diff --git a/src/EnergyPlus/IOFiles.cc b/src/EnergyPlus/IOFiles.cc index d5dd1e3ffc4..614a27abcfd 100644 --- a/src/EnergyPlus/IOFiles.cc +++ b/src/EnergyPlus/IOFiles.cc @@ -319,6 +319,14 @@ std::vector InputOutputFile::getLines() return std::vector(); } +bool IOFiles::OutputControl::writeTabular(EnergyPlusData &state) +{ + bool const htmlTabular = state.files.outputControl.tabular; + bool const jsonTabular = state.files.outputControl.json && state.dataResultsFramework->resultsFramework->timeSeriesAndTabularEnabled(); + bool const sqliteTabular = state.files.outputControl.sqlite; // && @JasonGlazer thinks something else maybe? + return (htmlTabular || jsonTabular || sqliteTabular); +} + void IOFiles::OutputControl::getInput(EnergyPlusData &state) { auto &ip = state.dataInputProcessing->inputProcessor; diff --git a/src/EnergyPlus/IOFiles.hh b/src/EnergyPlus/IOFiles.hh index 426313bd1fa..d93b8b67978 100644 --- a/src/EnergyPlus/IOFiles.hh +++ b/src/EnergyPlus/IOFiles.hh @@ -673,6 +673,7 @@ public: OutputControl() = default; void getInput(EnergyPlusData &state); + bool writeTabular(EnergyPlusData &state); bool csv = false; bool mtr = true; diff --git a/src/EnergyPlus/OutputReportTabular.cc b/src/EnergyPlus/OutputReportTabular.cc index 993db07fb51..9316ad16cda 100644 --- a/src/EnergyPlus/OutputReportTabular.cc +++ b/src/EnergyPlus/OutputReportTabular.cc @@ -323,7 +323,7 @@ void GetInputTabularMonthly(EnergyPlusData &state) // SUBROUTINE LOCAL VARIABLE DECLARATIONS: auto &ort = state.dataOutRptTab; - if (!(state.files.outputControl.tabular || state.files.outputControl.sqlite)) { + if (!state.files.outputControl.writeTabular(state)) { ort->WriteTabularFiles = false; return; } @@ -1053,7 +1053,7 @@ void GetInputTabularTimeBins(EnergyPlusData &state) auto &ort = state.dataOutRptTab; - if (!(state.files.outputControl.tabular || state.files.outputControl.sqlite)) { + if (!state.files.outputControl.writeTabular(state)) { ort->WriteTabularFiles = false; return; } @@ -1461,7 +1461,7 @@ void GetInputOutputTableSummaryReports(EnergyPlusData &state) auto &ort = state.dataOutRptTab; bool ErrorsFound = false; - if (!(state.files.outputControl.tabular || state.files.outputControl.sqlite)) { + if (!state.files.outputControl.writeTabular(state)) { ort->WriteTabularFiles = false; return; }