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

[Bug 43643] New: Cannot load records with empty fields
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43643 Bug ID: 43643 Summary: Cannot load records with empty fields Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Staff interface Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com Target Milestone: --- To recreate: Create a record with an empty field (from bug 35104) You can inject either into an existing record's stored metadata via the DB to set up test data: UPDATE biblio_metadata SET metadata = REPLACE(metadata, '</record>', '<datafield tag="500" ind1=" " ind2=" "></datafield></record>') WHERE biblionumber = <some biblionumber>; Try to view the record: Can't call method "leader" on an undefined value at /kohadevbox/koha/catalogue/detail.pl line 202 If you clear that error by editing the code: if( !$invalid_marc_record ){ my $leader = $marc_record->leader(); $dat->{bundlesEnabled} = ( ( substr( $leader, 7, 1 ) eq 'c' ) && C4::Context->preference('BundleNotLoanValue') ) ? 1 : 0; } You still get: Can't call method "as_xml" on an undefined value at /kohadevbox/koha/C4/XSLT.pm line 242 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 3

[Bug 43776] New: TransformHtmlToMarc should repair and report invalid characters rather than silently stripping them
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43776 Bug ID: 43776 Summary: TransformHtmlToMarc should repair and report invalid characters rather than silently stripping them Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Cataloging Assignee: koha-bugs(a)lists.koha-community.org Reporter: martin.renvoize(a)openfifth.co.uk QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl Target Milestone: --- Bug 35104 removed the inline StripNonXmlChars calls from C4::Biblio::TransformHtmlToMarc on the assumption that Koha::Biblio::Metadata->store() now strips centrally. TransformHtmlToMarc never calls store(): it builds a MARC::Record from submitted form values, and callers such as cataloguing/addbiblio.pl (FindDuplicate, duplicate redisplay), cataloguing/merge.pl and svc/cataloguing/automatic_linker.pl use that record without saving it. Control characters pasted into the form therefore reached MARC::Record/XML serialisation again, the same failure bug 34549 fixed. The RM follow-up on bug 35104 restored the two StripNonXmlChars calls as a stop-gap. That only covers non-XML characters in the first subfield of a new field and in control fields; it does not cover the other faults Koha::Biblio::Metadata->repair_marcxml handles (e.g. empty datafields), and it silently discards data without telling the cataloguer. Proposal: - Have TransformHtmlToMarc validate its result with the same logic as repair_marcxml instead of stripping inline. - Recoverable faults are repaired and reported to the caller (so the cataloguer can be shown what was changed). - Unrecoverable input throws Koha::Exceptions::Metadata::Invalid, as store() does. - Callers catch the exception and tell the user: addbiblio.pl and merge.pl show the message in the form/merge page, the automatic_linker svc returns a clear error. authorities/ callers need a decision, since authorities do not go through Koha::Biblio::Metadata. Test plan: 1. Apply the patches on top of bug 35104. 2. In the cataloguing editor, paste a control character (e.g. U+001F) into a subfield of a new record, then use Save/Duplicate. Confirm the record is saved or redisplayed with a warning describing what was stripped, and no 500 error. 3. Submit a form producing an empty datafield. Confirm it is repaired and reported. 4. Run prove t/db_dependent/Biblio/TransformHtmlToMarc.t Sponsored-by: OpenFifth -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 3

[Bug 29915] Anonymous session generates 1 new session ID per hit
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29915 Jonathan Druart <jonathan.druart(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |43683 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 39233] New: Non-patron guarantor info not dropped when patron category is updated
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39233 Bug ID: 39233 Summary: Non-patron guarantor info not dropped when patron category is updated 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.m.hall(a)gmail.com When child patrons category is updated via update_patrons_category.pl patrons relationship with their guarantor is dropped from borrower_relationships table. But non-patron guarantees info is still kept in borrowers table. To reproduce: 1. Find child patron with non-patron guarantor or create one. 2. Update patrons age so that their category is updated when you run following: perl misc/cronjobs/update_patrons_category.pl --confirm --too_old --from K --to PT 3. Check patrons info from borrowers table. => Note that values in contactname, contactfirstname and relationship columns were not dropped. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 19

