Description
Kelly McElligott
2019-09-25 00:35:13 UTC
*** Bug 22185 has been marked as a duplicate of this bug. *** Agreed. The Transactions table on boraccount.pl should allow one to add notes and/or edit existing notes. It's not a bad idea, but, as a system administrator for 51 libraries all using 1 shared Koha, I would want some control over who was able to edit notes. We use custom jQuery to add usernames and branchcodes to all notes so any staff member can easily look at an accountline and know who created it an where so that they don't have to run a report to see, for example, if someone at another library did a writeoff without permission from the library that owns that accountline. If anyone can edit any note, it makes it possible for someone to edit out the history of any undeserved writeoffs or credits they've created. I would want to be able to control who can edit the accountlines notes. A staff permission would be preferable, but a system preference would work too. Changed the bug title to reflect a broadened scope. Every accountlines entry has a notes field, so why not just put an Edit Note option on every line in the Transactions table. As George says, we'd want access to that button to be tied to a specific permission. Yes. Ability to add and edit notes is needed. Ability to extend "permissions" to that module for making/editing notes (and/or like in the Messages on patron's accounts where it gives the librarian, date, and time for the message when entered or edited) would be excellent, too [especially for the big wigs, like George ;-)] Created attachment 168035 [details] [review] Bug 23674: Add ability to add a note to APPLY DISCOUNT To test: 1. APPLY PATCH and restart_all 2. Apply a manual invoice to a patron account. 3. Go to the patron record -> accounting -> transaction tab 4. Look for the 'Apply discount' button. 5. When the 'Apply discount' modal appears notcie the 'Note' field. 6. Add a note, make sure it gets applied to this accountline. Created attachment 168036 [details] [review] Bug 23674: Add ability to add a note to pay individual fine from paycollect.tt Place where we need to have the ability to add a note: Pay individual Write-off individual Void Issue refund Issue payout Apply discount Created attachment 168174 [details] [review] Bug 23674: Add ability to add a note to pay individual and write-off individual To test: 1) Apply manual invoice to a patron account 2) Go to the patron record -> accounting -> transaction tab 3) Look for the "Pay" button under the actions column. Click this 4) Notice no place to enter note 5) Click the "Write off" button. Notice there is no place to enter note 6) Apply patch, restart_all 7) Notice that a "Note: " input appears in both the 'Pay' and 'Write off' tabs. 8) Write a not in the pay tab and confirm. Make sure this not shows up in the table. 9) Add another invoice and then test the note functionallity for the write-offs. Created attachment 168177 [details] [review] Bug 23674: Add ability to add a note to issue refund Created attachment 168178 [details] [review] Bug 23674: Add ability to add a note to issue payout To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 3) Under the actions column, select 'Issue Payout' 4) You can now enter a note. Type something in and press confirm. 5) Ensure the note shows up in the note column. Created attachment 168180 [details] [review] Bug 23674: Add ability to add note to void payment To test: 1) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 2) Under the actions column, select "Void" 3) Notice no option for a note. Press cancel 4) Apply patch, restart_all 5) Press void again. This time, you should see a modal that gives you the option to type in a note. 6) Type something in and select submit. 7) Ensure the note shows up in the note column. *** Bug 24597 has been marked as a duplicate of this bug. *** Created attachment 168213 [details] [review] Bug 23674: Add ability to add a note to issue refund To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual invoice. Enter some amount and select save and pay. Press confirm. 3) In the transactions tab, for the invoice you just created, under the actions column select issue refund. 4) This should open a modal with the ability to enter a note for the refund. Type in a note and confirm. 5) Ensure the note correctly shows in the table. Created attachment 168329 [details] [review] Bug 23674: Add ability to add note to canceled charge To test, with this patchset applied: 1) From patron accounting page -> Add a manual invoice 2) Under the transaction tab click 'Cancel charge' 3) You should see a modal that gives you the option to type in a note. 4) Type something in and select submit. 5) Ensure the note shows up in the note column. Created attachment 168330 [details] [review] Bug 23674: Fix duplicate ID's and label Created attachment 169782 [details] [review] Bug 23674: Add ability to add a note to APPLY DISCOUNT To test: 1. APPLY PATCH and restart_all 2. Apply a manual invoice to a patron account. 3. Go to the patron record -> accounting -> transaction tab 4. Look for the 'Apply discount' button. 5. When the 'Apply discount' modal appears notcie the 'Note' field. 6. Add a note, make sure it gets applied to this accountline. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 169783 [details] [review] Bug 23674: Add ability to add a note to pay individual and write-off individual To test: 1) Apply manual invoice to a patron account 2) Go to the patron record -> accounting -> transaction tab 3) Look for the "Pay" button under the actions column. Click this 4) Notice no place to enter note 5) Click the "Write off" button. Notice there is no place to enter note 6) Apply patch, restart_all 7) Notice that a "Note: " input appears in both the 'Pay' and 'Write off' tabs. 8) Write a not in the pay tab and confirm. Make sure this not shows up in the table. 9) Add another invoice and then test the note functionallity for the write-offs. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 169784 [details] [review] Bug 23674: Add ability to add a note to issue payout To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 3) Under the actions column, select 'Issue Payout' 4) You can now enter a note. Type something in and press confirm. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 169785 [details] [review] Bug 23674: Add ability to add note to void payment To test: 1) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 2) Under the actions column, select "Void" 3) Notice no option for a note. Press cancel 4) Apply patch, restart_all 5) Press void again. This time, you should see a modal that gives you the option to type in a note. 6) Type something in and select submit. 7) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 169786 [details] [review] Bug 23674: Add ability to add a note to issue refund To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual invoice. Enter some amount and select save and pay. Press confirm. 3) In the transactions tab, for the invoice you just created, under the actions column select issue refund. 4) This should open a modal with the ability to enter a note for the refund. Type in a note and confirm. 5) Ensure the note correctly shows in the table. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 169787 [details] [review] Bug 23674: Add ability to add note to canceled charge To test, with this patchset applied: 1) From patron accounting page -> Add a manual invoice 2) Under the transaction tab click 'Cancel charge' 3) You should see a modal that gives you the option to type in a note. 4) Type something in and select submit. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 169788 [details] [review] Bug 23674: Fix duplicate ID's and label Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Test notes: Test for "Bug 23674: Add ability to add a note to issue payout" works as described at the end, with full patchset. QA script: WARN members/boraccount.pl WARN tidiness The file is less tidy than before (bad/messy lines before: 45, now: 46) WARN members/cancel-charge.pl WARN tidiness The file is less tidy than before (bad/messy lines before: 4, now: 6) Patch no longer applies due to Bootstrap 5 changes Created attachment 174300 [details] [review] Bug 23674: Account for Bootstrap5 changes Created attachment 174301 [details] [review] Bug 23674: Add ability to add a note to APPLY DISCOUNT To test: 1. APPLY PATCH and restart_all 2. Apply a manual invoice to a patron account. 3. Go to the patron record -> accounting -> transaction tab 4. Look for the 'Apply discount' button. 5. When the 'Apply discount' modal appears notcie the 'Note' field. 6. Add a note, make sure it gets applied to this accountline. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174302 [details] [review] Bug 23674: Add ability to add a note to pay individual and write-off individual To test: 1) Apply manual invoice to a patron account 2) Go to the patron record -> accounting -> transaction tab 3) Look for the "Pay" button under the actions column. Click this 4) Notice no place to enter note 5) Click the "Write off" button. Notice there is no place to enter note 6) Apply patch, restart_all 7) Notice that a "Note: " input appears in both the 'Pay' and 'Write off' tabs. 8) Write a not in the pay tab and confirm. Make sure this not shows up in the table. 9) Add another invoice and then test the note functionallity for the write-offs. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174303 [details] [review] Bug 23674: Add ability to add a note to issue payout To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 3) Under the actions column, select 'Issue Payout' 4) You can now enter a note. Type something in and press confirm. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174304 [details] [review] Bug 23674: Add ability to add note to void payment To test: 1) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 2) Under the actions column, select "Void" 3) Notice no option for a note. Press cancel 4) Apply patch, restart_all 5) Press void again. This time, you should see a modal that gives you the option to type in a note. 6) Type something in and select submit. 7) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174305 [details] [review] Bug 23674: Add ability to add a note to issue refund To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual invoice. Enter some amount and select save and pay. Press confirm. 3) In the transactions tab, for the invoice you just created, under the actions column select issue refund. 4) This should open a modal with the ability to enter a note for the refund. Type in a note and confirm. 5) Ensure the note correctly shows in the table. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174306 [details] [review] Bug 23674: Add ability to add note to canceled charge To test, with this patchset applied: 1) From patron accounting page -> Add a manual invoice 2) Under the transaction tab click 'Cancel charge' 3) You should see a modal that gives you the option to type in a note. 4) Type something in and select submit. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174307 [details] [review] Bug 23674: Fix duplicate ID's and label Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 174308 [details] [review] Bug 23674: Account for Bootstrap5 changes Created attachment 177225 [details] [review] Bug 23674: Add ability to add a note to APPLY DISCOUNT To test: 1. APPLY PATCH and restart_all 2. Apply a manual invoice to a patron account. 3. Go to the patron record -> accounting -> transaction tab 4. Look for the 'Apply discount' button. 5. When the 'Apply discount' modal appears notcie the 'Note' field. 6. Add a note, make sure it gets applied to this accountline. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177226 [details] [review] Bug 23674: Add ability to add a note to pay individual and write-off individual To test: 1) Apply manual invoice to a patron account 2) Go to the patron record -> accounting -> transaction tab 3) Look for the "Pay" button under the actions column. Click this 4) Notice no place to enter note 5) Click the "Write off" button. Notice there is no place to enter note 6) Apply patch, restart_all 7) Notice that a "Note: " input appears in both the 'Pay' and 'Write off' tabs. 8) Write a not in the pay tab and confirm. Make sure this not shows up in the table. 9) Add another invoice and then test the note functionallity for the write-offs. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177227 [details] [review] Bug 23674: Add ability to add a note to issue payout To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 3) Under the actions column, select 'Issue Payout' 4) You can now enter a note. Type something in and press confirm. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177228 [details] [review] Bug 23674: Add ability to add note to void payment To test: 1) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 2) Under the actions column, select "Void" 3) Notice no option for a note. Press cancel 4) Apply patch, restart_all 5) Press void again. This time, you should see a modal that gives you the option to type in a note. 6) Type something in and select submit. 7) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177229 [details] [review] Bug 23674: Add ability to add a note to issue refund To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual invoice. Enter some amount and select save and pay. Press confirm. 3) In the transactions tab, for the invoice you just created, under the actions column select issue refund. 4) This should open a modal with the ability to enter a note for the refund. Type in a note and confirm. 5) Ensure the note correctly shows in the table. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177230 [details] [review] Bug 23674: Add ability to add note to canceled charge To test, with this patchset applied: 1) From patron accounting page -> Add a manual invoice 2) Under the transaction tab click 'Cancel charge' 3) You should see a modal that gives you the option to type in a note. 4) Type something in and select submit. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177231 [details] [review] Bug 23674: Fix duplicate ID's and label Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177232 [details] [review] Bug 23674: Account for Bootstrap5 changes Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 177233 [details] [review] Bug 23674: (QA follow-up) Re-introduce cud-cancel op Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> This is a nice enhancements, makes the UI consistent and clean. Created attachment 178059 [details] [review] Bug 23674: Add ability to add a note to APPLY DISCOUNT To test: 1. APPLY PATCH and restart_all 2. Apply a manual invoice to a patron account. 3. Go to the patron record -> accounting -> transaction tab 4. Look for the 'Apply discount' button. 5. When the 'Apply discount' modal appears notcie the 'Note' field. 6. Add a note, make sure it gets applied to this accountline. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178060 [details] [review] Bug 23674: Add ability to add a note to pay individual and write-off individual To test: 1) Apply manual invoice to a patron account 2) Go to the patron record -> accounting -> transaction tab 3) Look for the "Pay" button under the actions column. Click this 4) Notice no place to enter note 5) Click the "Write off" button. Notice there is no place to enter note 6) Apply patch, restart_all 7) Notice that a "Note: " input appears in both the 'Pay' and 'Write off' tabs. 8) Write a not in the pay tab and confirm. Make sure this not shows up in the table. 9) Add another invoice and then test the note functionallity for the write-offs. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178061 [details] [review] Bug 23674: Add ability to add a note to issue payout To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 3) Under the actions column, select 'Issue Payout' 4) You can now enter a note. Type something in and press confirm. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178062 [details] [review] Bug 23674: Add ability to add note to void payment To test: 1) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 2) Under the actions column, select "Void" 3) Notice no option for a note. Press cancel 4) Apply patch, restart_all 5) Press void again. This time, you should see a modal that gives you the option to type in a note. 6) Type something in and select submit. 7) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178063 [details] [review] Bug 23674: Add ability to add a note to issue refund To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual invoice. Enter some amount and select save and pay. Press confirm. 3) In the transactions tab, for the invoice you just created, under the actions column select issue refund. 4) This should open a modal with the ability to enter a note for the refund. Type in a note and confirm. 5) Ensure the note correctly shows in the table. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178064 [details] [review] Bug 23674: Add ability to add note to canceled charge To test, with this patchset applied: 1) From patron accounting page -> Add a manual invoice 2) Under the transaction tab click 'Cancel charge' 3) You should see a modal that gives you the option to type in a note. 4) Type something in and select submit. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178065 [details] [review] Bug 23674: Fix duplicate ID's and label Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178066 [details] [review] Bug 23674: Account for Bootstrap5 changes Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178067 [details] [review] Bug 23674: (QA follow-up) Re-introduce cud-cancel op Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Rebased with auto_rebase.pl script Sorry Lucas, can you have another go? Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 23674: Add ability to add a note to APPLY DISCOUNT Applying: Bug 23674: Add ability to add a note to pay individual and write-off individual Applying: Bug 23674: Add ability to add a note to issue payout Applying: Bug 23674: Add ability to add note to void payment error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt). error: could not build fake ancestor Patch failed at 0001 Bug 23674: Add ability to add note to void payment hint: Use 'git am --show-current-patch=diff' to see the failed patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-23674-Add-ability-to-add-note-to-void-payment-u228tvwc.patch I recently had this issue on another patch and it applied nicely from a remote branch. Happy to try that here as well if you want to provide a link. Created attachment 178791 [details] [review] Bug 23674: Add ability to add a note to APPLY DISCOUNT To test: 1. APPLY PATCH and restart_all 2. Apply a manual invoice to a patron account. 3. Go to the patron record -> accounting -> transaction tab 4. Look for the 'Apply discount' button. 5. When the 'Apply discount' modal appears notcie the 'Note' field. 6. Add a note, make sure it gets applied to this accountline. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178792 [details] [review] Bug 23674: Add ability to add a note to pay individual and write-off individual To test: 1) Apply manual invoice to a patron account 2) Go to the patron record -> accounting -> transaction tab 3) Look for the "Pay" button under the actions column. Click this 4) Notice no place to enter note 5) Click the "Write off" button. Notice there is no place to enter note 6) Apply patch, restart_all 7) Notice that a "Note: " input appears in both the 'Pay' and 'Write off' tabs. 8) Write a not in the pay tab and confirm. Make sure this not shows up in the table. 9) Add another invoice and then test the note functionallity for the write-offs. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178793 [details] [review] Bug 23674: Add ability to add a note to issue payout To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 3) Under the actions column, select 'Issue Payout' 4) You can now enter a note. Type something in and press confirm. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178794 [details] [review] Bug 23674: Add ability to add note to void payment To test: 1) From patron accounting page -> Create manual credit. Put in some amount and press add credit. 2) Under the actions column, select "Void" 3) Notice no option for a note. Press cancel 4) Apply patch, restart_all 5) Press void again. This time, you should see a modal that gives you the option to type in a note. 6) Type something in and select submit. 7) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178795 [details] [review] Bug 23674: Add ability to add a note to issue refund To test: 1) Apply patch, restart_all 2) From patron accounting page -> Create manual invoice. Enter some amount and select save and pay. Press confirm. 3) In the transactions tab, for the invoice you just created, under the actions column select issue refund. 4) This should open a modal with the ability to enter a note for the refund. Type in a note and confirm. 5) Ensure the note correctly shows in the table. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178796 [details] [review] Bug 23674: Add ability to add note to canceled charge To test, with this patchset applied: 1) From patron accounting page -> Add a manual invoice 2) Under the transaction tab click 'Cancel charge' 3) You should see a modal that gives you the option to type in a note. 4) Type something in and select submit. 5) Ensure the note shows up in the note column. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178797 [details] [review] Bug 23674: Fix duplicate ID's and label Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178798 [details] [review] Bug 23674: Account for Bootstrap5 changes Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 178799 [details] [review] Bug 23674: (QA follow-up) Re-introduce cud-cancel op Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Trivial rebase this time. Can be reset to PQA I think. I did some testing in the GUI on this patch set and come back with some questions: 1) Partial payment and refund - Create a manual invoice, 10.00 - Pay using the pay button in the transactions table, 1:00 + note - Payment note is empty when checking the payments tab: It looks like the note used there now and the existing payment notes use different fields - should we not use the same for the payment note no matter where it's paid from? - Now issuing a refund explodes into: Exception 'Koha::Exceptions::ParameterTooHigh' thrown 'Amount to payout (3.00) is higher than amountoutstanding (0)' at /usr/share/perl5/Exception/Class/Base.pm line 88 - new bug? 2) Manual invoice + discount - Create a manual invoice, 10.00 - Apply discount: 1.00 The note field is before the amount, on the other forms it's the last input field on the form. Should we move it to make it more consistent? Can you please check? Void, refund and cancel all worked as expected in testing. > - Create a manual invoice, 10.00 > - Pay using the pay button in the transactions table, 1:00 + note > - Payment note is empty when checking the payments tab: It looks like the > note used there now and the existing payment notes use different fields - > should we not use the same for the payment note no matter where it's paid > from? > - Now issuing a refund explodes into: > Exception 'Koha::Exceptions::ParameterTooHigh' thrown 'Amount to payout > (3.00) is higher than amountoutstanding (0)' > at /usr/share/perl5/Exception/Class/Base.pm line 88 - new bug? Part of this is a new bug, it happens without this patchset applied: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39219 I'll look into why the note is empty. Thanks, Cait! > - Payment note is empty when checking the payments tab: It looks like the
> note used there now and the existing payment notes use different fields -
> should we not use the same for the payment note no matter where it's paid
> from?
I think its the same accountline note and we should display it on the payment tab if it exists.
Created attachment 178884 [details] [review] Bug 23674: Display note of the pay.tt when it exists
> - Create a manual invoice, 10.00
> - Apply discount: 1.00 The note field is before the amount, on the other
> forms it's the last input field on the form. Should we move it to make it
> more consistent?
Yes.
Created attachment 178885 [details] [review] Bug 23674: Add note field the last field in discount modal I think I've addressed all your concerns here? Setting back to PQA. (In reply to Lucas Gass (lukeg) from comment #68) > > - Payment note is empty when checking the payments tab: It looks like the > > note used there now and the existing payment notes use different fields - > > should we not use the same for the payment note no matter where it's paid > > from? > > I think its the same accountline note and we should display it on the > payment tab if it exists. I think if we also have a payment note, it's a little confusing that it's not available on paying. And having 2 notes later (adding the payment note) is not ideal. If we start out with this, we will have a bit of a mixed data situation later. I'd like to get another set of eyes if possible on the notes for payment. (In reply to Katrin Fischer from comment #73) > (In reply to Lucas Gass (lukeg) from comment #68) > > > - Payment note is empty when checking the payments tab: It looks like the > > > note used there now and the existing payment notes use different fields - > > > should we not use the same for the payment note no matter where it's paid > > > from? > > > > I think its the same accountline note and we should display it on the > > payment tab if it exists. > > I think if we also have a payment note, it's a little confusing that it's > not available on paying. And having 2 notes later (adding the payment note) > is not ideal. If we start out with this, we will have a bit of a mixed data > situation later. I'd like to get another set of eyes if possible on the > notes for payment. I re-tested and I think I got it now - the payment note and the note from transactions are indeed the same field in the database: accountlines.note. I have a feeling that the notes fields for paying and writeoff on the 'Make a payment' tab could also need some thought, but that's out of scope here! Thanks! Pushed for 25.05! Well done everyone, thank you! |