From 8e6d009d676c9c2dc10e59fd56df6a977d5b6da6 Mon Sep 17 00:00:00 2001
From: Vinay Rao <workingmonk@gmail.com>
Date: Thu, 2 Feb 2017 18:52:23 -0800
Subject: [PATCH] changes suggested by Tom and Zach

---
 .../zigbee-button.src/zigbee-button.groovy    | 25 ++++++++-----------
 1 file changed, 10 insertions(+), 15 deletions(-)

diff --git a/devicetypes/smartthings/zigbee-button.src/zigbee-button.groovy b/devicetypes/smartthings/zigbee-button.src/zigbee-button.groovy
index 97948d1..47dd320 100644
--- a/devicetypes/smartthings/zigbee-button.src/zigbee-button.groovy
+++ b/devicetypes/smartthings/zigbee-button.src/zigbee-button.groovy
@@ -119,36 +119,31 @@ private Map parseNonIasButtonMessage(Map descMap){
     def buttonState = ""
     def buttonNumber = 0
     if ((device.getDataValue("model") == "3460-L") &&(descMap.clusterInt == 0x0006)) {
-        if (descMap.command == "01") {
+        if (descMap.commandInt == 1) {
             getButtonResult("press")
         }
-        else if (descMap.command == "00") {
+        else if (descMap.commandInt == 0) {
             getButtonResult("release")
         }
     }
     else if ((device.getDataValue("model") == "3450-L") && (descMap.clusterInt == 0x0006)) {
-    	log.info "For 3450-L"
-        log.trace "descMap : $descMap"
-        if (descMap.command == "01") {
+        if (descMap.commandInt == 1) {
             getButtonResult("press")
         }
-        else if (descMap.command == "00") {
-        	def button = 1
+        else if (descMap.commandInt == 0) {
+            def button = 1
             switch(descMap.sourceEndpoint) {
-            	case "01":
-                	button = 4
+                case "01":
+                    button = 4
                     break
                 case "02":
-                	button = 3
+                    button = 3
                     break
                 case "03":
-                	button = 1
+                    button = 1
                     break
                 case "04":
-                	button = 2
-                    break
-                default:
-                	button = 1
+                    button = 2
                     break
             }