[Bug 43775] New: Koha/SuggestionEngine.pm uses incorrect bitwise OR
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43775 Bug ID: 43775 Summary: Koha/SuggestionEngine.pm uses incorrect bitwise OR Initiative type: --- Sponsorship --- status: Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Searching Assignee: koha-bugs(a)lists.koha-community.org Reporter: dcook(a)prosentient.com.au QA Contact: testopia(a)bugs.koha-community.org Target Milestone: --- Do you ever see the following in your plack-error.log file? Use of strings with code points over 0xFF as arguments to bitwise or (|) operator is not allowed at /usr/share/koha/lib/Koha/SuggestionEngine.pm line 153. Well, that's because we're using the wrong operator in the following line: $suggestions{ $suggestion->{'search'} }->{'label'} |= $suggestion->{'label'}; I suspect the author meant to use ||= but accidentally added |=. Looks like it was added by Jared in 2012. I'm surprised he'd make a typo like that, but we all make mistakes... -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 2

[Bug 8209] "Did you mean?" suggestions based on authorities
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8209 Jonathan Druart <jonathan.druart(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |43775 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43775 [Bug 43775] Koha/SuggestionEngine.pm uses incorrect bitwise OR -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 37710] New: Remove the marc link when removing an item from a bundle
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37710 Bug ID: 37710 Summary: Remove the marc link when removing an item from a bundle Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: koha-bugs(a)lists.koha-community.org Reporter: baptiste.wojtkowski(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl A new functionality to automatically add a new MARC link has been added in Bug 29560 (Add option to create MARC links when adding items to bundles). There should also be a way to remove this link when removing the item from the bundle. Moreover I don't know if this should be a second bug but since the link is not removed, there can be multiple links if the item is added to a new bundle. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 30

[Bug 29033] New: Add C4::Context->multivalue_preference
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29033 Bug ID: 29033 Summary: Add C4::Context->multivalue_preference 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: tomascohen(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org I've seen many places in which a syspref is read, and then split used to convert the contents into a list. It seems it would be simple to add a method to do that. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 46

[Bug 43336] New: Remove support for LibrisSpellcheck
by bugzilla-daemon＠bugs.koha-community.org 12 Oct '26

12 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43336 Bug ID: 43336 Summary: Remove support for LibrisSpellcheck Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: magnus(a)libriotech.no QA Contact: testopia(a)bugs.koha-community.org Target Milestone: --- The Swedish National Library has discontinued the "bibspell" API that used to live at http://api.libris.kb.se/bibspell/ on which the LibrisSpellcheck feature was built, so we need to remove this feature from Koha. This should include at least: - The code - The sysprefs LibrisKey and LibrisURL - Settings for LibrisSpellcheck in the syspref OPACdidyoumean MariaDB [koha_kohadev]> select * from systempreferences where variable = 'opacdidyoumean'\G *************************** 1. row *************************** variable: OPACdidyoumean value: [{ "name": "AuthorityFile"}, { "name": "ExplodedTerms"}, { "name": "LibrisSpellcheck"}] options: NULL explanation: Did you mean? configuration for the OPAC. Do not change, as this is controlled by /cgi-bin/koha/admin/didyoumean.pl. type: Free $ grep -rn LibrisSpell * Koha/SuggestionEngine/Plugin/LibrisSpellcheck.pm:1:package Koha::SuggestionEngine::Plugin::LibrisSpellcheck; Koha/SuggestionEngine/Plugin/LibrisSpellcheck.pm:27: return 'LibrisSpellcheck'; Koha/SuggestionEngine/Plugin/LibrisSpellcheck.pm:68:Koha::SuggestionEngine::Plugin::LibrisSpellcheck koha-tmpl/intranet-tmpl/prog/en/modules/admin/didyoumean.tt:24: [% CASE 'LibrisSpellcheck' %] -- You are receiving this mail because: You are watching all bug changes.

1 35

[Bug 43041] Uncaught exceptions and DBI errors are not timestamped in plack-error.log
by bugzilla-daemon＠bugs.koha-community.org 11 Oct '26

11 Oct '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43041 --- Comment #7 from David Cook <dcook(a)prosentient.com.au> --- Yeah having those timestamps would've been really handy Monday morning heh -- You are receiving this mail because: You are watching all bug changes.

1 0