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

[Bug 11999] Add two checks in CanBookBeReserved and CanItemBeReserved
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11999 Julian Maurice <julian.maurice(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #50224|0 |1 is obsolete| | --- Comment #44 from Julian Maurice <julian.maurice(a)biblibre.com> --- Created attachment 58741 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58741&action=ed… Bug 11999: Add two checks in CanBookBeReserved and CanItemBeReserved - Check if borrower already has reserved the same biblio (or item). - Check if borrower has reached the maximum number of holds allowed (syspref maxreserves) The goal of this patch is to do these checks also when using ILS-DI services HoldTitle and HoldItem Test plan: ---------- Before patch: 1/ Set syspref maxreserves to 1 2/ Place some holds through ILS-DI and note that you can place more than 1 hold for a borrower 3/ Try to hold the same biblio/item with the same borrower through ILS-DI and note that you can Apply the patch 4/ Place some holds through ILS-DI, you shouldn't be able to place more than 1 hold for a borrower 5/ Try to hold the same biblio/item with the same borrower through ILS-DI. You shouldn't be able to do that. 6/ Try to place holds on staff interface and OPAC. The behaviour must be identical than before the patch. Maxreserves and alreadyreserved works on ILSDI Maxreserves works also on staff and opac Already reserves works also on staff and opac tests on t/db_dependent/Reserves.t and t/db_dependent/Holds.t passe (using koha_ut db) Signed-off-by: Alex Sassmannshausen <alex.sassmannshausen(a)ptfs-europe.com> Signed-off-by: Alex Arnaud <alex.arnaud(a)biblibre.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11580] returnBeforeExpiry and closed day
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580 Claire Gravely <claire_gravely(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57562|0 |1 is obsolete| | --- Comment #2 from Claire Gravely <claire_gravely(a)hotmail.com> --- Created attachment 58740 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58740&action=ed… Bug 11580 - If returnBeforeExpiry is ON, use calendar to calculate return date not on closed day (depending on useDaysMode) Test plan: 1. ReturnBeforeExpiry is activated 2. useDaysMode is different from "circulation rules only" 3. Set expiry date of a patron to a near date 4. Set a closed day on calendar for this date 5. Do a checkout Without patch, return date will be patron expiration date With the patch, return date will be last open day before patron expiration day Signed-off-by: Claire Gravely <claire_gravely(a)hotmail.com> -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 11580] returnBeforeExpiry and closed day
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580 Claire Gravely <claire_gravely(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 14570] Make it possible to add multiple guarantors to a record
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14570 --- Comment #23 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 58739 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58739&action=ed… Bug 14570 - Followup Fix small errors in update child script -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8290] Replace usage of Greybox modals with Bootstrap modals
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8290 Josef Moravec <josef.moravec(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |josef.moravec(a)gmail.com -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14187] branchtransfer needs a primary key (id) for DBIx and common sense.
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14187 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58124|0 |1 is obsolete| | Attachment #58646|0 |1 is obsolete| | --- Comment #16 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 58730 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58730&action=ed… Bug 14187 - branchtransfer needs a primary key (id) for DBIx and common sense. * C4/Circulation.pm (GetTransfers, GetTransfersFromTo): Also return branchtransfer_id in return columns. * installer/data/mysql/atomicupdate/14187.perl: New file. * installer/data/mysql/kohastructure.sql: Modify branchtransfers structure. * t/db_dependent/Circulation/transfers.t: Update tests to expect branchtransfer_id. Signed-off-by: Owen Leonard <oleonard(a)myacpl.org> Test plan successful on all steps. Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> Amended patch: Remove Schema changes from this patch -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14187] branchtransfer needs a primary key (id) for DBIx and common sense.
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14187 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7710] multiple holds per title
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7710 --- Comment #91 from Olli-Antti Kivilahti <olli-antti.kivilahti(a)jns.fi> --- Since the other one went to master, I dont see a point in this bug. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14747] RESTful API with reverse proxy configuration
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14747 Bug 14747 depends on bug 14448, which changed state. Bug 14448 Summary: Hypnotoad and Nginx config for REST API https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14448 What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14448] Hypnotoad and Nginx config for REST API
by bugzilla-daemon＠bugs.koha-community.org 10 Jan '17

10 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14448 Olli-Antti Kivilahti <olli-antti.kivilahti(a)jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WONTFIX Status|NEW |RESOLVED --- Comment #11 from Olli-Antti Kivilahti <olli-antti.kivilahti(a)jns.fi> --- Hi! I managed to get starman, the plack-server to do hot-reloads, so we might not be interested in maintaining Hypnotoad. Community runs the REST API on starman and this is the most convenient thing to do. I tried to get rid of Apache by running only starman behind haproxy, but starman couldn't beat apache2 in static content delivery so we still have apace2 around. The less services we need to run Koha, the better. So this feature is not maintained by us. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 0