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.

Sign up for GitHub

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

Jump to bottom

Error applying user defined filters with colon in property name after reopening the case #1392

Closed
lfcnassif opened this issue Oct 20, 2022 · 0 comments · Fixed by #1393
Closed

Error applying user defined filters with colon in property name after reopening the case #1392

lfcnassif opened this issue Oct 20, 2022 · 0 comments · Fixed by #1393
Assignees
@lfcnassif
Labels
bug

Comments

@lfcnassif
Copy link
Member

After creating a new filter in analysis UI using some property name with colon, if user closes and reopens the same (or another) case, trying to apply the new filter results in error below:
image

That's because the escape char is being lost.
@lfcnassif lfcnassif added the bug label Oct 20, 2022
@lfcnassif lfcnassif self-assigned this Oct 20, 2022
lfcnassif added a commit that referenced this issue Oct 20, 2022
@lfcnassif
'#1392: fix UTF8Properties class to not replace \: by :
f4c1f9d
lfcnassif added a commit that referenced this issue Oct 20, 2022
@lfcnassif
'#1392: back compat code to fix old filters saved by old versions
6c9c6b4
lfcnassif added a commit that referenced this issue Oct 20, 2022
@lfcnassif
'#1392: replace \\: by \: in DefaultFilters.txt config file
7ca1e97
@lfcnassif lfcnassif mentioned this issue Oct 20, 2022
Merged
@lfcnassif lfcnassif changed the title Error applying user defined filters with colon in key name after reopening the case Error applying user defined filters with colon in property name after reopening the case Oct 20, 2022
lfcnassif added a commit that referenced this issue Oct 20, 2022
@lfcnassif
'#1392: fix UTF8Properties class to not replace \: by :
1474152
lfcnassif added a commit that referenced this issue Oct 20, 2022
@lfcnassif
'#1392: back compat code to fix old filters saved by old versions
6b26eb9
lfcnassif added a commit that referenced this issue Oct 20, 2022
@lfcnassif
'#1392: replace \\: by \: in DefaultFilters.txt config file
650881a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lfcnassif lfcnassif

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#1392 fix user filters sepinf-inc/IPED
1 participant
@lfcnassif