addAlarm method in position class must be refactor

    public void addAlarm(String alarm) {
        if (alarm != null) {
            if (hasAttribute(KEY_ALARM)) {
                set(KEY_ALARM, getAttributes().get(alarm) + "," + alarm);
            } else {
                set(KEY_ALARM, alarm);
            }
        }
    }

we should have get(alarm) instead get(KEY_ALARM)

What? Why?

for example we want to have this item in attributes: {"alarm": "powerCut,1") is it ok addAlarm method in traacar with this?

How is this related to official Traccar?

so why we use this method in protocol decoder? where the result of method store?