wallet: properly resolve wallet ids.

handshake-org · Dec 4, 2023 · 06d8fd1 · 06d8fd1
1 parent b12650d
commit 06d8fd1
Showing 1 changed file with 12 additions and 6 deletions.
diff --git a/lib/wallet/walletdb.js b/lib/wallet/walletdb.js
@@ -273,7 +273,7 @@ class WalletDB extends EventEmitter {
     const wallets = await this.getWallets();
 
     for (const wname of wallets)
-      await this._get(wname);
+      await this.get(wname);
   }
 
   /**
@@ -917,7 +917,7 @@ class WalletDB extends EventEmitter {
   /**
    * Map wallet id to wid.
    * @param {String|Number} id
-   * @returns {Promise} - Returns {Number}.
+   * @returns {Promise<Number>}
    */
 
   async ensureWID(id) {
@@ -933,7 +933,7 @@ class WalletDB extends EventEmitter {
   /**
    * Map wallet id to wid.
    * @param {String} id
-   * @returns {Promise} - Returns {Number}.
+   * @returns {Promise<Number>}
    */
 
   async getWID(id) {
@@ -950,7 +950,7 @@ class WalletDB extends EventEmitter {
   /**
    * Map wallet wid to id.
    * @param {Number} wid
-   * @returns {Promise} - Returns {String}.
+   * @returns {Promise<String?>}
    */
 
   async getID(wid) {
@@ -965,7 +965,7 @@ class WalletDB extends EventEmitter {
   /**
    * Get a wallet from the database, setup watcher.
    * @param {Number|String} id
-   * @returns {Promise} - Returns {@link Wallet}.
+   * @returns {Promise<Wallet?>}
    */
 
   async get(id) {
@@ -987,10 +987,11 @@ class WalletDB extends EventEmitter {
    * Get a wallet from the database without a lock.
    * @private
    * @param {Number} wid
-   * @returns {Promise} - Returns {@link Wallet}.
+   * @returns {Promise<Wallet?>}
    */
 
   async _get(wid) {
+    console.trace('Getting: ', wid);
     const cache = this.wallets.get(wid);
 
     if (cache)
@@ -2676,6 +2677,11 @@ function fromString(str) {
   return buf;
 }
 
+/**
+ * @param {Buffer} buf
+ * @returns {String}
+ */
+
 function toString(buf) {
   assert(buf.length > 0);
   assert(buf[0] === buf.length - 1);