-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Description
While reviewing #190, I noticed that the state-handling of implements is neither coherent nor working correctly.
All implements know whether they are activated or not and that state can be changed by Implement.activate
and Implement.deactivate
. They are however not used by the base navigation class, but called by the navigations that extend each other, which leads to activate
being called multiple times at different stages of the preparation process.
Additionally WeedingImplent
tracks it's own state which should not be the case. Especially because it's never stopped after being started.
When updating the KPIs, implements should either check if they are active or not. Or maybe better not handle the KPIs at all and let the KPI-provider do that.
Metadata
Metadata
Assignees
Labels
No labels