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

[Bug 41603] New: Plugin hook causing DB locks when cancelling holds
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41603 Bug ID: 41603 Summary: Plugin hook causing DB locks when cancelling holds Initiative type: --- Sponsorship --- status: Product: Koha Version: 24.11 Hardware: All OS: All Status: NEW Severity: critical Priority: P5 - low Component: Plugin architecture Assignee: koha-bugs(a)lists.koha-community.org Reporter: tomascohen(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org The Rapido ILL plugin [1] has a simple `after_hold_action` hook that checks if the current hold is linked to an ILL request [2], and in the positive case, it inserts a row on a `task_queue` table for the action to be processed asynchronously by a separate daemon/worker. Even though the plugin itself is not doing any heavy action on the DB, this causes DB locks that are easily reproducible. This seems to be a core Koha issue as it makes the holds cancelling process (and probably others) rather fragile. To reproduce: 1. Clone Rapido ILL: $ mkdir -p ~/git/koha-plugins && cd ~/git/koha-plugins $ git clone https://github.com/bywatersolutions/koha-plugin-rapido-ill.git rapido-ill 2. Start a KTD instance: $ ktd --proxy --name rapido --single-plugin ~/git/koha-plugins/rapido-ill up -d $ ktd --name rapido --wait-ready 200 3. Install the plugin config $ ktd --name rapido --shell k$ cd /kohadevbox/plugins/rapido-ill/scripts k$ perl bootstrap_rapido_testing.pl k$ restart_all 4. Create a hold for a known patron 5. Run this: k$ perl -e 'use Carp::Always; use Koha::Holds; my $hold = Koha::Holds->find(1);$hold->cancel({cancellation_reason => "OVER_365"})' => FAIL: It takes a while! It eventually fails with a lock wait timeout!: ``` C4::Reserves::_FixPriority(): DBI Exception: DBD::mysql::st execute failed: Lock wait timeout exceeded; try restarting transaction at /kohadevbox/koha/Koha/Hold.pm line 933 ``` => FACTS: There's a hold, being cancelled. There is no ILL request on the system. The only thing that is happening is that the plugin hook is being called, and is doing this query: ``` MariaDB [koha_cuyahoga]> SELECT me.* FROM illrequests me JOIN illrequestattributes ON illrequestattributes.illrequest_id = me.illrequest_id WHERE illrequestattributes.type = 'hold_id' AND illrequestattributes.value = '1' AND me.backend = 'RapidoILL'; Empty set (0.001 sec) ``` Moving the plugin hook outside the transaction fixes it. [1] https://github.com/bywatersolutions/koha-plugin-rapido-ill [2] Linked means checking if it has an extended attribute with the hold_id value. No FK checks. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 26

[Bug 42545] New: Koha::Calendar::days_between skips holiday subtraction for end date if time is early
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42545 Bug ID: 42545 Summary: Koha::Calendar::days_between skips holiday subtraction for end date if time is early Initiative type: --- Sponsorship --- status: Product: Koha Version: 24.11 Hardware: All OS: All Status: NEW Severity: critical Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: apirak(a)punsarn.asia QA Contact: testopia(a)bugs.koha-community.org Created attachment 198697 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198697&action=e… This patch truncates both dates to the beginning of the day after cloning and setting the time zone, ensuring the loop correctly processes all calendar days between the two dates. -- You are receiving this mail because: You are watching all bug changes.

1 21

[Bug 34000] Don't allow auto-generated cardnumbers to be re-used, it may give access of services to the next patron created
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34000 Caroline Cyr La Rose <caroline.cyr-la-rose(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED --- Comment #90 from Caroline Cyr La Rose <caroline.cyr-la-rose(a)inlibro.com> --- Bug fix, nothing to add or edit in the Koha manual. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 41045] New: Suggestions manage permissions added to patrons who previously had no permissions in that category
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41045 Bug ID: 41045 Summary: Suggestions manage permissions added to patrons who previously had no permissions in that category Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Patrons Assignee: koha-bugs(a)lists.koha-community.org Reporter: emmi.takkinen(a)koha-suomi.fi QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle(a)bywatersolutions.com Bug 33363 separated suggestions_manage permission in three permissions for creating, updating, and deleting suggetions. In bugs test plan and in its atomicupdate file it is stated that these permissions should have been added only for those users who previously had suggestions_manage permissions. However there is a typo on SQL insert statement and new permissions have been added to all users with full catalogue permissions: >INSERT IGNORE INTO user_permissions (borrowernumber, module_bit, code) SELECT borrowernumber, 12, 'suggestions_create' FROM borrowers WHERE flags & (1 << 2) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 36

[Bug 42606] New: Error when importing a record using Z39.50 from the cataloging home page
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42606 Bug ID: 42606 Summary: Error when importing a record using Z39.50 from the cataloging home page Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Cataloging Assignee: koha-bugs(a)lists.koha-community.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: andrew(a)bywatersolutions.com, angela.berrett(a)familysearch.org, blawlor(a)clamsnet.org, bugzilla(a)koha-us.org, carthur(a)slolibrary.org, cbrannon(a)cdalibrary.org, david(a)davidnind.com, dcook(a)prosentient.com.au, echandler(a)clamsnet.org, george(a)nekls.org, jonathan.druart(a)gmail.com, lauren_denny(a)sil.org, lisette(a)bywatersolutions.com, lucas(a)bywatersolutions.com, m.de.rooy(a)rijksmuseum.nl, mike.grgurev(a)mainlib.org, mnero(a)oslri.net, mspinney(a)clamsnet.org, nmumford(a)clamsnet.org Depends on: 32773 >From the cataloging home page, click "New from Z39.50" without choosing a framework in the dropdown menu. Perform a Z39.50 search in the popup window, then click "Import" for one of the results. This triggers an error in the parent window: Can't call method "is_fast_add" on an undefined value at /kohadevbox/koha/cataloguing/addbiblio.pl line 518 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32773 [Bug 32773] Have the ability to have more than 1 Fast Add framework -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 30

[Bug 41932] New: Allow optional zero-padding in items.cn_sort
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41932 Bug ID: 41932 Summary: Allow optional zero-padding in items.cn_sort Initiative type: --- Sponsorship --- status: Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: koha-bugs(a)lists.koha-community.org Reporter: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl We have callnumbers like 1 A 1, 10 A 10, 100 A 100 and 1000 A 100. Alphabetically sorting those does not work (100 A 100 < 2 A 2) This proposal includes adding a syspref to define callnumber formats that also specify the desired length when zero-padding cn_sort. This would be active when using C4::ClassSortRoutine::Generic. In the above case we would like to sort them with 0001 A 001, 0010 A 010, 0100 A 100 and 1000 A 100. So the first digit block has length 4, the second block 3. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 88

[Bug 42374] New: DB Upgrade from the UI is broken
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42374 Bug ID: 42374 Summary: DB Upgrade from the UI is broken Initiative type: --- Sponsorship --- status: Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: blocker Priority: P5 - low Component: Installation and upgrade (command-line installer) Assignee: koha-bugs(a)lists.koha-community.org Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: tomascohen(a)gmail.com >From bug 17438 comment 17 Thats how i reproduce it: - git checkout v25.11.03-2 - reset_all - git checkout main - restart_all - open koha and run web installer -> error occurs If you run updatedatabase in shell the error is not thrown ISBN_RANGE_MESSAGE is set to [] but that file does not exist! Trying to use the default locations -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 42

[Bug 8088] Png-images of covers lost transparency
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8088 Caroline Cyr La Rose <caroline.cyr-la-rose(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED --- Comment #24 from Caroline Cyr La Rose <caroline.cyr-la-rose(a)inlibro.com> --- Bug fix, nothing to add or edit in the manual. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 10198] Add the ability to import and export authorized values
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10198 --- Comment #10 from Laurence Rault <laurence.rault(a)biblibre.com> --- We consider the fields in the authorised_values table. So branch restrictions are not included. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 0

[Bug 8137] Checkout limit for all libraries
by bugzilla-daemon＠bugs.koha-community.org 10 Aug '26

10 Aug '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8137 --- Comment #169 from Thibaud Guillot (thibaud_g) <thibaud.guillot(a)biblibre.com> --- (In reply to David Nind from comment #167) > Happy to sign off as things seem to work as expected - see my testing > notes.[1] > > However, I have Failed QA because of the qa script FAILS and WARNINGS - see > qa script FAIL and WARNINGS.[2] > > [1] Testing notes (using KTD) > ============================= > > 1. Rules set up after patches applied, and so on: > > All All rule for all libraries: > - Current checkouts allowed: 2 > - Current on-site checkouts allowed: 2 > > Kid All rule for all libraries: > - Current checkouts allowed: 5 > - Current on-site checkouts allowed: 5 > > All All rule for Centerville: > - Current checkouts allowed: 5 > - Current on-site checkouts allowed: 5 > > All All rule for Midway: > - Current checkouts allowed: 5 > - Current on-site checkouts allowed: 5 > > New "Global checkout limits" section: > - All All: 1 for both options > - Kid All: 2 for both options > > 2. Testing checkouts (check items back in after each step)/ > > 2.1 . Library is set to Centerville: > . Mary Burton - with the above rules, should only be able to check out > one item in total > . Items to check out: 39999000011418, 39999000000498 > ==> can't checkout the second item (as expected), message is: > > Please confirm checkout > - Too many checked out. 1 checked out, only 1 are allowed. > - Maximum checkouts calculated from the circulation rule for > . Item type: All item types > . Patron category: All patron categories > . Library: All libraries > > 2.2 Library is set to Midway: > . Lisa Charles - with the above rules, should only be able to checkout > two items in total > . Items to check out: 39999000001532, 39999000006155, 39999000005097 > ==> can't check out the third item (as expected), message is: > > Please confirm checkout > - Too many checked out. 2 checked out, only 2 are allowed. > - Maximum checkouts calculated from the circulation rule for > . Item type: All item types > . Patron category: Kid > . Library: All libraries > > 2.3 Remove global rules, then attempt these check outs: > > - Centerville: attempt to check out 6 items to Mary (39999000011418, > 39999000000498, 39999000005134, 39999000005776,39999000011111, > 39999000011135), should only be able to checkout 5, message is: > > Please confirm checkout > - Too many checked out. 5 checked out, only 5 are allowed. > - Maximum checkouts calculated from the circulation rule for > . Item type: All item types > . Patron category: All patron categories > . Library: Centerville > > - Then attempt to check out 6 items to Mary with Midway as the library > (39999000001532, 39999000006155, 39999000005097, 39999000007077, > 39999000004595, 39999000004618), should only be able to checkout 5, message > on 6th item is: > > Please confirm checkout > - Too many checked out. 5 checked out, only 5 are allowed. > - Maximum checkouts calculated from the circulation rule for > . Item type: All item types > . Patron category: All patron categories > . Library: Midway > > - Then attempt to checkout 3 items to Mary with Fairview as the library > (39999000008197, 39999000000818, 39999000001518), not able to check out any > as the All All for all libraries is 2 - have exceeded that as have checkout > 10 already (5 at Centerville, 5 at Midway), message is: > > Please confirm checkout > - Too many checked out. 10 checked out, only 2 are allowed. > - Maximum checkouts calculated from the circulation rule for > . Item type: All item types > . Patron category: All patron categories > . Library: All libraries > > [2] qa script fails and warnings > ================================ > > [FAIL] Koha/CirculationRules.pm > FAIL pod_coverage > sub _type has no pod coverage. > ... > [FAIL] admin/smart-rules.pl > FAIL tidiness > File is not tidy, please run `perl misc/devel/tidy.pl admin/smart-rules.pl` > ... > [FAIL] installer/data/mysql/kohastructure.sql > FAIL tinyint_has_boolean_flag > The new column circulation_rules.has_priority is using TINYINT(1) as type > but is not defined as boolean in the schema file > (Koha/Schema/Result/CirculationRule.pm), see the SQL12 coding guideline > [FAIL] koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt > WARN spelling > librairies ==> libraries > librairies ==> libraries > FAIL tidiness > File is not tidy, please run `perl misc/devel/tidy.pl > koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt` Hello David, Thank you for this very detailed review. I push another commit to fix this, tidiness, spelling and boolean_flag is related to this bug, for pod_coverage I modify the file but the warn already exists before I think so I also fixed this to make this bug move forward :) Have a nice day -- You are receiving this mail because: You are watching all bug changes.

1 0