fix: unknown manufacturer

[robertsLando]

Fixes #373
Fixes #347

[coveralls]

Pull Request Test Coverage Report for Build 520238894

• 0 of 29 (0.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.04%) to 21.547%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
lib/ZwaveClient.js	0	29	0.0%

Totals
Change from base Build 520238236:	-0.04%
Covered Lines:	2044
Relevant Lines:	9713

---

💛 - Coveralls

[AlCalzone]

Like we discussed, I would do it differently. When node is ready, use getDefinedValueIDs() to

1. detect which values are added and add them
2. detect which values were removed and remove them
3. update the others.

[robertsLando]

Like we discussed, I would do it differently. When node is ready, use getDefinedValueIDs() to

Isn't the result of what I did the same?

[AlCalzone]

The way I'm seeing it, you do 3. but not 1. or 2.

Edit: you also do 1., but with value updated event

[robertsLando]

@AlCalzone When node is ready I get a reference of the existing node values, clear all actual node values , I get all the new defined values, I parse one by one and I check if it was existing (and if it has changed). The removed ones are not there so and the new ones are reported

[AlCalzone]

Ok then I didn't understand the consequences.