Improve backwards compatibility in regards to previous commit

- Restore fromEntryHash
- Make fromMultihash simply call fromCID

package.json

@@ -42,6 +42,7 @@
     "mocha-headless-chrome": "~2.0.1",
     "orbit-db-keystore": "github:orbitdb/orbit-db-keystore",
     "rimraf": "~2.6.1",
+    "sinon": "~7.2.2",
     "standard": "~12.0.1",
     "webpack": "~4.28.0",
     "webpack-cli": "~3.1.2"

src/entry.js

@@ -48,7 +48,8 @@ class Entry {
     entry.identity = identity.toJSON()
     entry.sig = signature
     entry.cid = await Entry.toCID(ipfs, entry)
-    return entry
+
+    return Entry.ensureInterop(entry)
   }
 
   /**
@@ -129,7 +130,7 @@ class Entry {
    */
   static async toMultihash (ipfs, entry) {
     if (!ipfs) throw IpfsNotDefinedError()
-    if (!Entry.isEntry(entry)) throw new Error('Invalid object format, cannot generate entry multihash')
+    if (!Entry.isEntry(entry)) throw new Error('Invalid object format, cannot generate entry CID')
 
     // Ensure `entry` follows the correct format
     const e = {
@@ -165,7 +166,7 @@ class Entry {
     const e = await dagNode.read(ipfs, cid, IPLD_LINKS)
 
     let entry = {
-      cid,
+      [getCidProp(e)]: cid,
       id: e.id,
       payload: e.payload,
       next: e.next,
@@ -177,7 +178,7 @@ class Entry {
     if (e.identity) Object.assign(entry, { identity: e.identity })
     if (e.sig) Object.assign(entry, { sig: e.sig })
 
-    return entry
+    return Entry.ensureInterop(entry)
   }
 
   /**
@@ -192,9 +193,6 @@ class Entry {
    * @deprecated
    */
   static async fromMultihash (ipfs, multihash) {
-    if (!ipfs) throw IpfsNotDefinedError()
-    if (!multihash) throw new Error(`Invalid multihash: ${multihash}`)
-
     return Entry.fromCID(ipfs, multihash)
   }
 
@@ -212,6 +210,32 @@ class Entry {
       obj.clock !== undefined
   }
 
+  /**
+   * Ensures that this entry is interoperable between earlier versions
+   * and the most recent one (and vice-versa).
+   * @param {Entry} entry The entry to ensure interoperability
+   * @return {Entry} entry The same entry but with backwards and forward interoperability
+   */
+  static ensureInterop (entry) {
+    if (entry.cid && entry.hash) {
+      return entry
+    }
+
+    const prop = getCidProp(entry)
+    const accessorProp = prop === 'hash' ? 'cid' : 'hash'
+
+    Object.defineProperty(entry, accessorProp, {
+      get () {
+        return this[prop]
+      },
+      set (value) {
+        this[prop] = value
+      }
+    })
+
+    return entry
+  }
+
   /**
    * Compares two entries.
    * @param {Entry} a

src/log-io.js

@@ -11,32 +11,32 @@ const last = (arr, n) => arr.slice(arr.length - n, arr.length)
 
 class LogIO {
   /**
-   * Get the multihash of a Log.
+   * Get the CID of a Log.
    * @param {IPFS} ipfs An IPFS instance
-   * @param {Log} log Log to get a multihash for
+   * @param {Log} log Log to get a CID for
    * @returns {Promise<string>}
-   * @deprecated
    */
-  static async toMultihash (ipfs, log) {
+  static async toCID (ipfs, log) {
     if (!isDefined(ipfs)) throw LogError.IPFSNotDefinedError()
     if (!isDefined(log)) throw LogError.LogNotDefinedError()
     if (log.values.length < 1) throw new Error(`Can't serialize an empty log`)
 
-    return dagNode.write(ipfs, 'dag-pb', log.toJSON(), IPLD_LINKS)
+    return dagNode.write(ipfs, 'dag-cbor', log.toJSON(), IPLD_LINKS)
   }
 
   /**
-   * Get the CID of a Log.
+   * Get the multihash of a Log.
    * @param {IPFS} ipfs An IPFS instance
-   * @param {Log} log Log to get a CID for
+   * @param {Log} log Log to get a multihash for
    * @returns {Promise<string>}
+   * @deprecated
    */
-  static async toCID (ipfs, log) {
+  static async toMultihash (ipfs, log) {
     if (!isDefined(ipfs)) throw LogError.IPFSNotDefinedError()
     if (!isDefined(log)) throw LogError.LogNotDefinedError()
     if (log.values.length < 1) throw new Error(`Can't serialize an empty log`)
 
-    return dagNode.write(ipfs, 'dag-cbor', log.toJSON(), IPLD_LINKS)
+    return dagNode.write(ipfs, 'dag-pb', log.toJSON(), IPLD_LINKS)
   }
 
   /**
@@ -76,19 +76,16 @@ class LogIO {
   }
 
   /**
-     * Create a log from a multihash.
-     * @param {IPFS} ipfs An IPFS instance
-     * @param {string} multihash Multihash (as a Base58 encoded string) to create the Log from
-     * @param {number} [length=-1] How many items to include in the log
-     * @param {Array<Entry>} [exclude] Entries to not fetch (cached)
-     * @param {function(cid, entry, parent, depth)} onProgressCallback
-     * @returns {Promise<Log>}
-     * @deprecated
-     */
+   * Create a log from a multihash.
+   * @param {IPFS} ipfs An IPFS instance
+   * @param {string} multihash Multihash (as a Base58 encoded string) to create the Log from
+   * @param {number} [length=-1] How many items to include in the log
+   * @param {Array<Entry>} [exclude] Entries to not fetch (cached)
+   * @param {function(cid, entry, parent, depth)} onProgressCallback
+   * @returns {Promise<Log>}
+   * @deprecated
+   */
   static async fromMultihash (ipfs, multihash, length = -1, exclude, onProgressCallback) {
-    if (!isDefined(ipfs)) throw LogError.IPFSNotDefinedError()
-    if (!isDefined(multihash)) throw new Error(`Invalid multihash: ${multihash}`)
-
     return LogIO.fromCID(ipfs, multihash, length, exclude, onProgressCallback)
   }
 
@@ -113,6 +110,7 @@ class LogIO {
 
   static async fromJSON (ipfs, json, length = -1, timeout, onProgressCallback) {
     if (!isDefined(ipfs)) throw LogError.IPFSNotDefinedError()
+    json.heads.forEach(Entry.ensureInterop)
     const headCids = json.heads.map(e => e.cid)
     const entries = await EntryIO.fetchParallel(ipfs, headCids, length, [], 16, timeout, onProgressCallback)
     const finalEntries = entries.slice().sort(Entry.compare)
@@ -144,6 +142,7 @@ class LogIO {
     if (!Array.isArray(sourceEntries)) {
       sourceEntries = [sourceEntries]
     }
+    sourceEntries.forEach(Entry.ensureInterop)
 
     // Fetch given length, return size at least the given input entries
     length = length > -1 ? Math.max(length, sourceEntries.length) : length