https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |tomascohen@gmail.com |ity.org | Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #2 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 197711 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197711&action=edit Bug 42418: Wire Koha::Checkin into AddReturn Every call to AddReturn now creates a Koha::Checkin record capturing the full context and outcomes of the return scan: - user_id and desk_id from C4::Context->userenv - library_id from the branch parameter - checkout_id from the issue being returned (old_issues) - transfer_id from $messages->{WasTransfered} - hold_id from $messages->{ResFound}{reserve_id} The checkin record is also passed to the after_circ_action plugin hook, so plugins can access it directly instead of reconstructing context from side effects. AddReturn now returns a fifth value: the Koha::Checkin object. Existing callers are unaffected — Perl silently ignores extra return values. Assisted-by: Sonnet 4.6 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lisette@bywatersolutions.co | |m -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197710|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #4 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 197715 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197715&action=edit Bug 42418: Add checkins table and Koha::Checkin object Introduces checkins as a first-class resource with its own table, tracking the full context and outcomes of a return scan: - who: user_id (staff member), desk_id (optional) - what: item_id - where: library_id - when: timestamp - outcomes: checkout_id (returned), transfer_id (triggered), hold_id (filled) New files: - Koha::Checkin / Koha::Checkins (Koha::Object classes) - Koha::Schema::Result::Checkin (DBIC, generated via dbic) - atomicupdate/bug_42418.pl + kohastructure.sql Test plan: 1. Apply patch 2. Run updatedatabase.pl 3. Verify checkins table exists with correct structure 4. Sign off :-D Assisted-by: Sonnet 4.6 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com --- Comment #6 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- (In reply to Lisette Scheer from comment #3)

Would be helpful to have a clear path to deleted_items so when an item is deleted the link isn't lost.

The deleteditems pattern is a known anti-pattern we'd like to move away from. I'm not sure we want to add a FK here perpetuating it. Happy to discuss it in a broader context.

Also might be worth adding a local_use column for flagging if a checkin was a local use. Would make tracking that more effectively, especially if there could be a checkbox or something so local use could be specifically set.

Done! Thanks for bringing that up! I totally missed it, and fits perfectly within the reasons we need a proper checkins table :-D

I know an adjustment to me when I came to Koha was that my old ils had a separate tab for local_use checkins and things like holds we'd pull and check in to fill the hold wouldn't count as local use since it wasn't really used locally, just pulled by staff to fill the hold.

This sounds great. File a bug depending on this one so we discuss it in context! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197716|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #8 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 197723 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197723&action=edit Bug 42418: Wire Koha::Checkin into AddReturn Every call to AddReturn now creates a Koha::Checkin record capturing the full context and outcomes of the return scan: - user_id and desk_id from C4::Context->userenv - library_id from the branch parameter - checkout_id from the issue being returned (old_issues) - transfer_id from $messages->{WasTransfered} - hold_id from $messages->{ResFound}{reserve_id} The checkin record is also passed to the after_circ_action plugin hook, so plugins can access it directly instead of reconstructing context from side effects. AddReturn now returns a fifth value: the Koha::Checkin object. Existing callers are unaffected — Perl silently ignores extra return values. Assisted-by: Sonnet 4.6 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@openfifth.c | |o.uk -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #11 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- (In reply to Katrin Fischer from comment #10)

Can you explain a bit how this will affect old_issues or if there are plans for old_issues down the road?

Another solution could have been to add additional columns to old issues as that represents a checkin currently. I guess you had reasons not to go this route, but am a bit worried about the duplicated data.

