[Koha-bugs] [Bug 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Status|Pushed to master|RESOLVED Resolution|--- |FIXED -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Bug 24300 depends on bug 27967, which changed state. Bug 27967 Summary: Modals on the borrower account page don't validate minimum values https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27967 What|Removed |Added Status|Pushed to master|RESOLVED Resolution|--- |FIXED -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #51 from Fridolin Somers --- Enhancement not pushed to 20.11.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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Bug 24300 depends on bug 27927, which changed state. Bug 27927 Summary: longoverdue cronjob renews items before marking lost when both RenewAccruingItemWhenPaid and WhenLostForgiveFine are enabled https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27927 What|Removed |Added Status|Pushed to oldstable |RESOLVED Resolution|--- |FIXED -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #50 from Jonathan Druart --- Pushed to master for 21.05, 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||21.05.00 released in|| -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #119200|0 |1 is obsolete|| --- Comment #49 from Martin Renvoize --- Created attachment 120021 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120021&action=edit Bug 24300: (QA follow-up) Enhance test and catch failures The tests did not include testing for offset lines. This patch adds such tests and updates the function to pass them 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #119199|0 |1 is obsolete|| --- Comment #48 from Martin Renvoize --- Created attachment 120020 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120020&action=edit Bug 24300: (QA follow-up) Update payout_amount to use internal methods Tomas pointed out that we could, and probably should, use the internal 'add_debit' method within this function. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #119197|0 |1 is obsolete|| --- Comment #46 from Martin Renvoize --- Created attachment 120018 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120018&action=edit Bug 24300: (QA follow-up) Add Unit Tests Doh! I forgot the unit tests first time around! 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #119198|0 |1 is obsolete|| --- Comment #47 from Martin Renvoize --- Created attachment 120019 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120019&action=edit Bug 24300: (follow-up) Set minimum for amount input Add a 'min' attribute for validation of the amount field in the payout modal so that negative amounts cannot be submitted. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #119196|0 |1 is obsolete|| --- Comment #45 from Martin Renvoize --- Created attachment 120017 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120017&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 4/ Signoff Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #119195|0 |1 is obsolete|| --- Comment #44 from Martin Renvoize --- Created attachment 120016 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120016&action=edit Bug 24300: Add payout_amount method to Koha::Account This patch adds a 'payout_amount' method to the Koha::Account class to allow payout of an amount against a list of credits (or any outstanding credits, oldest to newest). Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Status|BLOCKED |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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Jonathan Druart changed: What|Removed |Added Status|Passed QA |BLOCKED CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #43 from Jonathan Druart --- Blocked by bug 27927. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #42 from Martin Renvoize --- Tree re-arranged slightly to put bugs that need backporting before these enhancements that don't. Rebased to accommodate the above. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118612|0 |1 is obsolete|| --- Comment #41 from Martin Renvoize --- Created attachment 119200 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119200&action=edit Bug 24300: (QA follow-up) Enhance test and catch failures The tests did not include testing for offset lines. This patch adds such tests and updates the function to pass them 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118605|0 |1 is obsolete|| --- Comment #40 from Martin Renvoize --- Created attachment 119199 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119199&action=edit Bug 24300: (QA follow-up) Update payout_amount to use internal methods Tomas pointed out that we could, and probably should, use the internal 'add_debit' method within this function. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118604|0 |1 is obsolete|| --- Comment #39 from Martin Renvoize --- Created attachment 119198 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119198&action=edit Bug 24300: (follow-up) Set minimum for amount input Add a 'min' attribute for validation of the amount field in the payout modal so that negative amounts cannot be submitted. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118603|0 |1 is obsolete|| --- Comment #38 from Martin Renvoize --- Created attachment 119197 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119197&action=edit Bug 24300: (QA follow-up) Add Unit Tests Doh! I forgot the unit tests first time around! 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118602|0 |1 is obsolete|| --- Comment #37 from Martin Renvoize --- Created attachment 119196 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119196&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 4/ Signoff Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118601|0 |1 is obsolete|| --- Comment #36 from Martin Renvoize --- Created attachment 119195 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119195&action=edit Bug 24300: Add payout_amount method to Koha::Account This patch adds a 'payout_amount' method to the Koha::Account class to allow payout of an amount against a list of credits (or any outstanding credits, oldest to newest). Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Depends on||27927 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27927 [Bug 27927] longoverdue cronjob renews items before marking lost when both RenewAccruingItemWhenPaid and WhenLostForgiveFine are enabled -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118606|0 |1 is obsolete|| --- Comment #35 from Martin Renvoize --- Created attachment 118612 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118612&action=edit Bug 24300: (QA follow-up) Enhance test and catch failures The tests did not include testing for offset lines. This patch adds such tests and updates the function to pass them 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #34 from Tomás Cohen Arazi --- Created attachment 118606 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118606&action=edit Bug 24300: (QA follow-up) Enhance test and catch failures The tests did not include testing for offset lines. This patch adds such tests and updates the function to pass them -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #33 from Tomás Cohen Arazi --- Created attachment 118605 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118605&action=edit Bug 24300: (QA follow-up) Update payout_amount to use internal methods Tomas pointed out that we could, and probably should, use the internal 'add_debit' method within this function. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #32 from Tomás Cohen Arazi --- Created attachment 118604 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118604&action=edit Bug 24300: (follow-up) Set minimum for amount input Add a 'min' attribute for validation of the amount field in the payout modal so that negative amounts cannot be submitted. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #31 from Tomás Cohen Arazi --- Created attachment 118603 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118603&action=edit Bug 24300: (QA follow-up) Add Unit Tests Doh! I forgot the unit tests first time around! 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #30 from Tomás Cohen Arazi --- Created attachment 118602 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118602&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 4/ Signoff Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #29 from Tomás Cohen Arazi --- Created attachment 118601 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118601&action=edit Bug 24300: Add payout_amount method to Koha::Account This patch adds a 'payout_amount' method to the Koha::Account class to allow payout of an amount against a list of credits (or any outstanding credits, oldest to newest). Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118396|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118395|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118394|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118393|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118392|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118503|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #118464|0 |1 is obsolete|| --- Comment #28 from Martin Renvoize --- Created attachment 118503 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118503&action=edit Bug 24300: (QA follow-up) Enhance test and catch failures The tests did not include testing for offset lines. This patch adds such tests and updates the function to pass them -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks|27636 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27636 [Bug 27636] Replace Koha::Account::pay with a simpler method -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks||27971 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27971 [Bug 27971] The VOID method should be updated to respect double-entry accounting -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks|22435 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22435 [Bug 22435] account_offset types should be codes not descriptions -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks||27636 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27636 [Bug 27636] Replace Koha::Account::pay with a simpler method -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks||22435 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22435 [Bug 22435] account_offset types should be codes not descriptions -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks|27967 | Depends on||27967 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27967 [Bug 27967] Modals on the borrower account page don't validate minimum values -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Status|ASSIGNED|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #27 from Martin Renvoize --- Created attachment 118464 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118464&action=edit Bug 24300: (QA follow-up) Enhance test and catch failures The tests did not include testing for offset lines. This patch adds such tests and updates the function to pass them -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #26 from Martin Renvoize --- (In reply to Martin Renvoize from comment #25) > Oop, just found a bug in that final QA followup And.. I realised I'd missed tests for offset creations too.. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Status|Passed QA |ASSIGNED --- Comment #25 from Martin Renvoize --- Oop, just found a bug in that final QA followup -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #24 from Tomás Cohen Arazi --- (In reply to Martin Renvoize from comment #22) > I took the liberty of attaching the patches with Tomas's SO lines ;) Thanks! You saved me some time! -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #23 from Tomás Cohen Arazi --- (In reply to Owen Leonard from comment #16) > Tomás I think you forgot to add the signed-off patches You are right! -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #22 from Martin Renvoize --- I took the liberty of attaching the patches with Tomas's SO lines ;) -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #21 from Martin Renvoize --- Created attachment 118396 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118396&action=edit Bug 24300: (QA follow-up) Update payout_amount to use internal methods Tomas pointed out that we could, and probably should, use the internal 'add_debit' method within this function. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #20 from Martin Renvoize --- Created attachment 118395 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118395&action=edit Bug 24300: (follow-up) Set minimum for amount input Add a 'min' attribute for validation of the amount field in the payout modal so that negative amounts cannot be submitted. 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #19 from Martin Renvoize --- Created attachment 118394 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118394&action=edit Bug 24300: (QA follow-up) Add Unit Tests Doh! I forgot the unit tests first time around! 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #18 from Martin Renvoize --- Created attachment 118393 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118393&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 4/ Signoff Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #17 from Martin Renvoize --- Created attachment 118392 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118392&action=edit Bug 24300: Add payout_amount method to Koha::Account This patch adds a 'payout_amount' method to the Koha::Account class to allow payout of an amount against a list of credits (or any outstanding credits, oldest to newest). Signed-off-by: Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #16 from Owen Leonard --- Tomás I think you forgot to add the signed-off patches -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #15 from Tomás Cohen Arazi --- Works as expected. Tests pass. My QA concerns were promptly addressed by the follow-up. QA script happy. Good job -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #117919|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #117918|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #117917|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118335|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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks|22435 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22435 [Bug 22435] account_offset types should be codes not descriptions -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks||27967 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27967 [Bug 27967] Modals on the borrower account page don't validate minimum values -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #14 from Martin Renvoize --- Created attachment 118335 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118335&action=edit Bug 24300: (follow-up) Set minimum for amount input Add a 'min' attribute for validation of the amount field in the payout modal so that negative amounts cannot be submitted. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added CC||n...@bywatersolutions.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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Keywords|release-notes-needed| -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Text to go in the||This enhancement adds a release notes||'Payout amount' button to ||the borrowers account ||details page when there is ||outstanding credit. This ||allows a library to payout ||multiple outstanding ||credits in one action. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Status|Failed QA |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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #13 from Martin Renvoize --- Created attachment 117919 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117919&action=edit Bug 24300: (QA follow-up) Add Unit Tests Doh! I forgot the unit tests first time around! -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #116838|0 |1 is obsolete|| --- Comment #12 from Martin Renvoize --- Created attachment 117918 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117918&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 4/ Signoff Signed-off-by: Sally -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #116837|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize --- Created attachment 117917 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117917&action=edit Bug 24300: Add payout_amount method to Koha::Account This patch adds a 'payout_amount' method to the Koha::Account class to allow payout of an amount against a list of credits (or any outstanding credits, oldest to newest). Signed-off-by: Sally -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #10 from Martin Renvoize --- (In reply to Katrin Fischer from comment #9) > Hi Martin, please add unit tests! Dang, how did I forget that!.. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Katrin Fischer changed: What|Removed |Added Keywords||release-notes-needed -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #9 from Katrin Fischer --- Hi Martin, please add unit tests! -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Sally 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #8 from Sally --- Patch works as described. An account being more in debt is ok - there are sometimes circumstances where a patron might owe money, but they also need a refund for an unrelated transaction. With that in mind, it might be nice to have the option to pay out the credit despite the patron being in debt. e.g. Patron owes £20.00 for lost book - but let's say this is in dispute. Patron borrows 5 x DVDs at £2 each. Patron is given a credit for all 5 DVDs. Patron then decides to have a refund rather than replacement DVDs. There is no payout button because the account says £10.00 (£20 owed minus £10 credit). It would be nice to have a payout button for the £10 credit, rather than having to issue payout across 5 individual lines. Equally, on the payment page, it would be nice to be able to use the credit against selected lines - a bit like 'Pay Selected' or 'Write off Selected', it'd be nice to 'Use Credit against Selected' - so again, we can use the credit against the DVDs and not against the lost book. I get the feeling that's all out of scope of this bug though. But the patch works as described, so I'm signing 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 ByWater Sandboxes changed: What|Removed |Added Attachment #116413|0 |1 is obsolete|| --- Comment #7 from ByWater Sandboxes --- Created attachment 116838 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116838&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 4/ Signoff Signed-off-by: Sally -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 ByWater Sandboxes changed: What|Removed |Added Attachment #116412|0 |1 is obsolete|| --- Comment #6 from ByWater Sandboxes --- Created attachment 116837 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116837&action=edit Bug 24300: Add payout_amount method to Koha::Account Signed-off-by: Sally -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added CC||sally.healey@cheshireshared ||services.gov.uk, ||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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #5 from Martin Renvoize --- This will need some unit tests adding for QA, but it's ready for an initial testing phase. I'm not entirely sure I like how it works.. at the moment you can payout prior to paying off debts so you may result in an account that more in debt than before the payout. Feedback welcome. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #4 from Martin Renvoize --- Created attachment 116413 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116413&action=edit Bug 24300: Add payout amount to boraccount page Test plan 1/ Add some debts and credits to a patrons account, resulting in an overall credit 2/ Note the appearance of a 'Payout amount' button next to the total due line. 3/ Try paying out an amount with the button 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Attachment #116395|0 |1 is obsolete|| --- Comment #3 from Martin Renvoize --- Created attachment 116412 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116412&action=edit Bug 24300: Add payout_amount method to Koha::Account -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #2 from Martin Renvoize --- Created attachment 116395 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116395&action=edit Bug 24300: Add payout_amount method to Koha::Account -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks|17596 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17596 [Bug 17596] Simplify accounting UI in staff client [OMNIBUS] -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 --- Comment #1 from Martin Renvoize --- I'm thinking of adding a payout_amount method within Koha::Account that does similar to the offset_amount method I'm looking to add in bug 27636 but in the reverse direction.. i.e. it adds a PAYOUT debit line and offsets it against a set of passed CREDIT lines. -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=27636 -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks||22435 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22435 [Bug 22435] account_offset types should be codes not descriptions -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Bug 24300 depends on bug 24080, which changed state. Bug 24080 Summary: Add a 'payout' process to accounts https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24080 What|Removed |Added Status|Pushed to master|RESOLVED Resolution|--- |FIXED -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks|14825 |17596 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14825 [Bug 14825] Accounts Rewrite Omnibus https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17596 [Bug 17596] Simplify accounting UI in staff client -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Blocks||14825 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14825 [Bug 14825] Accounts Rewrite Omnibus -- 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 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 24300] Add a 'payout amount' option to accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24300 Martin Renvoize changed: What|Removed |Added Depends on||24080 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24080 [Bug 24080] Add a 'payout' process to accounts -- 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/