SMF Development > Bug Reports
Another paid subscriptions bug?
Arantor:
That's the joy of the sandbox mode, you can test it and test it and test it over and once you're satisfied it can be committed :)
Wazza:
Hi Krash,
Injected the code (don't worry I have a back up :P) into the test 2.0.2 we created.
Came up with this error.
Fatal error: Cannot redeclare paypal_payment::isSubscription() in /home/wazza/public_html/ATown/Sources/Subscriptions-PayPal.php on line 233
...I should point out to the others I agree totally with the extra comments about getting this wrong when its someones money we are working with. However currently left the way it is, this gets it wrong ???
"Recurring" whilst for me would be great to be working as optioned in SMF, I should say this is not imperative to me that this must work. I looked at this more as a time saver for members not needing to return and renew each month. I can live with out this, if this cannot be made functional. However for me I'm now more concerned for the future ramifications left this way in SMF if others make the same mistake?
I believe we ether need to fix this option or have this option removed :-\
Wazza
Krash.:
--- Quote from: Arantor on May 28, 2012, 02:43:56 PM ---Well, it definitely has worked for me in the past, so I'm intrigued as to why it hasn't for you.
--- End quote ---
Where is the code that recognizes subscr_signup? That's what has me stumped.
--- Quote from: emanuele on May 28, 2012, 02:48:15 PM ---To debug paypal issues I need to be in a very good mood (it has to do with money and I don't want to do something wrong with someone else's money) so...later.
--- End quote ---
Don't believe it's a PayPal problem. They're sending the correct IPN, and it's received by subscriptions.php, but SMF doesn't recognize the txn_type and subscription fails.
Krash.:
--- Quote from: Wazza on May 28, 2012, 05:17:15 PM ---Fatal error: Cannot redeclare paypal_payment::isSubscription() in /home/wazza/public_html/ATown/Sources/Subscriptions-PayPal.php on line 233
--- End quote ---
Don't see how the hack could cause that error. Attach your modified Subscriptions-Paypal.php.
Wazza:
Attached ;)
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version