snowplow-archive / codeigniter-paypal-ipn

A CodeIgniter library for working with the PayPal IPN (Instant Payment Notification) service
111 stars 34 forks source link

Duplicate rows in database #18

Closed jpbringhurst closed 11 years ago

jpbringhurst commented 11 years ago

After setting up, when I run a test order I get a successful order logged in the db and then right after I get another record of failed in the db. Any idea why this is happening?

alexanderdean commented 11 years ago

Weird... It's worth going through the debug steps if you haven't already:

https://github.com/orderly/codeigniter-paypal-ipn/issues/9

jpbringhurst commented 11 years ago

Thanks for the quick response. Do I have to be using the sandbox mode for the debug to work? Currently I am running tests live.

Date: Tue, 4 Dec 2012 16:07:29 -0800 From: notifications@github.com To: codeigniter-paypal-ipn@noreply.github.com CC: jpbringhurst@hotmail.com Subject: Re: [codeigniter-paypal-ipn] Duplicate rows in database (#18)

Weird... It's worth going through the debug steps if you haven't already:

9

          —

          Reply to this email directly or view it on GitHub.
alexanderdean commented 11 years ago

Debug mode should work with live... I'd always recommend getting the library working with the sandbox first, then migrating live...

jpbringhurst commented 11 years ago

I followed the instructions but I don't get any error on screen (there is a no post data error in the log). I have it redirecting to a dashboard if there is not a valid IPN. It acts like it is calling the ipn code twice so the first one comes through with valid data but the second one doesn't.

Date: Tue, 4 Dec 2012 16:15:43 -0800 From: notifications@github.com To: codeigniter-paypal-ipn@noreply.github.com CC: jpbringhurst@hotmail.com Subject: Re: [codeigniter-paypal-ipn] Duplicate rows in database (#18)

Debug mode should work with live... I'd always recommend getting the library working with the sandbox first, then migrating live...

          —

          Reply to this email directly or view it on GitHub.
jpbringhurst commented 11 years ago

Could it be due to the configuration in my paypal account? I have IPN notifications on as well as auto return and PDT enabled. Both have the same url for return and messages.

Date: Tue, 4 Dec 2012 16:15:43 -0800 From: notifications@github.com To: codeigniter-paypal-ipn@noreply.github.com CC: jpbringhurst@hotmail.com Subject: Re: [codeigniter-paypal-ipn] Duplicate rows in database (#18)

Debug mode should work with live... I'd always recommend getting the library working with the sandbox first, then migrating live...

          —

          Reply to this email directly or view it on GitHub.
alexanderdean commented 11 years ago

That sounds like the problem - two different things pinging your IPN handler. Your IPN handler should only be pinged with IPN confirmation messages - use a different controller for anything else...

A On Dec 5, 2012 12:43 AM, "jpbringhurst" notifications@github.com wrote:

Could it be due to the configuration in my paypal account? I have IPN notifications on as well as auto return and PDT enabled. Both have the same url for return and messages.

Date: Tue, 4 Dec 2012 16:15:43 -0800 From: notifications@github.com To: codeigniter-paypal-ipn@noreply.github.com CC: jpbringhurst@hotmail.com Subject: Re: [codeigniter-paypal-ipn] Duplicate rows in database (#18)

Debug mode should work with live... I'd always recommend getting the library working with the sandbox first, then migrating live...

Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/orderly/codeigniter-paypal-ipn/issues/18#issuecomment-11022719.

jpbringhurst commented 11 years ago

Great - I will try disabling the PDT portion of it.

Thank you for your help.

Date: Tue, 4 Dec 2012 16:48:18 -0800 From: notifications@github.com To: codeigniter-paypal-ipn@noreply.github.com CC: jpbringhurst@hotmail.com Subject: Re: [codeigniter-paypal-ipn] Duplicate rows in database (#18)

That sounds like the problem - two different things pinging your IPN

handler. Your IPN handler should only be pinged with IPN confirmation

messages - use a different controller for anything else...

A

On Dec 5, 2012 12:43 AM, "jpbringhurst" notifications@github.com wrote:

Could it be due to the configuration in my paypal account? I have IPN

notifications on as well as auto return and PDT enabled. Both have the same

url for return and messages.

Date: Tue, 4 Dec 2012 16:15:43 -0800

From: notifications@github.com

To: codeigniter-paypal-ipn@noreply.github.com

CC: jpbringhurst@hotmail.com

Subject: Re: [codeigniter-paypal-ipn] Duplicate rows in database (#18)

Debug mode should work with live... I'd always recommend getting the

library working with the sandbox first, then migrating live...

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/orderly/codeigniter-paypal-ipn/issues/18#issuecomment-11022719.

          —

          Reply to this email directly or view it on GitHub.
alexanderdean commented 11 years ago

Great, closing