Summary: | Item search in staff interface should call barcodedecode if the search index is a barcode | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Searching | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | e.betemps, michaela.sieber |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31252 | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
23.11.00,23.05.02,22.11.08
|
|
Circulation function: | |||
Bug Depends on: | 26351 | ||
Bug Blocks: | |||
Attachments: |
Bug 31253 - Item search in staff interface should call barcodedecode if the search index is a barcode
Bug 31253 - Item search in staff interface should call barcodedecode if the search index is a barcode Bug 31253: Item search in staff interface should call barcodedecode if the search index is a barcode |
Description
Kyle M Hall (khall)
2022-07-28 13:12:33 UTC
Created attachment 138205 [details] [review] Bug 31253 - Item search in staff interface should call barcodedecode if the search index is a barcode We should run any inputted barcode through barcodedecode before passing it to any subroutines. This was missed during the initial development of bug 26351. Test Plan: 1) Set itemBarcodeInputFilter to "Remove spaces from" 2) Create an item with the barcode "MYTEST" 3) Browse to the staff side advanced search 4) Run a barcode search for "MY TEST" 5) Note no results are round 6) Apply this patch 7) Restart all the things! 8) Repeat your search 9) Note the item was found! Is this specific to the search engine? Searching for MY TEST returns no results (In reply to Owen Leonard from comment #2) > Is this specific to the search engine? It should not be. It changes the search term that is used. Also item search is SQL based. No results match your search for 'bc,wrdl: "MY TEST"'. (In reply to Emmanuel Bétemps from comment #5) > No results match your search for 'bc,wrdl: "MY TEST"'. That is not a valid test. This patch is about item searching ( itemsearch.pl ) Created attachment 152909 [details] [review] Bug 31253 - Item search in staff interface should call barcodedecode if the search index is a barcode We should run any inputted barcode through barcodedecode before passing it to any subroutines. This was missed during the initial development of bug 26351. Test Plan: 1) Set itemBarcodeInputFilter to "Remove spaces from" 2) Create an item with the barcode "MYTEST" 3) Browse to the staff side advanced search 4) Run a barcode search for "MY TEST" 5) Note no results are round 6) Apply this patch 7) Restart all the things! 8) Repeat your search 9) Note the item was found! Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Created attachment 152937 [details] [review] Bug 31253: Item search in staff interface should call barcodedecode if the search index is a barcode We should run any inputted barcode through barcodedecode before passing it to any subroutines. This was missed during the initial development of bug 26351. Test Plan: 1) Set itemBarcodeInputFilter to "Remove spaces from" 2) Create an item with the barcode "MYTEST" 3) Browse to the staff side advanced search 4) Run a barcode search for "MY TEST" 5) Note no results are round 6) Apply this patch 7) Restart all the things! 8) Repeat your search 9) Note the item was found! Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Edit: fixed typo in comment Pushed to master for 23.11. Nice work everyone, thanks! Thanks for all the hard work! Pushed to 23.05.x for the next release Nice work everyone! Pushed to 22.11.x for next release |