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

[Bug 2499] Improvement on text wrapping algorithm needed
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2499 I'm just a bot <gitbot(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- When did the bot|2012-09-24 00:00:00 |2013-09-24 last check this| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2499] Improvement on text wrapping algorithm needed
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2499 I'm just a bot <gitbot(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gitbot(a)bugs.koha-community. | |org When did the bot| |2012-09-24 last check this| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2720] Overdues which debar automatically should undebar automatically when returned
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2720 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2720] Overdues which debar automatically should undebar automatically when returned
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2720 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #20658|0 |1 is obsolete| | --- Comment #40 from Kyle M Hall <kyle(a)bywatersolutions.com> --- Created attachment 21369 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21369&action=edit Bug 2720 - QA Followup 2 -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2720] Overdues which debar automatically should undebar automatically when returned
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2720 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #20657|0 |1 is obsolete| | --- Comment #39 from Kyle M Hall <kyle(a)bywatersolutions.com> --- Created attachment 21368 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21368&action=edit Bug 2720: Unit tests improvements Create the patron. Use a transaction. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2720] Overdues which debar automatically should undebar automatically when returned
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2720 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #20656|0 |1 is obsolete| | --- Comment #38 from Kyle M Hall <kyle(a)bywatersolutions.com> --- Created attachment 21367 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21367&action=edit Bug 2720 [FOLLOW-UP] Overdues which debar automatically should undebar automatically This follow-up to Bug 2720 makes some template changes which I think make the interface a little more consistent and streamlined. Instead of incorporating the manual entry form into the table it is broken out and follows standard form structure. This lets the table be hidden altogether if there are no existing restrictions. The manual entry form is hidden by default and shown when you click a link to add a manual restriction. These changes have been applied to both the include file used for circulation and patron detail and to the patron entry form template. To test, add and remove manual restrictions from the circulation page, the patron detail page, and the patron edit page. All operations should work correctly. Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2720] Overdues which debar automatically should undebar automatically when returned
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2720 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #20655|0 |1 is obsolete| | --- Comment #37 from Kyle M Hall <kyle(a)bywatersolutions.com> --- Created attachment 21366 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21366&action=edit Bug 2720 - Overdues which debar automatically should undebar automatically when returned This patch adds a more extensible and flexible debarments system to Koha. The fields borrowers.debarred and borrowers.debarredcomment are retained for compatibility and speed. This system supports having debarments for multiple reasons. There are currently three types of debarments: OVERDUES - Generated by overdue_notices.pl if the notice should debar a patron SUSPENSION - A punative debarment generated on checkin via an issuing rule MANUAL - A debarment created manually by a librarian OVERDUE debarments are cleared automatically when all overdue items have been returned, if the new system preference AutoRemoveOverduesRestrictions is enabled. It is disabled by default to retain current default functionality. Whenever a borrowers debarments are modified, the system updates the borrowers debarment fields with the highest expiration from all the borrowers debarments, and concatenates the comments from the debarments together. Test plan: 1) Apply patch 2) Run updatedatabase.pl 3) Verify the borrower_debarments table has been created and populated with the pre-existing debarments 4) Run t/db_dependent/Borrower_Debarments.t 5) Manually debar a patron, with an expiration date 6) Verify the patron cannot be issued to 7) Add another manual debarment with a different expiration date 8) Verify the 'restricted' message lists the date farthest into the future 9) Add another manual debarment with no expiration date 10) Verify the borrower is now debarred indefinitely 11) Delete the indefinite debarment 12) Verify the debarment message lists an expiration date dagain 13) Enable the new system preference AutoRemoveOverduesRestrictions 14) Set an overdue notice to debar after 1 day of being overdue 15) Check out an item to a patron and backdate the due date to yesterday 16) Run overdue_notices.pl 17) Verify the OVERDUES debarment was created 18) Return the item 19) Verify the OVERDUES debarment was removed 20) Disable AutoRemoveOverduesRestrictions 21) Repeat steps 15 though 18, verify the OVERDUES debarment was *not* removed 22) Add issuing rules so that an overdue item causes a temporary debarment 23) Check out an item to a patron and backdate the due date by a year 24) Return the item 25) Verify the SUSPENSION debarment was added to the patron Signed-off-by: Owen Leonard <oleonard(a)myacpl.org> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 10503] New: incorrect initialization parameters passed Memoize::Memcached
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10503 Bug ID: 10503 Summary: incorrect initialization parameters passed Memoize::Memcached Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: gmcharlt(a)gmail.com Currently, Memoize::Memcached is initialized like this: eval { if (C4::Context->ismemcached) { require Memoize::Memcached; import Memoize::Memcached qw(memoize_memcached); memoize_memcached( 'GetMarcStructure', memcached => C4::Context->memcached); } }; C4::Context->memcached returns a Cache::Memcached object. However, that's not what Memoize::Memcached expects: This function also accepts a memcached option, which expects a hashref. This is de-referenced and passed directly into an internal function which sets up the memcached configuration for that function. This contents of this hashref are mostly options passed to Cache::Memcached, with a few exceptions. However, passing a Cache::Memcached happened to work, by coincidence, as Memoize::Memcached was calling UNIVERSAL::isa (in a deprecated fashion [0]) to check to see whether the passed argument is a HASH. Nowadays, though, since UNIVERSAL->import is deprecated, the version of Memoize::Memcached packaged for Debian Wheezy now uses ref() to check whether the argument is a hash [1]. That means that the coincidence no longer works, and consequently MARC frameworks don't get cached via memcached in Debian Wheezy, leading to a performance degradation. The solution is to pass Memoize::Memcached what it needs -- a hashref with the Cache::Memcached arguments -- and not a Cache::Memcached object. [0] https://rt.cpan.org/Public/Bug/Display.html?id=68170 [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=614868 -- You are receiving this mail because: You are watching all bug changes.

1 16

[Bug 10487] New: UT: GetMaxPickupDate needs unit tests.
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '13

23 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10487 Bug ID: 10487 Summary: UT: GetMaxPickupDate needs unit tests. Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: gmcharlt(a)gmail.com Reporter: jonathan.druart(a)biblibre.com GetMaxPickupDate is introduced by bug 8367. It needs unit tests. -- You are receiving this mail because: You are watching all bug changes.

1 10

[Bug 4456] Enable addition of PO Number to order
by bugzilla-daemon＠bugs.koha-community.org 22 Sep '13

22 Sep '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4456 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |10530 -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0