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

[Bug 30119] New: Additional fields in acquisitions should not automatically generate an entry called "All"
by bugzilla-daemon＠bugs.koha-community.org 17 Feb '22

17 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30119 Bug ID: 30119 Summary: Additional fields in acquisitions should not automatically generate an entry called "All" Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Acquisitions Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org When linking an additional field for a basket to an authorised value there is a "all" entry generated instead of an empty one. This makes sense on the search form, but not when entering the data. For some values "all" just doesn't fit. Like for a list of priorities or a list of faculties ordering. It would be better to have an empty entry here and keep the all option only for search. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 29865] New: Wrong includes in circ/returns.pl
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29865 Bug ID: 29865 Summary: Wrong includes in circ/returns.pl Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: minor 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 Super trivial, duplicated lines. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 12

[Bug 11750] Overdue report does not limit patron attributes
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11750 Andrew Fuerste-Henry <andrew(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Pushed to oldstable CC| |andrew(a)bywatersolutions.com Version(s)|22.05.00,21.11.03 |22.05.00,21.11.03,21.05.11 released in| | --- Comment #9 from Andrew Fuerste-Henry <andrew(a)bywatersolutions.com> --- Pushed to 21.05.x for 21.05.11 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 25670] New: Add a method to search by ordernumber on transfered ones as well to K:A:Orders
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25670 Bug ID: 25670 Summary: Add a method to search by ordernumber on transfered ones as well to K:A:Orders Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Acquisitions Assignee: koha-bugs(a)lists.koha-community.org Reporter: tomascohen(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org This is something that happens in C4::Acquisition::SearchOrders and is needed to be used on the API controller. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 39

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 The Minh Luong <the-minh.luong(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 The Minh Luong <the-minh.luong(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |the-minh.luong(a)inlibro.com --- Comment #61 from The Minh Luong <the-minh.luong(a)inlibro.com> --- I have tested before applying the patch and everything seems to work WITHOUT it. The tests seems to work fine without the patch, but one fail when applying the patch. Here is what I did: Before applying the patch: 1) Select a user with active restrictions (manual restriction works) and note his category. 2) Make sure finesday=0 for the user's category noted in step 1. 3) Checkout and return an item (not overdue). A previous restriction reminder will appear (Message in yellow box) 4) Checkout and return an overdue item (change the date at checkout) A previous restriction reminder will also appear (Message in yellow box) 5) Run prove -l t/db_dependent/Circulation.t (ALL tests should pass) After applying patch: Same steps from 1 to 5 , but at step 5, one test fails (AddReturn + CumulativeRestrictionPeriods). -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29820] New: Print summary just show 20 items
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Bug ID: 29820 Summary: Print summary just show 20 items Change sponsored?: --- Product: Koha Version: 21.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: mhby87(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Created attachment 129190 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129190&action=e… List print summary After upgrade from 21.05 to 21.11, print summary for check out items just show 20 rows only. For example, patron borrow 500 item, but print summary show 20 items list. How to show all check out items in patron print summary. Thank you -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 13

[Bug 29889] New: Incorrect library check in patron message deletion logic
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29889 Bug ID: 29889 Summary: Incorrect library check in patron message deletion logic 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: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Depends on: 21757 If you DON'T have AllowAllMessageDeletion enabled, message deletion is supposed to be limited by library: Staff from the same library can delete each other's messages, but not messages left by other libraries. At some point the logic around the display of the "Delete" link got changed, I think by Bug 21757. Now the check compares the branchcode of the staff member who left the message with the branchcode of the patron: patron_message.branchcode == patron.branchcode I think this is meant to compare patron_message.branchcode to the logged-in library. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21757 [Bug 21757] Patron detail script (moremember.pl) cleanup -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 15

[Bug 10756] Carousel display of new titles on OPAC home page
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10756 Ray Delahunty <r.delahunty(a)arts.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |r.delahunty(a)arts.ac.uk --- Comment #40 from Ray Delahunty <r.delahunty(a)arts.ac.uk> --- As part of the wishlist for when the carousel might be coded into the community version of Koha may I ask that the code deal well with keyboard navigation. We are testing functionality of the 21.11 Koha OPAC on our upgrade server with minimal customisations other than colours etc., and find that the keyboard navigation jumps from the links on the left hand side of the OPAC page into individual titles of records in the carousel, rather than into the carousel heading. This means repeated tabbing to move from the carousel to the next OPAC feature. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 30017] New: Should we prefer iterator.first over calling variable.count to reduce our DB hits
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '22

16 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30017 Bug ID: 30017 Summary: Should we prefer iterator.first over calling variable.count to reduce our DB hits 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: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org I noticed a pattern recently whereby we often call a count prior to iterating over the resultset: [% IF things.count %] <table> . . . [% FOREACH thing IN things %] <td>[% thing.whatever %]</td> [% END %] </table> [% END %] This introduces a count(*) DB call. We could instead use the first iteration of the iterator to setup the boilerplate and reduce our DB hits. [% FOREACH thing IN things %] [% IF thing.first %] <table> [% END %] <td>[% thing.whatever %]</td> [% IF thing.last %] </table> [% END %] [% END %] -- You are receiving this mail because: You are watching all bug changes.

1 11