Thanks @kvvaradha and @notrinos. Will fix this later.

Joe

This has been fixed and committed to core. Thanks @notrinos for detecting and solving the issue.

Joe

Yes, but we can't fix the db change during 2.4.X.

Can it be used without the db change?

Joe

Thanks @notrinos.

We can fix this to the next major, 2.5. It is not possible to change db structure during minor releases.

Joe

@rafat

If you only need to see the total debits and credits you can use the Trial Balance Inquiry in Banking and General Ledger and the trial Balance Report in General Ledger Reports. They will show the total debits and credits for the selected period.

/Joe

It is automatically mirrored to /FrontaccountingErp/FA.

Joe

This has been solved by including a checkbox in the Customer Transactions 'Zero values'. Marking this will show the voided and zero valued records.

Will be committed in a while.

/Joe

This has been updated. Will be committed in a while.

/Joe

Committed

Yes, maybe a company flag. Will have a look.

Joe

Will have a look later.

Joe

I guess the dimension fields in the bank enquiries are redundant. In my opinion it gives no meaning to use dimensions in the bank enquiry.

Joe

No, not really. However in normal conditions the old code should work.

Joe

This item has been rolled back due to new bugs in the fixed solution.

Committed with whitespace fixes.

/Joe

@kvvaradha

There seems to be a bug in our commitment. Do you have a solution or should we roll it back?

Joe

Ok, let me investigate this a little more.

Joe

But isn't the voided entries all set to 0 amounts? So in my eyes this is a redundant change, right?

Joe

@kvvaradha

I am out of office until Friday. Will fix and commit it to stable then. Thanks.

Joe

@boxygen

I am out of office until Friday. Will fix it then and commit. Thanks.

Joe

Fixed and committed to stable.

Joe

Fixed and commited to stable.

Joe

If this has been tested for a while I think we can include it. Is that so @kvvaradha?

Joe

722

(2 replies, posted in Setup)

Committed to stable repo. Can be downloaded here.

/Joe

723

(2 replies, posted in Setup)

Will do later. Thanks.

Joe

I don't know. I am out of office a week. Will look at it when I am back.

Joe

If this test is positive we will implement it immediately.

Joe