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

[Bug 35942] OPAC user can enroll several times to the same club
by bugzilla-daemon＠bugs.koha-community.org 07 Jul '24

07 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35942 Phil Ringnalda <phil(a)chetcolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |37031 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37031 [Bug 37031] Club enrollment doesn't complete in staff interface -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 5561] Help on summary field in authority type editor
by bugzilla-daemon＠bugs.koha-community.org 06 Jul '24

06 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5561 Phil Ringnalda <phil(a)chetcolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |phil(a)chetcolibrary.org --- Comment #7 from Phil Ringnalda <phil(a)chetcolibrary.org> --- Sort of still valid, but not exactly. At first the feature applied to both MARC21 and UNIMARC, but then it became UNIMARC only. Janusz wanted the documentation to say that it was UNIMARC only, but now the documentation no longer exists at all. Which is fine, it was wrong. At one point you could only have four characters (not three) before and after the field number and subfield code, but that limit is gone, and it was never limited to digits, because the whole point was punctuation. You can see examples by looking at a UNIMARC install, because we actually ship it by default for UNIMARC. Administration - Authority types, for any type other than Default look at the Summary column. It's an attempt at dealing with the fact that UNIMARC tends to use multiple subfields, and to not put the ISBD punctuation that goes between them into the authority record. So while a MARC21 authority would have 100$a as "Smith, Bessie," and 100$d as "1894-1937" and if you just stick them together with a space between, you get the correct punctuation, UNIMARC would have 200$a "Smith" 200$b "Bessie" 200$f "1894-1937" so just sticking them together doesn't look good, or correct, since correct for UNIMARC would be "Smith, Bessie (1894-1937)". So in a UNIMARC install the default Summary for Personal Name is the template "[200a][, 200b][ 200d][ ; 200c][ (200f)]" which says to show 200$a then if there's a 200$b show it after comma space, then if there's a 200$d show it after a space, then if there's a 200$c show it after space semicolon space, then if there's a 200f show it after space ( with a ) after it. That's all fine, it creates a properly punctuated heading, but it doesn't go far enough, and nothing stops the default summary that was intended to only be created when there's no template from showing. The default summary is a link to the authority details page wrapped around the heading, followed by (on new lines) see-from, see-also, and other scripts. The other thing that's untrue in what the former documentation said is "things outside [] are kept as is (including HTML)" because now we escape anything and everything, lots of things twice. So you can't imitate the link to details in a Summary template, and if you want to include see-from, see-also, other-scripts, and notes, you're going to have to do it all in one run-on line. And there's also currently no way to hide the default summary even if you could completely replace it (other than hiding it with CSS, and that may be what people supporting UNIMARC installs do instead of upstreaming a fix). Once upon a time, we only built the default one if you didn't have a template, but now we build them both. And although jcamins did see that showing both was silly, and he added an "UNLESS ( summary.summaryonly )" block to allow the default one to be hidden, he must have then realized there wasn't any way to actually replace it, because he didn't add any code that would ever set summary.summaryonly. Sorry, I don't know how to distill that down to a manual entry. "If you like the better punctuated version and can live with having to go to the Actions menu to get to MARC preview, you might like to have something like #authorities_searchresultlist_results .authorizedheading { display: none; } in your IntranetUserCSS and something similar in OPACUserCSS so the unpunctuated version doesn't show."? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 36701] New: Adjust hold confirmation to avoid showing empty div
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36701 Bug ID: 36701 Summary: Adjust hold confirmation to avoid showing empty div Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Templates Assignee: emily.lamancusa(a)montgomerycountymd.gov Reporter: emily.lamancusa(a)montgomerycountymd.gov QA Contact: testopia(a)bugs.koha-community.org Depends on: 33174 Bug 33174 added a warning when cancelling multiple holds from request.pl, clearly indicating how many holds would be cancelled. The added div is empty when directly cancelling a single hold (by clicking the FA trash can icon next to it), so it makes sense to hide it in that case. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33174 [Bug 33174] Have better indication when one is cancelling multiple holds on a record -- You are receiving this mail because: You are watching all bug changes.

1 16

[Bug 26793] New: Hidden 'more" menu if many informations displayed on header
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26793 Bug ID: 26793 Summary: Hidden 'more" menu if many informations displayed on header Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: severine.queune(a)bulac.fr QA Contact: testopia(a)bugs.koha-community.org Created attachment 112228 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112228&action=e… Hidden 'More' menu If there are many informations in the header (long librarian name, using desk option, add some local dropdown menu, etc.), the "More" menu is hidden by librarian name and can't be use. See screenshot attached. -- You are receiving this mail because: You are watching all bug changes.

1 2

[Bug 34478] Full CSRF protection
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34478 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |37264 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37264 [Bug 37264] Fix delete button on staff interface's suggestion detail page -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 36786] New: (Bug 31791 follow-up) Koha explodes when trying to edit a biblio with invalid biblionumber
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36786 Bug ID: 36786 Summary: (Bug 31791 follow-up) Koha explodes when trying to edit a biblio with invalid biblionumber Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Cataloging Assignee: januszop(a)gmail.com Reporter: januszop(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl After changes made to addbiblio with the bug 31791, when trying to open the editor with biblionumber=<invalid_number> (e.g. a deleted biblionumber) Koha explodes with a message << Can't call method "can_be_edited" on an undefined value at /kohadevbox/koha/cataloguing/addbiblio.pl line 555 >> -- You are receiving this mail because: You are watching all bug changes.

1 22

[Bug 33739] New: ModItemTransfer triggers indexing twice
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33739 Bug ID: 33739 Summary: ModItemTransfer triggers indexing twice 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 With the changes from bug 31212, it is happening again, because the implicit call to `ModDateLastSeen` is not passed the skip_record_index flag. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 17

[Bug 33043] New: Use process_tt in SIP modules
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33043 Bug ID: 33043 Summary: Use process_tt in SIP modules 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: kyle(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org Bug 33030 implements a new helper subroutine to standardize processing of Template Toolkit syntax outside slips and notices. We should use this subroutine in the various parts of the SIP server code where we are currently using Template::process directly. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 19

[Bug 36419] New: Ambiguous return value in GetPlugins
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36419 Bug ID: 36419 Summary: Ambiguous return value in GetPlugins Change sponsored?: --- Product: Koha Version: master 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 Bug 35536 introduced some better error handling, but made the return value more erratic. We you previously had code like this: ```perl my @plugins = Koha::Plugins->new()->GetPlugins( { method => 'ill_backend', metadata => { name => $backend_id }, all => 1, errors => 1 } ); foreach my $plugin (@plugins) { ... } ``` And now the new return value for GetPlugins is: ```perl return $errors ? ( \@plugins, \@failing ) : @plugins; ``` This means the caller will need to check the return value, and if the arity is two (?), check that the second element has some specific structure so we determine... it is not actually a plugin! I propose we return a hashref with the plugins and (possibly) the errors. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 12

[Bug 34427] New: Fix responsive table style in the OPAC for items tables
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '24

05 Jul '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34427 Bug ID: 34427 Summary: Fix responsive table style in the OPAC for items tables Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: fridolin.somers(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org Like Bug 31248 fixed some tables in OPAC, items (holdings) tables needs to be fixed to be responsive like before. -- You are receiving this mail because: You are watching all bug changes.

1 8