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!

CommentFileSizeAuthor
#7 stock_d7_upgrade_v2.patch16.78 KBdrclaw
#4 stock_dev_convert_to_d7.patch16.72 KBdrclaw
stock_convert_to_d7.patch16.63 KBdrclaw

Comments

drclaw’s picture

drclaw commented

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

drclaw’s picture

drclaw commented
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

kbahey commented
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

drclaw commented
StatusFileSize
new stock_dev_convert_to_d7.patch16.72 KB

Try this one.

kbahey’s picture

kbahey commented

Still the same.

drclaw’s picture

drclaw commented
StatusFileSize
new stock_d7_upgrade_v2.patch16.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

kbahey commented
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

drclaw commented

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.