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

[Bug 43663] New: Add Koha::Objects::Mixin::Polymorphic for polymorphic subclass dispatch
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43663 Bug ID: 43663 Summary: Add Koha::Objects::Mixin::Polymorphic for polymorphic subclass dispatch Initiative type: --- Sponsorship --- status: Product: Koha Version: Main 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)openfifth.co.uk QA Contact: testopia(a)bugs.koha-community.org Target Milestone: --- Several Koha::Objects plural classes independently implement the same pattern: given a row, decide which Koha::Object subclass should represent it, based on the value of one column (a hand-rolled _polymorphic_field/_polymorphic_map pair plus an ~8-line object_class method). Koha::File::Transports (already in main) dispatches on the 'transport' column to Koha::File::Transport::{SFTP,FTP,Local}. Koha::Auth::Identity::Providers (bug 24880, not yet merged) dispatches on 'protocol' to Koha::Auth::Identity::Provider::{OAuth,OIDC,SAML2}. The same need came up again during review of bug 42870 (pluggable email transports). Koha::Auth::Identity::Provider::OAuth additionally hand-types its own dispatch key a second time, independently, inside its own new() ($params->{protocol} = 'OAuth';) - nothing ties that string to the _polymorphic_map entry in the plural class that has to match it, so the two can silently drift apart. This bug extracts the shared behaviour into a proper Koha::Objects::Mixin::* (the same convention Koha already uses for Koha::Objects::Mixin::AdditionalFields, composed via 'use parent qw( Koha::Objects Koha::Objects::Mixin::Foo )', used by 20+ plural classes), and migrates the one already-in-main consumer (Koha::File::Transports) onto it as proof it is behaviour-preserving. A consuming plural class declares: sub _polymorphic_field { return 'transport' } sub _polymorphic_classes { return (List::Of::Class::Names) } sub _polymorphic_base_class { return 'Fallback::Class::Name' } Each subclass self-declares its own dispatch key: sub _polymorphic_key { return 'sftp' } The mixin builds _polymorphic_map from _polymorphic_classes plus each class's own _polymorphic_key, and provides object_class() itself, so the copy-pasted dispatch logic disappears from every consumer, not just the map. A plural class needing genuinely custom mapping logic can still override _polymorphic_map (or object_class) directly - ordinary Perl inheritance makes that a free escape hatch. This is a proposal filed ahead of code, based on a design worked out with the QA team. It is deliberately narrow: it introduces no mechanism for Koha Plugins to register additional polymorphic subclasses, no Plugin management UI changes, and no koha-conf.xml changes - that is a separate, later concern once a concrete capability needs it. It does not touch Koha::Auth::Identity::Providers (bug 24880) or bug 42870 directly; both could adopt this mixin as a follow-up once merged. Patches to follow. Test plan (once patches are attached): 1. Apply the patch(es). 2. Run prove t/db_dependent/Koha/Objects/Mixin/Polymorphic.t and confirm all tests pass. 3. Run prove t/db_dependent/Koha/File/Transport.t t/db_dependent/Koha/File/Transports.t t/db_dependent/Koha/File/Transport/FTP.t t/db_dependent/Koha/File/Transport/Local.t t/db_dependent/Koha/File/Transport/SFTP.t and confirm they all still pass unmodified against the migrated Koha::File::Transports, proving the migration is behaviour-preserving. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 13

