What would be the best way to query more data fields from yahoo?

For example, I would like to query '52 weeks high' as well ('k' in the field list). For this to work I need to change the module code and the underlying table. Would you accept this kind of patch? I wouldn't want to change the module code on my local install, but didn't find a better way for this either.

Comments

kbahey’s picture

kbahey commented
Status: Active » Needs work

Yes, I would accept such a patch.

Here is my current thinking on that:

Ideally, there would be a list of fields that are available (current ones plus the new ones you add) and the admin would enable the fields she wants. They are retrieved and become visible from that setting.

The database table should have all the fields and they are populated only if retrieved. Or perhaps retrieve and populate all the fields regardless, and display only the ones the admin selects.

yhager’s picture

yhager commented
Status: Needs work » Active

ok, thanks.
I'll see if I can work on that, and update here if I do.

matt westgate’s picture

matt westgate commented
Version: 5.x-1.1 » master
Status: Active » Fixed

All possible fields from Yahoo! have now been added to the database schema. On the stockapi settings page you can choose which set of fields you wish to use: basic, extended, or real-time if you have a paid account with Yahoo!

yhager’s picture

yhager commented

WooHoo!

Thanks :)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.