[Bug 8421] New: patron images fail when barcode for patron has a +
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Priority: P5 - low Change sponsored?: --- Bug ID: 8421 Assignee: gmcharlt@gmail.com Summary: patron images fail when barcode for patron has a + Severity: minor Classification: Unclassified OS: All Reporter: elliott@bywatersolutions.com Hardware: All Status: NEW Version: master Component: Architecture, internals, and plumbing Product: Koha It appears that the cardnumber is not url encoded when it comes from patronimage.pl. Should be a simple fix. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Elliott Davis <elliott@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|gmcharlt@gmail.com |elliott@bywatersolutions.co | |m -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 --- Comment #1 from Elliott Davis <elliott@bywatersolutions.com> --- Created attachment 10778 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10778&action=edit Added a URI filter to encode a cardnumber for borrowers as to not break patronimage.pl To Test: Add a + (or any char that would need to be encoded) to a card number of a patron with an image attached. Go to member/moremember.pl. If the patrons image is there, you are golden. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Elliott Davis <elliott@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #10778|0 |1 is obsolete| | --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 10909 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10909&action=edit [SIGNED-OFF] Added a URI filter to encode a cardnumber for borrowers as to not break patronimage.pl To Test: Add a + (or any char that would need to be encoded) to a card number of a patron with an image attached. Go to member/moremember.pl. If the patrons image is there, you are golden. http://bugs.koha-community.org/show_bug.cgi?id=8421 Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |paul.poulain@biblibre.com Version|master |rel_3_8 --- Comment #3 from Paul Poulain <paul.poulain@biblibre.com> --- QA comment: tiny template-only patch, passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8421 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |chris@bigballofwax.co.nz --- Comment #4 from Chris Cormack <chris@bigballofwax.co.nz> --- Pushed to 3.8.x, will be in 3.8.4 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org