Bug 30840 - Add support for barcode filters to course reserves
Summary: Add support for barcode filters to course reserves
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Course reserves (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Nick Clemens (kidclamp)
QA Contact: Joonas Kylmälä
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-24 18:28 UTC by Nick Clemens (kidclamp)
Modified: 2023-06-08 22:26 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00, 22.05.01, 21.11.07


Attachments
Bug 30840: Add support for barcode filters to course reserves (3.37 KB, patch)
2022-05-24 18:31 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 30840: Add support for barcode filters to course reserves (3.41 KB, patch)
2022-05-24 19:13 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 30840: Add support for barcode filters to course reserves (3.55 KB, patch)
2022-05-26 10:58 UTC, Joonas Kylmälä
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2022-05-24 18:28:01 UTC

    
Comment 1 Nick Clemens (kidclamp) 2022-05-24 18:31:45 UTC
Created attachment 135328 [details] [review]
Bug 30840: Add support for barcode filters to course reserves

This bug adds support for barcode filters on the inputs in the course reserves module.

To test:
1 - Set itemBarcodeInputFilter to 'Remove spaces from'
2 - Find a barcode in your system and add spaces between each characters
    e.g. 3 9 9 9 9 0 0 0 0 0 0 8 1 8
3 - Past that barcode into the checkin box, success
4 - Browse to course reserves, ensure you have a course or create on
5 - Test adding a reserve, batch adding a reserve, and batch removing reserves
6 - The pasted barcode does not in any
7 - Apply patch and restart
8 - Pasted barcode with spaces can now successfuly be added, batch added, and batch removed from courses
Comment 2 ByWater Sandboxes 2022-05-24 19:13:40 UTC
Created attachment 135329 [details] [review]
Bug 30840: Add support for barcode filters to course reserves

This bug adds support for barcode filters on the inputs in the course reserves module.

To test:
1 - Set itemBarcodeInputFilter to 'Remove spaces from'
2 - Find a barcode in your system and add spaces between each characters
    e.g. 3 9 9 9 9 0 0 0 0 0 0 8 1 8
3 - Past that barcode into the checkin box, success
4 - Browse to course reserves, ensure you have a course or create on
5 - Test adding a reserve, batch adding a reserve, and batch removing reserves
6 - The pasted barcode does not in any
7 - Apply patch and restart
8 - Pasted barcode with spaces can now successfuly be added, batch added, and batch removed from courses

Signed-off-by: Jason Robb <jrobb@sekls.org>
Comment 3 Joonas Kylmälä 2022-05-26 10:58:44 UTC
Created attachment 135366 [details] [review]
Bug 30840: Add support for barcode filters to course reserves

This bug adds support for barcode filters on the inputs in the course reserves module.

To test:
1 - Set itemBarcodeInputFilter to 'Remove spaces from'
2 - Find a barcode in your system and add spaces between each characters
    e.g. 3 9 9 9 9 0 0 0 0 0 0 8 1 8
3 - Past that barcode into the checkin box, success
4 - Browse to course reserves, ensure you have a course or create on
5 - Test adding a reserve, batch adding a reserve, and batch removing reserves
6 - The pasted barcode does not in any
7 - Apply patch and restart
8 - Pasted barcode with spaces can now successfuly be added, batch added, and batch removed from courses

Signed-off-by: Jason Robb <jrobb@sekls.org>

Signed-off-by: Joonas Kylmälä <joonas.kylmala@iki.fi>
Comment 4 Tomás Cohen Arazi 2022-06-02 17:08:56 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 5 Lucas Gass 2022-06-15 15:57:46 UTC
pushed to 22.05.x for 22.05.01
Comment 6 Arthur Suzuki 2022-06-21 13:42:55 UTC
thx pushed to 21.11.x for 21.11.07
Comment 7 Victor Grousset/tuxayo 2022-06-26 00:00:36 UTC
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed.