Patch #5441
closedInitialize TimeEntry attributes with params[:time_entry]
0%
Description
I've made a one-liner patch to fill the new TimeEntry attributes created in IssuesController#update_issue_from_param with the params[:time_entry].
This allows to pre-fill the spent time field when accessing the issue editing page.
I want to use this feature for my time tracker plugin. When the user stops his time tracker, I want to redirect him to the issue editing page to let him change what he wants (%done ratio for example) and enter a time log comment. However, I need to be able to fill the spent time field for him with the time tracker value.
Files
Updated by Ludovic Gasc about 14 years ago
- Assignee changed from Eric Davis to Jean-Philippe Lang
+1
I valid this patch, it's useful with this plugin: http://github.com/delaitre/redmine_time_tracker
Moreover, some other plugins could use this feature to pre-populate the timesheet part in the edit form.
Somebody in the Redmine team could integrate this patch in the Redmine trunk ?
Thanks.
Updated by Jérémie Delaitre almost 14 years ago
I still need this patch (1 line!). Is there any chance to get it included?
Updated by Jean-Philippe Lang almost 14 years ago
Could you update the patch for current trunk and include a functional test ?
Updated by Jérémie Delaitre almost 14 years ago
- File time_entry_init.diff time_entry_init.diff added
I've updated the patch according to current github master.
The 'test_get_edit_with_params' functional test has been updated too to check the time_entry params.
Updated by Jérémie Delaitre almost 14 years ago
I forgot to ask you to execute the updated test as I'm not confident in my redmine installation...
Updated by Jérémie Delaitre almost 14 years ago
- File time_entry_init.diff time_entry_init.diff added
Ok, I finally have a working redmine install for testing purpose. It appears that my previous patch contains a bug in the functional test. I fixed it in this new patch (which should be the last one!).
Updated by Jérémie Delaitre over 13 years ago
Any chance to have this included in 1.0.4 or 1.1.0 ?
Updated by Jean-Philippe Lang over 13 years ago
- Status changed from New to Closed
- Target version set to 1.1.0
Patch commited in r4411.
As it's not a fix, it won't be merged in 1.0-stable but in 1.1.
Thanks.