Defect #32973
closed
Editing a time entry for a locked user changes the user to the current user
0%
Description
Steps to reproduce:
1. Add a time entry for another user (B)
2. Lock that user (B)
3. Edit the time entry
4. Observe that field User is selected with "me".
Saving the time entry, it will lose the user.
Files
Related issues
Updated by Marius BĂLTEANU almost 5 years ago
- Related to Feature #3848: Permission to log time for another user added
Updated by Marius BĂLTEANU almost 5 years ago
I see two options to fix this:
1. We don't allow changing the user when the time entry is logged on a locked user.
2. We add the locked user to the list of users.
Updated by Go MAEDA almost 5 years ago
Marius BALTEANU wrote:
I see two options to fix this:
1. We don't allow changing the user when the time entry is logged on a locked user.
2. We add the locked user to the list of users.
I think the second option is better because the limitation suggested in option 1 may confuse users. In my opinion, there is no reason to disallow changing the user when the user of the time entry is locked.
Updated by Marius BĂLTEANU almost 5 years ago
- File 0002-Reorder-and-rename-test-added-in-32959.patch 0002-Reorder-and-rename-test-added-in-32959.patch added
- File 0001-Allow-editing-a-time-entry-logged-on-a-locked-user.patch 0001-Allow-editing-a-time-entry-logged-on-a-locked-user.patch added
- Assignee deleted (
Marius BĂLTEANU)
Here is the fix for the second option, but I would like a double check on the solution. The second patch renames and reorders the test added in #32959.
All tests pass: https://gitlab.com/redmine-org/redmine/pipelines/116292306
Updated by Jean-Philippe Lang almost 5 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Committed and merged, thanks!
Updated by Go MAEDA over 4 years ago
- Related to Defect #33341: Time entry user is shown twice in the User drop-down when editing spent time added