Fix symbol conversion for Yahoo Finance

apps/api/src/services/data-provider/yahoo-finance/yahoo-finance.service.ts

@@ -262,16 +262,21 @@ export const convertFromYahooFinanceSymbol = (aYahooFinanceSymbol: string) => {
  *                  SOL1USD -> SOL1-USD
  */
 export const convertToYahooFinanceSymbol = (aSymbol: string) => {
-  if (isCurrency(aSymbol)) {
-    if (isCrypto(aSymbol) || isCrypto(aSymbol.replace('1', ''))) {
+  if (
+    (aSymbol.includes('CHF') ||
+      aSymbol.includes('EUR') ||
+      aSymbol.includes('USD')) &&
+    aSymbol.length >= 6
+  ) {
+    if (isCurrency(aSymbol.substring(0, aSymbol.length - 3))) {
+      return `${aSymbol}=X`;
+    } else if (isCrypto(aSymbol) || isCrypto(aSymbol.replace('1', ''))) {
       // Add a dash before the last three characters
       // BTCUSD  -> BTC-USD
       // DOGEUSD -> DOGE-USD
       // SOL1USD -> SOL1-USD
       return aSymbol.replace('USD', '-USD');
     }
-
-    return `${aSymbol}=X`;
   }
 
   return aSymbol;

libs/common/src/lib/helper.ts

@@ -1,9 +1,11 @@
+import * as currencies from '@dinero.js/currencies';
 import { getDate, getMonth, getYear, parse, subDays } from 'date-fns';
 
 import { ghostfolioScraperApiSymbolPrefix } from './config';
 
 const cryptocurrencies = require('cryptocurrencies');
 
+
 export const DEMO_USER_ID = '9b112b4d-3b7d-4bad-9bdd-3b0f7b4dac2f';
 
 export function capitalize(aString: string) {
@@ -85,12 +87,7 @@ export function isCrypto(aSymbol = '') {
 }
 
 export function isCurrency(aSymbol = '') {
-  return (
-    (aSymbol.includes('CHF') ||
-      aSymbol.includes('EUR') ||
-      aSymbol.includes('USD')) &&
-    aSymbol.length >= 6
-  );
+  return currencies[aSymbol];
 }
 
 export function isGhostfolioScraperApiSymbol(aSymbol = '') {

package.json

@@ -58,6 +58,7 @@
     "@angular/platform-browser-dynamic": "12.2.4",
     "@angular/router": "12.2.4",
     "@codewithdan/observable-store": "2.2.11",
+    "@dinero.js/currencies": "2.0.0-alpha.8",
     "@nestjs/common": "7.6.18",
     "@nestjs/config": "0.6.3",
     "@nestjs/core": "7.6.18",

yarn.lock

@@ -1681,6 +1681,11 @@
     debug "^3.1.0"
     lodash.once "^4.1.1"
 
+"@dinero.js/currencies@2.0.0-alpha.8":
+  version "2.0.0-alpha.8"
+  resolved "https://registry.yarnpkg.com/@dinero.js/currencies/-/currencies-2.0.0-alpha.8.tgz#aa02a04ce3685a9b06a7ce12f8c924726386c3fd"
+  integrity sha512-zApiqtuuPwjiM9LJA5/kNcT48VSHRiz2/mktkXjIpfxrJKzthXybUAgEenExIH6dYhLDgVmsLQZtZFOsdYl0Ag==
+
 "@discoveryjs/json-ext@0.5.3":
   version "0.5.3"
   resolved "https://registry.yarnpkg.com/@discoveryjs/json-ext/-/json-ext-0.5.3.tgz#90420f9f9c6d3987f176a19a7d8e764271a2f55d"