statistics: reverse chart selection logic

The old ways was to select the chart first, then depending on
the chart choose the binning.

Willem says that it should work the other way round: select
the binning (or operation) and make the charts depend on
that.

I'm not arguing one way or the other, just note that the new
way is much more tricky, because it is easy to get unsupported
combinations. For example, there is no chart where the
first variable is unbinned, but the second axis is binned
or has an operation. This makes things distinctly more tricky
and this code still needs a thorough audit.

Since this is all more tricky, implement a "invalid" chart
state. Ideally that should be never shown to the user, but
let's try to be defensive.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 2 insertions, 0 deletions

diff --git a/stats/statsview.cpp b/stats/statsview.cpp
index ba5e8c24e..7df8589b9 100644
--- a/stats/statsview.cpp
+++ b/stats/statsview.cpp
@@ -200,6 +200,8 @@ void StatsView::plot(const StatsState &stateIn)
 		return plotHistogramBoxChart(dives, state.var1, state.var1Binner, state.var2);
 	case ChartType::ScatterPlot:
 		return plotScatter(dives, state.var1, state.var2);
+	case ChartType::Invalid:
+		return;
 	default:
 		qWarning("Unknown chart type: %d", (int)state.type);
 		return;