Bug 25261 - Multiple parts handling - confirmation alert
Summary: Multiple parts handling - confirmation alert
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Martin Renvoize
QA Contact: Testopia
URL:
Keywords: Manual-updated
Depends on:
Blocks: 27976 30519 29504
  Show dependency treegraph
 
Reported: 2020-04-23 09:38 UTC by Jonathan Field
Modified: 2024-02-23 10:35 UTC (History)
12 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: Medium patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This enhancement adds the option to require staff members to confirm that an item contains all its listed parts at check-in/check-out time. New system preference: `CircConfirmItemParts`
Version(s) released in:
20.11.00


Attachments
Bug 25261: Add system preference (3.29 KB, patch)
2020-04-28 12:57 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (8.77 KB, patch)
2020-04-28 12:57 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.44 KB, patch)
2020-04-30 10:07 UTC, Frédéric Demians
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (8.88 KB, patch)
2020-04-30 10:08 UTC, Frédéric Demians
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.19 KB, patch)
2020-04-30 10:08 UTC, Frédéric Demians
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.44 KB, patch)
2020-05-06 08:28 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (8.89 KB, patch)
2020-05-06 08:28 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.19 KB, patch)
2020-05-06 08:28 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (4.94 KB, patch)
2020-05-06 08:28 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.70 KB, patch)
2020-05-06 08:28 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.50 KB, patch)
2020-05-06 09:09 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (8.95 KB, patch)
2020-05-06 09:09 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.19 KB, patch)
2020-05-06 09:09 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (4.94 KB, patch)
2020-05-06 09:09 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.70 KB, patch)
2020-05-06 09:09 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.66 KB, patch)
2020-05-06 09:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.50 KB, patch)
2020-07-23 08:44 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (8.95 KB, patch)
2020-07-23 08:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.19 KB, patch)
2020-07-23 08:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (4.94 KB, patch)
2020-07-23 08:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.70 KB, patch)
2020-07-23 08:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.67 KB, patch)
2020-07-23 08:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.57 KB, patch)
2020-07-23 12:21 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.57 KB, patch)
2020-07-23 12:25 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.02 KB, patch)
2020-07-23 12:25 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.26 KB, patch)
2020-07-23 12:25 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.01 KB, patch)
2020-07-23 12:25 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.77 KB, patch)
2020-07-23 12:25 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.74 KB, patch)
2020-07-23 12:25 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.30 KB, patch)
2020-07-24 08:34 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.71 KB, patch)
2020-07-24 11:58 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (3.60 KB, patch)
2020-08-05 09:55 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (4.68 KB, patch)
2020-08-05 10:10 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.67 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.12 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.42 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.06 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.82 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.79 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.37 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.77 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (4.74 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.29 KB, patch)
2020-08-13 17:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.38 KB, patch)
2020-08-13 17:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.03 KB, patch)
2020-08-13 17:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.58 KB, patch)
2020-08-13 17:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.67 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.12 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.42 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.06 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.82 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.79 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.37 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.77 KB, patch)
2020-08-14 06:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (5.47 KB, patch)
2020-08-14 06:16 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.52 KB, patch)
2020-08-14 06:16 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.61 KB, patch)
2020-08-14 06:16 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.22 KB, patch)
2020-08-14 06:16 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.86 KB, patch)
2020-08-14 06:16 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.67 KB, patch)
2020-09-02 10:33 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.12 KB, patch)
2020-09-02 10:33 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.42 KB, patch)
2020-09-02 10:33 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.06 KB, patch)
2020-09-02 10:33 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.82 KB, patch)
2020-09-02 10:33 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.81 KB, patch)
2020-09-02 10:33 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.55 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.77 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (5.47 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.52 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.61 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.22 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.86 KB, patch)
2020-09-02 10:34 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.67 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.12 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.42 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.06 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.82 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.81 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.55 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.77 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (5.47 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.52 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.61 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.22 KB, patch)
2020-09-02 11:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.86 KB, patch)
2020-09-02 11:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Capitalize return of needsconfirm (7.69 KB, patch)
2020-09-02 11:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Restore previously returned list (2.19 KB, patch)
2020-09-02 11:22 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.67 KB, patch)
2020-09-14 10:22 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.12 KB, patch)
2020-09-14 10:22 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.42 KB, patch)
2020-09-14 10:22 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.06 KB, patch)
2020-09-14 10:22 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.82 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.81 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.55 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.77 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (5.47 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.52 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.61 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.22 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.86 KB, patch)
2020-09-14 10:23 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Capitalize return of needsconfirm (7.69 KB, patch)
2020-09-14 10:24 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Restore previously returned list (2.19 KB, patch)
2020-09-14 10:24 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Terminology: checkin/checkout (3.47 KB, patch)
2020-09-15 21:30 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent double dialogue (1.31 KB, patch)
2020-09-16 12:19 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Make issue and returns messages consistent (3.12 KB, patch)
2020-09-17 11:24 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch attepts to return via SCO (1.46 KB, patch)
2020-09-17 12:00 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent server error on bad barcode (1.11 KB, patch)
2020-09-17 14:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.67 KB, patch)
2020-09-18 10:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.12 KB, patch)
2020-09-18 10:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.42 KB, patch)
2020-09-18 10:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.06 KB, patch)
2020-09-18 10:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.82 KB, patch)
2020-09-18 10:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.81 KB, patch)
2020-09-18 10:02 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.55 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.77 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (5.47 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.52 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.59 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.22 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.86 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Capitalize return of needsconfirm (7.67 KB, patch)
2020-09-18 10:03 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Restore previously returned list (2.19 KB, patch)
2020-09-18 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Terminology: checkin/checkout (3.47 KB, patch)
2020-09-18 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent double dialogue (1.31 KB, patch)
2020-09-18 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Make issue and returns messages consistent (3.12 KB, patch)
2020-09-18 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch attepts to return via SCO (1.44 KB, patch)
2020-09-18 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent server error on bad barcode (1.11 KB, patch)
2020-09-18 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.75 KB, patch)
2020-09-18 10:56 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.20 KB, patch)
2020-09-18 10:56 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.47 KB, patch)
2020-09-18 10:56 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.15 KB, patch)
2020-09-18 10:56 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.88 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.88 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.61 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.84 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (5.52 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.57 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.65 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.28 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.93 KB, patch)
2020-09-18 10:57 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Capitalize return of needsconfirm (7.76 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Restore previously returned list (2.24 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Terminology: checkin/checkout (3.52 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent double dialogue (1.37 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Make issue and returns messages consistent (3.19 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch attepts to return via SCO (1.49 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent server error on bad barcode (1.17 KB, patch)
2020-09-18 10:58 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25261: Add system preference (3.72 KB, patch)
2020-09-29 14:49 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Require confirmation of multiple parts items (9.17 KB, patch)
2020-09-29 14:49 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Checkout confirmation depending on syspref (1.47 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts (5.11 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (follow-up) item.material may contain non-numerics (1.87 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add unit tests (2.87 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Simplify tests and conditions (4.61 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Non-confirm on return (5.83 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add confirmation to issues table (4.87 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkout (1.57 KB, patch)
2020-09-29 14:50 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCO (2.65 KB, patch)
2020-09-29 14:51 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch check in SIP Checkin (2.28 KB, patch)
2020-09-29 14:51 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Add handling to SCI (2.92 KB, patch)
2020-09-29 14:51 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Capitalize return of needsconfirm (7.73 KB, patch)
2020-09-29 14:51 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Restore previously returned list (2.24 KB, patch)
2020-09-29 14:51 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Terminology: checkin/checkout (3.52 KB, patch)
2020-09-29 14:52 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent double dialogue (1.37 KB, patch)
2020-09-29 14:52 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Make issue and returns messages consistent (3.18 KB, patch)
2020-09-29 14:52 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Catch attepts to return via SCO (1.49 KB, patch)
2020-09-29 14:52 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: (QA follow-up) Prevent server error on bad barcode (1.17 KB, patch)
2020-09-29 14:52 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 25261: Prevent tests to fail if CircConfirmItemParts is on (1.71 KB, patch)
2020-10-01 09:40 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Field 2020-04-23 09:38:53 UTC
Koha already has a field in the item record to identify multi-part material. Materials specified (bound volume or other part) or 952$3. When there is a value in this field (e.g. 2) Koha will prompt you at checkout and checkin with a message dialog which says:
 
Note about the accompanying materials: 2

This is helpful but can be missed by library staff. This enhancement takes this one step further to introduce a confirmation box to acknowledge that the parts have been checked on issue and return. This prevents staff from proceeding before checking the item in their hands.  

The reason for this is to prevent arguments with library patrons that they did not originally take all the parts or that they did not return all the parts. For example, a CD with an accompanying booklet or a string quartet with 4 parts.
Comment 1 Martin Renvoize 2020-04-28 12:57:06 UTC
Created attachment 103856 [details] [review]
Bug 25261: Add system preference
Comment 2 Martin Renvoize 2020-04-28 12:57:09 UTC
Created attachment 103857 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff
Comment 3 Frédéric Demians 2020-04-30 09:43:09 UTC
Use of uninitialized value $no_of_parts in numeric gt (>) at /C4/Circulation.pm line 848.

You should replace:
  my $no_of_parts = $item_object->materials;
with:
  my $no_of_parts = $item_object->materials || 0;
Comment 4 Frédéric Demians 2020-04-30 09:55:15 UTC
The confirmation need is controlled by a new syspref. Why not a per user permission like?

  CAN_user_circulate_force_checkout_materials
Comment 5 Frédéric Demians 2020-04-30 10:02:40 UTC
You enhancement works great for check in: playing with CircConfirmParts, the confirmation if the return is required or not. But for checkout, independently of CircConfirmParts, the confirmation is required. I will sign your patch and add a follow-up to fix that.
Comment 6 Frédéric Demians 2020-04-30 10:07:53 UTC
Created attachment 104011 [details] [review]
Bug 25261: Add system preference

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 7 Frédéric Demians 2020-04-30 10:08:00 UTC
Created attachment 104012 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 8 Frédéric Demians 2020-04-30 10:08:04 UTC
Created attachment 104013 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 9 Jonathan Druart 2020-04-30 10:44:23 UTC
Is "Parts" meaningful enough?
As a non-native English speaker it seems vague to me, "parts" of what?
Maybe adding "Item" in the syspref's name?

Please ignore if it does not make sense :)
Comment 10 Jonathan Druart 2020-04-30 10:46:44 UTC
Also, from QA script:

 OK     C4/Circulation.pm
 FAIL   circ/returns.pl
   FAIL   spelling
                 recieved  ==> received

 OK     installer/data/mysql/sysprefs.sql
 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
   FAIL   filters
                missing_filter at line 203 (                                        Please confirm that the accompanying materials are present: [% additional_materials %])


And I spotted a 8 indentation level in returns.pl
304                 $needs_confirm = 1;
Comment 11 Martin Renvoize 2020-05-06 08:28:08 UTC
Created attachment 104404 [details] [review]
Bug 25261: Add system preference

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 12 Martin Renvoize 2020-05-06 08:28:11 UTC
Created attachment 104405 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 13 Martin Renvoize 2020-05-06 08:28:15 UTC
Created attachment 104406 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 14 Martin Renvoize 2020-05-06 08:28:18 UTC
Created attachment 104407 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?
Comment 15 Martin Renvoize 2020-05-06 08:28:21 UTC
Created attachment 104408 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.
Comment 16 Martin Renvoize 2020-05-06 08:29:37 UTC
QA failures corrected and system preference updated to include 'Item' to clarify it's use.

I've also added a followup to allow for arbitrary notes in the materials field.. they are not always numeric it appears.
Comment 17 Martin Renvoize 2020-05-06 09:09:46 UTC
Created attachment 104409 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 18 Martin Renvoize 2020-05-06 09:09:50 UTC
Created attachment 104410 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 19 Martin Renvoize 2020-05-06 09:09:53 UTC
Created attachment 104411 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 20 Martin Renvoize 2020-05-06 09:09:56 UTC
Created attachment 104412 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?
Comment 21 Martin Renvoize 2020-05-06 09:09:59 UTC
Created attachment 104413 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.
Comment 22 Martin Renvoize 2020-05-06 09:34:08 UTC
Created attachment 104414 [details] [review]
Bug 25261: (QA follow-up) Add unit tests
Comment 23 Martin Renvoize 2020-07-23 08:44:59 UTC
Created attachment 107217 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 24 Martin Renvoize 2020-07-23 08:45:03 UTC
Created attachment 107218 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 25 Martin Renvoize 2020-07-23 08:45:06 UTC
Created attachment 107219 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 26 Martin Renvoize 2020-07-23 08:45:09 UTC
Created attachment 107220 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?
Comment 27 Martin Renvoize 2020-07-23 08:45:13 UTC
Created attachment 107221 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.
Comment 28 Martin Renvoize 2020-07-23 08:45:16 UTC
Created attachment 107222 [details] [review]
Bug 25261: (QA follow-up) Add unit tests
Comment 29 Kyle M Hall 2020-07-23 12:21:20 UTC
Created attachment 107228 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 30 Kyle M Hall 2020-07-23 12:25:00 UTC
Created attachment 107229 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 31 Kyle M Hall 2020-07-23 12:25:10 UTC
Created attachment 107230 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 32 Kyle M Hall 2020-07-23 12:25:13 UTC
Created attachment 107231 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 33 Kyle M Hall 2020-07-23 12:25:16 UTC
Created attachment 107232 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 34 Kyle M Hall 2020-07-23 12:25:19 UTC
Created attachment 107233 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 35 Kyle M Hall 2020-07-23 12:25:22 UTC
Created attachment 107234 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 36 Jonathan Druart 2020-07-24 08:28:43 UTC
Found 2 problems:
1. When checking in, "Please confirm checkin" click "No" will check the item in
2. If you checkin from the checkouts page there is no alert
Comment 37 Jonathan Druart 2020-07-24 08:34:36 UTC
Created attachment 107285 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation
Comment 38 Martin Renvoize 2020-07-24 11:58:44 UTC
Created attachment 107331 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in
Comment 39 Martin Renvoize 2020-07-24 12:06:42 UTC
(In reply to Jonathan Druart from comment #36)
> Found 2 problems:
> 1. When checking in, "Please confirm checkin" click "No" will check the item in

Fixed in followup, good catch, not sure how my brain missed that :(

> 2. If you checkin from the checkouts page there is no alert

Can you clarify how to achieve this.. I can't replicate it at present.
Comment 40 Jonathan Druart 2020-07-24 12:25:06 UTC
(In reply to Martin Renvoize from comment #39)
> (In reply to Jonathan Druart from comment #36)
> > 2. If you checkin from the checkouts page there is no alert
> 
> Can you clarify how to achieve this.. I can't replicate it at present.

from the checkouts view: circ/circulation.pl?borrowernumber=42
Comment 41 Martin Renvoize 2020-08-04 15:24:17 UTC
(In reply to Jonathan Druart from comment #40)
> (In reply to Martin Renvoize from comment #39)
> > (In reply to Jonathan Druart from comment #36)
> > > 2. If you checkin from the checkouts page there is no alert
> > 
> > Can you clarify how to achieve this.. I can't replicate it at present.
> 
> from the checkouts view: circ/circulation.pl?borrowernumber=42

Nope, still not sure what you mean.

I checkout an item to person X.. and then try to checkout the same item to Person Y... the alert is triggered.. is that not what you mean?

How else can you perform a check-in on the checkout page?
Comment 42 Jonathan Druart 2020-08-04 15:36:21 UTC
https://snipboard.io/RsyS8W.jpg
last column
Comment 43 Martin Renvoize 2020-08-04 20:42:07 UTC
Found it.. the action was hidden by the checkout table column settings.. grr.

Looking at how the code in the works, this is going to be rather challenging to add 😑
Comment 44 Martin Renvoize 2020-08-05 09:55:56 UTC
Created attachment 107822 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.
Comment 45 Martin Renvoize 2020-08-05 10:10:01 UTC
Created attachment 107823 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.
Comment 46 Martin Renvoize 2020-08-05 10:11:27 UTC
Let me know if you're happy or not with that followup Jonathan.. was an interesting area to dig into :)
Comment 47 Jonathan Druart 2020-08-07 08:37:45 UTC
What about self checkout? Has it been tested?
Comment 48 Jonathan Druart 2020-08-07 08:38:46 UTC
(In reply to Jonathan Druart from comment #47)
> What about self checkout? Has it been tested?

Reading the code I think it's missing from opac/sco/sco-main.pl line 154.
Comment 49 Katrin Fischer 2020-08-07 09:30:17 UTC
I wonder about making it visible in sco/sci, as we had a lengthy discussion about libraries using this field for information that should not be seen by users: bug 13547

I would suggest to deal with self-checkout/checkin separately, maybe with a pref.
Comment 50 Martin Renvoize 2020-08-07 09:34:35 UTC
There is the option to just block in SCO by the looks.. 

'Please see a member of the library staff.'

I wonder if that's the approach to take.?
Comment 51 Katrin Fischer 2020-08-07 09:39:17 UTC
(In reply to Martin Renvoize from comment #50)
> There is the option to just block in SCO by the looks.. 
> 
> 'Please see a member of the library staff.'
> 
> I wonder if that's the approach to take.?

I think that's a common approach for multi-part items. 

Only problem might be that it causes a change in behaviour we should probably document clearly in the pref description (items with materials no will no longer be able to be returned on self check). Hate to ask, but if we go there... what about SIP?
Comment 52 Martin Renvoize 2020-08-13 17:03:25 UTC
Created attachment 108196 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 53 Martin Renvoize 2020-08-13 17:03:28 UTC
Created attachment 108197 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 54 Martin Renvoize 2020-08-13 17:03:33 UTC
Created attachment 108198 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 55 Martin Renvoize 2020-08-13 17:03:36 UTC
Created attachment 108199 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 56 Martin Renvoize 2020-08-13 17:03:40 UTC
Created attachment 108200 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 57 Martin Renvoize 2020-08-13 17:03:44 UTC
Created attachment 108201 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 58 Martin Renvoize 2020-08-13 17:03:47 UTC
Created attachment 108202 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 59 Martin Renvoize 2020-08-13 17:03:51 UTC
Created attachment 108203 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 60 Martin Renvoize 2020-08-13 17:03:55 UTC
Created attachment 108204 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 61 Martin Renvoize 2020-08-13 17:03:59 UTC
Created attachment 108205 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 62 Martin Renvoize 2020-08-13 17:04:03 UTC
Created attachment 108206 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 63 Martin Renvoize 2020-08-13 17:04:06 UTC
Created attachment 108207 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 64 Martin Renvoize 2020-08-13 17:04:10 UTC
Created attachment 108208 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 65 Martin Renvoize 2020-08-13 17:05:10 UTC
Bumping back to the NQA queue as we had a number of QA followups and I feel like a final QA run would be a good idea.
Comment 66 Katrin Fischer 2020-08-13 18:31:39 UTC
What is the expected behavior on SIP, SCO and SCI? I think updating the commit messages for the last 3 patches could be really helpful.
Comment 67 Martin Renvoize 2020-08-14 06:15:17 UTC
Created attachment 108230 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 68 Martin Renvoize 2020-08-14 06:15:24 UTC
Created attachment 108231 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 69 Martin Renvoize 2020-08-14 06:15:30 UTC
Created attachment 108232 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 70 Martin Renvoize 2020-08-14 06:15:36 UTC
Created attachment 108233 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 71 Martin Renvoize 2020-08-14 06:15:41 UTC
Created attachment 108234 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 72 Martin Renvoize 2020-08-14 06:15:47 UTC
Created attachment 108235 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 73 Martin Renvoize 2020-08-14 06:15:52 UTC
Created attachment 108236 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 74 Martin Renvoize 2020-08-14 06:15:56 UTC
Created attachment 108237 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 75 Martin Renvoize 2020-08-14 06:16:02 UTC
Created attachment 108238 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 76 Martin Renvoize 2020-08-14 06:16:07 UTC
Created attachment 108239 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 77 Martin Renvoize 2020-08-14 06:16:13 UTC
Created attachment 108240 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 78 Martin Renvoize 2020-08-14 06:16:18 UTC
Created attachment 108241 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 79 Martin Renvoize 2020-08-14 06:16:23 UTC
Created attachment 108242 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 80 Martin Renvoize 2020-08-14 06:18:50 UTC
(In reply to Katrin Fischer from comment #66)
> What is the expected behavior on SIP, SCO and SCI? I think updating the
> commit messages for the last 3 patches could be really helpful.

As requested, I've added a test plan per follow-up patch to clarify behaviour.
Comment 81 Katrin Fischer 2020-08-22 23:43:26 UTC
Could someone help me out here by at least testing the SIP bit? I can take care of the other parts.
Comment 82 Nick Clemens 2020-08-28 13:53:50 UTC
SIP checkin fails - the message for needsconfirmation of 'additional_materials' is lowercased, but the test is UPPER CASED.

Additionally the sub _can_we_issue deletes any lowercased keys - there is half an explanation in a comment

One more thing:
If i check in an item that needs confirmation the checkin list is cleared after confirmation

To recreate:
0 - Enable ShowAllCheckins
1 - Check in several regular items
2 - Check in an item with parts
3 - Confirm the checkin in
4 - The list of checkins contains only last item
Comment 83 Martin Renvoize 2020-09-02 10:33:36 UTC
Created attachment 109478 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 84 Martin Renvoize 2020-09-02 10:33:40 UTC
Created attachment 109479 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 85 Martin Renvoize 2020-09-02 10:33:44 UTC
Created attachment 109480 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 86 Martin Renvoize 2020-09-02 10:33:48 UTC
Created attachment 109481 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 87 Martin Renvoize 2020-09-02 10:33:52 UTC
Created attachment 109482 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 88 Martin Renvoize 2020-09-02 10:33:56 UTC
Created attachment 109483 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 89 Martin Renvoize 2020-09-02 10:34:00 UTC
Created attachment 109484 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 90 Martin Renvoize 2020-09-02 10:34:04 UTC
Created attachment 109485 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 91 Martin Renvoize 2020-09-02 10:34:08 UTC
Created attachment 109486 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 92 Martin Renvoize 2020-09-02 10:34:12 UTC
Created attachment 109487 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 93 Martin Renvoize 2020-09-02 10:34:16 UTC
Created attachment 109488 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 94 Martin Renvoize 2020-09-02 10:34:20 UTC
Created attachment 109489 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 95 Martin Renvoize 2020-09-02 10:34:24 UTC
Created attachment 109490 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 96 Martin Renvoize 2020-09-02 11:02:10 UTC
Created attachment 109491 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 97 Martin Renvoize 2020-09-02 11:02:15 UTC
Created attachment 109492 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 98 Martin Renvoize 2020-09-02 11:02:19 UTC
Created attachment 109493 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 99 Martin Renvoize 2020-09-02 11:02:23 UTC
Created attachment 109494 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 100 Martin Renvoize 2020-09-02 11:02:27 UTC
Created attachment 109495 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 101 Martin Renvoize 2020-09-02 11:02:32 UTC
Created attachment 109496 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 102 Martin Renvoize 2020-09-02 11:02:36 UTC
Created attachment 109497 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 103 Martin Renvoize 2020-09-02 11:02:40 UTC
Created attachment 109498 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 104 Martin Renvoize 2020-09-02 11:02:44 UTC
Created attachment 109499 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 105 Martin Renvoize 2020-09-02 11:02:48 UTC
Created attachment 109500 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 106 Martin Renvoize 2020-09-02 11:02:53 UTC
Created attachment 109501 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 107 Martin Renvoize 2020-09-02 11:02:59 UTC
Created attachment 109502 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 108 Martin Renvoize 2020-09-02 11:03:03 UTC
Created attachment 109503 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 109 Martin Renvoize 2020-09-02 11:03:07 UTC
Created attachment 109504 [details] [review]
Bug 25261: (QA follow-up) Capitalize return of needsconfirm

This patch capitalizes the 'ADDITIONAL_MATERIALS' key in the
needsconfirm hash return value for CanBookBeIssued to restore
consistency with previous code and ensure SIP Checkout works as
expected.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 110 Martin Renvoize 2020-09-02 11:04:08 UTC
Final patch should restore consistency in the return value for CanBookBeIssued such that SIP Checkout will perform as expected.
Comment 111 Martin Renvoize 2020-09-02 11:22:01 UTC
Created attachment 109508 [details] [review]
Bug 25261: (QA follow-up) Restore previously returned list

Test plan
0 - Enable ShowAllCheckins
1 - Check in several regular items
2 - Check in an item with parts
3 - Confirm the checkin in
4 - The list of checkins contains all the previous checkins and the
  multi-part checkin too.
Comment 112 Martin Renvoize 2020-09-02 11:22:42 UTC
All issues highlighted in QA should now be resolved :), Thanks for testing Nick :)
Comment 113 Nick Clemens 2020-09-02 15:24:04 UTC
(In reply to Katrin Fischer from comment #81)
> Could someone help me out here by at least testing the SIP bit? I can take
> care of the other parts.

SIP bits are all good with latest patches - do you plan to test the rest?
Comment 114 Katrin Fischer 2020-09-13 08:50:31 UTC
Can you please rebase?

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 25261: Add system preference
Using index info to reconstruct a base tree...
M	installer/data/mysql/sysprefs.sql
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
Auto-merging installer/data/mysql/sysprefs.sql
Applying: Bug 25261: Require confirmation of multiple parts items
Applying: Bug 25261: Checkout confirmation depending on syspref
Applying: Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts
error: sha1 information is lacking or useless (circ/returns.pl).
error: could not build fake ancestor
Patch failed at 0001 Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem run "git bz apply --continue".
If you would prefer to skip this patch, instead run "git bz apply --skip".
To restore the original branch and stop patching run "git bz apply --abort".
Patch left in /tmp/Bug-25261-follow-up-Rename-CircConfirmParts-to-Cir-9FLtrb.patch
Comment 115 Martin Renvoize 2020-09-14 10:22:37 UTC
Created attachment 110038 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 116 Martin Renvoize 2020-09-14 10:22:42 UTC
Created attachment 110039 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 117 Martin Renvoize 2020-09-14 10:22:48 UTC
Created attachment 110040 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 118 Martin Renvoize 2020-09-14 10:22:55 UTC
Created attachment 110041 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 119 Martin Renvoize 2020-09-14 10:23:00 UTC
Created attachment 110042 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 120 Martin Renvoize 2020-09-14 10:23:09 UTC
Created attachment 110043 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 121 Martin Renvoize 2020-09-14 10:23:16 UTC
Created attachment 110044 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 122 Martin Renvoize 2020-09-14 10:23:23 UTC
Created attachment 110045 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 123 Martin Renvoize 2020-09-14 10:23:29 UTC
Created attachment 110046 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 124 Martin Renvoize 2020-09-14 10:23:35 UTC
Created attachment 110047 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 125 Martin Renvoize 2020-09-14 10:23:41 UTC
Created attachment 110048 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 126 Martin Renvoize 2020-09-14 10:23:48 UTC
Created attachment 110049 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 127 Martin Renvoize 2020-09-14 10:23:57 UTC
Created attachment 110050 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 128 Martin Renvoize 2020-09-14 10:24:03 UTC
Created attachment 110051 [details] [review]
Bug 25261: (QA follow-up) Capitalize return of needsconfirm

This patch capitalizes the 'ADDITIONAL_MATERIALS' key in the
needsconfirm hash return value for CanBookBeIssued to restore
consistency with previous code and ensure SIP Checkout works as
expected.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 129 Martin Renvoize 2020-09-14 10:24:15 UTC
Created attachment 110052 [details] [review]
Bug 25261: (QA follow-up) Restore previously returned list

Test plan
0 - Enable ShowAllCheckins
1 - Check in several regular items
2 - Check in an item with parts
3 - Confirm the checkin in
4 - The list of checkins contains all the previous checkins and the
  multi-part checkin too.
Comment 130 Katrin Fischer 2020-09-15 21:30:54 UTC
Created attachment 110133 [details] [review]
Bug 25261: (QA follow-up) Terminology: checkin/checkout
Comment 131 Katrin Fischer 2020-09-15 21:32:25 UTC
When checking out an item with $3 set and the syspref active, I see both the old blue 'information dialog' and the new yellow 'confirmation dialog'.
The information shown is doubled up this way - I believe if we ask for confirmation the blue one should not appear?
Comment 132 Martin Renvoize 2020-09-16 12:19:29 UTC
Created attachment 110184 [details] [review]
Bug 25261: (QA follow-up) Prevent double dialogue

This patch removes the blue information message box if we are using
multi-part confirmations.
Comment 133 Martin Renvoize 2020-09-16 12:20:06 UTC
Corrected in followup.
Comment 134 Katrin Fischer 2020-09-17 06:26:42 UTC
1) Check in via check in page

With confirmation off:
When returning an item with $3 the dialog is blue. With the patch, the dialog is yellow and has a list item. I believe that the previous formatting should be kept in this case, so that checkout and checkin match up.

With confirmation on:
There is a similar problem as before with the modal AND the normal message showing in the background. I think we should only show the modal in this case.

Visually I would maybe make the $3 content in the message stand out a bit more, making it <strong> maybe?

2) Self check out (sco-main)

On self check out with returns allowed (SCOAllowCheckin):
When you click on "renew item", it asks if you want to renew or return it. You can then successfully return the item from this dialog. 

Note: There are some issues with the CSS there - the Renew item link is not obvious as a button - and I am not sure why the normal return option doesn't display :(
Comment 135 Katrin Fischer 2020-09-17 06:32:26 UTC
> 2) Self check out (sco-main)

I also had trouble with the 'renew' option from here - could you try that please?
Comment 136 Katrin Fischer 2020-09-17 06:35:48 UTC
sci-main is ok.
Comment 137 Martin Renvoize 2020-09-17 11:24:28 UTC
Created attachment 110258 [details] [review]
Bug 25261: (QA follow-up) Make issue and returns messages consistent

This patch makes the confirmation dialogue handing and visuality
consistent between issues and returns.
Comment 138 Martin Renvoize 2020-09-17 12:00:51 UTC
Created attachment 110259 [details] [review]
Bug 25261: (QA follow-up) Catch attepts to return via SCO

This patch adds a block to capture attempts to return items with
attached materials when checking in via the self checkout when the
relevant system preference is set.
Comment 139 Martin Renvoize 2020-09-17 12:22:31 UTC
(In reply to Katrin Fischer from comment #135)
> > 2) Self check out (sco-main)
> 
> I also had trouble with the 'renew' option from here - could you try that
> please?

This is a bit weird; but I don't think it's anything to do with this bug.  The renewal goes through as expected if you look at the data, but I would expect the page to redirect back to the logged in SCO session rather than just leave you on the confirmation page. The behaviour pre-exists this bug though, so I'd prefer to fix it in it's own bug.
Comment 140 Martin Renvoize 2020-09-17 12:23:36 UTC
(In reply to Katrin Fischer from comment #134)
> Note: There are some issues with the CSS there - the Renew item link is not
> obvious as a button - and I am not sure why the normal return option doesn't
> display :(

Not related to this bug; certainly one to raise on it's own bug.
Comment 141 Katrin Fischer 2020-09-17 13:55:14 UTC
(In reply to Martin Renvoize from comment #140)
> (In reply to Katrin Fischer from comment #134)
> > Note: There are some issues with the CSS there - the Renew item link is not
> > obvious as a button - and I am not sure why the normal return option doesn't
> > display :(
> 
> Not related to this bug; certainly one to raise on it's own bug.

Forgot to update here, already filed 2 new bugs for this:

Bug 26479
Bug 26478
Comment 142 Katrin Fischer 2020-09-17 13:56:15 UTC
> > I also had trouble with the 'renew' option from here - could you try that
> > please?
> 
> This is a bit weird; but I don't think it's anything to do with this bug. 
> The renewal goes through as expected if you look at the data, but I would
> expect the page to redirect back to the logged in SCO session rather than
> just leave you on the confirmation page. The behaviour pre-exists this bug
> though, so I'd prefer to fix it in it's own bug.

It worked for me without the patches - the page did then reload for me. Same item as with the confirmation. I can retest, but it might be related.
Comment 143 Martin Renvoize 2020-09-17 14:02:57 UTC
Created attachment 110268 [details] [review]
Bug 25261: (QA follow-up) Prevent server error on bad barcode

Prior to this patch if you had CircConfirmParts enabled and you
attempted to checkin a deleted item then you would be met with a server
error.
Comment 144 Katrin Fischer 2020-09-17 20:08:29 UTC
I retested this:

- Turn on multi part checking and edit 3999900000018 to have $3
- Check out 3999900000018 in staff
- Go to sco/sco-main.pl
- The checkout shows in the table of checkouts
- The last column is "Renew item"
- Click on it, click "Renew" - page refreshes, all is good
- Apply patches, repeat
- Renew doesn't direct back to the overview page anymore
Comment 145 Martin Renvoize 2020-09-18 10:00:23 UTC
Right.. you were getting caught by bug 26301.. which got pushed to master last night!
Comment 146 Martin Renvoize 2020-09-18 10:02:27 UTC
Created attachment 110322 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 147 Martin Renvoize 2020-09-18 10:02:34 UTC
Created attachment 110323 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 148 Martin Renvoize 2020-09-18 10:02:40 UTC
Created attachment 110324 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 149 Martin Renvoize 2020-09-18 10:02:46 UTC
Created attachment 110325 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 150 Martin Renvoize 2020-09-18 10:02:51 UTC
Created attachment 110326 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 151 Martin Renvoize 2020-09-18 10:02:57 UTC
Created attachment 110327 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 152 Martin Renvoize 2020-09-18 10:03:02 UTC
Created attachment 110328 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 153 Martin Renvoize 2020-09-18 10:03:08 UTC
Created attachment 110329 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 154 Martin Renvoize 2020-09-18 10:03:16 UTC
Created attachment 110330 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 155 Martin Renvoize 2020-09-18 10:03:24 UTC
Created attachment 110331 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 156 Martin Renvoize 2020-09-18 10:03:32 UTC
Created attachment 110332 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 157 Martin Renvoize 2020-09-18 10:03:47 UTC
Created attachment 110333 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 158 Martin Renvoize 2020-09-18 10:03:52 UTC
Created attachment 110334 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 159 Martin Renvoize 2020-09-18 10:03:59 UTC
Created attachment 110335 [details] [review]
Bug 25261: (QA follow-up) Capitalize return of needsconfirm

This patch capitalizes the 'ADDITIONAL_MATERIALS' key in the
needsconfirm hash return value for CanBookBeIssued to restore
consistency with previous code and ensure SIP Checkout works as
expected.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 160 Martin Renvoize 2020-09-18 10:04:07 UTC
Created attachment 110336 [details] [review]
Bug 25261: (QA follow-up) Restore previously returned list

Test plan
0 - Enable ShowAllCheckins
1 - Check in several regular items
2 - Check in an item with parts
3 - Confirm the checkin in
4 - The list of checkins contains all the previous checkins and the
  multi-part checkin too.
Comment 161 Martin Renvoize 2020-09-18 10:04:17 UTC
Created attachment 110337 [details] [review]
Bug 25261: (QA follow-up) Terminology: checkin/checkout
Comment 162 Martin Renvoize 2020-09-18 10:04:27 UTC
Created attachment 110338 [details] [review]
Bug 25261: (QA follow-up) Prevent double dialogue

This patch removes the blue information message box if we are using
multi-part confirmations.
Comment 163 Martin Renvoize 2020-09-18 10:04:36 UTC
Created attachment 110339 [details] [review]
Bug 25261: (QA follow-up) Make issue and returns messages consistent

This patch makes the confirmation dialogue handing and visuality
consistent between issues and returns.
Comment 164 Martin Renvoize 2020-09-18 10:04:45 UTC
Created attachment 110340 [details] [review]
Bug 25261: (QA follow-up) Catch attepts to return via SCO

This patch adds a block to capture attempts to return items with
attached materials when checking in via the self checkout when the
relevant system preference is set.
Comment 165 Martin Renvoize 2020-09-18 10:04:51 UTC
Created attachment 110341 [details] [review]
Bug 25261: (QA follow-up) Prevent server error on bad barcode

Prior to this patch if you had CircConfirmParts enabled and you
attempted to checkin a deleted item then you would be met with a server
error.
Comment 166 Martin Renvoize 2020-09-18 10:15:55 UTC
(In reply to Katrin Fischer from comment #144)
> I retested this:
> 
> - Turn on multi part checking and edit 3999900000018 to have $3
> - Check out 3999900000018 in staff
> - Go to sco/sco-main.pl
> - The checkout shows in the table of checkouts
> - The last column is "Renew item"
> - Click on it, click "Renew" - page refreshes, all is good
> - Apply patches, repeat
> - Renew doesn't direct back to the overview page anymore

Rebased on master should have fixed this.. you were seeing a manifestation of bug 26301 which was pushed to master over night.
Comment 167 Katrin Fischer 2020-09-18 10:56:35 UTC
Created attachment 110356 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 168 Katrin Fischer 2020-09-18 10:56:43 UTC
Created attachment 110357 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 169 Katrin Fischer 2020-09-18 10:56:49 UTC
Created attachment 110358 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 170 Katrin Fischer 2020-09-18 10:56:55 UTC
Created attachment 110359 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 171 Katrin Fischer 2020-09-18 10:57:01 UTC
Created attachment 110360 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 172 Katrin Fischer 2020-09-18 10:57:08 UTC
Created attachment 110361 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 173 Katrin Fischer 2020-09-18 10:57:14 UTC
Created attachment 110362 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 174 Katrin Fischer 2020-09-18 10:57:21 UTC
Created attachment 110363 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 175 Katrin Fischer 2020-09-18 10:57:27 UTC
Created attachment 110364 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 176 Katrin Fischer 2020-09-18 10:57:35 UTC
Created attachment 110365 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 177 Katrin Fischer 2020-09-18 10:57:43 UTC
Created attachment 110366 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 178 Katrin Fischer 2020-09-18 10:57:50 UTC
Created attachment 110367 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 179 Katrin Fischer 2020-09-18 10:57:56 UTC
Created attachment 110368 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 180 Katrin Fischer 2020-09-18 10:58:03 UTC
Created attachment 110369 [details] [review]
Bug 25261: (QA follow-up) Capitalize return of needsconfirm

This patch capitalizes the 'ADDITIONAL_MATERIALS' key in the
needsconfirm hash return value for CanBookBeIssued to restore
consistency with previous code and ensure SIP Checkout works as
expected.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 181 Katrin Fischer 2020-09-18 10:58:11 UTC
Created attachment 110370 [details] [review]
Bug 25261: (QA follow-up) Restore previously returned list

Test plan
0 - Enable ShowAllCheckins
1 - Check in several regular items
2 - Check in an item with parts
3 - Confirm the checkin in
4 - The list of checkins contains all the previous checkins and the
  multi-part checkin too.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 182 Katrin Fischer 2020-09-18 10:58:17 UTC
Created attachment 110371 [details] [review]
Bug 25261: (QA follow-up) Terminology: checkin/checkout

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 183 Katrin Fischer 2020-09-18 10:58:25 UTC
Created attachment 110372 [details] [review]
Bug 25261: (QA follow-up) Prevent double dialogue

This patch removes the blue information message box if we are using
multi-part confirmations.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 184 Katrin Fischer 2020-09-18 10:58:32 UTC
Created attachment 110373 [details] [review]
Bug 25261: (QA follow-up) Make issue and returns messages consistent

This patch makes the confirmation dialogue handing and visuality
consistent between issues and returns.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 185 Katrin Fischer 2020-09-18 10:58:46 UTC
Created attachment 110374 [details] [review]
Bug 25261: (QA follow-up) Catch attepts to return via SCO

This patch adds a block to capture attempts to return items with
attached materials when checking in via the self checkout when the
relevant system preference is set.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 186 Katrin Fischer 2020-09-18 10:58:54 UTC
Created attachment 110375 [details] [review]
Bug 25261: (QA follow-up) Prevent server error on bad barcode

Prior to this patch if you had CircConfirmParts enabled and you
attempted to checkin a deleted item then you would be met with a server
error.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 187 Martin Renvoize 2020-09-29 14:49:45 UTC
Created attachment 110915 [details] [review]
Bug 25261: Add system preference

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 188 Martin Renvoize 2020-09-29 14:49:53 UTC
Created attachment 110916 [details] [review]
Bug 25261: Require confirmation of multiple parts items

Test plan
1/ Catalogue an item to contain multiple parts by populating 'Materials
specified (bound volume or other part)'
2/ Enable the new system preference 'CircConfirmParts'
3/ Attempt to checkout the item created in step 1 to a user and note
that confirmation is now required.
4/ Checkout the item
5/ Attempt to checkin the item you have just checked out and note that
confirmation is required.
6/ Signoff

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 189 Martin Renvoize 2020-09-29 14:50:00 UTC
Created attachment 110917 [details] [review]
Bug 25261: Checkout confirmation depending on syspref

With this follow-up patch, the intended confirmation of checkin based on
CircConfirmParts value is effective.

Sponsored-by: Royal College of Music [https://www.rcm.ac.uk/]
Sponsored-by: PTFS Europe [https://ptfs-europe.com/]
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 190 Martin Renvoize 2020-09-29 14:50:09 UTC
Created attachment 110918 [details] [review]
Bug 25261: (follow-up) Rename CircConfirmParts to CircConfirmItemParts

This clarifies the preference name to make it clear we are talking about
the 'parts' that make up an 'item'. 'Part' is a well known term in
british english libraries and I think perhaps 'Materials' may be
confused with other terms?

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 191 Martin Renvoize 2020-09-29 14:50:19 UTC
Created attachment 110919 [details] [review]
Bug 25261: (follow-up) item.material may contain non-numerics

Whilst QAing bug 13547 it was highlighted to me the at the 952$3 field,
and thus the item.materials field, may contain arbitrary notes about the
material rather than just numeric values. As such we need to check for
the field being defined as aposed to greater than '0'.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 192 Martin Renvoize 2020-09-29 14:50:29 UTC
Created attachment 110920 [details] [review]
Bug 25261: (QA follow-up) Add unit tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 193 Martin Renvoize 2020-09-29 14:50:40 UTC
Created attachment 110921 [details] [review]
Bug 25261: (QA follow-up) Simplify tests and conditions

Use build_sample_item in tests
Simplify tests for the confirmation

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 194 Martin Renvoize 2020-09-29 14:50:46 UTC
Created attachment 110922 [details] [review]
Bug 25261: (QA follow-up) Non-confirm on return

This patch updates the confirmation from an alert to a dismissable modal
which allows for optionally not checking the item in

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 195 Martin Renvoize 2020-09-29 14:50:53 UTC
Created attachment 110923 [details] [review]
Bug 25261: (QA follow-up) Add confirmation to issues table

The issues table allows for checkins and was not appropriately
requireing confirmation for the multi-part confirmation added in this
bug.

Test plan
1/ Enable the feature as per previous patches
2/ Checkot an item with attached materials
3/ Navigate to a page that display your users issues table (the checkout
page is a reasonable example)
4/ You should have the option to select items for return in the table
(If not, use the column settings to enable the feature)
5/ Select at least the item with attached materials to return
6/ Upon clicking the return buttton you should find that items without
additional materials are returned as expected, but rows with additional
materials turn yellow and contain a message and additional checkbox for
confirmation in the table.
7/ Ensure the checkbox is selected and click the return button again
8/ This item should have been returned.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 196 Martin Renvoize 2020-09-29 14:50:59 UTC
Created attachment 110924 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkout

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via a SIP unit
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 197 Martin Renvoize 2020-09-29 14:51:07 UTC
Created attachment 110925 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCO

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to checkout an item with materials attached via the SCO
3/ Note that the checkut is blocked with the message 'Item must be
issued at a circulation desk'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 198 Martin Renvoize 2020-09-29 14:51:16 UTC
Created attachment 110926 [details] [review]
Bug 25261: (QA follow-up) Catch check in SIP Checkin

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via a SIP unit
3/ Note that the return is blocked with an alert type of '99'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 199 Martin Renvoize 2020-09-29 14:51:28 UTC
Created attachment 110927 [details] [review]
Bug 25261: (QA follow-up) Add handling to SCI

Test plan
1/ Enable the feature as per previous patches
2/ Attempt to return an item with materials attached via the SCI
3/ Note that the return is blocked with the message '(The item
cannot be returned at a self service terminal, please see library
staff for assistance)'
4/ Signoff

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 200 Martin Renvoize 2020-09-29 14:51:40 UTC
Created attachment 110928 [details] [review]
Bug 25261: (QA follow-up) Capitalize return of needsconfirm

This patch capitalizes the 'ADDITIONAL_MATERIALS' key in the
needsconfirm hash return value for CanBookBeIssued to restore
consistency with previous code and ensure SIP Checkout works as
expected.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 201 Martin Renvoize 2020-09-29 14:51:53 UTC
Created attachment 110929 [details] [review]
Bug 25261: (QA follow-up) Restore previously returned list

Test plan
0 - Enable ShowAllCheckins
1 - Check in several regular items
2 - Check in an item with parts
3 - Confirm the checkin in
4 - The list of checkins contains all the previous checkins and the
  multi-part checkin too.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 202 Martin Renvoize 2020-09-29 14:52:00 UTC
Created attachment 110930 [details] [review]
Bug 25261: (QA follow-up) Terminology: checkin/checkout

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 203 Martin Renvoize 2020-09-29 14:52:09 UTC
Created attachment 110931 [details] [review]
Bug 25261: (QA follow-up) Prevent double dialogue

This patch removes the blue information message box if we are using
multi-part confirmations.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 204 Martin Renvoize 2020-09-29 14:52:22 UTC
Created attachment 110932 [details] [review]
Bug 25261: (QA follow-up) Make issue and returns messages consistent

This patch makes the confirmation dialogue handing and visuality
consistent between issues and returns.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 205 Martin Renvoize 2020-09-29 14:52:30 UTC
Created attachment 110933 [details] [review]
Bug 25261: (QA follow-up) Catch attepts to return via SCO

This patch adds a block to capture attempts to return items with
attached materials when checking in via the self checkout when the
relevant system preference is set.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 206 Martin Renvoize 2020-09-29 14:52:39 UTC
Created attachment 110934 [details] [review]
Bug 25261: (QA follow-up) Prevent server error on bad barcode

Prior to this patch if you had CircConfirmParts enabled and you
attempted to checkin a deleted item then you would be met with a server
error.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 207 Martin Renvoize 2020-09-29 14:53:16 UTC
Minor rebase for conflict caused by bug 25321.
Comment 208 Jonathan Druart 2020-10-01 09:40:18 UTC
Created attachment 111027 [details] [review]
Bug 25261: Prevent tests to fail if CircConfirmItemParts is on

Setting items.materials to NULL at TestBuilder level will (certainly) prevent some
tests to fail if the pref is on

In t/db_dependent/selenium/basic_workflow.t we set 952$3 to an empty
string. The tests will pass even if the pref is turned on.
Comment 209 Jonathan Druart 2020-10-01 09:44:40 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 210 Lucas Gass 2020-10-20 18:10:47 UTC
enhancement will not be backported to 20.05.x
Comment 211 Jonathan Druart 2024-02-23 10:35:51 UTC
Bug 25261: (QA follow-up) Add confirmation to issues table

I don't understand this patch.
1. The JS code is added in the "renew" function but we are talking about checkin
2. Why don't we simply don't remove the checkbox to prevent checkin (like we do for item on hold)?
3. The "Confirm" will no longer be displayed because of the table refresh, see bug 28653 and bug 36131.