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

[Bug 36592] New: Cannot save Default display length or Default sort order in Table settings
by bugzilla-daemon＠bugs.koha-community.org 30 Aug '25

30 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36592 Bug ID: 36592 Summary: Cannot save Default display length or Default sort order in Table settings Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: System Administration Assignee: koha-bugs(a)lists.koha-community.org Reporter: lucas(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com To recreate: 1. Go to Admin > Table settings 2. Find a table that includes Default display length or Default sort order. 3. Try to save a new value. 4. When the page reloads the new values have not been saved. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 18

[Bug 39368] New: Warnings from t::lib::Mocks::Zebra because of statement after exec
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39368 Bug ID: 39368 Summary: Warnings from t::lib::Mocks::Zebra because of statement after exec Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org # Previous test 1217 'POD coverage for t/lib/Mocks/Logger.pm' # Statement unlikely to be reached at /kohadevbox/koha/t/lib/Mocks/Zebra.pm line 119. # at /kohadevbox/koha/t/lib/Mocks/Zebra.pm line 119. # require t/lib/Mocks/Zebra.pm called at (eval 4894) line 1 # eval ' require t::lib::Mocks::Zebra ' called at /usr/share/perl5/Pod/Coverage.pm line 308 # Pod::Coverage::_get_syms(Pod::Coverage=HASH(0x569d3a0bbf50), "t::lib::Mocks::Zebra") called at /usr/share/perl5/Pod/Coverage.pm line 152 # Pod::Coverage::coverage(Pod::Coverage=HASH(0x569d3a0bbf50)) called at /usr/share/perl5/Pod/Coverage.pm line 209 # Pod::Coverage::naked(Pod::Coverage=HASH(0x569d3a0bbf50)) called at xt/author/pod_coverage.t line 40 # main::check_pod_coverage("t/lib/Mocks/Zebra.pm") called at xt/author/pod_coverage.t line 30 exec will replace the current process, it does not make sense to have another statement after. -- You are receiving this mail because: You are watching all bug changes.

1 13

[Bug 39325] New: Run codespell successfully on the whole codebase
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39325 Bug ID: 39325 Summary: Run codespell successfully on the whole codebase Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Blocks: 39324 Codespell is run by the QA script, but we have a lot of typos. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39324 [Bug 39324] [OMNIBUS] All QA tests should pass -- You are receiving this mail because: You are watching all bug changes.

1 101

[Bug 39705] New: Write POD when missing
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39705 Bug ID: 39705 Summary: Write POD when missing Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Bug 39367 added a POD entry to all subroutines: "Missing POD for" We should write correct documentation for those subroutines, maybe using AI? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 4

[Bug 39367] New: Add POD to all subroutines/methods
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39367 Bug ID: 39367 Summary: Add POD to all subroutines/methods Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Depends on: 39319 Blocks: 39324 In order to have a POD coverage test we need a starting point where all subroutines and methods will have a POD entry, even if empty. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39319 [Bug 39319] xt/author/podcorrectness.t only test POD for perl files within C4 and Koha https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39324 [Bug 39324] [OMNIBUS] All QA tests should pass -- You are receiving this mail because: You are watching all bug changes.

1 43

[Bug 39700] New: Fix test case t/db_dependent/Authority/Merge.t broken in 34739
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39700 Bug ID: 39700 Summary: Fix test case t/db_dependent/Authority/Merge.t broken in 34739 Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: domm(a)plix.at QA Contact: testopia(a)bugs.koha-community.org Bug 34739 adds some new tests to t/db_dependent/Authority/Merge.t, which work when run alone. But when running the whole test suite, some weird fails show up. This patch fixes that bug in the test suite The new test case calls AddBiblio. And AddBiblio returns ($biblionumber, $biblioitemnumber). But the test calls it like this: my $biblionumber = AddBiblio; AddBiblio returns a list, but the calling code only stores a single value, so it get's the last value, i.e. the biblioitemnumber. Now, when running the test alone, the DB sequences for biblio and biblioitem are in sync and both return the same value so the test works (by accident) But some tests add biblios with multiple items (or biblios without items), thus the sequences are no longer in sync and when the test tries to load a biblio via a biblioitemid, it finds nothing, which causes the fail bad: my $biblionumber = AddBiblio( $biblio_record, '', { skip_record_index => 1 } ); ok-ish: my ($biblionumber) = AddBiblio( $biblio_record, '', { skip_record_index => 1 } ); -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 39874] New: Template error prevents authority merging
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39874 Bug ID: 39874 Summary: Template error prevents authority merging Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Authentication Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: dpavlin(a)rot13.org Depends on: 39325 Bug 39325 renamed the variable used inside of the FOREACH, but one of the instances was unchanged: [% FOREACH line IN result %] <tr data-authid="[% result.authid | html %]"> Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39325 [Bug 39325] Run codespell successfully on the whole codebase -- You are receiving this mail because: You are watching all bug changes.

1 16

[Bug 39319] New: xt/author/podcorrectness.t only test POD for perl files within C4 and Koha
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39319 Bug ID: 39319 Summary: xt/author/podcorrectness.t only test POD for perl files within C4 and Koha Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org It should also test controllers and tests. -- You are receiving this mail because: You are watching all bug changes.

1 26

[Bug 38461] New: Table features needs to be covered by e2e tests - Cypress
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38461 Bug ID: 38461 Summary: Table features needs to be covered by e2e tests - Cypress Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Depends on: 36640 We need UI tests for the table features. We have different kinds of table and a lot of features that must be covered by tests. 3 types of table: * Tables using the REST API (kohaTable) * Tables not using the REST API (KohaTable) * Tables using dataTables constructor directly (should be removed by bug 38255) Features: * Save state * Copy shareable link * Save configuration state on page change * Save search state on page change: * Export * Search (and the 'Clear filters' button) * Columns filters * External filters * etc. Specific tables: * Patrons search * Items list (catalogue/detail.pl) * Items search * Pending requests (should not be different but it is) * Libraries ("classic" using kohaTable) Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36640 [Bug 36640] Upgrade DataTables from 1.13.6 to 2.x -- You are receiving this mail because: You are watching all bug changes.

1 88

[Bug 38503] New: Add a Cypress task to generate objects based on its swagger def spec
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38503 Bug ID: 38503 Summary: Add a Cypress task to generate objects based on its swagger def spec Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org We have TestBuilder for the Perl tests. However for the Cypress tests we have to: 1. either mock all the routes (which is painful and time-consuming), 2. or rely on the existing data in DB (which will produce failures if data has been inserted, and tests will have to cleanup before/after, which is painful, time-consuming and not robust) We should have a way to generate objects easily, in order to mock the response of a GET route easily. We can actually rely on the swagger spec files (the definition file) to generate random data and mock the response of routes. The first intended goal is to provide tests for the tables (bug 38461) and allow future enhancements in this area (see dependent bugs of bug 36640). -- You are receiving this mail because: You are watching all bug changes.

1 72