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

[Bug 11999] Add two checks in CanBookBeReserved and CanItemBeReserved
by bugzilla-daemon＠bugs.koha-community.org 07 Dec '25

07 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11999 --- Comment #144 from David Nind <david(a)davidnind.com> --- To clarify: 1. prove t/db_dependent/Reserves.t fails 2. prove t/db_dependent/Holds.t passes prove t/db_dependent/Reserves.t t/db_dependent/Reserves.t .. 68/71 # Looks like you planned 71 tests but ran 72. t/db_dependent/Reserves.t .. Dubious, test returned 255 (wstat 65280, 0xff00) All 71 subtests passed Test Summary Report ------------------- t/db_dependent/Reserves.t (Wstat: 65280 (exited 255) Tests: 72 Failed: 1) Failed test: 72 Non-zero exit status: 255 Parse errors: Bad plan. You planned 71 tests but ran 72. Files=1, Tests=72, 8 wallclock secs ( 0.03 usr 0.01 sys + 5.49 cusr 1.97 csys = 7.50 CPU) Result: FAIL kohadev-koha@kohadevbox:koha(bz11999)$ prove t/db_dependent/Holds.t t/db_dependent/Holds.t .. ok All tests successful. Files=1, Tests=74, 6 wallclock secs ( 0.03 usr 0.00 sys + 4.31 cusr 1.48 csys = 5.82 CPU) Result: PASS -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11999] Add two checks in CanBookBeReserved and CanItemBeReserved
by bugzilla-daemon＠bugs.koha-community.org 07 Dec '25

