Hello,

I did some work on this module to get it working on D7. There was some interest in the Stock API module issue queue so I'm just posting the patch. Unfortunately I've only tested it on D7 beta1 and beta2. I don't think anything's changed that will affect these changes, but I haven't had a chance to test it on a proper Drupal 7 v1.0 yet.

Place the patch in the root of the the stock module folder (e.g. stock/stock_convert_to_d7.patch) and, from the command line, use the command patch < stock_convert_to_d7.patch to patch it.

Hope this helps someone!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drclaw’s picture

Oh, if you need the stockapi patch, it's in the stockapi issue queue: #936240: Upgrade to D7?

drclaw’s picture

Version: 6.x-1.x-dev » 6.x-1.2

Also, the patch is actually for version 1.2, not the dev version. Sorry!

kbahey’s picture

Status: Active » Needs work

Patch fails to apply.

I tried against HEAD as well as the tag DRUPAL-6--1. Same results.

$ patch --dry-run < stock_convert_to_d7.patch 
patching file stock.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file stock.info.rej
patching file stock.install
Reversed (or previously applied) patch detected!  Assume -R? [n] 
Apply anyway? [n] 
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file stock.install.rej
patching file stock.module
Reversed (or previously applied) patch detected!  Assume -R? [n] 
Apply anyway? [n] 
Skipping patch.
15 out of 15 hunks ignored -- saving rejects to file stock.module.rej

Please change it to be against 6.x-1.x-dev, since that may have had some fixes since 1.2.

drclaw’s picture

Try this one.

kbahey’s picture

Still the same.

drclaw’s picture

FileSize
16.78 KB

Damn, just realized I had the diffs backwards! Total rookie mistake, I'm sorry. This one is better. It's for the dev version.

kbahey’s picture

Version: 6.x-1.2 » 7.x-1.x-dev
Status: Needs work » Fixed

Thanks. That worked, with a modification to the .info part which was not right.

There is now a release for 7.x, but it will not appear until about 6 hours from now.

Remind me tomorrow, by commenting in this issue, if you do not see a 7.x release on the project's main page.

drclaw’s picture

Glad to see that 7.x release! Will you also do the stockapi module?

Status: Fixed » Closed (fixed)

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