Filters do not take into account transactions where the categories are split

[Dellyes]

Dans la Liste des opérations le résultats des filtres ne tiennent pas compte des opérations où la catégorie a été répartie

---

Version: 1.5.15 64-bit
• Build: Juin 16 2022 09:13:44
• db 7 • (aes128cbc)
• git e7bb96c (2022-06-16)

Libs:
• wxWidgets 3.1.5 (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
• French (windows-1252)
• 1360x768 32bit 96x96ppi

[tactilis]

The title says: "In the list of operations, the results of the filters do not take into account operations where the category has been distributed"

@Dellyes

1. You will need to provide more information. The nature of the problem you are having is not clear.
   Please report bugs on this form https://github.com/moneymanagerex/moneymanagerex/issues/new/choose and provide the information requested.

[whalley]

There was an error with split transactions, that was fixed here: whalley@9a6b832

Can you try the latest beta build. https://moneymanagerex.org/docs/downloads/development/

#4662

[Dandy64]

Sorry, but the problem still exists.
Last version 1.5.18, Windows, Portable, 64 bit

Steps.

1. Created a new DB with just 1 account - Bank
2. Created 2 transactions:

1 transaction - 200:

2 Transaction 100+50 =150:

Total - 350:

3. Created Transaction filter
   

Expected result: 150 + 200 = 350
Actual result: 200

The split transaction was not taken into account.
p.s. All the transactions were reconciled.
p.s.1 If you need it, I can send the Test DB.

[whalley]

Adding
if (dlg.get()->mmIsCategorySubCatChecked()) value = value + ".*";

after:

moneymanagerex/src/reports/transactions.cpp

Line 398 in 6349819

const auto& value = catFilter ? dlg.get()->mmGetCategoryPattern() : "";

should possibly fix it (need to remove the const also)

[vomikan]

Adding if (dlg.get()->mmIsCategorySubCatChecked()) value = value + ".*";

after:

moneymanagerex/src/reports/transactions.cpp

Line 398 in 6349819

const auto& value = catFilter ? dlg.get()->mmGetCategoryPattern() : "";

should possibly fix it (need to remove the const also)

I think I did it in my own way.

[tauchmi]

It seems with 1.6.2 (64 bit) this is again an issue. You can simulate it easily with a new database.

It only happens when I filter by a top category and select "Include all sub-categories".
test.zip

[n-stein]

@vomikan I see this is closed again but I didn't see a fix. If it is already handled let me know and I'll close PR #5515.

[vomikan]

@vomikan I see this is closed again but I didn't see a fix. If it is already handled let me know and I'll close PR #5515.

Unfortunately, I got confused in this case. I saw your comment after that I closed the case.
If something went wrong, feel free to fix it.