| Summary: | Bad cardnumber causes 'Borrower' search to fail | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Mason James <mtj> |
| Component: | Staff interface | Assignee: | Bugs List <koha-bugs> |
| Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | gmcharlt, jonathan.druart, ldjamison |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: | pic | ||
Hi Mason, can you provide clarification for your step 1? I'm confused what you mean by a leading tab character because pressing the TAB key in the cardnumber field of memberentry.pl?op=modify simply changes the focus to the Library drop down field. Are you referring to the escaped '\t' character? > ...because pressing the TAB key in the cardnumber field
> of memberentry.pl?op=modify simply changes the focus to the Library drop
> down field.
I copy/pasted the problematic string from a text-editor into the field
We can do that:
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
@@ -14,7 +14,7 @@
"<input type='checkbox' class='selection' name='borrowernumber' value='[% data.borrowernumber %]' />",
[% END %]
"dt_cardnumber":
- "[% data.cardnumber | html %]",
+ "[% To.json( data.cardnumber ) %]",
But we should do it for all the data.
I'm not able to save a card number with a tab character, so I think this can be considered fixed. |
Created attachment 64122 [details] pic to test bug.. 1/ add a leading TAB character to a borrower's cardnumber 2/ attempt to search for that user 3/ observe that no results are displayed (see pic)