mirror of
https://github.com/actualbudget/actual.git
synced 2026-07-15 22:54:03 -05:00
[GH-ISSUE #3347] [Bug]: Some filter types return no results #50496
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/actual#50496
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @oddstr13 on GitHub (Sep 2, 2024).
Original GitHub issue: https://github.com/actualbudget/actual/issues/3347
Verified issue does not already exist?
What happened?
Running
edge(due to CSV import not working inlatest), I pulled and re-tested just now, so should be the most recent nightly build.Create rule from transaction does not actually create a rule that matches the selected transaction.
At least the
isandmatchesfilters don't behave as expected.matcheswith.*onimported payeepulls up transactions, but.*Store.*does not.The
isfilter does not match the exact payee string populated by the «create rule» button, butcontainsdoes match it.Where are you hosting Actual?
Docker
What browsers are you seeing the problem on?
Firefox
Operating System
Linux
@bvanelli commented on GitHub (Sep 8, 2024):
Have been having weird issues too (version 24.9.0):
The following matches:
But one more character and it does not work:
Perhaps related to https://github.com/actualbudget/actual/issues/3392
@oddstr13 commented on GitHub (Sep 8, 2024):
Yes, #3392 looks like the exact same issue, except I was seeing it with more than only regex queries
@juanrapmc commented on GitHub (Sep 15, 2024):
I think it's not just because of case-sensitivity
Example that gets matches -- removed the first letter of the payee
Example that uses all small
Example that matches casing
@jfdoming commented on GitHub (Sep 15, 2024):
It is due to case sensitivity, but in a weird way (value is converted to lowercase, but the regex isn't, and vice versa in some cases). Sorry for the delay on pushing this PR out. Working on getting tests to pass, then will post.
@jfdoming commented on GitHub (Sep 15, 2024):
hey folks, any chance you could try out the linked PR (deployed here) and let me know if it fixes the issue you were seeing?
@juanrapmc commented on GitHub (Sep 15, 2024):
Looks good to me! Thanks for this!
@oddstr13 commented on GitHub (Sep 16, 2024):
I tested with the demo data, so haven't tested the imported payee field (as it doesn't seem to be populated there), but both exact matches and regex matches on the notes field seems to be working just fine on the deployed test instance.