From 39b1ae0fbfa9cdfb0cfff3ec577d5e42ddcc0b1a Mon Sep 17 00:00:00 2001
From: KuhnChris <kuhnchris@kuhnchris.eu>
Date: Wed, 1 May 2019 17:03:51 +0200
Subject: [PATCH 1/4] uport-project -> 3box

Even more changes!
---
 package.json | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/package.json b/package.json
index ae72ab87..8341b22f 100644
--- a/package.json
+++ b/package.json
@@ -34,14 +34,14 @@
   },
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/uport-project/3box-js.git"
+    "url": "git+https://github.com/3box/3box-js.git"
   },
   "author": "3box <team@3box.io>",
   "license": "Apache-2.0",
   "bugs": {
-    "url": "https://github.com/uport-project/3box-js/issues"
+    "url": "https://github.com/3box/3box-js/issues"
   },
-  "homepage": "https://github.com/uport-project/3box-js#readme",
+  "homepage": "https://github.com/3box/3box-js#readme",
   "dependencies": {
     "@babel/runtime": "^7.1.2",
     "did-jwt": "^0.1.1",

From 837d6337ee987c91a2ef3bf8d8e76131ce5d95b0 Mon Sep 17 00:00:00 2001
From: Jonathan Schwartz <jpschwartz2@uwalumni.com>
Date: Mon, 29 Apr 2019 15:35:42 -0400
Subject: [PATCH 2/4] feat(api): add setMultiple

---
 readme-template.md                  | 15 +++++++++++-
 src/__mocks__/keyValueStore.js      | 12 ++++++++++
 src/__tests__/keyValueStore.test.js |  8 +++++++
 src/__tests__/privateStore.test.js  | 17 ++++++++++++++
 src/__tests__/publicStore.test.js   | 16 ++++++++++++-
 src/__tests__/space.test.js         | 36 +++++++++++++++++++++++++----
 src/keyValueStore.js                | 26 ++++++++++++++++++++-
 src/privateStore.js                 |  7 ++++++
 src/publicStore.js                  |  8 ++++++-
 src/space.js                        | 13 ++++++++++-
 src/utils/index.js                  | 10 ++++++++
 11 files changed, 159 insertions(+), 9 deletions(-)

diff --git a/readme-template.md b/readme-template.md
index 12477186..4ad0bef5 100644
--- a/readme-template.md
+++ b/readme-template.md
@@ -88,6 +88,19 @@ await box.private.set('email', 'oed@email.service')
 await box.private.remove('email')
 ```
 
+##### Set multiple fields at once:
+```js
+const fields = ['name', 'website', 'employer']
+const values = ['Jon Schwartz', 'openworklabs.com', 'Open Work Labs']
+
+await box.public.setMultiple(fields, values)
+
+const privateFields = ['age', 'coinBalance']
+const privateValues = ['xxx', 'yyy']
+
+await box.private.setMultiple(privateFields, privateValues)
+```
+
 <!-- commenting this out for now, not really needed when we're not using the iframe
 #### IPFS Configs
 
@@ -143,7 +156,7 @@ For the fully detailed spec, view the [documentation](https://github.com/3box/3b
 **WARNING: this is an experimental feature, the API will likely change in the future!**
 
 #### Viewing a Thread
-You can get all posts made in a thread without opening a space. This is great for allowing visitors of your site view comments made by other users. This is achieved by calling the `getThread` method on the Box object. 
+You can get all posts made in a thread without opening a space. This is great for allowing visitors of your site view comments made by other users. This is achieved by calling the `getThread` method on the Box object.
 ```js
 const posts = await Box.getThread(spaceName, threadName)
 console.log(posts)
diff --git a/src/__mocks__/keyValueStore.js b/src/__mocks__/keyValueStore.js
index abc75e5d..8a578ede 100644
--- a/src/__mocks__/keyValueStore.js
+++ b/src/__mocks__/keyValueStore.js
@@ -1,3 +1,5 @@
+import { throwIfNotEqualLenArrays } from '../utils'
+
 class KeyValueStore {
   constructor (orbitdb, name, ensureConnected, threeId) {
     this._orbitdb = orbitdb
@@ -17,6 +19,16 @@ class KeyValueStore {
     return true
   }
 
+  async setMultiple(keys, values) {
+    this._requireLoad()
+    throwIfNotEqualLenArrays(keys, values)
+    try {
+      keys.map((key, i) => this._db.set(key, values[i]))
+    } catch (error) {
+      throw new Error(error)
+    }
+  }
+
   async remove (key) {
     this._requireLoad()
     this._db.remove(key)
diff --git a/src/__tests__/keyValueStore.test.js b/src/__tests__/keyValueStore.test.js
index 2dfac8f8..7f293052 100644
--- a/src/__tests__/keyValueStore.test.js
+++ b/src/__tests__/keyValueStore.test.js
@@ -34,6 +34,7 @@ describe('KeyValueStore', () => {
 
   it('should throw if not synced', async () => {
     expect(keyValueStore.set('key', 'value')).rejects.toThrow(/_load must/)
+    expect(keyValueStore.setMultiple(['keys'], ['values'])).rejects.toThrow(/_load must/)
     expect(keyValueStore.get('key')).rejects.toThrow(/_load must/)
     expect(keyValueStore.remove('key')).rejects.toThrow(/_load must/)
   })
@@ -58,6 +59,13 @@ describe('KeyValueStore', () => {
     expect(ensureConnected).toHaveBeenCalledTimes(3)
   })
 
+  it('should set and get multiple values correctly', async () => {
+    await keyValueStore.setMultiple(['key4', 'key5'], ['yoyo', 'ma'])
+    expect(await keyValueStore.get('key4')).toEqual('yoyo')
+    expect(await keyValueStore.get('key5')).toEqual('ma')
+    expect(ensureConnected).toHaveBeenCalledTimes(1)
+  })
+
   it('should remove values correctly', async () => {
     await keyValueStore.remove('key3')
     expect(await keyValueStore.get('key3')).toBeUndefined()
diff --git a/src/__tests__/privateStore.test.js b/src/__tests__/privateStore.test.js
index 70498a31..7811528d 100644
--- a/src/__tests__/privateStore.test.js
+++ b/src/__tests__/privateStore.test.js
@@ -88,6 +88,17 @@ describe('PrivateStore', () => {
     expect(privateStore._decryptEntry(retreived)).toEqual('12345')
   })
 
+  it('should set multiple values correctly', async () => {
+    await privateStore._load()
+    await privateStore.setMultiple(['key4', 'key5'], ['yoyo', 'ma'])
+    let dbkey = privateStore._genDbKey('key4')
+    let retreived = privateStore._db.get(dbkey)
+    expect(privateStore._decryptEntry(retreived)).toEqual('yoyo')
+    dbkey = privateStore._genDbKey('key5')
+    retreived = privateStore._db.get(dbkey)
+    expect(privateStore._decryptEntry(retreived)).toEqual('ma')
+  })
+
   it('should get values correctly', async () => {
     let value = await privateStore.get('key1')
     expect(value).toEqual('value1')
@@ -97,6 +108,12 @@ describe('PrivateStore', () => {
 
     value = await privateStore.get('key3')
     expect(value).toEqual('12345')
+
+    value = await privateStore.get('key4')
+    expect(value).toEqual('yoyo')
+
+    value = await privateStore.get('key5')
+    expect(value).toEqual('ma')
   })
 
   it('should remove values correctly', async () => {
diff --git a/src/__tests__/publicStore.test.js b/src/__tests__/publicStore.test.js
index 566a7456..583715cd 100644
--- a/src/__tests__/publicStore.test.js
+++ b/src/__tests__/publicStore.test.js
@@ -37,7 +37,21 @@ describe('PublicStore', () => {
     expect(publicStore.set()).rejects.toEqual(new Error('key is a required argument'))
   })
 
+  it('should set multiple keys', async () => {
+    await publicStore._load()
+    await publicStore.setMultiple(['key2', 'key3'], ['value2', 'value3'])
+    const value2 = await publicStore.get('key2')
+    const value3 = await publicStore.get('key3')
+    expect(value2).toEqual('value2')
+    expect(value3).toEqual('value3')
+  })
+
+  it('should throw an error if multiple keys are not arrays of equal length arrays', async () => {
+    expect(publicStore.setMultiple(['key'], ['value', 'value1'])).rejects.toEqual(new Error('Arrays must be of the same length'))
+    expect(publicStore.setMultiple('key', ['value', 'value1'])).rejects.toEqual(new Error('One or more arguments are not an array'))
+  })
+
   it('should return profile correctly', async () => {
-    expect(await publicStore.all()).toEqual({ key1: 'value1' })
+    expect(await publicStore.all()).toEqual({ key1: 'value1', key2: 'value2', key3: 'value3' })
   })
 })
diff --git a/src/__tests__/space.test.js b/src/__tests__/space.test.js
index fb3fd949..b9319617 100644
--- a/src/__tests__/space.test.js
+++ b/src/__tests__/space.test.js
@@ -98,8 +98,20 @@ describe('Space', () => {
       expect(await space.public.get('k2')).toBeUndefined()
     })
 
+    it('setMultiple works correctly', async () => {
+      await space.public.setMultiple(['k4', 'k5', 'k6'], ['v4', 'v5', 'v6'])
+      expect(await space.public.get('k4')).toEqual('v4')
+      expect(await space.public.get('k5')).toEqual('v5')
+      expect(await space.public.get('k6')).toEqual('v6')
+      await space.public.remove('k6')
+      expect(await space.public.get('k6')).toBeUndefined()
+
+      expect(space.public.setMultiple(['key'], ['value', 'value1'])).rejects.toEqual(new Error('Arrays must be of the same length'))
+      expect(space.public.setMultiple('key', ['value', 'value1'])).rejects.toEqual(new Error('One or more arguments are not an array'))
+    })
+
     it('log should only return public values', async () => {
-      const refLog = [ { key: 'k1', op: 'PUT', timeStamp: 123, value: 'v1' }, { key: 'k3', op: 'PUT', timeStamp: 123, value: 'v3' } ]
+      const refLog = [{ key: 'k1', op: 'PUT', timeStamp: 123, value: 'v1' }, { key: 'k3', op: 'PUT', timeStamp: 123, value: 'v3' }, { key: 'k4', op: 'PUT', timeStamp: 123, value: 'v4' }, { key: 'k5', op: 'PUT', timeStamp: 123, value: 'v5' } ]
       const log1 = space.public.log
       expect(log1).toEqual(refLog)
       space._store.set('key', 'value')
@@ -110,7 +122,9 @@ describe('Space', () => {
     it('all should return all values from public store', async () => {
       expect(await space.public.all()).toEqual({
         k1: 'v1',
-        k3: 'v3'
+        k3: 'v3',
+        k4: 'v4',
+        k5: 'v5',
       })
     })
 
@@ -132,8 +146,20 @@ describe('Space', () => {
       expect(await space.private.get('k2')).toEqual(null)
     })
 
+    it('setMultiple works correctly', async () => {
+      await space.private.setMultiple(['k4', 'k5', 'k6'], ['sv4', 'sv5', 'sv6'])
+      expect(await space.private.get('k4')).toEqual('sv4')
+      expect(await space.private.get('k5')).toEqual('sv5')
+      expect(await space.private.get('k6')).toEqual('sv6')
+      await space.private.remove('k6')
+      expect(await space.private.get('k6')).toEqual(null)
+
+      expect(space.private.setMultiple(['key'], ['value', 'value1'])).rejects.toEqual(new Error('Arrays must be of the same length'))
+      expect(space.private.setMultiple('key', ['value', 'value1'])).rejects.toEqual(new Error('One or more arguments are not an array'))
+    })
+
     it('log should only return private values', async () => {
-      const refLog = [ { key: 'k1', op: 'PUT', timeStamp: 123, value: 'sv1' }, { key: 'k3', op: 'PUT', timeStamp: 123, value: 'sv3' } ]
+      const refLog = [{ key: 'k1', op: 'PUT', timeStamp: 123, value: 'sv1' }, { key: 'k3', op: 'PUT', timeStamp: 123, value: 'sv3' }, { key: 'k4', op: 'PUT', timeStamp: 123, value: 'sv4' }, { key: 'k5', op: 'PUT', timeStamp: 123, value: 'sv5' } ]
       const log1 = space.private.log
       expect(log1).toEqual(refLog)
       space._store.set('key', 'value')
@@ -144,7 +170,9 @@ describe('Space', () => {
     it('all should return all values from private store', async () => {
       expect(await space.private.all()).toEqual({
         k1: 'sv1',
-        k3: 'sv3'
+        k3: 'sv3',
+        k4: 'sv4',
+        k5: 'sv5'
       })
     })
 
diff --git a/src/keyValueStore.js b/src/keyValueStore.js
index b9adc19c..888dd7d0 100644
--- a/src/keyValueStore.js
+++ b/src/keyValueStore.js
@@ -1,4 +1,4 @@
-const { throwIfUndefined } = require('./utils/index')
+const { throwIfUndefined, throwIfNotEqualLenArrays } = require('./utils/index')
 
 class KeyValueStore {
   /**
@@ -57,6 +57,30 @@ class KeyValueStore {
     return true
   }
 
+  /**
+  * Set multiple values for multiple keys
+  *
+  * @param     {Array<String>}    keys                     the keys
+  * @param     {Array<String>}    values                   the values
+  * @return    {Boolean}                                  true if successful, throw error if not
+  */
+  async setMultiple (keys, values) {
+    throwIfNotEqualLenArrays(keys, values)
+    this._requireLoad()
+    this._ensureConnected()
+    try {
+      await keys.reduce(async (previousPromise, nextKey, index) => {
+        await previousPromise
+        throwIfUndefined(nextKey, 'key')
+        const timeStamp = new Date().getTime()
+        return this._db.put(nextKey, { value: values[index], timeStamp })
+      }, Promise.resolve())
+      return true
+    } catch (error) {
+      throw new Error(error)
+    }
+  }
+
   /**
    * Remove the value for the given key
    *
diff --git a/src/privateStore.js b/src/privateStore.js
index 61718a47..5555ae6a 100644
--- a/src/privateStore.js
+++ b/src/privateStore.js
@@ -26,6 +26,13 @@ class PrivateStore extends KeyValueStore {
     return super.set(key, value)
   }
 
+  async setMultiple (keys, values) {
+    utils.throwIfNotEqualLenArrays(keys, values)
+    const dbKeys = keys.map(this._genDbKey, this)
+    const encryptedValues = values.map(this._encryptEntry, this)
+    return super.setMultiple(dbKeys, encryptedValues)
+  }
+
   async remove (key) {
     key = this._genDbKey(key)
     return super.remove(key)
diff --git a/src/publicStore.js b/src/publicStore.js
index 0e10d881..4c026290 100644
--- a/src/publicStore.js
+++ b/src/publicStore.js
@@ -1,5 +1,5 @@
 const KeyValueStore = require('./keyValueStore')
-const { throwIfUndefined } = require('./utils/index')
+const { throwIfUndefined, throwIfNotEqualLenArrays } = require('./utils/index')
 
 class ProfileStore extends KeyValueStore {
   constructor (orbitdb, name, linkProfile, ensureConnected, _3id) {
@@ -13,6 +13,12 @@ class ProfileStore extends KeyValueStore {
     if (!opts.noLink) this._linkProfile()
     return super.set(key, value)
   }
+
+  async setMultiple(keys, values) {
+    throwIfNotEqualLenArrays(keys, values)
+    this._linkProfile()
+    return super.setMultiple(keys, values)
+  }
 }
 
 module.exports = ProfileStore
diff --git a/src/space.js b/src/space.js
index 0f84bfad..ba873117 100644
--- a/src/space.js
+++ b/src/space.js
@@ -1,6 +1,6 @@
 const KeyValueStore = require('./keyValueStore')
 const Thread = require('./thread')
-const { sha256Multihash, throwIfUndefined } = require('./utils')
+const { sha256Multihash, throwIfUndefined, throwIfNotEqualLenArrays } = require('./utils')
 
 const ENC_BLOCK_SIZE = 24
 const nameToSpaceName = name => `3box.space.${name}.keyvalue`
@@ -122,6 +122,11 @@ const publicStoreReducer = (store) => {
       throwIfUndefined(key, 'key')
       store.set(PREFIX + key, value)
     },
+    setMultiple: async (keys, values) => {
+      throwIfNotEqualLenArrays(keys, values)
+      const prefixedKeys = keys.map(key => PREFIX + key)
+      return store.setMultiple(prefixedKeys, values)
+    },
     remove: async key => {
       throwIfUndefined(key, 'key')
       store.remove(PREFIX + key)
@@ -170,6 +175,12 @@ const privateStoreReducer = (store, keyring) => {
     },
     getMetadata: async key => store.getMetadata(dbKey(key)),
     set: async (key, value) => store.set(dbKey(key), encryptEntry({ key, value })),
+    setMultiple: async (keys, values) => {
+      throwIfNotEqualLenArrays(keys, values)
+      const dbKeys = keys.map(dbKey)
+      const encryptedEntries = values.map((value, index) => encryptEntry({ key: keys[index], value}))
+      return store.setMultiple(dbKeys, encryptedEntries)
+    },
     remove: async key => store.remove(dbKey(key)),
     get log () {
       return store.log.reduce((newLog, entry) => {
diff --git a/src/utils/index.js b/src/utils/index.js
index 1c0f9849..ef6782ef 100644
--- a/src/utils/index.js
+++ b/src/utils/index.js
@@ -119,6 +119,16 @@ module.exports = {
     }
   },
 
+  throwIfNotEqualLenArrays: (arr1, arr2) => {
+    if (!Array.isArray(arr1) || !Array.isArray(arr2)) {
+      throw new Error('One or more arguments are not an array')
+    }
+
+    if (arr1.length !== arr2.length) {
+      throw new Error('Arrays must be of the same length')
+    }
+  },
+
   sha256Multihash: str => {
     const digest = Buffer.from(sha256.digest(str))
     return Multihash.encode(digest, 'sha2-256').toString('hex')

From e2b5dcb5369b5c3b76b880c119b841e6bc33d1c0 Mon Sep 17 00:00:00 2001
From: Joel Torstensson <oed3933@gmail.com>
Date: Wed, 8 May 2019 16:19:26 +0200
Subject: [PATCH 3/4] chore(readme): Generated readme.

---
 README.md          | 29 ++++++++++++++++++++++++++++-
 src/publicStore.js |  2 +-
 src/space.js       |  2 +-
 3 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 6a0e6ecc..4ef79129 100644
--- a/README.md
+++ b/README.md
@@ -88,6 +88,19 @@ await box.private.set('email', 'oed@email.service')
 await box.private.remove('email')
 ```
 
+##### Set multiple fields at once:
+```js
+const fields = ['name', 'website', 'employer']
+const values = ['Jon Schwartz', 'openworklabs.com', 'Open Work Labs']
+
+await box.public.setMultiple(fields, values)
+
+const privateFields = ['age', 'coinBalance']
+const privateValues = ['xxx', 'yyy']
+
+await box.private.setMultiple(privateFields, privateValues)
+```
+
 <!-- commenting this out for now, not really needed when we're not using the iframe
 #### IPFS Configs
 
@@ -143,7 +156,7 @@ For the fully detailed spec, view the [documentation](https://github.com/3box/3b
 **WARNING: this is an experimental feature, the API will likely change in the future!**
 
 #### Viewing a Thread
-You can get all posts made in a thread without opening a space. This is great for allowing visitors of your site view comments made by other users. This is achieved by calling the `getThread` method on the Box object. 
+You can get all posts made in a thread without opening a space. This is great for allowing visitors of your site view comments made by other users. This is achieved by calling the `getThread` method on the Box object.
 ```js
 const posts = await Box.getThread(spaceName, threadName)
 console.log(posts)
@@ -518,6 +531,7 @@ Check if the given address is logged in
     * [.get(key)](#KeyValueStore+get) ⇒ <code>String</code>
     * [.getMetadata(key)](#KeyValueStore+getMetadata) ⇒ <code>Metadata</code>
     * [.set(key, value)](#KeyValueStore+set) ⇒ <code>Boolean</code>
+    * [.setMultiple(keys, values)](#KeyValueStore+setMultiple) ⇒ <code>Boolean</code>
     * [.remove(key)](#KeyValueStore+remove) ⇒ <code>Boolean</code>
 
 <a name="new_KeyValueStore_new"></a>
@@ -577,6 +591,19 @@ Set a value for the given key
 | key | <code>String</code> | the key |
 | value | <code>String</code> | the value |
 
+<a name="KeyValueStore+setMultiple"></a>
+
+#### keyValueStore.setMultiple(keys, values) ⇒ <code>Boolean</code>
+Set multiple values for multiple keys
+
+**Kind**: instance method of [<code>KeyValueStore</code>](#KeyValueStore)  
+**Returns**: <code>Boolean</code> - true if successful, throw error if not  
+
+| Param | Type | Description |
+| --- | --- | --- |
+| keys | <code>Array.&lt;String&gt;</code> | the keys |
+| values | <code>Array.&lt;String&gt;</code> | the values |
+
 <a name="KeyValueStore+remove"></a>
 
 #### keyValueStore.remove(key) ⇒ <code>Boolean</code>
diff --git a/src/publicStore.js b/src/publicStore.js
index 4c026290..bb7e454d 100644
--- a/src/publicStore.js
+++ b/src/publicStore.js
@@ -14,7 +14,7 @@ class ProfileStore extends KeyValueStore {
     return super.set(key, value)
   }
 
-  async setMultiple(keys, values) {
+  async setMultiple (keys, values) {
     throwIfNotEqualLenArrays(keys, values)
     this._linkProfile()
     return super.setMultiple(keys, values)
diff --git a/src/space.js b/src/space.js
index ba873117..5c21c626 100644
--- a/src/space.js
+++ b/src/space.js
@@ -178,7 +178,7 @@ const privateStoreReducer = (store, keyring) => {
     setMultiple: async (keys, values) => {
       throwIfNotEqualLenArrays(keys, values)
       const dbKeys = keys.map(dbKey)
-      const encryptedEntries = values.map((value, index) => encryptEntry({ key: keys[index], value}))
+      const encryptedEntries = values.map((value, index) => encryptEntry({ key: keys[index], value }))
       return store.setMultiple(dbKeys, encryptedEntries)
     },
     remove: async key => store.remove(dbKey(key)),

From 2793273b56616e5058d9066b6302118951a950e6 Mon Sep 17 00:00:00 2001
From: Joel Torstensson <oed3933@gmail.com>
Date: Wed, 8 May 2019 16:28:09 +0200
Subject: [PATCH 4/4] Release v1.8.0

---
 RELEASE-NOTES.md | 5 +++++
 package.json     | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/RELEASE-NOTES.md b/RELEASE-NOTES.md
index 213055c4..a3e95f04 100644
--- a/RELEASE-NOTES.md
+++ b/RELEASE-NOTES.md
@@ -1,5 +1,10 @@
 # Release Notes
 
+## v1.8.0 - 2019-05-09
+* Feature: Add `setMultiple` method, enables multiple fields to be set at once.
+
+Special thanks to @Schwartz10 for contributing this feature!
+
 ## v1.7.2 - 2019-04-30
 * Fix: Don't allow setting values wihtout a 'key'
 * Fix: Ensure that linkProfile only happens once
diff --git a/package.json b/package.json
index 8341b22f..7e4b3e74 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "3box",
-  "version": "1.7.2",
+  "version": "1.8.0",
   "description": "Interact with user data",
   "main": "lib/3box.js",
   "directories": {