-
Notifications
You must be signed in to change notification settings - Fork 416
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Yahoo finance.keystats is broken #447
Comments
I agree with you I believe that should fix the issue. The new tbody tag has to be what is throwing off all of the keystats queries. |
+1 I've made similar changes and more to another file (analystestimates) in PR 499. |
These changes should fix the problem described in yql#447
These changes should fix the problem described in yql#447
I tried to run YQL query to table "Yahoo finance.keystats" but it still returns only symbols. If I understnad correctly, I need to use Xpath, and not such YQL selects? |
anyone get this working? I'm still getting back only symbols. Yahoo console keeps giving me a connect failure when i try it there too. |
Still broken would be awesome if something would get it fixed |
Any update? |
Nope - still broken for me. I tried @johnmwilson82's fix from above, but not working for me still |
yahoo.finance.keystats just returns the symbol. The old xpath is:
var keystatsquery = y.xpath(
rawresult,
"//table[@Class='yfnc_datamodoutline1']/tr/td/table/tr" +
"/td[@Class='yfnc_tabledata1' or @Class='yfnc_tablehead1']"
);
I think it needs to be changed to:
var keystatsquery = y.xpath(
rawresult,
"//table[@Class='yfnc_datamodoutline1']/tbody/tr/td/table/tbody/tr" +
"/td[@Class='yfnc_tabledata1' or @Class='yfnc_tablehead1']"
);
it looks like there are tbody tags that are throwing it off. i haven't contributed to this project yet, and so i'm a little hesitant to make the change myself. can someone comment as to whether they agree with this?
The text was updated successfully, but these errors were encountered: