From 47eee217fba71f97d651f747f120d23f869bcd3c Mon Sep 17 00:00:00 2001
From: Devon Rifkin <drifkin@drifkin.net>
Date: Sat, 8 Jun 2013 18:42:12 -0700
Subject: [PATCH] Consider group 0 to be a valid group id

This allow us to use setLightState() to modify all lights
---
 hue-api/api.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/hue-api/api.js b/hue-api/api.js
index 9a50526..69f56e7 100644
--- a/hue-api/api.js
+++ b/hue-api/api.js
@@ -246,6 +246,11 @@ HueApi.prototype.updateGroup = function (groupId, name, lightIds) {
         throw new errors.ApiError("Group Id '" + groupId + "' is not valid for the bridge");
     }
 
+    // Protect the default implicit all lights group from being updated
+    if (groupId === 0 || groupId === "0") {
+        throw new errors.ApiError("Cannot update the 'All Lights' default group");
+    }
+
     var parseResults = function (result) {
             if (!_wasSuccessful(result)) {
                 throw new errors.ApiError(_parseErrors(result).join(", "));
@@ -315,7 +320,7 @@ HueApi.prototype.createGroup = function (name, lightIds) {
  */
 HueApi.prototype.deleteGroup = function (id) {
     // Protect the default implicit all lights group from being deleted (not sure if this is even possible, but do not want to risk it)
-    if (id === 0) {
+    if (id === 0 || id === "0") {
         throw new errors.ApiError("Cannot delete the 'All Lights' default group");
     }
 
@@ -750,7 +755,7 @@ function _isLightIdValid(id) {
 }
 
 function _isGroupIdValid(id) {
-    if (parseInt(id, 10) > 0) {
+    if (parseInt(id, 10) >= 0) {
         //TODO: check group is valid
         //Keep in mind, group 0 is always valid, stands for ALL lights known by hue bridge
         return true;