Defect #33083
Projects filter "Subproject of" does not work when the given value is "My projects" or "My bookmarks"
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | % Done: | 0% | ||
Category: | Projects | |||
Target version: | 4.1.1 | |||
Resolution: | Fixed | Affected version: | 4.1.0 |
Description
Reproduction steps:
- Use PostgreSQL
- View projects/index
- Add "Subproject of" filter
- Set the operator of the "Subproject of" filter to "is" or "is not"
- Set the value of the "Subproject of" filter to << my projects >> or << my bookmarks >>
- Apply => An exception occurs
Error message:
ActiveRecord::StatementInvalid (PG::InvalidTextRepresentation: ERROR: invalid input syntax for integer: "mine" LINE 1: ...rojects.status <> 9) AND ((projects.parent_id IN ('mine'))) ... ^ : SELECT "projects".* FROM "projects" WHERE (projects.status <> 9) AND ((projects.parent_id IN ('mine'))) ORDER BY projects.lft ASC): app/controllers/projects_controller.rb:57:in `block (2 levels) in index' app/controllers/projects_controller.rb:53:in `index' lib/redmine/sudo_mode.rb:64:in `sudo_mode'
Executing the SELECT statement without converting << my projects >>('mine') and << my bookmarks >>('bookmarks') to actual ids is the cause of this problem.
Related issues
Associated revisions
Projects filter "Subproject of" does not work when the given value is "My projects" or "My bookmarks" (#33083).
Patch by Mizuki ISHIKAWA.
Fix a RuboCop offense Style/MultipleComparison (#33083).
History
#1
Updated by Mizuki ISHIKAWA 11 months ago
- File fix-33083.patch
added
I have attached a patch to fix this problem.
#2
Updated by Go MAEDA 11 months ago
- Related to Feature #29482: Query system for Projects page added
#5
Updated by Go MAEDA 11 months ago
- Subject changed from Filtering the parent_id field in Project by "My projects" or "My bookmarks" raises an exception to Projects filter "Subproject of" does not work when the given value is "My projects" or "My bookmarks"
- Status changed from Confirmed to Resolved
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the fix. Thank you for catching and fixing the issue.