Here's a patch for a Drupal 6 version of the currency module. This is my first patch submitted, so if I need to change anything in the formatting, please let me know.

Tested against 6.3.

CommentFileSizeAuthor
#5 currency-patch.zip1.58 KBtr
#1 currency6.patch8.25 KBscott falconer
currency6.patch0 bytesscott falconer

Comments

scott falconer’s picture

scott falconer commented
StatusFileSize
new currency6.patch8.25 KB

Not sure why that attached as 0 bytes...let me try again.

kbahey’s picture

kbahey commented
Version: 5.x-1.1 » 6.x-1.x-dev
Status: Needs review » Fixed

Thank you for the patch.

I created a 6.x dev release that will be available in 24 hours.

Please test it.

tr’s picture

tr commented

This patch didn't change the .install file for the currency_api? The new D6 Schema API is a big part of the changes needed for a D6 version...

kbahey’s picture

kbahey commented
Status: Fixed » Needs work

@TR

Can you please submit a patch for it here in this issue?

tr’s picture

tr commented
Status: Needs work » Needs review
StatusFileSize
new currency-patch.zip1.58 KB

Attached is a .zip with two things in it:

1) currency.install
A new file needed for the currency module. Holds hook_uninstall() which removes variables set by this module.

2) currency_api_install.patch
A patch to the DRUPAL-6--1 branch, porting currency_api.install to D6.

Both these have been tested under D6.2. The modules install/uninstall cleanly under D6 without leaving behind any dreck in the database. The modules function correctly under D6 as far as I have tested.

kbahey’s picture

kbahey commented
Status: Needs review » Fixed

Both committed.

We still have the bad tag on HEAD being DRUPAL-5, which I will get to in a week or so.

Anonymous’s picture

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

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