feat(property): return properties as object instead of array. resolves #20

Author: missinglink

module/property/StatementFetch.js

@@ -16,6 +16,13 @@ class StatementFetch extends SqliteStatement {
       this.error('PREPARE STATEMENT', e)
     }
   }
+  // reduce results (convert from array to object)
+  _transform (res) {
+    return res.reduce((memo, item) => {
+      memo[ item.key ] = item.value
+      return memo
+    }, {})
+  }
 }
 
 module.exports = StatementFetch

module/property/StatementFetch.test.js

@@ -53,13 +53,10 @@ module.exports.tests.function = (test, common) => {
     })
 
     // test response structure
-    t.deepEqual(rows, [{
-      key: 'example_key_1',
-      value: 'example_value_1'
-    }, {
-      key: 'example_key_2',
-      value: 'example_value_2'
-    }], 'read')
+    t.deepEqual(rows, {
+      example_key_1: 'example_value_1',
+      example_key_2: 'example_value_2'
+    }, 'read')
 
     t.end()
   })

server/demo/assets/links.js

@@ -21,12 +21,8 @@ function decorateLink (el) {
 
   api.property({ source: source, id: id }, {}, function (err, res) {
     if (err) { console.error(err) } else {
-      (res || []).forEach(function (prop) {
-        if (prop.key === 'name') {
-          el.attr('href', '/demo/place/' + encodeURIComponent(source) + '/' + encodeURIComponent(id))
-          el.text(prop.value)
-        }
-      })
+      el.attr('href', '/demo/place/' + encodeURIComponent(source) + '/' + encodeURIComponent(id))
+      el.text(_.get(res, 'name', 'unknown'))
     }
   })
 }

server/demo/views/pages/place.hbs

@@ -72,7 +72,7 @@
       <table class="table is-fullwidth">
         <tbody>
           {{#each params.place.property}}
-            <tr><th>{{this.key}}</th><td>{{this.value}}</td></tr>
+            <tr><th>{{@key}}</th><td>{{this}}</td></tr>
           {{/each}}
         </tbody>
       </table>

sqlite/SqliteStatement.js

@@ -1,17 +1,18 @@
 const Sqlite = require('./Sqlite')
 class SqliteStatement extends Sqlite {
   _selectStatement () { return this.statement }
+  _transform (res) { return res }
   run () {
     let stmt = this._selectStatement.apply(this, arguments)
-    return stmt.run.apply(stmt, arguments)
+    return this._transform(stmt.run.apply(stmt, arguments))
   }
   get () {
     let stmt = this._selectStatement.apply(this, arguments)
-    return stmt.get.apply(stmt, arguments)
+    return this._transform(stmt.get.apply(stmt, arguments))
   }
   all () {
     let stmt = this._selectStatement.apply(this, arguments)
-    return stmt.all.apply(stmt, arguments)
+    return this._transform(stmt.all.apply(stmt, arguments))
   }
 }