Fix for FT symbols containing space

Author: cmallwitz

examples.ods

@@ -90,7 +90,7 @@ def __init__(self, ctx):
     @profile
     def getRealtime(self, ticker, datacode=None, source=None):
 
-        if ticker == 'SUPPORT':
+        if ticker == 'SUPPORT' or ticker == 'support':
             return self.support(datacode)
 
         try:

src/financials.py

@@ -12,6 +12,7 @@
 import os
 import re
 import time
+import urllib.parse
 
 import dateutil.parser
 
@@ -59,9 +60,6 @@ def getRealtime(self, ticker: str, datacode: int):
         :return:
         """
 
-        # remove white space
-        ticker = "".join(ticker.split())
-
         # use cached value for up to 60 seconds
         if ticker in self.realtime:
             tick = self.realtime[ticker]
@@ -77,7 +75,7 @@ def getRealtime(self, ticker: str, datacode: int):
 
         asset_class = self.guess_asset_class(ticker)
 
-        url = f'https://markets.ft.com/data/{asset_class}/tearsheet/summary?s={ticker}'
+        url = f'https://markets.ft.com/data/{asset_class}/tearsheet/summary?s={urllib.parse.quote_plus(ticker)}'
 
         try:
             text = self.urlopen(url, redirect=True, data=None, headers=None)

src/financials_ft.py

@@ -14,7 +14,7 @@
 cur_dir = os.getcwd()
 
 addin_id = "com.financials.getinfo"
-addin_version = "3.0.0"
+addin_version = "3.0.1"
 addin_displayname = "Financial Market Extension"
 addin_publisher_link = "https://github.com/cmallwitz/Financials-Extension"
 addin_publisher_name = "The Publisher"

src/generate_metainfo.py

@@ -254,6 +254,17 @@ def test_DE_equity(self):
         s = financials.getRealtime('ISHAX:GER', 'EX_DIV_DATE', 'FT')
         self.assertIsNone(s, 'test_DE_equity EX_DIV_DATE {}'.format(s))
 
+    def test_DK_equity(self):
+        s = financials.getRealtime('NOVO B:CPH', 'name', 'FT')
+        self.assertEqual('Novo Nordisk A/S', s, 'test_DK_equity NAME {}'.format(s))
+
+        s = financials.getRealtime('NOVO B:CPH', 'currency', 'FT')
+        self.assertEqual('DKK', s, 'test_DK_equity CURRENCY {}'.format(s))
+
+        s = financials.getRealtime('NOVO B:CPH', 'industry', 'FT')
+        self.assertEqual(str, type(s), 'test_DK_equity INDUSTRY {}'.format(s))
+        self.assertEqual('Pharmaceuticals & Biotechnology', s, 'test_DK_equity INDUSTRY {}'.format(s))
+
     def test_TY_equity(self):
         s = financials.getRealtime('6503:TYO', 'OPEN', 'FT')
         self.assertEqual(float, type(s), 'test_TY_equity OPEN {}'.format(s))

src/test_ft.py

@@ -195,6 +195,17 @@ def test_realtime_DE_equity(self):
         s = financials.getRealtime('C060.DE', Datacode.EXCHANGE.value, 'YAHOO')
         self.assertEqual(s, 'GER', 'test_realtime_DE_equity EXCHANGE')
 
+    def test_DK_equity(self):
+        s = financials.getRealtime('NOVO-B.CO', 'name', 'YAHOO')
+        self.assertEqual('Novo Nordisk A/S', s, 'test_DK_equity NAME {}'.format(s))
+
+        s = financials.getRealtime('NOVO-B.CO', 'currency', 'YAHOO')
+        self.assertEqual('DKK', s, 'test_DK_equity CURRENCY {}'.format(s))
+
+        s = financials.getRealtime('NOVO-B.CO', 'industry', 'YAHOO')
+        self.assertEqual(str, type(s), 'test_DK_equity INDUSTRY {}'.format(s))
+        self.assertEqual('Biotechnology', s, 'test_DK_equity INDUSTRY {}'.format(s))
+
     def test_realtime_TY_equity(self):
         s = financials.getRealtime('6503.T', Datacode.OPEN.value, 'YAHOO')
         self.assertEqual(float, type(s), 'test_TY_equity OPEN {}'.format(s))
@@ -261,7 +272,7 @@ def test_historic_US_equity(self):
 
         # Note: quarterly dividend and splits will change past adjusted prices - will fail after the next dividend
         s = financials.getHistoric('IBM', Datacode.ADJ_CLOSE.value, '2017-01-03', 'YAHOO')
-        self.assertEqual(137.772202, s, 'test_historic_US_equity ADJ_CLOSE {}'.format(s))
+        self.assertEqual(136.249847, s, 'test_historic_US_equity ADJ_CLOSE {}'.format(s))
 
     def test_historic_UK_ETF(self):