fix: issue #56

paul-shuvo · Oct 22, 2023 · 0377813 · 0377813
1 parent 6d1cd63
commit 0377813
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/index.js b/index.js
@@ -271,7 +271,7 @@ class CurrencyConverter {
         });
       } else {
         return new Promise((resolve, reject) => {
-            request(`https://www.google.com/search?q=${this.currencyFrom}+to+${this.currencyTo}+&hl=en`, function(error, response, body) {
+            request(`https://www.google.com/search?q=${this.currencyAmount}+${this.currencyFrom}+to+${this.currencyTo}+&hl=en`, function(error, response, body) {
               if (error) {
                 return reject(error);
               } else {
@@ -294,10 +294,11 @@ class CurrencyConverter {
               if (rates.includes(","))
                 rates = this.replaceAll(rates, ",", "")
             }
+            rates = parseFloat(rates) / this.currencyAmount
             if (this.isRatesCaching) {
-              this.addRateToRatesCache(currencyPair, parseFloat(rates));
+              this.addRateToRatesCache(currencyPair, rates);
             }
-            return parseFloat(rates)
+            return rates
           })
       }
     }

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "currency-converter-lt",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "A nodejs currency converter library that doesn't require subscribing to any API calls.",
   "main": "index.js",
   "dependencies": {