commit 99c58f55a3556a270e144a1cfb7016496e1e689d
author Paul Westbrook <pwestbro@google.com> Thu Oct 29 13:09:27 2015 -0700
committer Vitaly Buka <vitalybuka@google.com> Thu Oct 29 22:18:14 2015 +0000
tree 464b5ce188c0b6e74018925049e001634d2c970d
parent b70fd47becc57a280c4635e3d9925c62680eddd2

Update virtual light to standard schema

Change-Id: Ie51d16899bb6cbbcb8760bb450489f5f7c02e391
Reviewed-on: https://weave-review.googlesource.com/1440
Reviewed-by: Vitaly Buka <vitalybuka@google.com>

libweave/examples/daemon/light_handler.h

diff --git a/libweave/examples/daemon/light_handler.h b/libweave/examples/daemon/light_handler.h
index 08eb682..33b440a 100644
--- a/libweave/examples/daemon/light_handler.h
+++ b/libweave/examples/daemon/light_handler.h
@@ -30,20 +30,18 @@
     })",
                                        nullptr);
 
-    // Once bug b/25304415 is fixed, these should be changed
-    // to use the standard commands.
     device->AddCommandDefinitionsFromJson(R"({
       "onOff": {
-         "_setConfig":{
+         "setConfig":{
            "parameters": {
-             "_state": ["on", "standby"]
+             "state": ["on", "standby"]
            }
          }
        },
        "brightness": {
-         "_setConfig":{
+         "setConfig":{
            "parameters": {
-             "_brightness": {
+             "brightness": {
                "type": "integer",
                "minimum": 0,
                "maximum": 100
@@ -53,11 +51,11 @@
       }
     })");
     device->AddCommandHandler(
-        "onOff._setConfig",
+        "onOff.setConfig",
         base::Bind(&LightHandler::OnOnOffSetConfig,
                    weak_ptr_factory_.GetWeakPtr()));
     device->AddCommandHandler(
-        "brightness._setConfig",
+        "brightness.setConfig",
          base::Bind(&LightHandler::OnBrightnessSetConfig,
                     weak_ptr_factory_.GetWeakPtr()));
   }
@@ -69,7 +67,7 @@
       return;
     LOG(INFO) << "received command: " << cmd->GetName();
     int32_t brightness_value = 0;
-    if (cmd->GetParameters()->GetInteger("_brightness", &brightness_value)) {
+    if (cmd->GetParameters()->GetInteger("brightness", &brightness_value)) {
       // Display this command in terminal.
       LOG(INFO) << cmd->GetName() << " brightness: " << brightness_value;
 
@@ -92,7 +90,7 @@
       return;
     LOG(INFO) << "received command: " << cmd->GetName();
     std::string requested_state;
-    if (cmd->GetParameters()->GetString("_state", &requested_state)) {
+    if (cmd->GetParameters()->GetString("state", &requested_state)) {
       LOG(INFO) << cmd->GetName() << " state: " << requested_state;
 
       bool new_light_status = requested_state == "on";