From b7179fe373dedcdaf16e983fc343c8f74b4bbd5a Mon Sep 17 00:00:00 2001
From: Dave C-J <dave@conway-jones.co.uk>
Date: Fri, 13 Sep 2013 12:23:46 +0100
Subject: [PATCH] Better detection of BlinkStick - yet to do text colour values

---
 nodes/hardware/76-blinkstick.html |  4 ++--
 nodes/hardware/76-blinkstick.js   | 23 ++++++++++++++++++++---
 2 files changed, 22 insertions(+), 5 deletions(-)

diff --git a/nodes/hardware/76-blinkstick.html b/nodes/hardware/76-blinkstick.html
index fd21490b2..f3a6b0890 100644
--- a/nodes/hardware/76-blinkstick.html
+++ b/nodes/hardware/76-blinkstick.html
@@ -31,13 +31,13 @@
 <script type="text/javascript">
 	RED.nodes.registerType('blinkstick',{
 		category: 'advanced-output',
-		color:"cornsilk",
+		color:"GoldenRod",
 		defaults: {
 			name: {value:""}
 		},
 		inputs:1,
 		outputs:0,
-		icon: "arrow-in.png",
+		icon: "light.png",
 		align: "right",
 		label: function() {
 			return this.name||"blinkstick";
diff --git a/nodes/hardware/76-blinkstick.js b/nodes/hardware/76-blinkstick.js
index 3cb02d15f..9e76f638e 100644
--- a/nodes/hardware/76-blinkstick.js
+++ b/nodes/hardware/76-blinkstick.js
@@ -17,23 +17,40 @@
 var RED = require("../../red/red");
 var blinkstick = require("blinkstick");
 
+Object.size = function(obj) {
+	var size = 0, key;
+	for (key in obj) {
+		if (obj.hasOwnProperty(key)) size++;
+	}
+	return size;
+};
+
 function BlinkStick(n) {
 	RED.nodes.createNode(this,n);
-	var p1 = /^#.*/
+	var p1 = /^\#[A-Za-z0-9]{6}$/
 	var p2 = /[0-9]+,[0-9]+,[0-9]+/
-
-	this.led = new blinkstick.findFirst();
+	this.led = blinkstick.findFirst(); // maybe try findAll() ?
 	var node = this;
 
 	node.log("started");
 	this.on("input", function(msg) {
 		if (msg != null) {
+			if (Object.size(blinkstick.findFirst()) !== 0) {
+				if (p2.test(msg.payload)) {
+					var rgb = msg.payload.split(",");
+					node.led.setColor(parseInt(rgb[0]), parseInt(rgb[1]), parseInt(rgb[2]));
+				}
 				if ((p1.test(msg.payload))|(p2.test(msg.payload))) {
 					node.led.setColor(msg.payload);
 				}
+				// TODO - need to handle colour strings...
 				else {
 					node.error("Incorrect format: "+msg.payload);
 				}
+			}
+			else {
+				node.error("No BlinkStick found");
+			}
 		}
 	});