Skip to content

Implement activation handling flawed #192

@pascalzauberzeug

Description

@pascalzauberzeug

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

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions