September 2017 - Koha-bugs - lists.koha-community.org

[Bug 19392] New: auth_values_input_www.t does not clean up
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19392 Bug ID: 19392 Summary: auth_values_input_www.t does not clean up Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)bugs.koha-community.org Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org The two authorised value categories are not removed. -- You are receiving this mail because: You are watching all bug changes.

1 7

[Bug 18422] New: Add Select2 to authority editor
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18422 Bug ID: 18422 Summary: Add Select2 to authority editor Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: gmcharlt(a)gmail.com Reporter: synapse.ova(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl Overview: Repeat tag fails if authority field has select subfield (for example, UNIMARC 700$8, 800$a) Steps to Reproduce: In authority editor repeat field that has select subfield Actual Results: Field does not repeat (copy is not created). Console shows a js TypeError in cataloging.js: «$(...).select2 is not a function» Expected Results: Field will repeat (copy is created) Additional Information: Error happens in version 16.11+ after adding Select2 js functions. The easiest way to fix is to add Select2 to authority editor -- You are receiving this mail because: You are watching all bug changes.

1 14

[Bug 19344] New: DB fields login_attempts and lang may be inverted
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19344 Bug ID: 19344 Summary: DB fields login_attempts and lang may be inverted Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jonathan.druart(a)bugs.koha-community.org Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org Due to a bad rebase, the borrowers and deletedborrowers table structure may different from a new install and upgraded install -- You are receiving this mail because: You are watching all bug changes.

1 12

[Bug 19317] New: Move of checkouts - Remove leftover
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19317 Bug ID: 19317 Summary: Move of checkouts - Remove leftover Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jonathan.druart(a)bugs.koha-community.org Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org Nothing important here, but this line should have been removed by bug 18966: 2177 # Update the fines 2178 $dbh->do(q|UPDATE accountlines SET issue_id = ? WHERE issue_id = ?|, undef, $old_checkout->issue_id, $issue->issue_id); The issue_id is now the same when moved from issues to old_issues. We do not need to update the accountlines table -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 18999] New: Acq: Shipping cost not included in total spent on acq home and funds page
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18999 Bug ID: 18999 Summary: Acq: Shipping cost not included in total spent on acq home and funds page Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major 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 Both the table on the acq start page and the table on the funds page don't include the shipping costs. This means, that the total spent is not correct. You can easily check this by comparing with the full list shown when clicking on the spent amount from the acq start page table. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 34

[Bug 19135] New: AllowHoldsOnPatronsPossessions is not working
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19135 Bug ID: 19135 Summary: AllowHoldsOnPatronsPossessions is not working 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: kyle(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com Depends on: 14695 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14695 [Bug 14695] Add ability to place multiple item holds on a given record per patron -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 24

[Bug 19329] New: The automatic slip printing does not work when IntranetSlipPrinterJS defined
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19329 Bug ID: 19329 Summary: The automatic slip printing does not work when IntranetSlipPrinterJS defined Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: josef.moravec(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes.

1 18

[Bug 19296] New: Tax is being subtracted from orders when vendor price does not include gst and ordering from a file
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19296 Bug ID: 19296 Summary: Tax is being subtracted from orders when vendor price does not include gst and ordering from a file Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Acquisitions Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org In the case of vendor prices don't include gst we are removing the gst from the order price, passing it to populate_order_with_prices and calculating tax on the price with tax removed! Example: Order with price of $10 and tax rate of 10% We calculate ecost/rrp at 9.09 Then calculate tax on 9.09 We should be calculating tax on $10 In addorderiso2707.pl: 270 if ( $bookseller->listincgst ) { 271 if ( $c_discount ) { 272 $orderinfo{ecost} = $price; 273 $orderinfo{rrp} = $orderinfo{ecost} / ( 1 - $c ); 274 } else { 275 $orderinfo{ecost} = $price * ( 1 - $c ); 276 $orderinfo{rrp} = $price; 277 } 278 } else { 279 if ( $c_discount ) { 280 $orderinfo{ecost} = $price / ( 1 + $orderinfo{tax_rate} ); 281 $orderinfo{rrp} = $orderinfo{ecost} / ( 1 - $c ); 282 } else { 283 $orderinfo{rrp} = $price / ( 1 + $orderinfo{tax_rate} ); 284 $orderinfo{ecost} = $orderinfo{rrp} * ( 1 - $c ); 285 } 286 } ... 296 297 %orderinfo = %{ 298 C4::Acquisition::populate_order_with_prices( 299 { 300 order => \%orderinfo, 301 booksellerid => $booksellerid, 302 ordering => 1, 303 receiving => 1, 304 } 305 ) 306 }; Then in C4 Acquisition: 2943 if ($ordering) { 2944 $order->{tax_rate_on_ordering} //= $order->{tax_rate}; 2945 if ( $bookseller->listincgst ) { 2946 # The user entered the rrp tax included 2947 $order->{rrp_tax_included} = $order->{rrp}; 2948 2949 # rrp tax excluded = rrp tax included / ( 1 + tax rate ) 2950 $order->{rrp_tax_excluded} = $order->{rrp_tax_included} / ( 1 + $order->{tax_rate_on_ordering} ); 2951 2952 # ecost tax excluded = rrp tax excluded * ( 1 - discount ) 2953 $order->{ecost_tax_excluded} = $order->{rrp_tax_excluded} * ( 1 - $discount ); 2954 2955 # ecost tax included = rrp tax included ( 1 - discount ) 2956 $order->{ecost_tax_included} = $order->{rrp_tax_included} * ( 1 - $discount ); 2957 } 2958 else { 2959 # The user entered the rrp tax excluded 2960 $order->{rrp_tax_excluded} = $order->{rrp}; 2961 2962 # rrp tax included = rrp tax excluded * ( 1 - tax rate ) 2963 $order->{rrp_tax_included} = $order->{rrp_tax_excluded} * ( 1 + $order->{tax_rate_on_ordering} ); 2964 2965 # ecost tax excluded = rrp tax excluded * ( 1 - discount ) 2966 $order->{ecost_tax_excluded} = $order->{rrp_tax_excluded} * ( 1 - $discount ); 2967 2968 # ecost tax included = rrp tax excluded * ( 1 - tax rate ) * ( 1 - discount ) 2969 $order->{ecost_tax_included} = 2970 $order->{rrp_tax_excluded} * 2971 ( 1 + $order->{tax_rate_on_ordering} ) * 2972 ( 1 - $discount ); 2973 } 2974 2975 # tax value = quantity * ecost tax excluded * tax rate 2976 $order->{tax_value_on_ordering} = 2977 $order->{quantity} * $order->{ecost_tax_excluded} * $order->{tax_rate_on_ordering}; 2978 } -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 19

[Bug 18742] New: Circulation statistics wizard no longer exports the total row
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18742 Bug ID: 18742 Summary: Circulation statistics wizard no longer exports the total row Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Reports Assignee: gmcharlt(a)gmail.com Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org It seems like a few lines were lost from issues-stats.pl in bug 7679: # footer print "TOTAL"; $cols = @$results[0]->{loopfooter}; print map {$sep.$_->{totalcol}} @$cols; print $sep.@$results[0]->{total}; -- You are receiving this mail because: You are watching all bug changes.

1 19

[Bug 18389] New: bulkmarcimport.pl does not take notice of MARC modification templates
by bugzilla-daemon＠bugs.koha-community.org 28 Oct '17

28 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18389 Bug ID: 18389 Summary: bulkmarcimport.pl does not take notice of MARC modification templates Change sponsored?: --- Product: Koha Version: 16.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Bibliographic record staging/import Assignee: gmcharlt(a)gmail.com Reporter: J.P.Knight(a)lboro.ac.uk QA Contact: testopia(a)bugs.koha-community.org Created attachment 61907 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61907&action=ed… Patch for bulkmarcimport.pl When importing large numbers of MARC records from a legacy LMS to Koha we needed to convert some of 852 fields to 952 fields for item holdings. I'd created MARC modification templates in the web administration front end, but the bulkmarcimport.pl migration tool doesn't use these. Joubu on the #koha IRC channel told me that this was because bulkmarcimport.pl was written before C4::MarcModificationTemplates.pm Perl module was available. The change is actually relatively easy and I've attached a diff patch to this case for others to contemplate and critic. -- You are receiving this mail because: You are watching all bug changes.

1 40