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

[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 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

[Bug 37302] New: xt/api.t should fail if swagger-cli is missing
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37302 Bug ID: 37302 Summary: xt/api.t should fail if swagger-cli is missing Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org Currently, we skip if the swagger-cli utility is missing. This was hiding genuine issues in Jenkins, we should make it as a failure with a suitable message to the end user. -- You are receiving this mail because: You are watching all bug changes.

1 39

[Bug 35904] New: C4::Auth::checkauth cannot be tested easily
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35904 Bug ID: 35904 Summary: C4::Auth::checkauth cannot be tested easily Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Test Suite Assignee: jonathan.druart+koha(a)gmail.com Reporter: jonathan.druart+koha(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org We are printing headers and the tests are hard to write. -- You are receiving this mail because: You are watching all bug changes.

1 40

[Bug 38474] New: t/db_dependent/Letters.t can fail randomly
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38474 Bug ID: 38474 Summary: t/db_dependent/Letters.t can fail randomly Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org CC: jonathan.druart(a)gmail.com 17:00:43 koha_1 | # Failed test 'Time status changed equals time queued when created in message_queue table' 17:00:43 koha_1 | # at t/db_dependent/Letters.t line 156. 17:00:43 koha_1 | # got: '2024-11-15 16:00:00' 17:00:43 koha_1 | # expected: '2024-11-15 15:59:59' 17:00:43 koha_1 | # Looks like you failed 1 test of 103. 17:00:43 koha_1 | [16:00:09] t/db_dependent/Letters.t .................................................... It looks like it took a little longer than expected maybe and the test fails because of it. If this is the issue, we could make it a little more robust. -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 35625] New: Add support for system flag to additional fields
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35625 Bug ID: 35625 Summary: Add support for system flag to additional fields Change sponsored?: --- Product: Koha Version: master 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: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org Additional fields could be used to extend some multi-use tables by default without needing to extend the initial table. It would be helpful to be able to mark such additional fields with the is_system flag to prevent user deletion. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 38

[Bug 35044] New: Additional fields: Allow for multiple selection of values
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35044 Bug ID: 35044 Summary: Additional fields: Allow for multiple selection of values 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: pedro.amorim(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com Depends on: 32751 If the field is an AV field, there should be an option to allow for multiple selection. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32751 [Bug 32751] [Omnibus] Extend, improve and standardize additional fields -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 189