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

[Bug 15407] Move the patron categories related code to Koha::Patron::Categories - part 2
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15407 --- Comment #32 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 50171 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=50171&action=ed… Bug 15407: Koha::Patron::Categories - replace C4::Category->all The C4::Category module contained only 1 method to return the patron categories available for the logged in user. The new method Koha::Patron::Categories->search_limited does exactly the same thing (see tests) and must be used in place of it. Test plan: - Same prerequisite as before For the following pages, you should not see patron categories limited to other libraries. - On the 'Item circulation alerts' admin page (admin/item_circulation_alerts.pl), modify the settings for check-in and checkout (NOTE: Should not we display all patron categories on this page? If yes, it must be done in another bug report to ease backporting it). - Search for patrons in the admin (budget) and acquisition (order) module. - On the patron home page (search form in the header) -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15407] Move the patron categories related code to Koha::Patron::Categories - part 2
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15407 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #47754|0 |1 is obsolete| | Attachment #47755|0 |1 is obsolete| | Attachment #47756|0 |1 is obsolete| | Attachment #47757|0 |1 is obsolete| | Attachment #47758|0 |1 is obsolete| | --- Comment #31 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 50170 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=50170&action=ed… Bug 15407: Koha::Patron::Categories - remove sql queries in some pl and pm This patch replaces sql queries done in some pl script and in C4::Reports::Guided. Since we have now a Koha::Patron::Categories module, we should use it where it is possible. Test plan: - Prerequisite: Be sure you have several patron categories created, with different option enabled, and limit some to certain libraries. - On the 'Circulation and fine rules' admin page (admin/smart-rules.pl), all the patron categories should be displayed (even the ones limited to another library), ordered by description. Try to add/update existing rules. - On the overdue rules page (tools/overduerules.pl), all the patron categories with overduenoticerequired set should be displayed. Try to add/update existing rules. - On the following reports: reports/borrowers_stats.pl reports/issues_avg_stats.pl The patron categories should be displayed. Note that there is an inconsistency with these 2 reports: the patron categories limited to other libraries are displayed on them, when they are not on the other reports. This should certainly be fixed (on another bug report). -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15407] Move the patron categories related code to Koha::Patron::Categories - part 2
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15407 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12478] Elasticsearch support for Koha
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 --- Comment #225 from Chris Cormack <chris(a)bigballofwax.co.nz> --- Heh that was meant to be a ? Not a ! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12478] Elasticsearch support for Koha
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 --- Comment #224 from Chris Cormack <chris(a)bigballofwax.co.nz> --- Have you tried with bug 16249! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 16144] New: Patron search fails when using some languages (eg: korean)
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16144 Bug ID: 16144 Summary: Patron search fails when using some languages (eg: korean) Change sponsored?: --- Product: Koha Version: 3.20 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Staff Client Assignee: koha-bugs(a)lists.koha-community.org Reporter: mtj(a)kohaaloha.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com I bumped into a rather strange and obscure bug when using the 'Korean' language in staff Here are the steps to replicate... 1/ set you lang to 'en' 2/ do a member search on any string (that returns multiple results) observe that results are displayed OK (see pic 1) 3/ set your language to Korean 4/ repeat step 2, and observe the following datatables error, (see pic2) "DataTables warning: table id=memberresultst - Requested unknown parameter 'dt_cardnumber' for row 0. For more information about this error, please see http://datatables.net/tn/4" I noticed that the error does not occur using the 'french' language (see pic 3) Bug discovered in 3.20.6 - i've not confirmed in master (yet) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 18

[Bug 12478] Elasticsearch support for Koha
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 --- Comment #223 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Jonathan Druart from comment #215) > Note that the tests are completely out of sync and do not pass. That will certainly be a blocker for QA. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13216] fix koha-shell to work with git/dev installs
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13216 --- Comment #16 from Mason James <mtj(a)kohaaloha.com> --- (In reply to Owen Leonard from comment #15) > I would like to test this but the test plan is confusing to me. Could you > revise it with some more specific instructions? For instance: > > run a "koha-shell -c 'which koha-*'" on a Koha tool, with -d arg > > The actual command would make a lot more sense to me. I did provide an actual command - its the following line, after the line you quoted 2/ run a "koha-shell -c 'which koha-*'" on a Koha tool, with -d arg # perl ~/koha-master/debian/scripts/koha-shell -d ~/koha-master -c 'which koha-list ' moof -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14242] Use ISBN-field to fill out purchase suggestions (using an Ajax-request)
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14242 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick(a)bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12478] Elasticsearch support for Koha
by bugzilla-daemon＠bugs.koha-community.org 13 Apr '16

13 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 --- Comment #222 from Chris Cormack <chris(a)bigballofwax.co.nz> --- The new_12478_elasticsearch now has all the patches signed off -- You are receiving this mail because: You are watching all bug changes.

1 0