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

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52843|0 |1 is obsolete| | --- Comment #91 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53025 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53025&action=ed… Bug 14048: (QA followup) Fix typo on syspref value Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52842|0 |1 is obsolete| | --- Comment #90 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53024 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53024&action=ed… Bug 14048: (QA followup) Update syspref name for HEA Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52841|0 |1 is obsolete| | --- Comment #89 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53023 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53023&action=ed… Bug 14048 [QA Followup] - Delete only the removed system preference Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52840|0 |1 is obsolete| | --- Comment #88 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53022 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53022&action=ed… Bug 14048: Add syspref and atomic update This patch introduces the changes to the sysprefs and the atomic update that creates the new table. It picks the syspref value for RefundLostItemFeeOnReturn and uses it to create a default (branchcode='*') rule on the new table. Sponsored-by: DoverNet Sponsored-by: South-East Kansas Library System Sponsored-by: SWITCH Library Consortium Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52839|0 |1 is obsolete| | --- Comment #87 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53021 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53021&action=ed… Bug 14048: [QA Followup] Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52838|0 |1 is obsolete| | --- Comment #86 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53020 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53020&action=ed… Bug 14048: Hook new rules into C4::Circulation This patch makes AddIssue and AddReturn use the new implementation The behaviour should respect the specification. Sponsored-by: DoverNet Sponsored-by: South-East Kansas Library System Sponsored-by: SWITCH Library Consortium Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52837|0 |1 is obsolete| | --- Comment #85 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53019 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53019&action=ed… Bug 14048: Add a refund rules setting form in smart-rules This patch introduces the CRUD UI for managing the refund lost item fee rules on the general 'Circulation and fines rules' page. Verify that rules can be added and changed. Rules are correctly associated to the chosen branch (or the general default rule). To test: - Apply the patch - Open smart-rules.pl => SUCCESS: The text and refund rules section correctly refers to default global rules. The policy can be set to Yes and No, and works as expected. - Pick a specific branch => SUCCESS: The rule refers to the specific branch => SUCCESS: It detects the default rule is picked => SUCCESS: On changing the rules values are correctly retrieved if entering again to the page. => SUCCESS: The 'Use default' text is suffixed with the correct value set by the user in the 'All libraries' scenario. => Sign off :-D Sponsored-by: DoverNet Sponsored-by: South-East Kansas Library System Sponsored-by: SWITCH Library Consortium Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52836|0 |1 is obsolete| | --- Comment #84 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53018 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53018&action=ed… Bug 14048: Unit tests This patch introduces unit tests for the feature. To test: - Have all patches from this bug applied - Run: $ prove -v t/db_dependent/RefundLostItemFeeRule.t => SUCCESS: All tests pass! Sponsored-by: DoverNet Sponsored-by: South-East Kansas Library System Sponsored-by: SWITCH Library Consortium Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52835|0 |1 is obsolete| | --- Comment #83 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 53017 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53017&action=ed… Bug 14048: Add RefundLostItemFeeRule table and classes This patch introduces new classes for handling refund lost item fee rules. It introduces a new table for storing this rules. It is designed so it is possible to define a global rule, and then branch-specific ones. The specific is prefered if available. This behaviour is fully tested by unit tests introduced by the following patches. This cannot be tested on its own. Sponsored-by: DoverNet Sponsored-by: South-East Kansas Library System Sponsored-by: SWITCH Library Consortium Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> Signed-off-by: Jason Robb <jrobb(a)sekls.org> Signed-off-by: Jennifer Schmidt <jschmidt(a)switchinc.org> Signed-off-by: Margaret Thrasher <p.thrasher(a)dover.nh.gov> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14048] Change RefundLostItemFeeOnReturn to be branch specific
by bugzilla-daemon＠bugs.koha-community.org 01 Jul '16

01 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14048 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