From 80d1e85001578b6d1253da4b2d797895a2f2eb24 Mon Sep 17 00:00:00 2001 From: kaihs <> Date: Thu, 14 Aug 2014 19:04:34 +0000 Subject: [PATCH] WMBUS: better error checking and documenation fixes git-svn-id: https://svn.fhem.de/fhem/trunk@6406 2b470e98-0d58-463d-a4d8-8e2adae1ed80 --- fhem/FHEM/36_WMBUS.pm | 82 +++++++++++++++++++++++-------------------- fhem/FHEM/WMBus.pm | 7 ++-- 2 files changed, 47 insertions(+), 42 deletions(-) diff --git a/fhem/FHEM/36_WMBUS.pm b/fhem/FHEM/36_WMBUS.pm index 7d3e108a9..5114698df 100644 --- a/fhem/FHEM/36_WMBUS.pm +++ b/fhem/FHEM/36_WMBUS.pm @@ -69,20 +69,24 @@ WMBUS_Define($$) } else { # manual specification -# $a[2] =~ m/[A-Z]{3}/; -# return "$a[2] is not a valid WMBUS manufacturer id" if( !defined($1) ); + if ($a[2] !~ m/[A-Z]{3}/) { + return "$a[2] is not a valid WMBUS manufacturer id"; + } -# $a[3] =~ m/[0-9]{1,8}/; -# return "$a[3] is not a valid WMBUS serial number id" if( !defined($1) ); + if ($a[3] !~ m/[0-9]{1,8}/) { + return "$a[3] is not a valid WMBUS serial number"; + } -# $a[4] =~ m/[0-9]{1,2}/; -# return "$a[4] is not a valid WMBUS version" if( !defined($1) ); + if ($a[4] !~ m/[0-9]{1,2}/) { + return "$a[4] is not a valid WMBUS version"; + } -# $a[5] =~ m/[0-9]{1,2}/; -# return "$a[5] is not a valid WMBUS type" if( !defined($1) ); + if ($a[5] !~ m/[0-9]{1,2}/) { + return "$a[5] is not a valid WMBUS type"; + } $hash->{Manufacturer} = $a[2]; - $hash->{IdentNumber} = $a[3]; + $hash->{IdentNumber} = int($a[3]); $hash->{Version} = $a[4]; $hash->{DeviceType} = $a[5]; @@ -402,24 +406,24 @@ WMBUS_Attr(@)

WMBUS - Wireless M-Bus