Unexpected space added to Category filter when Regex is used to include multiple sub categories

[rmelillo76]

MMEX version:

• 1.6.1

I have a category named "Home Projects" with multiple sub-categories. I wanted to create a report that would show the "Porch 2021" and "Porch 2022" subcategories only. I also have some transactions categorized directly under "Home Projects" only that I do not want to include and have other sub categories, so the "Include all sub-categories" checkbox doesn't meet my needs.

So I created (and saved) the following transaction filter and it worked perfectly:

However the next time I ran the filter, no transactions were displayed. Upon looking closer, I discovered an extra space was added after the : delimiter when I ran or opened the filter:

I also confirmed this in the database (after saving):

My current workaround is to use this Regex which avoids the category delimiter and there is no issue. This works but I thought I would share incase others are impacted by this behavior

Reproduction

Is the bug reproducible?

• Always

Also see #5391

[n-stein]

Fixed in PR #5398

[n-stein]

I think this also fixes #5391

Does not.

Now it does.

[rmelillo76]

Thanks @n-stein, unfortunately the problem still persists in the latest build after your change.
To confirm, my original regex is

Home Projects: Porch.*

and my delimiter is ": "

The filter works initially but then it gets saved with an extra space

Causing the report to not work:

[rmelillo76]

And as I test this some more, I realize that each time I open the filter dialog an additional space is added:



etc...

[n-stein]

Which build did you test? You need to test 5928. Build 5929 contains a fix for a different issue, doesn't contain this code fix.

Here: https://ci.appveyor.com/project/moneymanagerex/moneymanagerex/builds/45583834

[rmelillo76]

Which build did you test? You need to test 5928. Build 5929 contains a fix for a different issue, doesn't contain this code fix.

Here: https://ci.appveyor.com/project/moneymanagerex/moneymanagerex/builds/45583834

Thanks @n-stein, that did the trick. Confirmed working now!