Field names are not correctly handled in view filters #2233

Closed
opened 2024-03-23 13:13:21 +00:00 by waza-ari · 3 comments
Contributor

Description

When trying to create a View including a filter, some field names are not translated correctly. The following filter for example is stored as is, which will yield an internal server error as the column doesn't exist:

done = false || (done = true && doneAt > 'now-14d')

Manually editing the filter to

done = false || (done = true && done_at > 'now-14d')

yields the expected result

Vikunja Version

Unstable

Browser and version

No response

Can you reproduce the bug on the Vikunja demo site?

Yes

Screenshots

No response

### Description When trying to create a View including a filter, some field names are not translated correctly. The following filter for example is stored as is, which will yield an internal server error as the column doesn't exist: ``` done = false || (done = true && doneAt > 'now-14d') ``` Manually editing the filter to ``` done = false || (done = true && done_at > 'now-14d') ``` yields the expected result ### Vikunja Version Unstable ### Browser and version _No response_ ### Can you reproduce the bug on the Vikunja demo site? Yes ### Screenshots _No response_
waza-ari added the
kind/bug
label 2024-03-23 13:13:21 +00:00
Owner

Which fields are not translated?

Which fields are not translated?
Author
Contributor

So far I've only noticed that for doneAt, but by no means I did exhaustive tests.

So far I've only noticed that for `doneAt`, but by no means I did exhaustive tests.
Owner

Should be fixed in 13cab62d14. Please check with the next unstable build if the problem really went away.

Should be fixed in 13cab62d14d826e04c9dbfaf29dcadbd362dabfa. Please check with the next unstable build if the problem really went away.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#2233
No description provided.