SMF Support > SMF 2.0.x Support

Paid Subscriptions - Help

<< < (7/9) > >>

Krash.:
Ok, looking in /Sources/Subscriptions-PayPal.php, it appears that the following txn_types are recognized:

  Refunded
  reversal
  subscr_payment
  web_accept

The following, which appear in your error log, are not:

  subscr_eot
  subscr_cancel
  subscr_signup

Guessing that subscr_eot is recurring payment.  You can confirm that by trying another subscription with recurring payment checked.  subscr_cancel is sent when you cancel a recurring payment.  Don't know what subscr_signup is.

The same file contains the following:


--- Code: ---
// Verify the currency!
if (strtolower($_POST['mc_currency']) != $modSettings['paid_currency_code'])
exit;


--- End code ---

Looks like it compares the mc_currency code from the IPN with the selected currency type in subscription settings.  Since AUD is not a choice, it will return false.  This hack may fix it:


--- Code: ---
// Verify the currency!
if ((strtolower($_POST['mc_currency']) != $modSettings['paid_currency_code']) && ($_POST['mc_currency']) != 'aud'))
exit;


--- End code ---

Don't think it's case sensitive.  Backup the original file before you try this.

Wazza:
Hi Krash,

Points taken

For me in each instance I had this error with the 'recurring payment' the money was taken from the members account and their PayPal account was set up with a 'recurring payment' to my PayPal 'Business' account. However in return this member never got what they paid for due to this error. So for responsible use I personally would prefer this working then removed. However at current it is much worse as this makes this open for others to make this mistake in taking money without giving a service.

Stopping a 'recurring payment'  was quite easy to do via their PayPal account or from my PayPal account I was also able to cancel or refund payment(s) made. So I don't see the problem if the site owner is responsible with his members, plus as this is only a function they can be used via a business and business PayPal Account then the consumer should have more rights if they need to pursue a business.

But at present this for SMF is a security issue and should be ether removed for the points you raised or made functional for responsible business use only. As to allow the program the option of 'recurring payment' being  currently created by SMF, but at current not being recognized by SMF in a return_url, is only going to cause future problems.
 
__________________________________

 
I will try the code changes to have this recognize the Australian Dollar "AUD"

And hope we can find an answer for above.

Thank you again for your help  :)

Wazza

Krash.:

Posted the recurring payments problem in Bug Reports - will see if anyone confirms it.

Let me know if the mc_currency hack works.

Wazza:
Will do,


...and please check and cancel any recurring payments from your PayPal account and I will refund the one that just went though.

Again thank you

Wazza

Wazza:
Famous last words back up  :D


Went to "Settings" Paid subscriptions and seen a white screen reading the following


Parse error: syntax error, unexpected ')' in /home/"VPS name removed by wazza"/public_html/ATown/Sources/Subscriptions-PayPal.php on line 203

...will restore back up now  ;)

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version