From 22a15f5ed67bd7577cf51fe1bcb0e0b2a5a7c277 Mon Sep 17 00:00:00 2001 From: Arthur Camouseigt Date: Wed, 29 Jul 2020 16:05:08 +0200 Subject: [PATCH] [Shared/store_parameter_controler] Fixed a bug In stats when selecting an action on a column, all actions listed after the one selected were applied. Adding breaks to the switch solved the issue Change-Id: I4a2f8a41f734a209abb17e76388eed551bf1769c --- apps/shared/store_parameter_controller.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/apps/shared/store_parameter_controller.cpp b/apps/shared/store_parameter_controller.cpp index 0bd90d14f..50212fee2 100644 --- a/apps/shared/store_parameter_controller.cpp +++ b/apps/shared/store_parameter_controller.cpp @@ -51,10 +51,12 @@ bool StoreParameterController::handleEvent(Ion::Events::Event event) { } else { m_store->resetColumn(m_series, !m_xColumnSelected); } + break; } case 1: { m_storeController->displayFormulaInput(); + break; } case 2: { @@ -85,6 +87,7 @@ bool StoreParameterController::handleEvent(Ion::Events::Event event) { } else { Poincare::Helpers::Sort(swapRows, compareY, (m_store->data() + m_series), m_store->numberOfPairsOfSeries(m_series)); } + break; } } assert(selectedRow() >= 0 && selectedRow() <= 2);