Ported to Drupal 6 - Code Needs Testing [#280102]

I've ported the Simple Paypal and Donation modules to Drupal 6 and added a handful of features that make the modules easier to configure. I'll submit patches to the issue queues for both modules, but I was first hoping to get some help testing the initial cuts.

In addition to Drupal 6 compatibility, this version of the Donation module provides the administrator with the opportunity to customize nearly all of the text strings associated with the Donate page. This should eliminate the need to edit the module code in order to customize things for your site.

Anyone game to try this out (along with the updated Simple Paypal module that I posted to that module's issue queue) and give me some feedback?

Comment	File	Size	Author
#7	donation_d6.patch	26.65 KB	mgifford
#7	donation_d6.tar_.gz	11.2 KB	mgifford
#3	rej.zip	3.92 KB	flickerfly
#2	donation_280102_0.patch	26.34 KB	roopletheme
	Donation-6.x-1.1-dev.tar_.gz	11.02 KB	roopletheme

Comments

Comment #1

kbahey commented 23 July 2008 at 03:06

Status:

Active

» Needs review

Comment #2

roopletheme commented 26 July 2008 at 10:34

Status	File	Size
new	donation_280102_0.patch	26.34 KB

Cleaned up some issues found by the Coder module. This is a patch against the 5.x HEAD.

Comment #3

flickerfly commented 28 January 2009 at 16:40

Status	File	Size
new	rej.zip	3.92 KB

$ patch -p0 < donation_280102_0.patch 
patching file donation/donation.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file donation/donation.info.rej
patching file donation/donation.install
patching file donation/donation.module
Hunk #1 FAILED at 1.
Hunk #2 succeeded at 251 with fuzz 2 (offset 100 lines).
Hunk #3 succeeded at 333 (offset 100 lines).
Hunk #4 succeeded at 431 (offset 100 lines).
Hunk #5 succeeded at 485 (offset 100 lines).
Hunk #6 FAILED at 511.
Hunk #7 succeeded at 554 (offset 100 lines).
Hunk #8 succeeded at 580 (offset 100 lines).
Hunk #9 FAILED at 604.
Hunk #10 FAILED at 615.
Hunk #11 succeeded at 638 (offset 100 lines).
Hunk #12 FAILED at 655.
Hunk #13 FAILED at 677.
Hunk #14 succeeded at 708 (offset 111 lines).
Hunk #15 FAILED at 718.
7 out of 15 hunks FAILED -- saving rejects to file donation/donation.module.rej

I don't know much about diff/patch and what to do with the rej files. I've attached them for someone who does.

Comment #4

green monkey commented 7 February 2009 at 01:27

any update on D6 version?

Comment #5

green monkey commented 15 February 2009 at 01:06

would be happy to help test. But not to good at patching.

Can I download a version that is patched with the above?

Comment #6

rvan1 commented 20 February 2009 at 21:02

I have Donation 6.x running OK with Simple PayPal Framework on Drupal 6.9, but haven't applied patches.

I didn't see anywhere where it says to set the paypal email address, and other content, under 'Admin > Site Configuration > Donations', or to navigate to '/donate' to get to the submit form, or what 'Admin > Site Building > Donations' List is for, so it took a little time to put things together.