Ported to Drupal 6 - Code Needs Testing

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'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

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.

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

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?

$ patch -p0 < simple_paypal_280101_0.patch 
patching file simple_paypal/simple_paypal.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file simple_paypal/simple_paypal.info.rej
patching file simple_paypal/simple_paypal.module
Hunk #3 FAILED at 63.
Hunk #4 succeeded at 143 (offset 40 lines).
Hunk #5 succeeded at 170 (offset 40 lines).
1 out of 5 hunks FAILED -- saving rejects to file simple_paypal/simple_paypal.module.rej

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.

no release yet

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