search

nanorepublica / gnome-shell-stock-ticker

gnome shell extension that lets you montior stock prices (based on google finance)
4 stars 7 forks source link

Doesn't work for European stock quotes #1

Open verthezp opened 11 years ago

verthezp commented 11 years ago

Looks good, but it chokes on European stock quotes, such as EPA:ALU.

If you add a try/catch around the load_contents, lg shows the following exception: Failed to convert UTF-8 string to JS string: invalid byte sequence in conversion input

If you call the URL manually in firefox (https://www.google.com/finance/info?q=EPA:ALU) you get:

// [ { "id": "673489" ,"t" : "ALU" ,"e" : "EPA" ,"l" : "0.949" ,"l_cur" : "€0.95" ,"s": "0" ,"ltt":"5:35PM GMT+02:00" ,"lt" : "Sep 19, 5:35PM GMT+02:00" ,"c" : "-0.005" ,"cp" : "-0.52" ,"ccol" : "chr" } ]

So I guess the Euro sign in l_cur is the problem. Any idea how to fix this?

nanorepublica commented 11 years ago

Hey sorry, this has been left for so long, was hoping this had been fixed in gnome-shell 3.6.2, but is hasn't. There doesn't seem to be much I can do about it as you correctly pointed out that load_contents just fails. I will be raising a bug on the gnome-shell bugzilla shortly and I will post the link here when I do so you can track it's progress.