search

ioBroker / ioBroker.type-detector

Helper module (not adapter) to detect types of devices
MIT License
8 stars 9 forks source link

Fixed problem when read/write properties are string values. #12

Closed marc2016 closed 4 years ago

marc2016 commented 4 years ago

Dieses Paket wird in https://github.com/ioBroker/ioBroker.material verwendet und ich hatte dort ein Problem mit Objekten, die von dem TP-Link HS100 Adapter erzeugt wurden (https://github.com/arteck/ioBroker.hs100). Sie wurden nicht als light erkannte, wenn ich zusätzlich dieses Rolle ergänzt habe. Bei dem Objekt "state" in HS100 werden die read und write Eigenschaften scheinbar als string und nicht als boolean angelegt. Durch die bisherige Verarbeitung hier im Paket wird "true" nicht als true erkannt. Mein commit sollte dabei Abhilfe schaffen. Ich weiß, dass dies besser in dem HS100 Adapter zu beheben ist :-) Aber man könnte das Problem hier zusätzlich abfangen.

blackduck-copilot[bot] commented 4 years ago

Black Duck Security Report

Merging #12 into master will not change security risk.

Added Components

Clean: 5

Removed Components

Clean: 6

Click here to see full report

Garfonso commented 4 years ago

read / write müssen boolean sein. Eigentlich sollte es dazu mittlerweile beim anlegen des Objekts eine Warnung geben. Ich würde daher eher eine Änderung / PR / Issue beim HS100 machen.

marc2016 commented 4 years ago

Ich sehe gerade, dass dies hier behoben wurde HS100 Commit Die Version habe ich bei mir noch nicht drauf :-)

Mit meiner Anpassung könnte man nur vorsorglich diesem Verhalten entgegen wirken. Aber es stimmt, dass man die Objekte korrekt anlegen muss. Pull request kann dann gerne geschlossen werden.

GermanBluefox commented 4 years ago

Das hier ist sowieso immer "false": !!objects[id].common.read === undefined