ledger123 / runmyaccounts

SQL-Ledger Customizations by Run My Accounts
GNU General Public License v2.0
16 stars 9 forks source link

SQL-Ledger Bug with Payment Date < Invoice Date #488

Open MariaHashimi opened 6 months ago

MariaHashimi commented 6 months ago

Hi @ledger123 ,

During the Qualibox migration we have noticed a bug with payment dates < invoice dates in SQL-Ledger. If this is fixed, we could stop testing these cases in the Qualibox and this would reduce the amount of work for the accountants quite significantly.

Tomi informed us that we test these bookings in the qualibox only, because SQL-Ledger in these cases is buggy - not because they are wrong from an accounting point of view. By fixing this, we would not have to migrate this test to the new Qualibox.

Description of the bug done by Tomi: 240403 SQL Ledger Issue with payment date smaller than Invoice date.docx

ledger123 commented 5 months ago

Hi @MariaHashimi ,

The existing report handles too many possibilities and same code is used for transactions list as well as for outstanding report. Changing the existing report can introduce unforeseen problems.

So far the best solution seems to be that we modify the existing report and add a section before the report which shows the invoices which have been paid in advance as shown in this screen shot.

image

Does this look ok?

Regards

ledger123 commented 2 months ago

Hi @MariaHashimi ,

This is fixed now. This proved to be a really hard nut to crack but finally it is resolved. Here are screen shots of testing from my server when I take report for 28th and then 29th and you can see how TESTRC invoice is shown whose payment date is a day before than invoice date.

Regards

image

image