https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23761 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |kyle@bywatersolutions.com Severity|enhancement |minor -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23761 --- Comment #1 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 93817 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=93817&action=edit Bug 23761 - Allow PayPal return url to be set based on current domain, not OPACBaseURL In some library systems, each library has a customized sub-opac and a separate PayPal account. This is accomplished using the css and syspref override directives in the Apache conf. This all works great until a patron wants to make a PayPal payment. Let us assume OPACBaseURL is set to catalog.mylib.org, the generic unstyled OPAC. Each library then has a 'sub-opac' such as labA.catalog.mylib.lorg. If the patron is on an OPAC at libA.catalog.mylib.org, and makes a paypal payment, the patron will not be returned to libA.catalog.mylib.org. Instead, the patron will be send back to catalog.mylib.org. The simple solution is to use the CGI module to build a URL using the current base URL: $cgi->url( -base => 1 ) Test Plan: 1) Set up sub-OPACs and PayPal ( production or sandbox mode ) 2) Initiate a payment from a sub-OPAC 3) Note you are returned to whatever is in OPACBaseURL 4) Apply this patch 5) Restart all the things! 6) Initiate a second payment from a sub-OPAC 7) Note that you are returned to the correct sub-OPAC url! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23761 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #93817|0 |1 is obsolete| | --- Comment #2 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 93818 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=93818&action=edit Bug 23761 - Allow PayPal return url to be set based on current domain, not OPACBaseURL In some library systems, each library has a customized sub-opac and a separate PayPal account. This is accomplished using the css and syspref override directives in the Apache conf. This all works great until a patron wants to make a PayPal payment. Let us assume OPACBaseURL is set to catalog.mylib.org, the generic unstyled OPAC. Each library then has a 'sub-opac' such as labA.catalog.mylib.lorg. If the patron is on an OPAC at libA.catalog.mylib.org, and makes a paypal payment, the patron will not be returned to libA.catalog.mylib.org. Instead, the patron will be send back to catalog.mylib.org. The simple solution is to use the CGI module to build a URL using the current base URL: $cgi->url( -base => 1 ) Test Plan: 1) Set up sub-OPACs and PayPal ( production or sandbox mode ) 2) Initiate a payment from a sub-OPAC 3) Note you are returned to whatever is in OPACBaseURL 4) Apply this patch 5) Restart all the things! 6) Initiate a second payment from a sub-OPAC 7) Note that you are returned to the correct sub-OPAC url! Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: David Kuhn <kuhn@monterey.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23761 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Kyle, could you please take a look at how bug 21701 might relate to this? We just got it signed off at the hackfest last week. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23761 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Kyle M Hall from comment #4)

(In reply to Katrin Fischer from comment #3)

...

Hi Kyle, could you please take a look at how bug 21701 might relate to this? We just got it signed off at the hackfest last week.

That solves the same problem, albeit in a different way. I'll close my bug and QA that one :)

*** This bug has been marked as a duplicate of bug 21701 ***

Cool - thx! -- You are receiving this mail because: You are watching all bug changes.