[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Blocks|23091 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23091 [Bug 23091] Add options to charge new or restore forgiven overdues when a lost item is returned -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #24 from Nick Clemens --- Created attachment 111309 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111309=edit Bug 25663: [20.05.x] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi Bug 25663: Add get_lostreturn_policy method to CirculationRules This patch adds a new get_lostreturn_policy method to Koha::CirculationRules which returns a boolean to the caller which denotes whether a refund should be applied or not on the return of a lost item. Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi https://bugs.koha-community.org/show_bug.cgi?id=25633 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #23 from Lucas Gass --- enhancement will not be backported to 20.05.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added See Also|https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=18501 | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Jonathan Druart changed: What|Removed |Added Version(s)||20.11.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #22 from Jonathan Druart --- Pushed to master for 20.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Jonathan Druart changed: What|Removed |Added Blocks||18501 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18501 [Bug 18501] Automatic refunds need protection from failure -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=18501 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #21 from Tomás Cohen Arazi --- I'm not convinced by the current approach, but consistency is a good stepping stone for evolving. The patches are clean, covered by tests and work as expected. QA scripts say this are fine as well. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #20 from Tomás Cohen Arazi --- Created attachment 108071 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108071=edit Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #18 from Tomás Cohen Arazi --- Created attachment 108069 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108069=edit Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #19 from Tomás Cohen Arazi --- Created attachment 108070 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108070=edit Bug 25663: Add get_lostreturn_policy method to CirculationRules This patch adds a new get_lostreturn_policy method to Koha::CirculationRules which returns a boolean to the caller which denotes whether a refund should be applied or not on the return of a lost item. Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Tomás Cohen Arazi changed: What|Removed |Added Attachment #107333|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Tomás Cohen Arazi changed: What|Removed |Added Attachment #107332|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Tomás Cohen Arazi changed: What|Removed |Added Attachment #107334|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Tomás Cohen Arazi changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Tomás Cohen Arazi changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |tomasco...@gmail.com |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Depends on||20815 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20815 [Bug 20815] Add ability to choose if lost fee is refunded based on length of time item has been lost -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #17 from Martin Renvoize --- Rebased for bug 20815 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #106825|0 |1 is obsolete|| --- Comment #16 from Martin Renvoize --- Created attachment 107334 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=107334=edit Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #106824|0 |1 is obsolete|| --- Comment #15 from Martin Renvoize --- Created attachment 107333 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=107333=edit Bug 25663: Add get_lostreturn_policy method to CirculationRules This patch adds a new get_lostreturn_policy method to Koha::CirculationRules which returns a boolean to the caller which denotes whether a refund should be applied or not on the return of a lost item. Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #106823|0 |1 is obsolete|| --- Comment #14 from Martin Renvoize --- Created attachment 107332 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=107332=edit Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #13 from Martin Renvoize --- Rebased -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #106106|0 |1 is obsolete|| --- Comment #10 from Martin Renvoize --- Created attachment 106823 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106823=edit Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #106108|0 |1 is obsolete|| --- Comment #12 from Martin Renvoize --- Created attachment 106825 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106825=edit Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #106107|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize --- Created attachment 106824 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106824=edit Bug 25663: Add get_lostreturn_policy method to CirculationRules This patch adds a new get_lostreturn_policy method to Koha::CirculationRules which returns a boolean to the caller which denotes whether a refund should be applied or not on the return of a lost item. Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Andrew Fuerste-Henry changed: What|Removed |Added CC||and...@bywatersolutions.com Status|Signed Off |Patch doesn't apply --- Comment #9 from Andrew Fuerste-Henry --- Patch no longer applies. Please rebase? Needed to test bug 23091 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #8 from Kyle M Hall --- Created attachment 106108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106108=edit Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #7 from Kyle M Hall --- Created attachment 106107 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106107=edit Bug 25663: Add get_lostreturn_policy method to CirculationRules This patch adds a new get_lostreturn_policy method to Koha::CirculationRules which returns a boolean to the caller which denotes whether a refund should be applied or not on the return of a lost item. Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Kyle M Hall changed: What|Removed |Added Attachment #106105|0 |1 is obsolete|| --- Comment #6 from Kyle M Hall --- Created attachment 106106 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106106=edit Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Kyle M Hall changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Kyle M Hall changed: What|Removed |Added Attachment #105524|0 |1 is obsolete|| Attachment #105525|0 |1 is obsolete|| Attachment #105527|0 |1 is obsolete|| --- Comment #5 from Kyle M Hall --- Created attachment 106105 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106105=edit Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added CC||josef.mora...@gmail.com, ||k...@bywatersolutions.com, ||n...@bywatersolutions.com, ||tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Attachment #105526|0 |1 is obsolete|| --- Comment #4 from Martin Renvoize --- Created attachment 105527 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105527=edit Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |martin.renvoize@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 Martin Renvoize changed: What|Removed |Added Blocks||23091 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23091 [Bug 23091] Restore Forgiven Overdue Fines When Lost Item Returned -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #1 from Martin Renvoize --- Created attachment 105524 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105524=edit Bug 25663: Add unit tests This patch adds unit tests for a new get_lostretun_policy method which is getting added to Koha::CirculationRules. Test plan: 1/ Read the additions to t/db_dependent/CirculationRules.t code and compare to the t/db_dependent/RefundLostItemFeeRule.t. 2/ Run the test after applying the patch that adds the new method 3/ If the tests all pass, signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #2 from Martin Renvoize --- Created attachment 105525 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105525=edit Bug 25663: Add get_lostreturn_policy method to CirculationRules This patch adds a new get_lostreturn_policy method to Koha::CirculationRules which returns a boolean to the caller which denotes whether a refund should be applied or not on the return of a lost item. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25663] Koha::RefundLostItemFeeRules should be merged into Koha::CirculationRules
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25663 --- Comment #3 from Martin Renvoize --- Created attachment 105526 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105526=edit Bug 25663: Remove Koha::RefundLostItemFeeRule and uses This patch replaces all calls to RefundLostItemFeeRules with Koha::CirculationRules->get_lostreturn_policy and removes the module it makes redundant. Test plan 1/ Confirm that there are no longer any uses of RefundLostItemFeeRules in the codebase 2/ Confirm circulation tests still all pass 3/ Confirm you can still set and unset the lost return rules 4/ Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/