Skip to content

Commit

Permalink
Merge pull request #427 from JaredF/Fix_Stocks
Browse files Browse the repository at this point in the history
Fixes Stocks.pl 'Missing Symbols List' Error
  • Loading branch information
JaredF committed Jul 16, 2014
2 parents a99d2fd + 125dfd7 commit 645800e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion code/common/stocks.pl
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

# More info on how this magic url was derived can be found here:
# http://www.padz.net/~djpadz/YahooQuote/
my $stock_url = 'http://quote.yahoo.com/d?f=snl1d1t1c1p2va2bapomwerr1dyj1x\&s=' . join('%20', @stock_symbols);
my $stock_url = 'http://download.finance.yahoo.com/d?f=snl1d1t1c1p2va2bapomwerr1dyj1x\&s=' . join('%20', @stock_symbols);
my @stock_keys = ('SName', 'LName', 'Last', 'Date', 'Time', 'Change', 'PChange',
'Volume', 'Avg Volume', 'Bid', 'Ask', 'Prev Close', 'Open',
'Day Range', '52-Week Range', 'EPS', 'P/E Ratio', 'Div Pay Date',
Expand Down Expand Up @@ -198,6 +198,8 @@
unless &trigger_get('get stocks');
}

# 07 Jul 14, Jared J. Fernandez
# Updated stocks URL due to change by Yahoo.

# 27 Dec 05, David Norwood
# Someone else also added back the stock alerts in the last release. I removed the duplicate code.
Expand Down

0 comments on commit 645800e

Please sign in to comment.