kelvinyap2014 / yahoo-finance-managed

Automatically exported from code.google.com/p/yahoo-finance-managed
0 stars 0 forks source link

Error on MarketQuotesDownloadSettings #46

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Another little bug ;):

on the GetUrl():
       protected override string GetUrl()
        {
            if (mSector.HasValue)
            {
                return "http://biz.yahoo.com/p/csv/" + (mSector.Value + 1).ToString() + FinanceHelper.MarketQuotesRankingTypeString(this.RankedBy) + FinanceHelper.MarketQuotesRankingDirectionString(this.RankDirection) + ".csv";
            }
....

should be changed in:
        protected override string GetUrl()
        {
            if (mSector.HasValue)
            {
                return "http://biz.yahoo.com/p/csv/" + ((int)mSector.Value).ToString() + FinanceHelper.MarketQuotesRankingTypeString(this.RankedBy) + FinanceHelper.MarketQuotesRankingDirectionString(this.RankDirection) + ".csv";
            }
....

The sector has to be an integer, and you do not need to add 1.
Regards /// Angel

Original issue reported on code.google.com by i...@advancedcomputing.ch on 22 Mar 2012 at 12:06

Attachments:

GoogleCodeExporter commented 8 years ago
This issue will be fixed in next version.

Thanks for reporting!

Maas

Original comment by Maas...@gmail.com on 22 Mar 2012 at 3:06

GoogleCodeExporter commented 8 years ago

Original comment by Maas...@gmail.com on 22 Mar 2012 at 3:24