fix: read only list values in UI and better logging (#102)

* fix: undefined values * fix: add more logging * fix(ui): show read only list values
zwave-js · Dec 22, 2020 · 03f5610 · 03f5610
1 parent ffa300b
commit 03f5610
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 8 deletions.
diff --git a/app.js b/app.js
@@ -170,7 +170,7 @@ function setupSocket (server) {
   })
 
   socketManager.on(inboundEvents.zwave, async function (socket, data) {
-    logger.info(`Zwave api call: ${data.api} ${data.args}`)
+    logger.info(`Zwave api call: ${data.api} ${JSON.stringify(data.args)}`)
     if (gw.zwave) {
       const result = await gw.zwave.callApi(data.api, ...data.args)
       result.api = data.api

diff --git a/lib/ZwaveClient.js b/lib/ZwaveClient.js
@@ -423,7 +423,7 @@ function onNodeValueRemoved (zwaveNode, args) {
 }
 
 function onNodeMetadataUpdated (zwaveNode, args) {
-  const valueId = initValue.call(this, zwaveNode, args, args.metadata)
+  const valueId = parseValue.call(this, zwaveNode, args, args.metadata)
   logger.info(`Node ${valueId.nodeId}: metadata updated: ${getValueID(args)}`)
   this.emit(
     'event',
@@ -691,7 +691,7 @@ function addValue (zwaveNode, zwaveValue) {
   } else {
     const zwaveValueMeta = zwaveNode.getValueMetadata(zwaveValue)
 
-    const valueId = initValue.call(this, zwaveNode, zwaveValue, zwaveValueMeta)
+    const valueId = parseValue.call(this, zwaveNode, zwaveValue, zwaveValueMeta)
 
     logger.info(
       `Node ${zwaveNode.id}: value added ${valueId.id} => ${valueId.value}`
@@ -701,7 +701,7 @@ function addValue (zwaveNode, zwaveValue) {
   }
 }
 
-function initValue (zwaveNode, zwaveValue, zwaveValueMeta) {
+function parseValue (zwaveNode, zwaveValue, zwaveValueMeta) {
   const node = this.nodes[zwaveNode.id]
   const valueId = updateValueMetadata.call(
     this,
@@ -1890,6 +1890,8 @@ ZwaveClient.prototype.callApi = async function (apiName, ...args) {
  */
 ZwaveClient.prototype.writeValue = async function (valueId, value) {
   if (this.driverReady) {
+    logger.info(`Writing ${value} to ${valueId.id}`)
+
     let result = false
 
     if (

diff --git a/src/components/ControlPanel.vue b/src/components/ControlPanel.vue
@@ -1329,6 +1329,7 @@ export default {
     },
     initNode (n) {
       var values = []
+      // transform object in array
       for (var k in n.values) {
         n.values[k].newValue = n.values[k].value
         values.push(n.values[k])

diff --git a/src/components/ValueId.vue b/src/components/ValueId.vue
@@ -1,5 +1,5 @@
 <template>
-  <div v-if="!value.writeable">
+  <div v-if="!value.writeable && !value.list">
     <v-text-field
       :label="value.label + ' (' + value.id + ')'"
       readonly
@@ -18,11 +18,11 @@
             value.type === 'any')
       "
       :label="value.label + ' (' + value.id + ')'"
-      :type="value.type === 'string' ? 'text' : 'number'"
+      :type="value.type === 'number' ? 'number' : 'text'"
       :append-outer-icon="!disable_send ? 'send' : null"
       :suffix="value.units"
       :min="value.min != value.max ? value.min : null"
-      :step="value.type == 'decimal' ? 0.1 : 1"
+      :step="1"
       :max="value.min != value.max ? value.max : null"
       :hint="value.description || ''"
       v-model="value.newValue"
@@ -34,8 +34,9 @@
       :items="value.states"
       :label="value.label + ' (' + value.id + ')'"
       :hint="value.description || ''"
-      :append-outer-icon="!disable_send ? 'send' : null"
+      :append-outer-icon="!disable_send || value.writeable ? 'send' : null"
       v-model="value.newValue"
+      :readonly="!value.writeable"
       @click:append-outer="updateValue(value)"
     ></v-select>