Features do not respect the UserFeature superclass `userEnabled` field about artemis

[kristofbolyai]

@P0keDev Oh, then can I assume that you will solve the issues I mentioned above, and not worry about them in the config command PR?

[magicus]

Should we unregister the feature as a listener for events when it is turned off?

[kristofbolyai]

Should we unregister the feature as a listener for events when it is turned off?

I think we used to, at least in my head :). Unregistering greatly reduces the overhead of the mod and improves the performance of the event bus.

[P0keDev]

We still do:

public final void disable() {
        if (!enabled) throw new IllegalStateException("Feature can not be disabled as it already is disabled");

        onDisable();

        enabled = false;

        if (isListener) {
            WynntilsMod.getEventBus().unregister(this);
        }
        for (KeyHolder key : keyMappings) {
            KeyManager.unregisterKeybind(key);
        }
    }

[P0keDev]

Regarding the userEnabled functionality - I plan on tackling this next

[kristofbolyai]

Then the issue is that it is not getting respected. See ItemHighlightFeature which, I tested, and did not work correctly. Or was it the config command not updating the state? If so, we need something to force a config save and feature state reload, which is missing from #112

[P0keDev]

fixed in #114