edge-currency-plugins 2.6.0 → 2.6.1

package/lib/common/utxobased/db/Processor.js

@@ -157,12 +157,17 @@ const BASELET_DIR = 'tables'
     },
 
     async dumpData() {
-      const allBases = Object.values(baselets.all)
+      
+      const allBases = Object.values(
+        baselets.all
+      )
       return await Promise.all(
-        allBases.map(async base => ({
-          databaseName: base.databaseName,
-          data: (await base.dumpData('')) 
-        }))
+        allBases.map(async base => {
+          return {
+            databaseName: base.databaseName,
+            data: (await base.dumpData()) 
+          }
+        })
       )
     },
 
@@ -399,6 +404,15 @@ const BASELET_DIR = 'tables'
 
     async saveAddress(address) {
       await baselets.address(async tables => {
+        // This variable is used to update the scriptPubkeyByPath table.
+        // The path table must be written after the address table because the
+        // path table is an index of the address table.
+        // This variable acts as a catch for that update to be done after the
+        // address table is written.
+        let indexTableUpdate
+
+
+
         const [existingAddress] = await tables.addressByScriptPubkey.query('', [
           address.scriptPubkey
         ])
@@ -415,14 +429,14 @@ const BASELET_DIR = 'tables'
               'Attempted to save address with an existing path, but different script pubkey'
             )
 
-          await tables.scriptPubkeyByPath.insert(
-            _baselet3.addressPathToPrefix.call(void 0, address.path),
-            address.path.addressIndex,
-            address.scriptPubkey
-          )
+          indexTableUpdate = {
+            path: address.path,
+            scriptPubkey: address.scriptPubkey
+          }
 
-          // check if this address is used and if so, whether it has a higher last used index
-          if (address.used || (_nullishCoalesce(_optionalChain([existingAddress, 'optionalAccess', _2 => _2.used]), () => ( false)))) {
+          // check if this address is used and if so, whether it has a higher
+          // last used index
+          if (address.used || _optionalChain([existingAddress, 'optionalAccess', _2 => _2.used]) === true) {
             let [lastUsed] = await tables.lastUsedByFormatPath.query('', [
               _baselet3.addressPathToPrefix.call(void 0, address.path)
             ])
@@ -438,6 +452,7 @@ const BASELET_DIR = 'tables'
           }
         }
 
+        // Update routine:
         if (existingAddress != null) {
           // Only update the lastQueriedBlockHeight on the address if one was given and is greater than the existing value
           if (
@@ -463,16 +478,21 @@ const BASELET_DIR = 'tables'
             existingAddress.balance = address.balance
           }
 
-          // Only update the path field if one was given and currently does not have one
-          // NOTE: Addresses can be stored in the db without a path due to the `EdgeCurrencyEngine.addGapLimitAddresses` function
-          //  Once an address path is known, it should never be updated
+          /*
+          Only update the path field if one was given and the existing address
+          currently does not have one. We never update paths for addresses, only
+          insert paths when they're not present.
+
+          NOTE: Addresses can be stored in the db without a path due to the
+          `EdgeCurrencyEngine.addGapLimitAddresses` function. Once an address
+          path is known, it should never be updated
+          */
           if (address.path != null && existingAddress.path == null) {
             existingAddress.path = address.path
-            await tables.scriptPubkeyByPath.insert(
-              _baselet3.addressPathToPrefix.call(void 0, address.path),
-              address.path.addressIndex,
-              address.scriptPubkey
-            )
+            indexTableUpdate = {
+              path: existingAddress.path,
+              scriptPubkey: existingAddress.scriptPubkey
+            }
           }
 
           // Only update the used flag if one was given and is true
@@ -495,18 +515,32 @@ const BASELET_DIR = 'tables'
               )
             }
           }
+
+          // Update the address table:
           await tables.addressByScriptPubkey.insert(
             '',
             existingAddress.scriptPubkey,
             existingAddress
           )
-          return
         }
-        await tables.addressByScriptPubkey.insert(
-          '',
-          address.scriptPubkey,
-          address
-        )
+
+        // Insert routine:
+        if (existingAddress == null) {
+          await tables.addressByScriptPubkey.insert(
+            '',
+            address.scriptPubkey,
+            address
+          )
+        }
+
+        // Insert the path into the index table:
+        if (indexTableUpdate != null) {
+          await tables.scriptPubkeyByPath.insert(
+            _baselet3.addressPathToPrefix.call(void 0, indexTableUpdate.path),
+            indexTableUpdate.path.addressIndex,
+            indexTableUpdate.scriptPubkey
+          )
+        }
       })
     },
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edge-currency-plugins",
-  "version": "2.6.0",
+  "version": "2.6.1",
   "description": "Edge currency plugins to fetch and process wallet accounts, balances and transactions.",
   "repository": "https://github.com/EdgeApp/edge-currency-plugins",
   "license": "MIT",
@@ -36,7 +36,7 @@
     "@bitcoin-js/tiny-secp256k1-asmjs": "^2.2.3",
     "altcoin-js": "^1.0.0",
     "async-mutex": "^0.2.6",
-    "baselet": "^0.2.4",
+    "baselet": "^0.3.0",
     "biggystring": "^4.1.3",
     "bip32": "^2.0.5",
     "bip32grs": "^2.0.5",
@@ -107,4 +107,4 @@
     "typescript": "^4.1.2",
     "wif": "^2.0.6"
   }
-}
+}