The attached patch updates paypal_subscription to the forms API, and converts its main admin function over to a proper set of _page() functions using the submit model.

I haven't done extensive testing of the functionality yet, but the UI works smoothly.

CommentFileSizeAuthor
#2 paypal_subscription.module_0.patch34.78 KBeaton
#1 paypal_subscription.install1.28 KBeaton
paypal_subscription.module.patch34.44 KBeaton

Comments

eaton’s picture

eaton commented
StatusFileSize
new paypal_subscription.install1.28 KB

Attached is an .install file for the module as well.

eaton’s picture

eaton commented
StatusFileSize
new paypal_subscription.module_0.patch34.78 KB

One of the form fields was accidentally dropped during conversion -- subscriptions would never be recognized without it. :)

New patch attached. With the latest CVS version of paypal_framework, this version has been tested and DOES work for basic 'make a donation, get a new role' functionality.

reg’s picture

reg commented

I found a bug that needed to be fixed after these patches are installed. You can see the fix at 79246.

Ariadoss’s picture

Ariadoss commented

Which version of PayPal subscription are these patches for? Also, is the patch at http://drupal.org/node/79246 in addition to this patch or by itself. I received errors while trying to apply both patches on the CVS of the PayPal Subscription module. Thanks. :)

dbr’s picture

dbr commented
Assigned: Unassigned » dbr
Status: Needs review » Fixed

This patch and the install file have been commited to CVS (HEAD only at the moment).

Anonymous’s picture

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