NPE in some converters when calculating updated attributes

[TomTravaglino]

Caused by #427 and #409

I'm experiencing a NPE in the method ZigBeeConverterMeasurementPower.attributeUpdated(...) thrown by this statement:

BigDecimal valueInWatt = BigDecimal.valueOf(value * multiplier / divisor);

The reason is that multiplier and divisor are both null. This happens because since #427 these variables are initialized in the method initializeDevice() which since #409 is only invoked on the very first initialization and then only initializeConverter() will be called on startup. Such variables should be better initialized in the method initializeConverter().

Other converters which currently use to initialize variables in initializeDevice() will need to be changed too (didn't check them all).

[cdjackson]

Please can you provide a PR for this?

…

On 3 May 2019, at 08:22, Tommaso Travaglino ***@***.***> wrote: Caused by #427 <#427> and #409 <#409> I'm experiencing a NPE in the method ZigBeeConverterMeasurementPower.attributeUpdated(...) thrown by this statement: BigDecimal valueInWatt = BigDecimal.valueOf(value * multiplier / divisor); The reason is that multiplier and divisor are both null. This happens because since #427 <#427> these variables are initialized in the method initializeDevice() which since #409 <#409> is only invoked on the very first initialization and then only initializeConverter() will be called on startup. Such variables should be better initialized in the method initializeConverter(). Other converters which currently use to initialize variables in initializeDevice() will need to be changed too (didn't check them all). — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#432>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAH6IQ27U67BLHGKOMREPTDPTPR4PANCNFSM4HKRW7GA>.

[cdjackson]

Duplicate of #430