July 2016 - Koha-bugs - lists.koha-community.org

[Bug 16855] New: Poor performance due to high overhead of SQL call in export.pl
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16855 Bug ID: 16855 Summary: Poor performance due to high overhead of SQL call in export.pl Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Tools Assignee: gmcharlt(a)gmail.com Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org Export.pl uses Koha::Biblioitems->search() to retrieve biblionumbers for the export process. The SQL code generated by this does a full dump of the the biblioitems table including the marcxml for each record. This table can be quite large and on smaller servers can actually cause debian to fire oom-killer on the process. As the results of the search are only used to push biblionumbers into an array it might make better sense to do a direct SQL query and not retrieve the marcxml column -- You are receiving this mail because: You are watching all bug changes.

1 12

[Bug 16854] New: request.tt: Logic to display messages broken
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16854 Bug ID: 16854 Summary: request.tt: Logic to display messages broken 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: veron(a)veron.ch QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com The logic to display messages is broken in koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt To verify, examin code in line 295 and following: [% IF ( expiry || diffbranch ) %] This hides messages after line 301: [% IF restricted %] ...and after line 304: [% IF amount_outstanding && Koha.Preference('maxoutstanding') && amount_outstanding > Koha.Preference('maxoutstanding') %] -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 12

[Bug 16459] New: Adding patrons to a patron card label batch requires 'routing' permission
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16459 Bug ID: 16459 Summary: Adding patrons to a patron card label batch requires 'routing' permission Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Label/patron card printing Assignee: cnighswonger(a)foundations.edu Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org When adding patrons to a patron card batch using the search for patrons feature, the required permission is 'routing'... which makes no sense. -- You are receiving this mail because: You are watching all bug changes.

1 15

[Bug 16772] New: Change label from 'For:' to 'Library:' to ease translation
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16772 Bug ID: 16772 Summary: Change label from 'For:' to 'Library:' to ease translation 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: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org Change the label from "For:" to "Library:" in the acquisition information filter on the suggestions page in staff. -- You are receiving this mail because: You are watching all bug changes.

1 10

[Bug 16770] New: Remove wrong uses of Memoize::Memcached
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16770 Bug ID: 16770 Summary: Remove wrong uses of Memoize::Memcached Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement 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 >From C4/languages.pm: 32 eval { 33 if (C4::Context->ismemcached) { 34 require Memoize::Memcached; 35 import Memoize::Memcached qw(memoize_memcached); 36 37 memoize_memcached('getTranslatedLanguages', memcached => C4::Context->memcached); 38 memoize_memcached('getFrameworkLanguages' , memcached => C4::Context->memcached); 39 memoize_memcached('getAllLanguages', memcached => C4::Context->memcached); 40 } 41 }; Looking at this code, you might think these subroutines are cached, but actually they are not. The eval surrounding the code hides a bug, if you remove it, you will get: Invalid memcached argument (expected a hash) -- You are receiving this mail because: You are watching all bug changes.

1 23

[Bug 16751] New: What is sitemaper?
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16751 Bug ID: 16751 Summary: What is sitemaper? Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes.

1 12

[Bug 16697] New: Translatability: Fix problem with isolated "'s"in request.tt
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16697 Bug ID: 16697 Summary: Translatability: Fix problem with isolated "'s"in request.tt Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: I18N/L10N Assignee: koha-bugs(a)lists.koha-community.org Reporter: veron(a)veron.ch QA Contact: testopia(a)bugs.koha-community.org CC: frederic(a)tamil.fr In koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt, an isolated genitiv-'s leads to translation issues. Line 296 should be re-worded: <li><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowerfirstname %] [% borrowersurname %]</a>'s <strong>account has expired</strong></li> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 11

[Bug 16742] New: Dead code (?) - catalogue/subject.tt seems never to be used
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16742 Bug ID: 16742 Summary: Dead code (?) - catalogue/subject.tt seems never to be used Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: veron(a)veron.ch QA Contact: testopia(a)bugs.koha-community.org It seems that catalogue/subject.tt is never used. There is a call in catalogue/moredetail.pl line 52 depending on a param 'subject', but catalogue/moredetail.pl is never called with such param. See also FIXME comment on line 45 and comment 2 in Bug 15927: Catalog -> subject.tt (is this template used? -- You are receiving this mail because: You are watching all bug changes.

1 16

[Bug 16644] New: Plack: Use to_app to remove warning about Plack::App::CGIBin instance
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16644 Bug ID: 16644 Summary: Plack: Use to_app to remove warning about Plack::App::CGIBin instance 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: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org Using Plack with the debian psgi file, I get lots of warnings like: WARNING: Automatically converting Plack::App::CGIBin instance to a PSGI code reference. If you see this warning for each request, you probably need to explicitly call to_app() i.e. Plack::App::CGIBin->new(...)->to_app in your PSGI file. This patch is aimed to eliminate the warns. -- You are receiving this mail because: You are watching all bug changes.

1 19

[Bug 16462] New: Change default sorting of circulation patron search results to patron name
by bugzilla-daemon＠bugs.koha-community.org 16 Aug '16

16 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16462 Bug ID: 16462 Summary: Change default sorting of circulation patron search results to patron name Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: oleonard(a)myacpl.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: 16336 At some point in the past the sorting of patron search results in circulation (checkout and placing holds) changed from being sorted by name to being sorted by card number. I think sorting by name is more useful in these interfaces. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16336 [Bug 16336] UX of holds patron search with long lists of results -- You are receiving this mail because: You are watching all bug changes.

1 12