Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: Trunk
    • Fix Version/s: None
    • Component/s: accounting
    • Labels:
      None
    • Sprint:
      Bug Crush Event - 21/2/2015

      Description

      Pay pal had many interfaces.
      Ofbiz started with IPN https://www.paypal.com/ipn
      Then went to Express check out but changed the IPN code. https://www.paypal.com/us/cgi-bin/webscr?cmd=_additional-payment-overview
      finally we have NVP API. https://cms.paypal.com/us/cgi-bin/?cmd=_render-content&content_ID=developer/e_howto_api_nvp_NVPAPIOverview

      so I am creating an IPN interface that goes back to 777794
      I am not familiar enough with express and NVP it they are separate or not.

        Issue Links

          Activity

          BJ Freeman created issue -
          BJ Freeman made changes -
          Field Original Value New Value
          Link This issue relates to OFBIZ-2693 [ OFBIZ-2693 ]
          Hide
          BJ Freeman added a comment -

          just for history

          Show
          BJ Freeman added a comment - just for history
          Hide
          BJ Freeman added a comment -

          will provide the data for config in another patch
          once this is put in will do a update to current usage.

          Show
          BJ Freeman added a comment - will provide the data for config in another patch once this is put in will do a update to current usage.
          BJ Freeman made changes -
          Attachment OFBIZ-4075paypalIPNEvents.pact [ 12466950 ]
          Jacopo Cappellato made changes -
          Fix Version/s Trunk [ 12311928 ]
          Hide
          Pierre Smits added a comment -

          Should this be closed?

          Show
          Pierre Smits added a comment - Should this be closed?
          Sharan Foga made changes -
          Sprint Bug Crush Event - 21/2/2015 [ 91 ]
          Sharan Foga made changes -
          Rank Ranked higher
          Sharan Foga made changes -
          Rank Ranked higher
          Sharan Foga made changes -
          Rank Ranked higher
          Hide
          Sebastian Wachinger added a comment - - edited

          Actually this is one of the most enlightening threads on this issue to be found. Is there a more detailed timeline available for the three stages of the PayPal integration as outlined in the description by BJ?
          I was not aware that IPN is not supported anymore, or is it? Anyway, using it as is in the trunk now, it always renders the error No valid orderId returned with PayPal Callback, as the IPN parameter invoice is used for orderID but for whatever reason is not transmitted and even does not show up in the otherwise complete parameter listing in console.log.
          I suspect this to be the reason for no order confirmation being sent out to the shop's admin eventually, which we always receive when payment methods other than PayPal Express Checkout are used.
          Are there any restricions on the API version number to be used here, and are there plans to switch to using the REST API?
          Any input on this issue is highly appreciated!

          Show
          Sebastian Wachinger added a comment - - edited Actually this is one of the most enlightening threads on this issue to be found. Is there a more detailed timeline available for the three stages of the PayPal integration as outlined in the description by BJ? I was not aware that IPN is not supported anymore, or is it? Anyway, using it as is in the trunk now, it always renders the error No valid orderId returned with PayPal Callback , as the IPN parameter invoice is used for orderID but for whatever reason is not transmitted and even does not show up in the otherwise complete parameter listing in console.log . I suspect this to be the reason for no order confirmation being sent out to the shop's admin eventually, which we always receive when payment methods other than PayPal Express Checkout are used. Are there any restricions on the API version number to be used here, and are there plans to switch to using the REST API? Any input on this issue is highly appreciated!
          Hide
          Pierre Smits added a comment -

          Sebastian,

          Given that this issue is already 4+ years old and no-one responded on BJ's postings it is valid to say that this didn't find the love it deserves within the project. So, your input is very valuable.

          Best regards,

          Pierre

          Show
          Pierre Smits added a comment - Sebastian, Given that this issue is already 4+ years old and no-one responded on BJ's postings it is valid to say that this didn't find the love it deserves within the project. So, your input is very valuable. Best regards, Pierre
          Hide
          Adrian Crum added a comment -

          There are two PayPal integrations in the trunk:

          org.ofbiz.accounting.thirdparty.paypal.PayPalEvents

          org.ofbiz.accounting.thirdparty.paypal.PayPalServices

          PayPalServices is the most recent and it uses the OFBiz payment processor API. PayPalEvents is older and it does not use the OFBiz payment processor API.

          The PayPalServices code is getting old, but it still works. It needs to be updated to a later PayPal API version.

          Show
          Adrian Crum added a comment - There are two PayPal integrations in the trunk: org.ofbiz.accounting.thirdparty.paypal.PayPalEvents org.ofbiz.accounting.thirdparty.paypal.PayPalServices PayPalServices is the most recent and it uses the OFBiz payment processor API. PayPalEvents is older and it does not use the OFBiz payment processor API. The PayPalServices code is getting old, but it still works. It needs to be updated to a later PayPal API version.

            People

            • Assignee:
              Unassigned
              Reporter:
              BJ Freeman
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Development

                  Agile