- Koha-bugs - lists.koha-community.org

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185118|0 |1 is obsolete| | --- Comment #90 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185323 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185323&action=e… Bug 9762: (follow-up) Add branchcode to Log circulation and add some override confirmations in action logs, fix typos -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181546|0 |1 is obsolete| | --- Comment #89 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185322 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185322&action=e… Bug 9762: (Alternative) Handle renewals -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181545|0 |1 is obsolete| | --- Comment #88 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185321 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185321&action=e… Bug 9762: (Alternative) Log circulation overrides An alternative approach to the original submission for bug 9762. Instead of recording a distinct action log line for a set of override messages in addition to the standard issue record, this patch updates the existing issue log to include any confirmation or impossible responses from the preceeding CanBookBeIssued call. We continue to record these as a JSON structure as the prior patch did, but instead of human messages we use the predefined codes as returned from the existing CanBookBeIssued response. To test: After each step, a single action log message should be displayed under `Tools > Log viewer`. The message itself will now consist of a JSON string with an array of 'confirmations' and 'forced' strings representing the overriden feedback from CanBookBeIssued. Circulation -- Checking out 1) Check out an item that is age-restricted (AgeRestrictionOverride syspref) 2) Check out an item that has a “not for loan” status (AllowNotForLoanOverride syspref) 3) Check out an item that has a “lost” status (IssueLostItem syspref) 4) Check out an item to a patron who has reached the checkout limit (AllowTooManyOverride syspref) 5) Check out an item to a patron who is not allowed to borrow this item type 6) Check out an item to a patron who has unpaid fines (AllFinesNeedOverride and/or AllowFineOverride + noissuescharge sysprefs) 7) Check out an item on hold for someone else 8) Check out an item on hold ready for pickup by someone else 9) Check out an item already checked out to someone else (AutoReturnCheckedOutItems syspref) 10) Check out to a patron who has restrictions Circulation -- Checking in 11) Ignore a hold upon check in 12) Ignore a hold and transfer upon check in Circulation – Renewing 13) Renew an item on hold for someone else (AllowRenewalOnHoldOverride syspref) 14) Renew an item that has reached the maximum number of renewals (AllowRenewalLimitOverride syspref) Holds 15) Place a hold for a patron who is not allowed to place this item type on hold (AllowHoldPolicyOverride syspref) 16) Place a hold for a patron who has reached the maximum number of holds 17) Place an item-level hold when rules dictate that this is not allowed -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181363|0 |1 is obsolete| | --- Comment #87 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185320 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185320&action=e… Bug 9762: (QA follow-up) Remove superflous string construction We never actually pass `$message` back to anywhere so there's no reason to create it in the first place. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181362|0 |1 is obsolete| | --- Comment #86 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185319 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185319&action=e… Bug 9762: (QA follow-up) Fix failing test -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181361|0 |1 is obsolete| | --- Comment #85 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185318 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185318&action=e… Bug 9762: (QA follow-up) Add missing issueconfirmed check -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181360|0 |1 is obsolete| | --- Comment #84 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185317 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185317&action=e… Bug 9762: (QA follow-up) Fix executable bit -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181359|0 |1 is obsolete| | --- Comment #83 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185316 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185316&action=e… Bug 9762: Test CanBookBeIssued.pm To test: 1) Apply the patch 2) run prove t/db_dependent/Circulation/CanBookBeIssued.t --After each step, a message will be displayed in the action_logs table and in Tools > Log viewer -- Circulation -- Checking out 1) Check out an item that is age-restricted (AgeRestrictionOverride syspref) 2) Check out an item that has a “not for loan” status (AllowNotForLoanOverride syspref) 3) Check out an item that has a “lost” status (IssueLostItem syspref) 4) Check out an item to a patron who has reached the checkout limit (AllowTooManyOverride syspref) 5) Check out an item to a patron who is not allowed to borrow this item type 6) Check out an item to a patron who has unpaid fines (AllFinesNeedOverride and/or AllowFineOverride + noissuescharge sysprefs) 7) Check out an item on hold for someone else 8) Check out an item on hold ready for pickup by someone else 9) Check out an item already checked out to someone else (AutoReturnCheckedOutItems syspref) 10) Check out to a patron who has restrictions Circulation -- Checking in 11) Ignore a hold upon check in 12) Ignore a hold and transfer upon check in Circulation – Renewing 13) Renew an item on hold for someone else (AllowRenewalOnHoldOverride syspref) 14) Renew an item that has reached the maximum number of renewals (AllowRenewalLimitOverride syspref) Holds 15) Place a hold for a patron who is not allowed to place this item type on hold (AllowHoldPolicyOverride syspref) 16) Place a hold for a patron who has reached the maximum number of holds 17) Place an item-level hold when rules dictate that this is not allowed (In circulation rules put "On shelf holds allowed" to "If any unavailable") Bug 9762: change permission -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181358|0 |1 is obsolete| | --- Comment #82 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185315 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185315&action=e… Bug 9762: Change how to pass the param -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 12 Aug '25

12 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 Hammat wele <hammat.wele(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181357|0 |1 is obsolete| | --- Comment #81 from Hammat wele <hammat.wele(a)inlibro.com> --- Created attachment 185314 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185314&action=e… Bug 9762: Log circulation overrides This patch allows you to save commands that have been overridden by someone. The patch has been updated but is not yet completed. Tests 10, 13, 14, and 17 are still not functional. Tests 15 and 16 need to be fixed as they do not display the barcode. All other tests should be functional. To test: --After each step, a message will be displayed in the action_logs table and in Tools > Log viewer -- Circulation -- Checking out 1) Check out an item that is age-restricted (AgeRestrictionOverride syspref) 2) Check out an item that has a “not for loan” status (AllowNotForLoanOverride syspref) 3) Check out an item that has a “lost” status (IssueLostItem syspref) 4) Check out an item to a patron who has reached the checkout limit (AllowTooManyOverride syspref) 5) Check out an item to a patron who is not allowed to borrow this item type 6) Check out an item to a patron who has unpaid fines (AllFinesNeedOverride and/or AllowFineOverride + noissuescharge sysprefs) 7) Check out an item on hold for someone else 8) Check out an item on hold ready for pickup by someone else 9) Check out an item already checked out to someone else (AutoReturnCheckedOutItems syspref) 10) Check out to a patron who has restrictions Circulation -- Checking in 11) Ignore a hold upon check in 12) Ignore a hold and transfer upon check in Circulation – Renewing 13) Renew an item on hold for someone else (AllowRenewalOnHoldOverride syspref) 14) Renew an item that has reached the maximum number of renewals (AllowRenewalLimitOverride syspref) Holds 15) Place a hold for a patron who is not allowed to place this item type on hold (AllowHoldPolicyOverride syspref) 16) Place a hold for a patron who has reached the maximum number of holds 17) Place an item-level hold when rules dictate that this is not allowed (In circulation rules put "On shelf holds allowed" to "If any unavailable") -- You are receiving this mail because: You are watching all bug changes.

1 0