[Bug 3758] Automatic update of exchange rates
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3758 Claire Gravely <claire_gravely(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |claire_gravely(a)hotmail.com --- Comment #7 from Claire Gravely <claire_gravely(a)hotmail.com> --- (In reply to Katrin Fischer from comment #6) > It looks like this has been abandoned, any chance to pick it up again? We would also be interested in this, unless it has been picked up elsewhere -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 41993] New: Add a datatable to list available bookings
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41993 Bug ID: 41993 Summary: Add a datatable to list available bookings Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: thibaud.guillot(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle(a)bywatersolutions.com Hello @ll Like the datatable containing pending bookings, it may be useful to add a table displaying the list of bookable items. This would take into account specific rules (lead and trail periods) and be joined with current checkouts to certify their availability. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 19

[Bug 41768] New: OPAC pickup location selector doesn't reflect available pickup location if you select a specific item
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41768 Bug ID: 41768 Summary: OPAC pickup location selector doesn't reflect available pickup location if you select a specific item Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: arthur.suzuki(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org As of now, if I go on the OPAC to place a hold, I'm presented a list of available pickup locations for that biblio. However, If I click on "more options" and select a specific item with limited transfers, I can still select any pickup location available for the biblio. This leads to a user being able to select a pickup location and be denied the hold request only after confirmation, and then have to restart again (without knowing which libraries will work). I think that can lead to some frustration for users. This is quite similar to BZ33302 or BZ10993 but I will provide a new patch with the use of Koha's api and make the OPAC more similar to the intranet interface. -- You are receiving this mail because: You are watching all bug changes.

1 58

[Bug 39865] New: Consider updating OpenLibrary covers search API
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39865 Bug ID: 39865 Summary: Consider updating OpenLibrary covers search API Change sponsored?: --- Product: Koha Version: 24.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: oliver.behnke(a)aei.mpg.de QA Contact: testopia(a)bugs.koha-community.org Koha currently uses OpenLibrary's "Legacy Book API" [1] to fetch book covers which "is a legacy endpoint and may be phased out in the future". Also, it's inefficient. The replacement seems to be the dedicated "Covers API" [2]. One drawback, though, it's currently rate limited to 100 requests per IP per five minutes. That might quickly be exhausted on the OPAC search result page. Thanks [1] https://openlibrary.org/dev/docs/api/books [2] https://openlibrary.org/dev/docs/api/covers -- You are receiving this mail because: You are watching all bug changes.

1 1

[Bug 29733] New: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder in sample frameworks
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29733 Bug ID: 29733 Summary: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder in sample frameworks Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Bibliographic data support Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org We do link the authority controlled fields to the authority linker by default, so it would make sense to link the fields used for linking between bibliographic records to the available helper plugin as well. At the moment the plugin is not well know, although it provides a great functionality, this could help advertise and improve it. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 15

[Bug 30303] New: Add ability to select which values to retain when merging patrons
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30303 Bug ID: 30303 Summary: Add ability to select which values to retain when merging patrons Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Patrons Assignee: koha-bugs(a)lists.koha-community.org Reporter: andrew(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com When merging two patron records, one simply picks one record to retain values from. It would be helpful to have an interface more like what we see when merging biblio records, where one can select values from either pre-merge record to be retained post-merge. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 63

[Bug 35333] New: Koha::Patron->add_guarantor not checking anything
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35333 Bug ID: 35333 Summary: Koha::Patron->add_guarantor not checking anything Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: tomascohen(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org This method should receive a `guarantor` parameter, and check if it is a valid guarantor, the same way we later check when saving patrons that the guaratonrs are valid (which I think was put in the wrong place). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 2

[Bug 43198] New: Update suggestions template to use grid layout for forms
by bugzilla-daemon＠bugs.koha-community.org 29 Sep '26

29 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43198 Bug ID: 43198 Summary: Update suggestions template to use grid layout for forms Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org Depends on: 39780 Target Milestone: --- This patch will update the suggestions template to use grid layout for the add/edit form. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39780 [Bug 39780] Update library groups form to use grid layout -- You are receiving this mail because: You are watching all bug changes.

1 8

[Bug 24608] New: Allow modification of checkout due date
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '26

28 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24608 Bug ID: 24608 Summary: Allow modification of checkout due date Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: andrew.isherwood(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Currently it is not possible to modify a checkout's due date. This bug proposes to allow such an action. It is only proposed to allow modification of a due date *forward* at this stage as moving a due date backwards may involve additional complication that will require further thought. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 71