Bug #64419 for Finance-Quote: Quote-lookup through morningstar.se is broken

Mon Jan 03 12:15:57 2011 feffe [...] mdfnet.se - Ticket created

Subject:	Quote-lookup through morningstar.se is broken
Date:	Mon, 03 Jan 2011 18:17:33 +0100
To:	<bug-Finance-Quote [...] rt.cpan.org>
From:	Fredrik Eriksson <feffe [...] mdfnet.se>

Hello, Just thought I'd let you know that Morningstar is not working anymore. Since I havn't looked at Finance::Quote or morningstar.se before today, I don't know when it broke. morningstar.se has probably redesigned their homepage since this module was written since the URL in the module ("http://morningstar.se/funds/overview.asp?cid=") gives a 404 error. Even if you change the url to a more correct one ("http://morningstar.se/Funds/Quicktake/Overview.aspx?perfid=") it still fails; probably due to some parsing error. I might have time to look deeper into it and write a fix at some point, but if anyone who knows this stuff can make a quick fix it would be great :) Regards -- Fredrik Eriksson Mälardalens Datorförening

Sun Feb 05 03:19:33 2012 http://claimid.com/simon-lindgren - Correspondence added

From:

Simon Lindgren

Vid Mon, 03 Jan 2011 kl. 12.15.57, skrev feffe@mdfnet.se: Show quoted text

> > Hello, > > Just thought I'd let you know that Morningstar is not working anymore. > Since I havn't looked at Finance::Quote or morningstar.se before today, I > don't know when it broke. > > morningstar.se has probably redesigned their homepage since this module > was written since the URL in the module > ("http://morningstar.se/funds/overview.asp?cid=") gives a 404 error. Even > if you change the url to a more correct one > ("http://morningstar.se/Funds/Quicktake/Overview.aspx?perfid=") it still > fails; probably due to some parsing error. I might have time to look

deeper Show quoted text

> into it and write a fix at some point, but if anyone who knows this stuff > can make a quick fix it would be great :) >

I have a branch here with a fix: https://github.com/riviti/finance-quote/branches/Morningstar Wasn't aware of this bug until now, though maybe you've already seen this branch on finance-quote-devel?

Sun Feb 05 03:19:36 2012 The RT System itself - Status changed from 'new' to 'open'

Fri Sep 18 18:04:34 2015 anton [...] antonanton.com - Correspondence added

Subject:	[rt.cpan.org #64419] patch
Date:	Sat, 19 Sep 2015 00:01:49 +0200
To:	bug-Finance-Quote [...] rt.cpan.org
From:	Anton Lindström <anton [...] antonanton.com>

Hi, I wanted to get this working so I've created a patch for morningstar.se. I also had to patch the test file since it looked for a fund that doesn't exist any more. best regards -- Anton

Message body is not shown because sender requested not to inline it.

Sat Sep 19 18:43:52 2015 eco [...] ecocode.net - Correspondence added 15 min

On Fri Sep 18 18:04:34 2015, anton@antonanton.com wrote: Show quoted text

> Hi, > > I wanted to get this working so I've created a patch for morningstar.se. > I also had to patch the test file since it looked for a fund that > doesn't exist any more. > > best regards

Thanks! patch applied and appended to master branch -- Erik

Sat Sep 19 18:44:03 2015 eco [...] ecocode.net - Status changed from 'open' to 'patched'

Sat Sep 19 18:44:03 2015 eco [...] ecocode.net - Taken