moneymanagerex / moneymanagerex

Money Manager Ex is an easy to use, money management application built with wxWidgets
http://moneymanagerex.org
GNU General Public License v2.0
1.79k stars 277 forks source link

Transaction report filters for "transfer out" and "transfer in" not working as expected #4684

Closed cmbarbu closed 2 years ago

cmbarbu commented 2 years ago

MMEX version:

Note: bug reporters are expected to have verified the bug still exists either in the last stable version of MMEX or on updated development code (master branch).

Operating System:

Description of the bug

Regardless of which box is checked ("Transfer out", "Transfer in", or both), both types of transfers are shown in the transaction report.

image

Reproduction

Is the bug reproducible?

Reproduction steps:

  1. Create a transaction that is a transfer between two accounts
  2. Create a transaction report and select Type: "Transfer in"

Expected result: Only one transaction ("in") should be displayed

Actual result: Both sides of the transaction ("in" and "out") are displayed

whalley commented 2 years ago

There was an extensive discussion on this here: https://github.com/moneymanagerex/moneymanagerex/issues/4239

@cmbarbu Can you confirm if you have All Accounts selected in the filter? In this case you will get Transfers In and Out as noted in above issue.

However I have spotted something that looks odd... When an empty filter is edited and type only selected the allows both Transfer Out / Transfer In to be specified which is wrong as 'All Accounts' is selected. CleanShot 2022-06-21 at 01 16 18

If I select the Account checkbox and unselect then only 'Transfer' is allowed - which is correct. CleanShot 2022-06-21 at 01 18 42

cmbarbu commented 2 years ago

@cmbarbu Can you confirm if you have All Accounts selected in the filter? In this case you will get Transfers In and Out as noted in above issue.

@whalley No, I had not filtered by Account. Otherwise I might have noticed this part as well:

If I select the Account checkbox and unselect then only 'Transfer' is allowed - which is correct.

Good catch!

Thank you for the quick fix!