update for 4.7

I am

hope to post an patch for review this week

Here is the api patch, i've included the full package too, as i'm new to this patching thing and we have new ".install" files...

Changes:
- new formAPI stuff.
- code changes to conform with drupal coding conventions.
- creation of .install file (4.7 way to create the mysql table's automatically).

Important, i've added 2 thing's on this module:
- drupal_set_message warning user that his portfolio was saved.
- when user is just quoting and he have an registered portfolio, we show to his the symbols he has in his porfolio.

These are just sugested features.

Khalid, please take those of, if you want so, ok?

Pacheco

by the way, sorry for my poor english, it's not native ;)

Hi ppacheco,

I used your new script for 4.7 and it works perfectly. I was just wondering if you can also put the chart value of whole day trading of that company as well. This you can find at http://ichart.finance.yahoo.com/t?s= e.g. http://ichart.finance.yahoo.com/t?s=MSN for each company.

regards,
--rafi

i tried this patch with my drupal 4.7 ...but its not fully working.. Data in database not refreshing and when quering quote for new new symbol it gives blank result.

cron is not updating the db as well.

regards,
--rafi

Hi rafi, i will check it out, tks!

About the graphs, it can be done but i don't now if Yahoo allow this kind of use...

Regards,
Pacheco

Final patch.

4.7 converstion, conding style, plus some interface improvements, like informing when the user entered a wrong symbol.

And the install file
mysql file is not needed anymore

Thanks pacheco.

Commited to HEAD and 4.7.