Defect #14817
closedRedmine loses filters after deleting a spent time
0%
Description
When you add filters in the spent time "Details" view, and then delete an entry, the refresh loses the filters you had.
Steps to reproduce:- Add a filter
- Delete a spent time
Expected result:
Refresh with the filter you had
Actual result:
Refresh with the default page
Files
Related issues
Updated by Marius BĂLTEANU about 8 years ago
This issue reproduces also on the latest trunk version.
Updated by Marius BĂLTEANU about 8 years ago
I think that this issue is generated by another issue: time spent filters are not stored in the session.
To reproduce:
- Go to the spent time tab
- Create a filter
- Navigate to issues tab
- Return to the spent time tab
- Observe that the filters are reseted.
Updated by Marius BĂLTEANU about 8 years ago
The below change seems to fix the issue. I'm going to make a patch with tests for this.
diff --git a/app/controllers/timelog_controller.rb b/app/controllers/timelog_controller.rb old mode 100644 new mode 100755 index 6c0cd3e..2edf556 --- a/app/controllers/timelog_controller.rb +++ b/app/controllers/timelog_controller.rb @@ -281,6 +281,6 @@ private end def retrieve_time_entry_query - retrieve_query(TimeEntryQuery, false) + retrieve_query(TimeEntryQuery, true) end end
Updated by Marius BĂLTEANU about 8 years ago
Attached is a patch that fixes the issue reported initially in this ticket and the issue from note-2. Please let me know if a should open a new ticket for the issue from note-2.
Updated by Go MAEDA about 8 years ago
- Target version set to 3.3.2
Thank you for the patch, it works fine for me and passed all tests.
In addition, the patch also fixes #13622.
Setting target version to 3.3.2.
Updated by Go MAEDA about 8 years ago
- Related to Defect #13622: "Clear" button in Spent Time Report tab also clears global filters added
Updated by Jean-Philippe Lang about 8 years ago
- Status changed from New to Resolved
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Marius BALTEANU wrote:
Attached is a patch that fixes the issue reported initially in this ticket and the issue from note-2. Please let me know if a should open a new ticket for the issue from note-2.
Storing the filters in session is a different behaviour, please open another ticket and we'll see if this is desirable, thanks.
I've have the reported issue in r16056 without changing this behaviour.
Updated by Marius BĂLTEANU about 8 years ago
Jean-Philippe Lang wrote:
Marius BALTEANU wrote:
Attached is a patch that fixes the issue reported initially in this ticket and the issue from note-2. Please let me know if a should open a new ticket for the issue from note-2.
Storing the filters in session is a different behaviour, please open another ticket and we'll see if this is desirable, thanks.
I've have the reported issue in r16056 without changing this behaviour.
Here is the new ticket #24581.
Updated by Jean-Philippe Lang about 8 years ago
- Subject changed from Redmine loses filters after deleting a spent time. to Redmine loses filters after deleting a spent time
- Status changed from Resolved to Closed
Updated by Go MAEDA almost 8 years ago
- Has duplicate Defect #9016: displayed page after deleting time_entry added
Updated by Marius BĂLTEANU almost 7 years ago
- Related to Defect #17302: Forwarding from issues time entries page to project time entries page added