I always think of `old_issues` as the `issues` table with information about a special status of the checkout: returned (which is inferred from a return date being set). With that in mind my only plan is to merge those tables in the mid (?) term. But -regardless of what we do with those tables- this development highlights that the checkin action can (or not) trigger some status change for a checkout, and many more things. All of those nicely handled and tested here :-). Taking accountlines as an example, I will propose we ditch the old_issues link and replace with a checkin link. That way we actually gain tracking possibilities for (say) refunding lost fines even when there might not be a linked checkout, and things like that. The link to old checkouts could be reconstructed through the checkin link. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197722|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #13 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 197863 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197863&action=edit Bug 42418: Add checkins table and Koha::Checkin object Introduces checkins as a first-class resource with its own table, tracking the full context and outcomes of a return scan: - who: user_id (staff member), desk_id (optional) - what: item_id - where: library_id - when: timestamp - outcomes: checkout_id (returned), transfer_id (triggered), hold_id (filled) New files: - Koha::Checkin / Koha::Checkins (Koha::Object classes) - Koha::Schema::Result::Checkin (DBIC, generated via dbic) - atomicupdate/bug_42418.pl + kohastructure.sql Test plan: 1. Apply patch 2. Run updatedatabase.pl 3. Verify checkins table exists with correct structure 4. Sign off :-D Assisted-by: Sonnet 4.6 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #15 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 197865 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197865&action=edit Bug 42418: (follow-up) Make checkins.user_id nullable SIP and CLI-driven returns have no userenv, so user_id must accept NULL. The FK action is also changed from CASCADE to SET NULL so deleting a staff patron preserves the checkin history. Changes: - kohastructure.sql: user_id DEFAULT NULL, FK ON DELETE SET NULL - atomicupdate: same - DBIC schema regenerated - Test updated to expect SET NULL behavior -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #17 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- This looks to be a solid concept to me! It will definitely be useful to know when an item was scanned in with or without a return being processed. The conversation has drifted a bit, so here are my two bits; If we created a new table for all of the $table/old_$table combos, we could we could recreate the existing current/old tables as views to avoid the need to rewrite reports. It looks like DBIC also supports views, but I haven't looked into it deeply. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197864|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #18 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198164 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198164&action=edit Bug 42418: DB changes Adds the checkins table to track individual checkin events with references to the item, staff member, library, desk, checkout, transfer, hold, recall, restriction, and return claim involved. Includes backfill of historical checkin records from old_issues. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #20 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198166 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198166&action=edit Bug 42418: Add Koha::Checkin and Koha::Checkins classes Koha::Object-based classes for the checkins table, with tests. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #22 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198168 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198168&action=edit Bug 42418: Wire Koha::Checkin into AddReturn Creates a Koha::Checkin record during AddReturn, capturing the checkin context (item, staff, library, desk, checkout, transfer, hold, recall, restriction, claim). The checkin_id is passed to the after_circ_action hook payload for plugin consumption. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198165|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198167|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #23 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198169 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198169&action=edit Bug 42418: DB changes Adds the checkins table to track individual checkin events with references to the item, staff member, library, desk, checkout, transfer, hold, recall, restriction, and return claim involved. Includes backfill of historical checkin records from old_issues. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #25 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198171 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198171&action=edit Bug 42418: Add Koha::Checkin and Koha::Checkins classes Koha::Object-based classes for the checkins table, with tests. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #27 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198173 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198173&action=edit Bug 42418: Wire Koha::Checkin into AddReturn Creates a Koha::Checkin record during AddReturn, capturing the checkin context (item, staff, library, desk, checkout, transfer, hold, recall, restriction, claim). The checkin_id is passed to the after_circ_action hook payload for plugin consumption. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #29 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- prove t/db_dependent/Circulation.t t/db_dependent/Circulation.t .. 30/87 # Transfer limit: Holding -> Return # Transfer limit: Return -> Home # Transfer limit: Return -> Holding # Looks like you planned 3 tests but ran 2. # Failed test 'AddReturn + TransferLimits' # at t/db_dependent/Circulation.t line 3217. Broken FK constraint # Failed test 'no warnings' # at /usr/share/perl/5.36/Test/Builder.pm line 193. # There were 1 warning(s) # Previous test 2 'Item cannot be returned if there is a transfer to do (item is floating)' # DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Cannot add or update a child row: a foreign key constraint fails (`koha_kohadev`.`checkins`, CONSTRAINT `checkins_ibfk_2` FOREIGN KEY (`user_id`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE SET NULL ON UPDATE CASCADE) at /kohadevbox/koha/Koha/Object.pm line 174 # at /kohadevbox/koha/Koha/Object.pm line 178. # Koha::Object::catch {...} (DBIx::Class::Exception=HASH(0xaaaad545efd8)) called at /usr/share/perl5/Try/Tiny.pm line 123 # Try::Tiny::try(CODE(0xaaaad5327910), Try::Tiny::Catch=REF(0xaaaad57ae988)) called at /kohadevbox/koha/Koha/Object.pm line 183 # Koha::Object::store(Koha::Checkin=HASH(0xaaaacf983888)) called at /kohadevbox/koha/C4/Circulation.pm line 2842 # C4::Circulation::AddReturn("rDenWVmihu00aHTJS", "UihEz4gC") called at t/db_dependent/Circulation.t line 3215 # main::__ANON__() called at /usr/share/perl/5.36/Test/Builder.pm line 374 # eval {...} called at /usr/share/perl/5.36/Test/Builder.pm line 374 # Test::Builder::subtest(Test::Builder=HASH(0xaaaac96e2c20), "AddReturn + TransferLimits", CODE(0xaaaad5381bd8)) called at /usr/share/perl/5.36/Test/More.pm line 809 # Test::More::subtest("AddReturn + TransferLimits", CODE(0xaaaad5381bd8)) called at t/db_dependent/Circulation.t line 3217 # # Looks like your test exited with 11 just after 34. t/db_dependent/Circulation.t .. Dubious, test returned 11 (wstat 2816, 0xb00) Failed 55/87 subtests Test Summary Report ------------------- t/db_dependent/Circulation.t (Wstat: 2816 (exited 11) Tests: 34 Failed: 2) Failed tests: 33-34 Non-zero exit status: 11 Parse errors: Bad plan. You planned 87 tests but ran 34. Files=1, Tests=34, 10 wallclock secs ( 0.03 usr 0.01 sys + 7.42 cusr 1.34 csys = 8.80 CPU) Result: FAIL -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #31 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- (In reply to Kyle M Hall (khall) from comment #29)

# DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Cannot add or update a child row: a foreign key constraint fails (`koha_kohadev`.`checkins`, CONSTRAINT `checkins_ibfk_2` FOREIGN KEY (`user_id`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE SET NULL ON UPDATE CASCADE) at /kohadevbox/koha/Koha/Object.pm line 174

That's because the mocked userenv (I reused) is pointing to a non-existent patron. I'll provide a fix, but the fallback to `borrowernumber=51` (the default) is a ticking bomb I'd say. Gonna file a separate (follow-up) bug report to make `mock_userenv` create a patron if not passed. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198169|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198171|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198173|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #33 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198556 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198556&action=edit Bug 42418: Add Koha::Schema::Result::Checkin DBIC Result class for the new checkins table. Includes relationships to items, borrowers, branches, desks, old_issues, branchtransfers, reserves, recalls, borrower_debarments, and return_claims. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #35 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198558 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198558&action=edit Bug 42418: DBIC schema changes [DO NOT PUSH] Auto-generated has_many relationships added to related Result classes for the new checkins table. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #37 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 198560 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198560&action=edit Bug 42418: (follow-up) Fix tests using invalid mock_userenv The checkins table has a FK on user_id pointing to borrowers. Several mock_userenv calls only passed branchcode, which defaults borrowernumber to the hardcoded value 51 — a non-existent patron. This causes FK violations when AddReturn creates a checkin record. This patch fixes the affected calls in Circulation.t and Branch.t by passing a valid patron to mock_userenv. It also adjusts the stock rotation transfer test to account for the new 5th return value from AddReturn (the Koha::Checkin object), and adds tests for the three checkin record user_id scenarios (valid patron, missing number, explicit undef number). A global fix to mock_userenv (auto-generating a patron when none is passed) would address the ~120 remaining calls across the test suite, but the risk surface warrants a separate bug. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|24401 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24401 [Bug 24401] Add POST /checkins endpoint with availability check and confirmation flow -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198556|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198558|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198560|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198632|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #41 from David Nind <david@davidnind.com> --- Created attachment 198678 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198678&action=edit Bug 42418: Add Koha::Schema::Result::Checkin DBIC Result class for the new checkins table. Includes relationships to items, borrowers, branches, desks, old_issues, branchtransfers, reserves, recalls, borrower_debarments, and return_claims. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #43 from David Nind <david@davidnind.com> --- Created attachment 198680 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198680&action=edit Bug 42418: DBIC schema changes [DO NOT PUSH] Auto-generated has_many relationships added to related Result classes for the new checkins table. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #45 from David Nind <david@davidnind.com> --- Created attachment 198682 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198682&action=edit Bug 42418: (follow-up) Fix tests using invalid mock_userenv The checkins table has a FK on user_id pointing to borrowers. Several mock_userenv calls only passed branchcode, which defaults borrowernumber to the hardcoded value 51 — a non-existent patron. This causes FK violations when AddReturn creates a checkin record. This patch fixes the affected calls in Circulation.t and Branch.t by passing a valid patron to mock_userenv. It also adjusts the stock rotation transfer test to account for the new 5th return value from AddReturn (the Koha::Checkin object), and adds tests for the three checkin record user_id scenarios (valid patron, missing number, explicit undef number). A global fix to mock_userenv (auto-generating a patron when none is passed) would address the ~120 remaining calls across the test suite, but the risk surface warrants a separate bug. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #47 from David Nind <david@davidnind.com> --- Created attachment 198684 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198684&action=edit Bug 42418: (follow-up) Fix Patrons.t for checkin record creation AddReturn now creates a checkin record which requires a valid library_id. The anonymize_last_borrowers subtests call AddReturn without a userenv set, causing a NOT NULL constraint violation. This adds mock_userenv before the AddReturn calls. Test plan: 1. Apply patch 2. Run: $ ktd --shell k$ prove t/db_dependent/Koha/Patrons.t => SUCCESS: All 52 tests pass! 3. Sign off :-D Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #198677|0 |1 is obsolete| | Attachment #198678|0 |1 is obsolete| | Attachment #198679|0 |1 is obsolete| | Attachment #198680|0 |1 is obsolete| | Attachment #198681|0 |1 is obsolete| | Attachment #198682|0 |1 is obsolete| | Attachment #198683|0 |1 is obsolete| | Attachment #198684|0 |1 is obsolete| | Attachment #201487|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #50 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201931 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201931&action=edit Bug 42418: Add Koha::Schema::Result::Checkin DBIC Result class for the new checkins table. Includes relationships to items, borrowers, branches, desks, old_issues, branchtransfers, reserves, recalls, borrower_debarments, and return_claims. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #52 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201933 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201933&action=edit Bug 42418: DBIC schema changes [DO NOT PUSH] Auto-generated has_many relationships added to related Result classes for the new checkins table. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #54 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201935 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201935&action=edit Bug 42418: (follow-up) Fix tests using invalid mock_userenv The checkins table has a FK on user_id pointing to borrowers. Several mock_userenv calls only passed branchcode, which defaults borrowernumber to the hardcoded value 51 — a non-existent patron. This causes FK violations when AddReturn creates a checkin record. This patch fixes the affected calls in Circulation.t and Branch.t by passing a valid patron to mock_userenv. It also adjusts the stock rotation transfer test to account for the new 5th return value from AddReturn (the Koha::Checkin object), and adds tests for the three checkin record user_id scenarios (valid patron, missing number, explicit undef number). A global fix to mock_userenv (auto-generating a patron when none is passed) would address the ~120 remaining calls across the test suite, but the risk surface warrants a separate bug. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #56 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201937 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201937&action=edit Bug 42418: (follow-up) Fix Patrons.t for checkin record creation AddReturn now creates a checkin record which requires a valid library_id. The anonymize_last_borrowers subtests call AddReturn without a userenv set, causing a NOT NULL constraint violation. This adds mock_userenv before the AddReturn calls. Test plan: 1. Apply patch 2. Run: $ ktd --shell k$ prove t/db_dependent/Koha/Patrons.t => SUCCESS: All 52 tests pass! 3. Sign off :-D Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #57 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201943 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201943&action=edit Bug 42418: DB changes Adds the checkins table to track individual checkin events with references to the item, staff member, library, desk, checkout, transfer, hold, recall, restriction, and return claim involved. Includes backfill of historical checkin records from old_issues. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #59 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201945 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201945&action=edit Bug 42418: Add Koha::Checkin and Koha::Checkins classes Koha::Object-based classes for the checkins table, with tests. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #61 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201947 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201947&action=edit Bug 42418: Wire Koha::Checkin into AddReturn Creates a Koha::Checkin record during AddReturn, capturing the checkin context (item, staff, library, desk, checkout, transfer, hold, recall, restriction, claim). The checkin_id is passed to the after_circ_action hook payload for plugin consumption. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42418 --- Comment #63 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 201949 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201949&action=edit Bug 42418: (follow-up) Skip old_issues with no library on backfill Both checkin_library and branchcode are nullable in old_issues. The COALESCE can return NULL, which violates the NOT NULL constraint on checkins.library_id. Filter out those rows. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.