Hello
- Tuan Pham Ngoc
- Offline
- Administrator
-
I haven't changed code in the component. It remains as how it was . I noticed that some customers reported this issues recently and it only happens with Paypal on TEST MODE, not LIVE MODE . Maybe Paypal changes something in Test Mode recently, I am not sure !
Regarding your question Because in your code, to accept the transaction you search for the word "VERIFIED" in the IPN from paypal and in the ipn from the sandbox it doesn't have this word in it. . The reason is for security purpose . If you just check payment_status variable, what happens if someone make a post request (not Paypal) to your site and has payment_status variable set to Completed ? The registration will be approved without clients making any payment.
Hope my explanation is clear to you !
Tuan
Please Log in or Create an account to join the conversation.
- robertsm
- Offline
- New Member
-
- Posts: 4
- Thank you received: 0
A couple of months ago PayPal changed how IPN is carried out. I had just purchased Joombah (joombah.com) and found that this change was causing problems with that component, necessitating that developer to release a new version of their component. At first that developer thought it was only a sandbox issue, but it proved to be a problem with live sites and sandbox.
Did JoomDonation update Event Booking to accomodate this change with the PayPal IPN?
Please Log in or Create an account to join the conversation.
- Tuan Pham Ngoc
- Offline
- Administrator
-
Tuan
Please Log in or Create an account to join the conversation.
- lebill
- Topic Author
- Offline
- New Member
-
- Posts: 7
- Thank you received: 0
I'm a developer my self and I fix your component for the sandbox and also fix the validation of the payment in the same time.
Very easy to do , it's only a line of code to change.
In your /components/com_eventbooking/payments/os_paypal.php
at line 177 you have this
For me this is not good because you validate the payment if you found the VERIFIED word in the response from Paypal
if the payment is pending with your code your component allow the sale.
just replace the line with this one
With that line it fix the sandbox issue and the transaction is not process if the payment is not completed
Hope it help someone here
Regards
Bill
Please Log in or Create an account to join the conversation.
- lebill
- Topic Author
- Offline
- New Member
-
- Posts: 7
- Thank you received: 0
To answer your explication on the previous post
The reason is for security purpose . If you just check payment_status variable, what happens if someone make a post request (not Paypal) to your site and has payment_status variable set to Completed ? The registration will be approved without clients making any payment.
I don't know if you did this code, but the modification is in the validate() function, in that function you call back paypal to verify if what your os_paypal.php receive is good. And you check if you have the word VERIFIED in the answer (you don;t have anymore this word in the sandbox answer). If what you send to paypal was not good, paypal doesn't return the payment_status or any of the normal variable paypal use, I don;t remember exactly (it's long time I did the test) what is the answer in that case, but I'm sure when you ask Paypal for a cmd=_notify-validate, Paypal never return Paypal's variables with information in it, like email ID's etc... if the infos you try validate are not good
Anyway, I'm just trying to help here
Bill
Please Log in or Create an account to join the conversation.
- robertsm
- Offline
- New Member
-
- Posts: 4
- Thank you received: 0
For those following this thread: I just ran a test event (for $1) and processed via live payment with PayPal. I can confirm that this seems to be mainly a sandbox problem, and that live PayPal payments still process and they will post correctly in the Joomla Admin.
Please Log in or Create an account to join the conversation.
- Tuan Pham Ngoc
- Offline
- Administrator
-
I will modify the Paypal payment plugin so that we will just check payment_status variable on the return. For Live Mode, I will keep the code as how it is at the moment and plus checking payment_status as suggested !
It should be done at this time on tomorrow or alitle later !
Tuan
Please Log in or Create an account to join the conversation.
- robertsm
- Offline
- New Member
-
- Posts: 4
- Thank you received: 0
Please Log in or Create an account to join the conversation.
- Tuan Pham Ngoc
- Offline
- Administrator
-
Tuan
Please Log in or Create an account to join the conversation.
- Tuan Pham Ngoc
- Offline
- Administrator
-
Tuan
Please Log in or Create an account to join the conversation.
Support
Documentation
Information
The Joomla! name and logo is used under a limited license granted by Open Source Matters the trademark holder in the United States and other countries.