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

[Bug 19460] New: Limit total number of checkouts allowed per patron by patron category code
by bugzilla-daemon＠bugs.koha-community.org 28 Feb '19

28 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19460 Bug ID: 19460 Summary: Limit total number of checkouts allowed per patron by patron category code Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: kyle(a)bywatersolutions.com Reporter: kyle(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org Goal of development: Limit total number of checkouts allowed per patron by patron category code Work to be done: 1) Add new circulation rule "Max checkouts ( global, most restrictive )" to circulation rules. 2) Modify C4::Circulation::TooMany to respect this new rule 3) Modify script and template to show errors relating to this new rule -- You are receiving this mail because: You are watching all bug changes.

1 1

[Bug 19778] New: Move template JavaScript to the footer: Serials, part 4
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19778 Bug ID: 19778 Summary: Move template JavaScript to the footer: Serials, part 4 Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: testopia(a)bugs.koha-community.org Blocks: 17858 +++ This bug was initially created as a clone of Bug #19777 +++ In order to conform to best practices for performance staff client template javascript should be in the footer instead of the header. This patch will do so for even more of the serials templates. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17858 [Bug 17858] Omnibus: Move staff client JavaScript to the footer -- You are receiving this mail because: You are watching all bug changes.

1 10

[Bug 20174] New: Remove xml_sax.pl target from Makefile.pl
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20174 Bug ID: 20174 Summary: Remove xml_sax.pl target from Makefile.pl Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Installation and upgrade (command-line installer) Assignee: koha-bugs(a)lists.koha-community.org Reporter: dcook(a)prosentient.com.au QA Contact: testopia(a)bugs.koha-community.org A line was added to Makefile.PL with Bug #5033 but it doesn't seem like it's used anymore? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 12

[Bug 19832] New: SIP checkout removes extra hold on same biblio
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19832 Bug ID: 19832 Summary: SIP checkout removes extra hold on same biblio Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: SIP2 Assignee: koha-bugs(a)lists.koha-community.org Reporter: boutrosboutrosboutros(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: colin.campbell(a)ptfs-europe.com C4/SIP/ILS/Transaction/Checkout.pm includes the following bit of code which remove any hold that that matches the biblio and bilbionumber in checkout: # Fill any reserves the patron had on the item. # TODO: this logic should be pulled internal to AddIssue for all Koha. $debug and warn "pending_queue: " . (@$pending) ? Dumper($pending) : '[]'; foreach (grep {$_->{borrowernumber} eq $self->{patron}->{borrowernumber}} @$pending) { $debug and warn "Filling reserve (borrowernumber,biblionumber,reservedate): " . sprintf("(%s,%s,%s)\n",$_->{borrowernumber},$_->{biblionumber},$_->{reservedate}); ModReserveFill($_); # TODO: adjust representation in $self->item } As far as I can tell, this code is completely unnecessary, since C4::Circulation::AddIssue allread calls C4::Reserves::MoveReserve which fullfills the hold. What's more, if the borrower has multiple holds on the same biblio, all of them will be removed. This is clearly wrong. I purpose to remove the code above, unless someone can explain why SIP checkout should behave different than normal checkout in this case. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 16

[Bug 18114] New: koha-start-* and koha-stop-* should be replaced with the new command style
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18114 Bug ID: 18114 Summary: koha-start-* and koha-stop-* should be replaced with the new command style Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Packaging Assignee: koha-bugs(a)lists.koha-community.org Reporter: kyle(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: mirko(a)abunchofthings.net Commands like koha-start-zebra and koha-stop-sip should be replaced with koha-zebra --start and koha-sip --stop to emulate the newer style used in script like koha-plack -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 8

[Bug 17347] New: 'Renew' tab should ignore whitespace at begining and end of barcode
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17347 Bug ID: 17347 Summary: 'Renew' tab should ignore whitespace at begining and end of barcode Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com When you copy a barcode from somewhere else, sometimes there is whitespace at the beginning or the end. The checkout tab ignores this, the renew tab should as well. If you enter " validbarcode" the barcode is not found at the moment. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 16

[Bug 19994] New: use Modern::Perl in Authorities perl scripts
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19994 Bug ID: 19994 Summary: use Modern::Perl in Authorities perl scripts Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Authority data support Assignee: koha-bugs(a)lists.koha-community.org Reporter: aleisha(a)catalyst.net.nz QA Contact: testopia(a)bugs.koha-community.org use Modern::Perl should be used instead of use strict; use warnings; Relevant in: auth_finder.pl authorities-home.pl authorities-list.pl authorities.pl blinddetail-biblio-search.pl detail-biblio-search.pl detail.pl export.pl merge.pl merge_ajax.pl -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 18

[Bug 19920] New: changepasswordis exported from C4:: Members but has been removed
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19920 Bug ID: 19920 Summary: changepasswordis exported from C4::Members but has been removed 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: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org Depends on: 16853 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16853 [Bug 16853] Move changepassword to Koha::Patron->update_password -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 14

[Bug 20274] New: itemtypes.plugin report: not handling item-level_itypes syspref
by bugzilla-daemon＠bugs.koha-community.org 25 Feb '19

25 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20274 Bug ID: 20274 Summary: itemtypes.plugin report: not handling item-level_itypes syspref Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Reports Assignee: koha-bugs(a)lists.koha-community.org Reporter: victor.grousset(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org == Reproduce == 1. Use staff:/cgi-bin/koha/reports/manager.pl?report_name=itemtypes 2. Change the syspref item-level_itypes 3. One case should give a 0 result == Cause == There is a conditional but part of the SQL query was copy pasted without being changed. == Fix == "SELECT itemtype" → "SELECT biblioitems.itemtype" in these two lines https://gitlab.com/koha-community/Koha/blob/673a40d2acc5c8c58b7f3e7e6d11c32… https://gitlab.com/koha-community/Koha/blob/673a40d2acc5c8c58b7f3e7e6d11c32… (or maybe the other ones) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 16

[Bug 15894] New: Unify all account fine payment functions
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '19

22 Feb '19

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15894 Bug ID: 15894 Summary: Unify all account fine payment functions Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: kyle(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org Right now, Koha has four different functions for making payments: recordpayment, makepayment, recordpayment_sellectaccts and makepartialpayment. We should replace these four functions with a single all-singing all-dancing method in the Koha::Namespace. This would help to reduce regressions and make future additions much simpler ( such as my planned re-implementation of account offsets ) and add consistency for logging and whatnot. -- You are receiving this mail because: You are watching all bug changes.

1 34