fromZigbee.tuya_switch_power_outage_memory doesn't work, because of attribute check

[gorbunovav]

There is a check for attribute 0x8002:

zigbee-herdsman-converters/converters/fromZigbee.js

Line 7276 in a88c04c

const property = 0x8002;

But this attribute is listed under name moesStartUpOnOff in cluster definition:
moesStartUpOnOff
so the method receives it as a string and the check always fails.

There is also newer moes_power_on_behavior converter, which looks similar, but uses correct attribute name:

zigbee-herdsman-converters/converters/fromZigbee.js

Line 2561 in a88c04c

moes_power_on_behavior: {

At the toZigbee side converters look close also, but the moes_power_on_behavior variant contains the get converter in addition:

zigbee-herdsman-converters/converters/toZigbee.js

Line 2580 in a88c04c

tuya_switch_power_outage_memory: {

zigbee-herdsman-converters/converters/toZigbee.js

Line 3583 in a88c04c

moes_power_on_behavior: {

[Koenkk]

Could you submit a pr to fix this?

[gorbunovav]

@Koenkk yes, I can.

The simple fix would be to just update the attribute reference in tuya_switch_power_outage_memory.

But I wanted to clarify first - do we need both moes_power_on_behavior and tuya_switch_power_outage_memory converters or can they be merged?

[Koenkk]

The simple fix would be to just update the attribute reference in tuya_switch_power_outage_memory.

Sounds good

But I wanted to clarify first - do we need both moes_power_on_behavior and tuya_switch_power_outage_memory converters or can they be merged?

Lets keep both since changing this would be somewhat a breaking change.