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

[Bug 36245] New: ILL - Custom backend form action is broken
by bugzilla-daemon＠bugs.koha-community.org 17 Apr '24

17 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36245 Bug ID: 36245 Summary: ILL - Custom backend form action is broken Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: ILL Assignee: pedro.amorim(a)ptfs-europe.com Reporter: pedro.amorim(a)ptfs-europe.com CC: pedro.amorim(a)ptfs-europe.com Depends on: 34478 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34478 [Bug 34478] Full CSRF protection -- You are receiving this mail because: You are watching all bug changes.

1 17

[Bug 36249] New: ILL - "Request from partners" action is broken
by bugzilla-daemon＠bugs.koha-community.org 17 Apr '24

17 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36249 Bug ID: 36249 Summary: ILL - "Request from partners" action is broken Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: ILL Assignee: pedro.amorim(a)ptfs-europe.com Reporter: pedro.amorim(a)ptfs-europe.com CC: david(a)davidnind.com, martin.renvoize(a)ptfs-europe.com, pedro.amorim(a)ptfs-europe.com, tomascohen(a)gmail.com, victor(a)tuxayo.net Depends on: 34478 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34478 [Bug 34478] Full CSRF protection -- You are receiving this mail because: You are watching all bug changes.

1 21

[Bug 36130] New: ILL batches table not showing all batches
by bugzilla-daemon＠bugs.koha-community.org 17 Apr '24

17 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36130 Bug ID: 36130 Summary: ILL batches table not showing all batches Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: ILL Assignee: koha-bugs(a)lists.koha-community.org Reporter: pedro.amorim(a)ptfs-europe.com CC: pedro.amorim(a)ptfs-europe.com Depends on: 30719 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30719 [Bug 30719] ILL should provide the ability to create batch requests -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 31

[Bug 35335] New: Circulation history tab in patron information causes 500 error
by bugzilla-daemon＠bugs.koha-community.org 17 Apr '24

