It would be nice to be able to select which payment method to allow so you can select only paypal and check.

It would also be nice to be able to just add a method that's Credit Card and when selected, drops a credit card # box, expiration, name on card, unique code, and type. Upon signup, the credit info will just be emailed to an email.

The email field specified for receiving CC info in signup pay config page - this email (ex:email for accounting) should be different from email for receiving registration since this should be done through SSL and person receiving this email should be the person handling payment so there will be no need to forward such sensitive info. This will be useful for companies processing CC manually. Additional email to person handling registration will also be sent to notify them that someone register, but are awaiting payment.

CommentFileSizeAuthor
#9 choose_allowed_payment_methods1.patch2.27 KBeojthebrave
#4 option_to_enable-disable_payment.patch7.1 KBeojthebrave
#4 combined1.patch8.2 KBeojthebrave

Comments

kbahey’s picture

kbahey commented

Limiting the list of payment types is an OK feature to have.

Sending credit card by email is absolutely not acceptable, and is a big security risk.

giggler’s picture

giggler commented

Thanks for clarifying that. I've tried the paypal payment and it doesn't seem to give the pay by credit card option. Usually there is an option to the left of the login to paypal box that has "Don't have a PayPal account?" that let's you select to pay by credit card without having a paypal account. Can that variable be added?

Not sure which value does that:
https://www.paypal.com/IntegrationCenter/ic_std-variable-reference.html

bsherwood’s picture

bsherwood commented

This is a great idea, a lot of admins work with just online registration and they don't accept check/MO/etc..

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
StatusFileSize
new combined1.patch8.2 KB
new option_to_enable-disable_payment.patch7.1 KB

This patch adds the ability to turn payment methods on/off. Defaults to having PayPal on and all others off. Can be changed in admin/settings/signup_pay

The second patch, combined1.patch includes this patch, and the interface enhancements from this issue. http://drupal.org/node/239006

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Status: Active » Needs review

See above for patch.

kbahey’s picture

kbahey commented
Status: Needs review » Needs work

I applied the other patch, and now this fails. Please re-roll it.

Thanks

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented

Which version do you want me to diff against? The current CVS? Or other? Sorry for the confusion. Also, the comment above already contains a patch that has both this, and the interface enhancements from the other thread combined into one if that helps at all.

kbahey’s picture

kbahey commented

Do you have a cvs checkout from the "DRUPAL-5" tag? If so, then do a "cvs up" and diff against that.

Otherwise, wait till tomorrow for the updated tarball and use that.

The first method is preferred if you are well versed in using cvs.

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Status: Needs work » Needs review
StatusFileSize
new choose_allowed_payment_methods1.patch2.27 KB

New patch applies against CVS "DRUPAL-5" tag.

kbahey’s picture

kbahey commented
Status: Needs review » Fixed

Committed.

For future patches, please watch out for hard tabs vs. just 2 spaces. The coder module should help with code style.

Thanks again.

Anonymous’s picture

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

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