Ported to Drupal 6 - Code Needs Testing

The updated and compatible Donations module can be found here.

Can you remove the _ after tar in the file name? Bash auto completion can't find the file when trying to tar xzvf it.

I think that's the upload module calling the file_munge_filename function (or the D5 equiv) as a security thing. My uploaded filenames are correct.

I'm game to try this out. It's come just at the time when i need to implement this functionality on my development site!

I've downloaded both modules and installed them fine. The configuration pages work fine, but i can't see how to add anything in my site or what to do next. Should there be a block for donations or a page? What am i missing?

(I've never used these modules on Drupal 5 btw)

Thanks

You should get a donate page at yoursite/donate, similar to the one at:

http://demo.roopletheme.com/newsflash/donate

Thanks. I do indeed have that page, should have checked URL's more closely! Module seems to be working ok, so far.

Is there a method to make this into a block though? It would be much nicer to be able to move this around the page and add to different pages.

Guys

Please stay on topic. This project is about Paypal as an API, not as an end product (donation, cart, ...etc.).

Let us focus on testing the D6 version of simple_paypal, and not stray into discussions about "how to ...", or other projects/modules, ...etc.

I ran your module against coder to check for bugs. It found the following:

Line 29: Menu item titles and descriptions should now no longer be wrapped in t() calls. (Drupal Docs)
'title' => t('Paypal'),
Line 106: Parameters of watchdog() changed, you must remove calls to t() (Drupal Docs)
watchdog('debug', t('Post variables from Paypal IPN @vars', array(
Line 106: Parameters of watchdog() changed, you use $severity WATCHDOG_DEBUG instead of $type 'debug' (Drupal Docs)
watchdog('debug', t('Post variables from Paypal IPN @vars', array(
Line 133: Parameters of watchdog() changed, you must remove calls to t() (Drupal Docs)
watchdog('paypal', t('Call to curl_exec() failed. url=@url vars=@vars', array(

I didn't know about your port and had started my own. I'm now in testing mode. Want to join forces here? I'm trying to combine simple_paypal, signup, and signup_pay.

Thanks.

Hey thanks. Will fix and repost.

I'd be happy to combine efforts on this... feel free to use my contact.

Cleaned up coder issues. This is a patch against the 5.x HEAD.

Once a couple of people test this and confirm it is working, I will commit it and create a DRUPAL-6--1 and a dev release.

Can someone please give me an update on where this is at? I already posted about this topic BEFORE I saw this list (sorry! experienced php programmer but new to Drupal!) and now I see that it may be close to usable already.

I would be up for trying this out on the Drupal 6.4 site that I am currently working on.

OFFTOPIC: I assume that this is just an API, right? So... what other Drupal modules use this API to implement solutions? I am looking for something straightforward where users pay a yearly fee (different fee for different user types) via PayPal, before their account is activated. I will write this myself if I have to but... why reinvent the wheel? :-)

thanks!
Peter 'Fish' Fisera
GoatVirus Technologies
peter@goatvirus.com

I've been using the version from reply #10 on two sites for a couple of months, along with a similar patch I submitted for the Donate module. I know that the Extended Paypal module, the Bounty module, and the Fee module use this module. For a subscription site, you might also check out the lm_paypal module.

thanks roopletheme! i will follow your suggestions above, and let you know how it goes.

so... i install the 5.x version of Simple Paypal, and then apply the patch from reply #10, is that correct?

cheers,
Fish
GoatVirus Technologies

subscribing

Nearest I can come to getting your patch to apply is to apply it against
this release:
http://drupal.org/node/290655
(I tried against the cvs 5.x HEAD and it didn't go too well :-( )

Even on this release, the patch does not seem to be clean on the .info file :-(
(However, the patch for that is easy to apply by hand:-) )

now, whether this module is what I need - thats another question - so I haven't been able to test the result I'm afraid
(though druapl was happy to install the module, and I could change the config settings :-) )

hi good people... sorry, i won't be one of the test rats for this module after all, i have installed a 6.x alpha of lm_paypal instead (see http://drupal.org/node/209287#comment-996562 ), since it integrates with lm_subscriptions (which is EXACTLY what our project needs).

but i will watch this thread to see how it goes with Simple Paypal. good luck all !

cheers
Fish

I've posted a bounty for this upgrade:

http://drupal.org/node/324987

whats the word on this?

Sorry I don't know what to do with the rej files. I've attached them in hopes that they may be useful to someone.

Ok, I'm pretty sure I've addressed the outstanding issues with the patch. Tried it against the latest CVS and re-rolled it.

Also included the tarball for those who just want to try the D6 version.

I cleaned up the currency thing again and brought it in - http://drupal.org/node/164807

Used a slightly different approach this time to rank the dropdown.

Donation module is upgraded too.

Committed. Thanks all. A release for D6 will be created tonight.

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

no release yet

Release for 6.x now shows on the project page.

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