17 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35335 Bug ID: 35335 Summary: Circulation history tab in patron information causes 500 error Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Patrons Assignee: stalkernoid(a)gmail.com Reporter: stalkernoid(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com To test: 1. Change syspref: RecordStaffUserOnCheckout to "record" 2. Choose a customer who has borrowed something before=has circulation history (e.g. customer 209 in koha-kktest https://koha-kktest.lib.helsinki.fi/cgi-bin/koha/mainpage.pl) and see that circulation history is can be seen. 3. Borrow something and see circulation history now -> got internal error (example customer id 232). 4. Change RecorsStaffUserOnCheckout to "don't record". 5. See the previous customer's circulation history now, should work as supposed. -- You are receiving this mail because: You are watching all bug changes.

1 20

[Bug 21104] New: oai.pl returns wrong baseURL
by bugzilla-daemon＠bugs.koha-community.org 17 Apr '24

17 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21104 Bug ID: 21104 Summary: oai.pl returns wrong baseURL Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P4 Component: Web services Assignee: koha-bugs(a)lists.koha-community.org Reporter: pablo.bianchi(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Blocks: 21102 Related/continuation of: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17785#c11 When using Plack Koha anwer a "cooler" (bug 18410) but invalid baseURL, /opac/oai.pl instead of given URI. As Tomas said on bug #17785: > Our fix for the XSLT masked the underlying bug (our code doesn't know > about the proxypass we do, and then Plack answers what he knows, > /opac/oai.pl). Originally deteced with official validator: https://www.openarchives.org/Register/ValidateSite Tested with: curl -s "baseURL?verb=Identify" | xpath -q -e "/OAI-PMH/Identify/baseURL" Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21102 [Bug 21102] OAI-PMH specification conformance [UMBRELLA] -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 8

[Bug 35481] New: Ignore whitespace when entering or scanning barcodes in trusted web checkout modal
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35481 Bug ID: 35481 Summary: Ignore whitespace when entering or scanning barcodes in trusted web checkout modal Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org Depends on: 30979 Like in other barcode fields we should ignore leading and trailing whitespace when entering barcodes for the new trusted self checkout feature. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30979 [Bug 30979] Add ability for OPAC users to checkout to themselves -- You are receiving this mail because: You are watching all bug changes.

1 1

[Bug 35482] New: Trusted self check modal too small for translations
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35482 Bug ID: 35482 Summary: Trusted self check modal too small for translations Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org Depends on: 30979 The modal for the trusted self checkout is quite small, but that doesn't work so well for languages with longer words (yes, German...). It would be nice if we made it slightly bigger to accomodate for variations like this. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30979 [Bug 30979] Add ability for OPAC users to checkout to themselves -- You are receiving this mail because: You are watching all bug changes.

1 3

[Bug 32711] New: Add biblio details to trusted self-checkout modal
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32711 Bug ID: 32711 Summary: Add biblio details to trusted self-checkout modal Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org We should add further biblio details to the summary table that appears in the OPAC trusted self-checkout modal feature. Title and callnumber for example -- You are receiving this mail because: You are watching all bug changes.

1 26

[Bug 35805] New: Two issues with HoldsQueue::MapItemsToHoldRequests
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35805 Bug ID: 35805 Summary: Two issues with HoldsQueue::MapItemsToHoldRequests Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs(a)lists.koha-community.org Reporter: andreas.jonsson(a)kreablo.se QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com 1. This clause from the below if statement must be incorrect: ( !$request->{itemtype} # If hold itemtype is set, item's itemtype must match || ( $request->{itemnumber} && ( $items_by_itemnumber{ $request->{itemnumber} }->{itype} eq $request->{itemtype} ) ) ) Because $request->{itemnumber} is guaranteed to be undefined due to this clause: next if defined($request->{itemnumber}); # already handled these The clause was probably ment to be this: ( !$request->{itemtype} || ( $items_by_itemnumber{ $item->{itemnumber} }->{itype} eq $request->{itemtype} ) ) foreach my $request (@$hold_requests) { last if $num_items_remaining == 0; next if $request->{allocated}; next if defined($request->{itemnumber}); # already handled these # look for local match first my $pickup_branch = $request->{branchcode} || $request->{borrowerbranch}; my ($itemnumber, $holdingbranch); my $holding_branch_items = $items_by_branch{$pickup_branch}; my $priority_branch = C4::Context->preference('HoldsQueuePrioritizeBranch') // 'homebranch'; if ( $holding_branch_items ) { foreach my $item (@$holding_branch_items) { next unless $items_by_itemnumber{ $item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } ); if ( $request->{borrowerbranch} eq $item->{$priority_branch} && _checkHoldPolicy($item, $request) # Don't fill item level holds that contravene the hold pickup policy at this time && ( !$request->{itemtype} # If hold itemtype is set, item's itemtype must match || ( $request->{itemnumber} && ( $items_by_itemnumber{ $request->{itemnumber} }->{itype} eq $request->{itemtype} ) ) ) && ( !$request->{item_group_id} # If hold item_group is set, item's item_group must match || ( $item->{_object}->item_group && $item->{_object}->item_group->id eq $request->{item_group_id} ) ) ) { $itemnumber = $item->{itemnumber}; last; } } $holdingbranch = $pickup_branch; 2. Then the code continues: } elsif ($transport_cost_matrix) { But surely this should be: } if (!defined $itemnumber && defined $transport_cost_matrix) { Because the important condition here is that we failed to allocate from the local holding branch. Surely the transport_cost_matrix should still be used just even if we tried allocating locally first, but failed? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 3

[Bug 24718] New: Introduce hour-based holds
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24718 Bug ID: 24718 Summary: Introduce hour-based holds Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs(a)lists.koha-community.org Reporter: aleisha(a)catalyst.net.nz QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com Some libraries would like to be able to specify the hours which an item will be reserved for (think: 'booking'), for example with 2-hour loans. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 95