Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Balance' column makes no sense anymore #5123

Closed
jpweaving opened this issue Sep 23, 2022 · 11 comments
Closed

'Balance' column makes no sense anymore #5123

jpweaving opened this issue Sep 23, 2022 · 11 comments

Comments

@jpweaving
Copy link

Hi.
The 'Balance' column makes no sense anymore, it no longer gives a running total. I think that it is now just freezing the figure at the balance shown when the transaction was entered into the account, then when listed in Date order, the closing balance for any particular day appears at the bottom entry, not the top. If this is a deliberate change, I may get used to it but it is of no use to me.
Thanks.
Please do not remove information attached below this text.


Version: 1.5.21 64-bit
• Build: Sep 21 2022 09:27:42
• db 7 • (aes128cbc)
• git cb708cc (2022-09-21)

Libs:
• wxWidgets 3.2.0 (wxMSW 10.0)
• wxSQLite3 4.6.0 (SQLite 3.31.1)
• RapidJSON 1.1.0
• Lua 5.3.5
• lunasvg v2.3.1
• libcurl/7.82.0-DEV Schannel
• gettext 0.19.8.1
• apexcharts.js

Build using:
• CMake 3.16.2
• MSBuild 15.9.21.664
• MSVS15.0
• MSVSC 19.16.27035.0
• Windows SDK 10.0.14393.0

OS:
• Windows 10 (build 19044), 64-bit edition
• English (United Kingdom) (windows-1252)
• 1920x1080 32bit 96x96ppi ![1 5 21](https://user-images.githubusercontent.com/67058137/191940670-ac2dbdae-498d-4466-a7a1-e0f46a24f57b.JPG)
@vomikan
Copy link
Member

vomikan commented Sep 23, 2022

Please note that the sorting algorithm has changed.
#5108

@n-stein
Copy link
Contributor

n-stein commented Sep 23, 2022

Also #5122

@PMaff
Copy link
Contributor

PMaff commented Sep 27, 2022

I do think that the balance column is an important feature.
Please keep it.
Although there is a bug in 1.5.20, 1.5.21 .
Will open an issue.

Edit: opened issue:
#5136

@whalley whalley closed this as completed Sep 29, 2022
@jpweaving
Copy link
Author

It's not fixed. It still displays the Balance incorrectly for multiple transactions on the same day if those transactions were entered in random order.

@whalley
Copy link
Member

whalley commented Sep 30, 2022

It's not fixed. It still displays the Balance incorrectly for multiple transactions on the same day if those transactions were entered in random order.

That has always been the case, we currently only have Date and order it was entered. We need #2729 to improve this.

@jpweaving
Copy link
Author

The sort order is not the problem in any way. In 1.5.20 the balance column always makes sense. each entry in the balance column accurately reflects the account balance after that transaction is taken into account. So the balance at the last transaction shown for any day accurately reflects the account balance at the end of that banking day, so the top figure in the balance column always matches the actual balance of that account. In 1.5.21 the balance doesn't make any sense unless you always enter transactions in the order that they occurred. You can end up with a situation where a transaction for a £1.00 withdrawal can result in a balance figure that is higher than the previous one. There is no point in that at all.

@whalley
Copy link
Member

whalley commented Sep 30, 2022

You refer to v1.5.21 as having an issue. We have made some fixes in the latest beta release to address this. You will either have to revert back to v1.5.20, take the latest beta, or wait for the next release.

@jpweaving
Copy link
Author

jpweaving commented Sep 30, 2022 via email

@jpweaving
Copy link
Author

jpweaving commented Sep 30, 2022 via email

@whalley
Copy link
Member

whalley commented Oct 1, 2022

For win64 - https://ci.appveyor.com/project/moneymanagerex/moneymanagerex/build/job/qp8sa43blbsx4nwk/artifacts

Note - the next build will be v1.6.0, please take a copy/backup of your database before trying beta builds. In particular for this release as it upgrades the database.

@jpweaving
Copy link
Author

jpweaving commented Oct 1, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants