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

[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

[Bug 22215] New: table_borrowers name is inconsistent with other tables
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22215 Bug ID: 22215 Summary: table_borrowers name is inconsistent with other tables Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Circulation 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, kyle.m.hall(a)gmail.com Target Milestone: --- This table should be renamed more like patrons-table or patron-results-table -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 10

[Bug 17353] New: Add phone number column to checkout search
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17353 Bug ID: 17353 Summary: Add phone number column to checkout search 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: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 26

[Bug 33612] New: Staff with limited permissions don't always get an expanded catalog_search bar
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '24

16 Apr '24

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33612 Bug ID: 33612 Summary: Staff with limited permissions don't always get an expanded catalog_search bar 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: lucas(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com To recreate: 1. Setup a staff member with staff access permission but no circulate permissions. 2. Log in as that and on the intranet-home page notice that 'Search patron' bar across the top is not visible by default. 3. Click the 'Search patrons' link at the top, now you see the search bar. 4. This is also true on the item search page. It is because we are using the old home-search.inc when it should be the cat-search.inc If you git grep 'home-search.inc' you'll see it on these two pages. koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt: [% INCLUDE 'home-search.inc' %] koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt: [% INCLUDE 'home-search.inc' %] We can change that to cat-search.inc and remove the unused home-search.inc, I think. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 13