[patch] 74_AMAD.pm: Use of uninitialized value in hash element

Begonnen von gandy, 09 Juni 2016, 10:37:56

Vorheriges Thema - Nächstes Thema

gandy

Hi CoolTux,

anbei ein kleiner Patch, der folgendes Problem behebt:

Devices mit Namen AMADCommBridge_{IPADDR}_{PORT} haben weder das Internal BRIDGE noch HOST gesetzt, weswegen es in AMAD_Undef() zu zahlreichen Logeinträgen folgender Art kommt:

2016.06.07 14:31:27.420 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.
2016.06.07 14:31:27.448 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.
2016.06.07 14:31:57.357 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.
2016.06.07 14:31:57.361 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.
2016.06.07 14:31:57.603 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.
2016.06.07 14:31:57.653 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.
2016.06.07 14:31:57.702 1: PERL WARNING: Use of uninitialized value in hash element at /opt/fhem/FHEM/74_AMAD.pm line 147.


Möglicherweise muss die Unterscheidungung anders lauten oder es bedarf einer weiteren Verzweigung zur Behandlung dieser Art Devices, das kann ich auf die Schnelle nicht beurteilen..

Beste Grüße,
Andy.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

CoolTux

#1
Hallo Andy,

Der Patch ist ja wirklich Süß. Aber vielen vielen Dank. Manchmal sieht man die einfachsten Sachen nicht. Hatte mal hin und wieder mich drum gekümmert aber nie wirklich intensiv Zeit gefunden.



Grüße
Leon
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

gandy

Wenn es Dir hilft, die Änderung einzupflegen und ich mich dadurch wieder auf Logeinträge konzentrieren kann, die mir bei meiner Problemsuche helfen, steuere ich gern auch mal einen süßen Einzeiler als Patch bei ;-)
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

CoolTux

Ist bereits eingepflegt und kommt mit Version 2.2.2 mit. Auch der Eintrag im Wiki wurde entsprechend angepasst.


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net