[
https://issues.apache.org/jira/browse/OFBIZ-2790?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ahmed Dini updated OFBIZ-2790:
------------------------------
Attachment: paymentTotals.patch
Hello Sumit,
Thanks for your comments and recommendations. As recommended I have made the
changes requested. Sorry about my mistakes. In my future developments, I will
make sure to check everything with the best practice conventions.
Attached is mended patch. Thanks again I appreciate your comments.
NB if there is more improvements needed please let me know, more than happy to
change it again.
> Payment totals in Accounting
> ----------------------------
>
> Key: OFBIZ-2790
> URL: https://issues.apache.org/jira/browse/OFBIZ-2790
> Project: OFBiz
> Issue Type: Improvement
> Components: accounting
> Affects Versions: SVN trunk
> Reporter: Ahmed Dini
> Assignee: Anil K Patel
> Fix For: SVN trunk
>
> Attachments: paymentTotals.patch, paymentTotals.patch
>
>
> Under Accounting Manager --> Payments, Find Payments. I thought it would be
> a handy improvement if I display a total of the Amount fields of all payments
> under the search result screen no matter what their status is. This new
> improvement might not makes sense when payment status are mixed i.e. The
> total of cancelled payments plus the total of received payments, but I thinks
> it might make sense if you want to filter the query. i.e if you want the
> total of all payments in NOT-PAID status by a particular party or customer.
> Attached is patch which introduces a new groovy file namely
> FindPaymentTotals.groovy and the modification of the existing
> PaymentForms.xml.
> NB. Since this is my first major improvement for Ofbiz project, I might have
> done things differently than most of the commiters or contributors expect.
> Please let me know if I break Ofbiz coding conventions, I am willing to
> improve the code. Otherwise the attached patch works fine and has passed all
> my tests
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