07 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11999 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This fixes placing holds release notes| |when using ILS-DI - the | |value set in maxreserves | |(for example 1) was | |ignored, and you could | |place more holds than | |maxreserves. | | | |Note: When placing holds | |using the OPAC or staff | |interface, the value set in | |maxreserves is taken into | |account. (That is, you | |can't place more holds than | |what is set in maxreserves | |using the OPAC or staff | |interface.) Status|Needs Signoff |Failed QA --- Comment #143 from David Nind <david(a)davidnind.com> --- The patches still apply, however the tests no longer pass (which is probably not surprising 8-)). Testing notes (using KTD): 1. Enable the ILS-DI system preference 2. Add 0.0.0.0/0 to the ILS-DI:AuthorizedIPs system preference 3. Set the maxreserves system preference to 1 4. Place holds with ILS-DI (Henry, Midway) - you should only be able to place 1, but you can place as many holds as you like: - Hold 1 (Bleak House): http://127.0.0.1:8080/cgi-bin/koha/ilsdi.pl?service=HoldTitle&patron_id=19&…'127.0.0.1' - Hold 2 (Programming the Perl DBI): http://127.0.0.1:8080/cgi-bin/koha/ilsdi.pl?service=HoldTitle&patron_id=19&…'127.0.0.1' - Hold 3 (The definitive guide to Catalyst): http://127.0.0.1:8080/cgi-bin/koha/ilsdi.pl?service=HoldTitle&patron_id=19&…'127.0.0.1' 5. Try to place a hold for Henry using the staff interface - you should get a message: Cannot place hold Too many holds: Henry Acevedo can only place a maximum of 1 total holds. 6. Apply the patch and restart everything 7. Try to place another hold using ILS-DI - you should get a - Hold 4 (Higher-order Perl): 8. You should get a this in response: <HoldTitle> <code>tooManyReserves</code> </HoldTitle> 9. Tests should pass: - prove t/db_dependent/Reserves.t - prove t/db_dependent/Holds.t -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2646] Add sort option for the series number
by bugzilla-daemon＠bugs.koha-community.org 07 Dec '25

07 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2646 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david(a)davidnind.com --- Comment #10 from David Nind <david(a)davidnind.com> --- This was discussed at the recent koha-US Cataloging Special Interest Group 4 December 2025 https://www.youtube.com/watch?v=4aLGMvjntDM (at various places) -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 14962] Temp Shelving Location
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14962 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david(a)davidnind.com --- Comment #38 from David Nind <david(a)davidnind.com> --- I'd be happy to test, but with so many patches I think this needs a nice consolidated test plan so that mere mortals like myself can test 8-). -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz ✔ ❤ ★
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 Lisette Scheer <lisette(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Comma delimited|First sponsor, Sponsor The |Athens County Public list of Sponsors|Third, Sponsor two |Libraries --- Comment #447 from Lisette Scheer <lisette(a)bywatersolutions.com> --- Bug 6473: Testing attach -e with new perl-git-bz Testing new perl-git-bz edit function -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz ✔ ❤ ★
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 --- Comment #446 from Lisette Scheer <lisette(a)bywatersolutions.com> --- Created attachment 190256 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190256&action=e… Bug 41142: Update jQuery-validate plugin to 1.21.0 This patch updates the jQuery validation plugin in both the staff interface and the OPAC to the latest version, 1.21.0. To test, apply the patch and clear your browser cache if necessary. Test various pages in the staff client to confirm that form validation works as expected: - Patron password change form: - Password must conform to minPasswordLength - Password must not contain leading/trailing spaces - Passwords must match - Administration -> Add or edit budget: - Description, start date, and end date are required - Start date must be before end date - Administration -> Add or edit Z39.50/SRU server: - Server name, hostname, port, and database are required - Port, rank, and timeout must be a number Perform the same check of the "Change password" form in the OPAC. Sponsored-by: Athens County Public Libraries -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz ✔ ❤ ★
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 --- Comment #445 from Lisette Scheer <lisette(a)bywatersolutions.com> --- Created attachment 190255 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190255&action=e… Bug 41364: Fix error in preservation module breadcrumb This patch makes the 'Processings' breadcrum element not be tied to a component but a plain string with the link disabled. This was causing errors: - Breadcrumb element empty - Error in the browser console To test: 1. Enable the Preservation module 2. Go to Home > Preservation > Settings 3. Use the 'Add a new processing' button => FAIL: The breadcrumb has 'Settings > > Add processing' (i.e. is missing 'Processings' between >) 4. Click on 'Edit this processing' => FAIL: Same error, different context 5. Check the browser console => FAIL: You see something like this: ``` [Vue Router warn]: No match found for location with path "/cgi-bin/koha/preservation/settings/processings" devtools-BLCumUwL.mjs:59:1 ``` 6. Run the cypress tests for this module: k$ run_cypress --spec t/cypress/integration/Preservation/Settings.ts => FAIL: The breadcrumb navigation is broken 7. Apply this patch 8. Rebuild things: $ ktd --shell k$ yarn build 9. Fully refresh the page (Use cmd+shift+r on macOS, ctrl+shift+r on Linux/Windows) 10. Repeat 2-6 => SUCCESS: The breadcrumb is no longer broken => SUCCESS: No errors/warnings about this in the browser console => SUCCESS: Tests pass! 11. Sign off :-D Signed-off-by: Tomás Cohen Arazi <tomascohen(a)theke.io> Signed-off-by: Lucas Gass <lucas(a)bywatersolutions.com> Signed-off-by: David Nind <david(a)davidnind.com> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> Signed-off-by: Lucas Gass <lucas(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 31281] New: Overdue notices reply-to email address of a branch not respected
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31281 Bug ID: 31281 Summary: Overdue notices reply-to email address of a branch not respected Change sponsored?: --- Product: Koha Version: 21.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Notices Assignee: koha-bugs(a)lists.koha-community.org Reporter: nason.bimbe(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org The overdue notification email is sent from the correct email address as set up for the branch, however the reply-to email address though set for the branch is not the one used. The email has reply-to address of a different branch and not even the default one as set up in system preferences. Below is set up (however I have renamed the branches and email addresses for the purpose of this bug report). So what happened is the overdue notification was sent using LibraryB email address however the reply-to email address on the notification is the reply-to email address of LibraryA instead of LibraryB one. This is on Koha 21.11.05.000 LibraryA settings Email: librarya(a)mlibary.whatever Reply-To: librarya(a)mlibrary.whatever Return-Path: librarya(a)mlibrary.whatever LibraryA settings Email: libraryb(a)mlibary.whatever Reply-To: libraryb(a)mlibrary.whatever Return-Path: libraryb(a)mlibrary.whatever LibraryC settings Email: libraryc(a)mlibary.whatever Reply-To: libraryc(a)mlibrary.whatever Return-Path: libraryc(a)mlibrary.whatever SysPref setting: KohaAdminEmailAddress: library(a)library.whatever ReplytoDefault: library(a)library.whatever -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 28

[Bug 6473] Test bug for Git-bz ✔ ❤ ★
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 --- Comment #444 from Tomás Cohen Arazi (tcohen) <tomascohen(a)gmail.com> --- Created attachment 190253 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190253&action=e… Bug 40041: (follow-up) Fix periods and add Robert Fox We introduced some errant periods in the mailmap which cause parsing issues. I also fix the removal of the '=' author by replacing it with a mapping to the original author of the patch from bug 6979, Robert Fox Signed-off-by: Lucas Gass <lucas(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz ✔ ❤ ★
by bugzilla-daemon＠bugs.koha-community.org 06 Dec '25

06 Dec '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 --- Comment #443 from Tomás Cohen Arazi (tcohen) <tomascohen(a)gmail.com> --- Created attachment 190252 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190252&action=e… Bug 40524: Update print format to use built-in formatter This change updates the print format to use the built-in formatter for operations like trim, strip HTML, strip newlines, and not decoding entities. Test plan: 0. Apply the patch 1. Go to http://localhost:8081/cgi-bin/koha/acqui/basket.pl?basketno=1 2. Click "add to basket" 3. Type "test" next to "From an existing record" and click "Submit" 4. On the "Gairm" row, click "Add order" 5. Fill in the required fields 6. In the "Internal note" put the following: abc123 7. Click "Save" 8. In the Orders table, click "Export" and then "Print" 9. Note that the print page loads correctly, you can see the internal note, and you cannot see lines for "Add internal note", "Edit internal note", or "Add vendor note" Signed-off-by: Brendan Lawlor <blawlor(a)clamsnet.org> Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Signed-off-by: Lucas Gass <lucas(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0