http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11622 Kyle M Hall <kyle.m.hall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle.m.hall@gmail.com --- Comment #17 from Kyle M Hall <kyle.m.hall@gmail.com> --- (In reply to Jonathan Druart from comment #14)
(In reply to Galen Charlton from comment #5)
Comment on attachment 24754 [details] [review] [review] Bug 11622 - Add ability to pay fees and fines from OPAC via PayPal
Review of attachment 24754 [details] [review] [review]: -----------------------------------------------------------------
::: opac/opac-account-pay-paypal-return.pl @@ +20,5 @@
+ +use strict; +use warnings; +use utf8; +
Modern::Perl I see no non-ASCII characters in the source.
Still exists.
I'm not sure what this means. Can you explain?
::: opac/opac-account-pay.pl @@ +19,5 @@
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + +use strict; +use warnings; +use utf8;
Modern::Perl I see no non-ASCII characters in the source.
Still exists.
Same as above
@@ +94,5 @@
+ 'PAYMENTREQUEST_0_PAYMENTACTION' => 'Sale', + 'PAYMENTREQUEST_0_ALLOWEDPAYMENTMETHOD' => 'InstantPaymentOnly', + 'PAYMENTREQUEST_0_DESC' => + C4::Context->preference('PayPalChargeDescription'), + };
It seems like using Business::PayPal::API might offer some protection against API changes.
Answer needed.
That's what I tried first when I began writing this feature. It did not go well. PayPal doesn't change existing APIs anyway. They have an API versioning system so that applications don't break with changes to the API.
+use CGI;
Should be replaced by CGI qw( -utf8 ) unless you have a good reason.
Fixed!
3) Create a paypal developer account
How? Should I need a paypal account to do so?
I don't think you need a regular paypal account, but you do need an account on https://developer.paypal.com/ -- You are receiving this mail because: You are watching all bug changes.