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

[Bug 13990] ILS-DI LookupPatron Requries ID Type
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13990 Jon Knight <J.P.Knight(a)lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |In Discussion CC| |J.P.Knight(a)lboro.ac.uk --- Comment #2 from Jon Knight <J.P.Knight(a)lboro.ac.uk> --- I assume the obvious fix to this is to say that the id_type is required rather than optional in the POD? Alternative there could be a set of well defined id_types that are cycled through to find matches (that would be a pretty easy bit of code). On a similar note, we've noticed that "email" also appears to be a valid id_type but isn't in the POD list. A quick look in the code (hoorah for open source!) shows that any field in the borrower table can be looked up with this call. For example "phone", "phonepro" or "mobile" let me find myself using my primary, secondary and "other" phone number. This begs the question: should Koha be constraining the list of id_type's that can be passed into this routine from the client? Or is being able to search on any column in the borrowers table regarded as a useful feature (we've found "email" to be handy for sure). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 14180] Make "Always show checkouts immediately" a global setting
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14180 Séverine Queune <severine.queune(a)bulac.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |severine.queune(a)bulac.fr --- Comment #10 from Séverine Queune <severine.queune(a)bulac.fr> --- We added this code on 'IntranetUserJS' syspref and it works perfectly for us :) (Found on Biblibre's documentation) $(document).ready(function() { if ( typeof script !== 'undefined' && ! $.cookie("issues-table-load-immediately-" + script) ) { $.cookie("issues-table-load-immediately-" + script, true, { expires: 365 }); } }); -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 --- Comment #185 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- (In reply to Jonathan Druart from comment #182) > 1/ Error handling on borrowernumber when creating or updating a request > (backend issue?) This is definitely a backend issue. The Dummy backend does no check, it just passes the borrowernumber from the UI to the Koha::Illrequest->store method with no error catching. The FreeForm plugin does some checks before the ->store, though. Out of the scope of the bug anyway. > 6/ Should not we use message_queue instead of sendmail directly? There's a bug filed for this: bug 18588. > 7/ Maybe we should add a warning to the about page if the partner_code > defined in the config does not match a category_code (?) Done in one of the recent followups. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 --- Comment #184 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 68989 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68989&action=ed… Bug 7317: (followup) Make tests independent of data on the DB Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 --- Comment #183 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 68988 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68988&action=ed… Bug 7317: Add partner_code checks to abot page This patch adds checks to the configured (or fallback) partner_code. It produces two different warnings: - Configured partner_code doesn't match an existing patron category code. - partner_code is not defined, a fallback to ILLLIBS is done Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15522] New interface for revamped circulation rules
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15522 Patrick Robitaille <patrick.robitaille(a)collecto.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |patrick.robitaille@collecto | |.ca -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7882] Add ability to move and reorder fields in MARC editor
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7882 --- Comment #43 from Marjorie Barry-Vila <marjorie.barry-vila(a)collecto.ca> --- (In reply to Kyle M Hall from comment #38) > Created attachment 24228 [details] [review] > Bug 7882: Inputs should be editable > > disableSelection is deprecated and cause an issue on editing an input > (input, select, textarea, etc.) > see http://api.jqueryui.com/disableselection/ > > Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> Hi Kyle, Do you think you can rebase the patch for the master version? Regards, Marjorie -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 10748] Add option to block return of lost items
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10748 --- Comment #13 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Marcel de Rooy from comment #9) > Adding a comment in a follow-up. Thanks a lot Marcel! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13915] Inventory tool no longer hides items with date last seen greater than the inventory date
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13915 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy(a)rijksmuseum.nl Status|Failed QA |RESOLVED Resolution|--- |FIXED --- Comment #6 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Seems this could be closed in favor of bugs 12913/14399. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13560] need an add option in marc modification templates
by bugzilla-daemon＠bugs.koha-community.org 06 Nov '17

06 Nov '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13560 Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply CC| |victor.grousset(a)biblibre.co | |m --- Comment #7 from Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> --- Conflict when applying the patch. In t/db_dependent/MarcModificationTemplates.t <<<<<<< ff82c9aadc441f9ca18dcad2e58783e733cdd839 (from November 3 on master) use Test::More tests => 96; ||||||| merged common ancestors use Test::More tests => 95; ======= use Test::More tests => 114; >>>>>>> Bug 13560 - need an add option in marc modification templates (this patch's version) -- You are receiving this mail because: You are watching all bug changes.

1 0