https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292008-07-15T16:36:30ZRedmineRedmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=39152008-07-15T16:36:30ZArtem Vasiliev
<ul><li><strong>File</strong> <a href="/attachments/773">timelogging.PNG</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/773/timelogging.PNG">timelogging.PNG</a> added</li></ul> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=39342008-07-16T14:43:17ZArtem Vasiliev
<ul></ul><p>Notice that you might want to apply the patch at Windows machine via TortoiseSVN. It applies it just fine without any errors, while trying to apply the patch with 'patch -p0 file' on Linux gives me 3 errors like this:<br /><pre>patching file redmine/app/views/timelog/edit.rhtml
Hunk #1 FAILED at 3.
1 out of 1 hunk FAILED -- saving rejects to file redmine/app/views/timelog/edit.rhtml.rej
</pre><br />I tried 'diff -Nur' patch approach with the same result. The most sane solution in my case was to apply the patch on Windows and then move zip of the whole patched dir to Linux machine.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=41342008-07-29T13:13:23ZArtem Vasiliev
<ul></ul><p><a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Use Rails timezones support (New)" href="https://www.redmine.org/issues/1705">#1705</a> is needed to be applied for this patch to work in geographically distributed environment (user in one time zone, Redmine in another)</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=44112008-08-20T00:00:44ZMischa The Evil
<ul></ul><p><cite>Patch <a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Start/end time tracking for timelogging (New)" href="https://www.redmine.org/issues/1650">#1650</a></cite></p>
<blockquote>
<p><cite>Start/end time tracking for timelogging</cite></p>
</blockquote>
<p>Great patch I must say. I really like the approach this patch takes.</p>
<p>Now I've merged it with my personal branch (contains trunk @1745) doing some manual edits to resolve some minor conflicts I've got it running like described.<br />Though, I think I have just found one small error in the patch:</p>
<ul>
<li>The intersection-links, which are rendered while I recieve a flashnotice of an intersection, are missing the project-component. The correct url should be like:<pre>
http://xxx.xxx.xxx/projecta/timelog/edit/1
</pre>though it is now generated as:<pre>
http://xxx.xxx.xxx/timelog/edit/1</pre></li>
</ul>
<p>I've already tracked it down to this snippet in the <code>timelog_controller</code> (<em>../app/controllers/timelog_controller.rb</em>):<br /><pre><code class="ruby syntaxhl"> <span class="n">url_writer</span> <span class="o">=</span> <span class="nb">lambda</span> <span class="k">do</span> <span class="o">|</span><span class="n">entry</span><span class="o">|</span>
<span class="s2">"<a href = </span><span class="se">\"</span><span class="si">#{</span><span class="n">url_path</span><span class="p">(</span><span class="ss">:controller</span> <span class="o">=></span> <span class="ss">:timelog</span><span class="p">,</span> <span class="ss">:action</span> <span class="o">=></span> <span class="ss">:edit</span><span class="p">,</span> <span class="ss">:id</span> <span class="o">=></span> <span class="n">entry</span><span class="p">.</span><span class="nf">id</span><span class="p">)</span><span class="si">}</span><span class="se">\"</span><span class="s2">>#</span><span class="si">#{</span><span class="n">entry</span><span class="p">.</span><span class="nf">issue_id</span><span class="si">}</span><span class="s2">-</span><span class="si">#{</span><span class="n">entry</span><span class="p">.</span><span class="nf">id</span><span class="si">}</span><span class="s2"></a>"</span>
<span class="k">end</span></code></pre></p>
<p>At the moment I don't know precisely how to correct this code, to make the controller generate the correct URL's, due to my very-short time of basic experience with Ruby, RoR, Redmine.</p>
<p>I'll try to find my way around this... :-)</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=44232008-08-20T18:23:13ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>Great patch I must say. I really like the approach this patch takes.</p>
</blockquote>
<p>Thanks man!</p>
<blockquote>
<p>Though, I think I have just found one small error in the patch:</p>
<ul>
<li>The intersection-links, which are rendered while I recieve a flashnotice of an intersection, are missing the project-component. The correct url should be like: <a class="external" href="http://xxx.xxx.xxx/projecta/timelog/edit/1">http://xxx.xxx.xxx/projecta/timelog/edit/1</a></li>
</ul>
</blockquote>
though it is now generated as: <a class="external" href="http://xxx.xxx.xxx/timelog/edit/1">http://xxx.xxx.xxx/timelog/edit/1</a>
<p>Well, why do you think time entry url without project component is an error? It works fine because entry IDs are cross-project (it's plain artificial PK as usual in RoR).</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=44262008-08-20T23:07:54ZMischa The Evil
<ul></ul><p>At first thanks for the quick reply...</p>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>Mischa The Evil wrote:</p>
<blockquote>
<p>Though, I think I have just found one small error in the patch:</p>
<ul>
<li>The intersection-links, which are rendered while I recieve a flashnotice of an intersection, are missing the project-component. The correct url should be like: <a class="external" href="http://xxx.xxx.xxx/projecta/timelog/edit/1">http://xxx.xxx.xxx/projecta/timelog/edit/1</a></li>
</ul>
</blockquote>
<p>though it is now generated as: <a class="external" href="http://xxx.xxx.xxx/timelog/edit/1">http://xxx.xxx.xxx/timelog/edit/1</a></p>
<p>Well, why do you think time entry url without project component is an error? It works fine because entry IDs are cross-project (it's plain artificial PK as usual in RoR).</p>
</blockquote>
<p>Ok, I have to revise my "error-report" to be correct after looking at the problem with a clear mind...</p>
<p>I am running Redmine (on Apache + REE + Passenger) in a <strong>sub-URI</strong> of a domain (so xxx.xxx.xxx is home and xxx.xxx.xxx/redmine is where my Redmine is located).<br />After looking at my error again I notice that it is not missing any project-part in the URL (since, like you have stated before it works using the entry-ID's and not with project-ID's) but it <strong>is missing the sub-URI</strong> in the URL's of the intersection-links.</p>
<p>That would mean that <code>url_path</code> is returning only the <code>host.domain.tld</code>-part of the URL without passing any sub-URI.</p>
<p>I'll do some more research on this myself either...</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=44312008-08-21T05:59:09ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>That would mean that <code>url_path</code> is returning only the <code>host.domain.tld</code>-part of the URL without passing any sub-URI.</p>
</blockquote>
<p>That's interesting, and strange, as url_path just asks ::ActionController::Routing::Routes to generate the URL. Please tell if you find the cause of (or the solution to :) this.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=44452008-08-21T20:54:29ZMischa The Evil
<ul></ul><p>Artem Vasiliev wrote:</p>
<blockquote>
<p>Please tell if you find the cause of (or the solution to :) this.</p>
</blockquote>
<p>After several hours (again) of bug-hunting, code observation and reinterpretation, and trial-and-error-ing I have given up this issue.<br />It is not possible for me (according to my current knowledge of Ruby on Rails) to fix this error myself in near future.</p>
<p>Also due to some earlier problems I had running apps on sub-uri's I am now going to switch to my backup deployment paper. I'll now move the RoR-apps into seperated and dedicated vhosts.<br />I already lost too much time on fixing errors with apps installed in a sub-uri of a vhost (incl. other RM-instances).</p>
<p>I'll see in future if such an setup will become possible without problems, for now I'll drop it.</p>
<p><em><a class="user active" href="https://www.redmine.org/users/14013">Artem sviridov</a></em>: My earlier conclusion still holds though: "Great patch!". I even switch (lightweight) deployement architecture for it.. :)</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=44502008-08-22T06:32:25ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>Also due to some earlier problems I had running apps on sub-uri's I am now going to switch to my backup deployment paper. I'll now move the RoR-apps into seperated and dedicated vhosts.<br />I already lost too much time on fixing errors with apps installed in a sub-uri of a vhost (incl. other RM-instances).</p>
</blockquote>
<p>I guess that's much more popular environment, may be there are just some bugs in RoR when it's used in sub-uri environment.</p>
<blockquote>
<p><em><a class="user active" href="https://www.redmine.org/users/14013">Artem sviridov</a></em>: My earlier conclusion still holds though: "Great patch!". I even switch (lightweight) deployement architecture for it.. :)</p>
</blockquote>
<p>Thanks again, very nice to hear! ) <br />Be sure to check <a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Use Rails timezones support (New)" href="https://www.redmine.org/issues/1705">#1705</a> if you are working in timezone-distributed environment.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45102008-08-26T05:35:35ZMischa The Evil
<ul></ul><p>After a while of using my vendor-branch including this patch I found another issue regarding this patch...</p>
When using context-menu (right-click menu) in the issuelist, every change (even a more simpler priority-change) initiated through an action selected from the context-menu, brings me to the edit view for the issue showing me two errors saying:
<ol>
<li>« Project » is invalid</li>
<li>« Date » can't be blank</li>
</ol>
<p>In this view the requested changes are preselected. When I now click submit the change gets applied though without any problems.</p>
<p>It looks like, if using the context-menu, those two parameters aren't passed to the controller somehow...</p>
<p>What can be the cause or solution of this side-effect?</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45122008-08-26T08:11:19ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>After a while of using my vendor-branch including this patch I found another issue regarding this patch...</p>
When using context-menu (right-click menu) in the issuelist, every change (even a more simpler priority-change) initiated through an action selected from the context-menu, brings me to the edit view for the issue showing me two errors saying:
<ol>
<li>« Project » is invalid</li>
<li>« Date » can't be blank</li>
</ol>
</blockquote>
<p>Hm, looks strange to me. In our version of Redmine having this patch applied this works fine without any error messages, I've just tried it. And I don't remember fixing this :) Sorry, no clue.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45142008-08-26T10:27:56ZMischa The Evil
<ul></ul><p>Artem Vasiliev wrote:</p>
<blockquote>
<p>In our version of Redmine having this patch applied this works fine without any error messages, I've just tried it.</p>
</blockquote>
<p>What exact version of Redmine are you running? Then I'll search through following revisions for changes to the issue-controller to try to find the change of validation...</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45232008-08-26T13:55:11ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>In our version of Redmine having this patch applied this works fine without any error messages, I've just tried it.</p>
</blockquote>
<p>What exact version of Redmine are you running? Then I'll search through following revisions for changes to the issue-controller to try to find the change of validation...</p>
</blockquote>
<p>We run <a class="changeset" title="Fixes link in README." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1694">r1694</a> of <a class="external" href="http://redmine.rubyforge.org/svn/trunk">http://redmine.rubyforge.org/svn/trunk</a></p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45332008-08-27T12:43:28ZMischa The Evil
<ul></ul><p>Ok, after some thorough research and testing I think I have found the cause of the reported anomaly regarding the context-menu;</p>
<p>While updating my test-setup revision by revision (with and without the patch) I first noticed that the error started to show-up after applying <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a>.</p>
<p>In <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> timelog custom-fields are introduced.<br />It is that revision which adds a new line 163 to the <code>edit</code>-method in <em>../app/controllers/issues_controller.rb</em> which looks like:</p>
<pre><code class="ruby syntaxhl"> <span class="vi">@time_entry</span> <span class="o">=</span> <span class="no">TimeEntry</span><span class="p">.</span><span class="nf">new</span></code></pre>
<p>Now if I run a patched (this patch only) instance of <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> while commented-out the above mentioned new line I don't get the earlier mentioned errors while using the context-menu to modify an issue's priority. This result confirms my suspicion that it is indeed that line which causes the 2 errors.</p>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p><cite>In our version of Redmine having this patch applied this works fine without any error messages, I've just tried it.</cite></p>
</blockquote>
<blockquote>
<p><cite>We run <a class="changeset" title="Fixes link in README." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1694">r1694</a> of <a class="external" href="http://redmine.rubyforge.org/svn/trunk">http://redmine.rubyforge.org/svn/trunk</a></cite></p>
</blockquote>
<p>With the info I have posted above in the back of my mind, I am quite curious how the file <em>../app/controllers/issues_controller.rb</em> looks in your deployment. If it is indeed the line I mentioned which is causing the errors than it looks most likely that you've modified the file after posting the patch here.</p>
<p>I also compared the changes from your patch against the mentioned line to see if I could apply the patch'es changes onto that single line but I couldn't get it right... Though I see that your patch changes/extends more than once lines containing a <code>TimeEntry.new</code>-part.</p>
<p>Would it be possible for you to maybe update the patch for (full) compatibility with > <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> or post the related snippet of <em>../app/controllers/issues_controller.rb</em> from your deployment of <a class="changeset" title="Fixes link in README." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1694">r1694</a>?</p>
<p>Thanks in advance also for your great support upto now... :)</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45422008-08-27T21:39:20ZArtem Vasiliev
<ul></ul><p>Wow! Hats off to your investigation effort, and it's surprising to find myself in the middle of detective story )</p>
<blockquote>
<p>In <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> timelog custom-fields are introduced.<br />It is that revision which adds a new line 163 to the <code>edit</code>-method in <em>../app/controllers/issues_controller.rb</em> which looks like:<br />@time_entry = TimeEntry.new</p>
</blockquote>
<p>Same in our code! Now I tried this for issue without spent time history, and it does complain, though only about Time field. With some spent time log like yesterday was the case, everything was fine.<br />So, after timelogging was introduced we have such problem. I'll look at it more closely later.</p>
<p>Thanks for commenting whith some source code - this is important for us hungry code-eaters!</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45462008-08-28T01:07:33ZMischa The Evil
<ul></ul><a name="Re-offtopic"></a>
<h3 >Re (offtopic)<a href="#Re-offtopic" class="wiki-anchor">¶</a></h3>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>Hats off to your investigation effort</p>
</blockquote>
<p>Thanks :)</p>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>Thanks for commenting whith some source code - this is important for us hungry code-eaters!</p>
</blockquote>
<p>Your welcome. I'm willing to help as much as I can. Though it's sometimes difficult for me to catch-up the skills since I only have some more basic knowledge of structural PHP... :)<br />But, reading and trying to interpret other's code is a great way of leaning RoR to the skill I'd prefer myself to have...</p>
It's also a hobby, curiosity... I want to know what happens at which time... And like I said earlier: I really like this patch.<br />It is the only feature I am really missing in the core. Imho the time-tracking functionalities are there <strong>to track my spent time</strong> not only to collect my individual calculations of how long I worked in total on an issue.<br />When I'm 'working' on an issue I don't want to keep track of exact (in minutes) time of starting and ending. When I start working on an issue it is much more natural to 'inform' the tracking-system: "I'm started working on issue xxx!" or "I'm done working on this issue for now/this activity/this ..." than to 'post-log' the amount of time I've worked on an issue.<br />Besides the fact that this patch provides the above described start-/endtime functionality, it also maintains the flexibility of still being able to log time "manually".<br />Plus it adds some small, but quite neat features like:
<ul>
<li>time in-progress status (which I'd love to see extended with filters for issuelist - I understand that such things shouldn't be done using generic patches -> severity is too high imho)</li>
<li>Log Time link loads current In Progress time entry (if any) for current user</li>
<li>Same goes for Issue Edit form as it also has Log Time fields group</li>
<li>Intersection links</li>
</ul>
<p>This all makes this patch a kind of requirement for me to be able to do what I want using Redmine. It's a key feature to me.. :-)</p>
<a name="Continuation-and-temporarily-final-of-the-detective-story-ontopic"></a>
<h3 >Continuation and temporarily final of the detective story (ontopic)<a href="#Continuation-and-temporarily-final-of-the-detective-story-ontopic" class="wiki-anchor">¶</a></h3>
<p>Now, I'll finish my detective story with this post for the mean time :-)</p>
<p>Mischa The Evil wrote:</p>
<blockquote>
<p>Now if I run a patched (this patch only) instance of <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> while commented-out the above mentioned new line I don't get the earlier mentioned errors while using the context-menu to modify an issue's priority.</p>
</blockquote>
<p>This change <strong>does trigger also</strong> a side-effect which is doing more bad than good; it causes timelogs with zero hours (0.0) [issue <a class="issue tracker-1 status-5 priority-5 priority-high2 closed" title="Defect: Updating tickets add a time log with zero hours (Closed)" href="https://www.redmine.org/issues/1147">#1147</a>] to be created while updating an issue using the context-menu.</p>
<p>This behaviour made me thinking: everytime I manually update an issue the timelog hour field is default filled with 0.0 due to config of the DB. I have to manually remove it everytime I update the issue without adding a time-entry. Now it is written down as an empty timelog while updating using the context-menu.<br />That led me to the conclusion that the earlier mentioned line is there (logical since it is defined in the <code>edit</code>-method) to make sure that no timelog is created while an issue is updated via an action triggered from the context-menu.</p>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>Now I tried this for issue without spent time history, and it does complain, though only about Time field.</p>
</blockquote>
<p>This distinction makes it even more difficult to find the exact cause of these errors :S</p>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>So, after timelogging was introduced we have such problem.</p>
</blockquote>
<p>I now have isolated the cause of the error to revision <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> indeed (plus the related revisions <a class="changeset" title="Adds custom fields to the time entries csv export." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1690">r1690</a> & <a class="changeset" title="Adds boolean and list custom fields for time entries as criteria on timelog report." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1691">r1691</a>). <br />I've now reverted these revisions (and these revisions only) manually on a fresh checkout of the trunk at <a class="changeset" title="Extracted rcov options and removed gems from the rcov report." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1745">r1745</a> - in fact undoing all the changes made for the implementation of timelog custom fields - which solves the caused errors.<br />It works flawlessly after those changes (though I loose the support for this type of custom fields). It's actually quite easy to do...</p>
<p>Artem Vasiliev wrote:</p>
<blockquote>
<p>I'll look at it more closely later.</p>
</blockquote>
<p>That would be great! It would be superb if this patch could be kept/made "compatible" against more recent trunk untill comparable features are added to the Redmine core.<br />Though it's not a major issue for me now since it is still possible to revert the timelog custom field without creating hugh conflicts. Since I'm (currently) not using timelog custom fields I've chosen to stick with that solution for the time being. The provided functionality of this patch is more important <strong>to me</strong> than the ability to use custom fields for timelogs.</p>
<p>Again: thanks for all the help and support...</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45702008-08-29T05:01:07ZMischa The Evil
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>Mischa The Evil wrote:</p>
<blockquote>
<p>Now if I run a patched (this patch only) instance of <a class="changeset" title="Adds custom fields on time entries (#772)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1689">r1689</a> while commented-out the above mentioned new line I don't get the earlier mentioned errors while using the context-menu to modify an issue's priority.</p>
</blockquote>
<p>This change <strong>does trigger also</strong> a side-effect which is doing more bad than good; it causes timelogs with zero hours (0.0) [issue <a class="issue tracker-1 status-5 priority-5 priority-high2 closed" title="Defect: Updating tickets add a time log with zero hours (Closed)" href="https://www.redmine.org/issues/1147">#1147</a>] to be created while updating an issue using the context-menu.</p>
</blockquote>
<p>This has nothing to do with the change of commenting-out that line. See my <a href="http://www.redmine.org/issues/show/1147#note-11" class="external">note</a> on issue <a class="issue tracker-1 status-5 priority-5 priority-high2 closed" title="Defect: Updating tickets add a time log with zero hours (Closed)" href="https://www.redmine.org/issues/1147">#1147</a>.</p>
<p>This behaviour is probably created by this patch with that line commented-out or not...</p>
<p>Artem, could you please test this (lookup for 0.00 timelogs) and report back here?</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45912008-09-01T12:07:26ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
I want to know what happens at which time... And like I said earlier: I really like this patch.<br />It is the only feature I am really missing in the core. Imho the time-tracking functionalities are there <strong>to track my spent time</strong> not only to collect my individual calculations of how long I worked in total on an issue.<br />When I'm 'working' on an issue I don't want to keep track of exact (in minutes) time of starting and ending. When I start working on an issue it is much more natural to 'inform' the tracking-system: "I'm started working on issue xxx!" or "I'm done working on this issue for now/this activity/this ..." than to 'post-log' the amount of time I've worked on an issue.<br />Besides the fact that this patch provides the above described start-/endtime functionality, it also maintains the flexibility of still being able to log time "manually".<br />Plus it adds some small, but quite neat features like:
<ul>
<li>time in-progress status (which I'd love to see extended with filters for issuelist - I understand that such things shouldn't be done using generic patches -> severity is too high imho)</li>
<li>Log Time link loads current In Progress time entry (if any) for current user</li>
<li>Same goes for Issue Edit form as it also has Log Time fields group</li>
<li>Intersection links</li>
</ul>
<p>This all makes this patch a kind of requirement for me to be able to do what I want using Redmine. It's a key feature to me.. :-)</p>
</blockquote>
<p>Thanks Mischa! That's also exactly why we did this patch :) Glad to see that someone understood this feature so well.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=45922008-09-01T12:33:45ZArtem Vasiliev
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote><blockquote>
<p>This change <strong>does trigger also</strong> a side-effect which is doing more bad than good; it causes timelogs with zero hours (0.0) [issue <a class="issue tracker-1 status-5 priority-5 priority-high2 closed" title="Defect: Updating tickets add a time log with zero hours (Closed)" href="https://www.redmine.org/issues/1147">#1147</a>] to be created while updating an issue using the context-menu.</p>
</blockquote>
<p>This has nothing to do with the change of commenting-out that line. See my <a href="http://www.redmine.org/issues/show/1147#note-11" class="external">note</a> on issue <a class="issue tracker-1 status-5 priority-5 priority-high2 closed" title="Defect: Updating tickets add a time log with zero hours (Closed)" href="https://www.redmine.org/issues/1147">#1147</a>.</p>
<p>This behaviour is probably created by this patch with that line commented-out or not...</p>
<p>Artem, could you please test this (lookup for 0.00 timelogs) and report back here?</p>
</blockquote>
<p>Thanks for the link! I'm glad you noticed the issue was gone with <a class="changeset" title="Sligth changes to issue comments quoting links." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1773">r1773</a>. We're going to publish our Redmine fork at GitHub, and rebase it against latest trunk. This will hopefully make future rebasings against Redmine trunk easier.<br />Also you'll be welcome to try our other changes to Redmine, like diffs for issue descriptions (<a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Show diff for issue description change (Closed)" href="https://www.redmine.org/issues/1717">#1717</a>) and remaining time calculation for version (<a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Show a breakdown of estimated/spent/remaining time for a version (New)" href="https://www.redmine.org/issues/1671">#1671</a>). Our future Redmine development is going to be in GitHub, duplicated by SVN patches.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=46022008-09-02T18:04:00ZArtem Vasiliev
<ul></ul><p>Artem Vasiliev wrote:</p>
<blockquote>
<p>We're going to publish our Redmine fork at GitHub, and rebase it against latest trunk. This will hopefully make future rebasings against Redmine trunk easier.</p>
</blockquote>
<p>Allright, here it is: <a class="external" href="http://github.com/artemv/redmine_tt/tree/master">http://github.com/artemv/redmine_tt/tree/master</a>. You can use 'download' link or use git client.</p>
<p>This is basically what we use, but upgraded to <a class="changeset" title="Adds cross-project time reports support (#994)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/1778">r1778</a>. I tried changing issue priority by context menu in issue list - works like a charm, and no zero timelog entry created.</p>
<p>Our version appears to have some unit tests broken, I going to look at that. More development to come soon, first we're going to improve <a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Show a breakdown of estimated/spent/remaining time for a version (New)" href="https://www.redmine.org/issues/1671">#1671</a> (resolved issues shouldn't add to 'remaining' time in our opinion).</p>
<p>Looks like we need to move this thread to forums..</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=48052008-09-16T01:41:13ZMischa The Evil
<ul></ul><p><strong><em>Artem Vasiliev wrote:</em></strong></p>
<blockquote>
<p>Allright, here it is: <a class="external" href="http://github.com/artemv/redmine_tt/tree/master">http://github.com/artemv/redmine_tt/tree/master</a>.</p>
</blockquote>
<p>Great!!! Thanks for the taken actions to further develop your (TT's) Redmine fork in a public git-repo...</p>
<p><strong><em>Artem Vasiliev wrote:</em></strong></p>
<blockquote>
<p>Looks like we need to move this thread to forums..</p>
</blockquote>
<p>I just created <a href="http://www.redmine.org/boards/1/topics/show/2352" class="external">a new forum-thread</a> as a continuation of the discussion here regarding the patch & as an extension to the discussion regarding other changes in the fork by TT.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=60662008-12-10T17:40:26ZRiccardo Delpopolo
<ul></ul><p>Will this patch applied to the trunk???<br />Can i eventually add them to the trunk?</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=60672008-12-10T18:07:42ZRiccardo Delpopolo
<ul></ul><p>Wold be useful to have a list of started spent time.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=70892009-02-03T15:25:05ZJase Roberts
<ul></ul><p>We've just started using Redmine in our web shop, and also would really like to see this feature be officially implemented in the trunk. Discussion on this is currently fragmented here, plus:<br />Patch: <a class="external" href="http://www.redmine.org/issues/show/1650">http://www.redmine.org/issues/show/1650</a><br />Patch/fork discussion: <a class="external" href="http://www.redmine.org/boards/1/topics/show/2352">http://www.redmine.org/boards/1/topics/show/2352</a><br />Idea for feature: <a class="external" href="http://www.redmine.org/boards/1/topics/show/1027">http://www.redmine.org/boards/1/topics/show/1027</a></p>
<p>Having only skimmed these latter three, I'm not completely clear on the status of the patch (but very much appreciate the effort that has gone into it). Would be great to see this in trunk. We're currently also using SlimTimer, but repetition of data entry is a pain and prone to error.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=71162009-02-04T17:08:31ZArtem Vasiliev
<ul></ul><p>Thanks Jase!</p>
<p>And you see, it also supports <a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Show a breakdown of estimated/spent/remaining time for a version (New)" href="https://www.redmine.org/issues/1671">#1671</a> (Show a breakdown of estimated/spent/remaining time for a version) quite well: for <a class="issue tracker-3 status-1 priority-4 priority-default" title="Patch: Show a breakdown of estimated/spent/remaining time for a version (New)" href="https://www.redmine.org/issues/1671">#1671</a> it's crucial to have spent time carefully entered for tickets, and if you forgot to enter spent time at some point it's much easier to remember how much did you miss if you have start/end times and view on person's spent time during the day showing it in right order.</p>
<p>Best regards,<br />Artem</p>
<blockquote>
<p>We've just started using Redmine in our web shop, and also would really like to see this feature be officially implemented in the trunk. Discussion on this is currently fragmented here, plus:<br />Patch: <a class="external" href="http://www.redmine.org/issues/show/1650">http://www.redmine.org/issues/show/1650</a><br />Patch/fork discussion: <a class="external" href="http://www.redmine.org/boards/1/topics/show/2352">http://www.redmine.org/boards/1/topics/show/2352</a><br />Idea for feature: <a class="external" href="http://www.redmine.org/boards/1/topics/show/1027">http://www.redmine.org/boards/1/topics/show/1027</a></p>
<p>Having only skimmed these latter three, I'm not completely clear on the status of the patch (but very much appreciate the effort that has gone into it). Would be great to see this in trunk. We're currently also using SlimTimer, but repetition of data entry is a pain and prone to error.</p>
</blockquote> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=73772009-02-17T06:49:41ZPetr Cervenka
<ul></ul><p>+1 for some support for 3rd party software (SlimTimer or Any other Desktop/Wep Application) which can update/modify spend time on various tickets.</p>
<p>That would be awesome, <br />Petr</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=81262009-03-18T18:59:16ZRiccardo Delpopolo
<ul></ul><p>I've modified the patch to work with the <a class="changeset" title="Fixes that user's last_login_on was not set when using registration with automatic activation." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/2594">r2594</a><br />Where can i upload it?</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=81432009-03-19T06:52:22ZArtem Vasiliev
<ul></ul><p>Riccardo Delpopolo wrote:</p>
<blockquote>
<p>I've modified the patch to work with the <a class="changeset" title="Fixes that user's last_login_on was not set when using registration with automatic activation." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/2594">r2594</a><br />Where can i upload it?</p>
</blockquote>
<p>Cool!<br />Uploading patch here is fine for me; also consider making a separate branch at GitHub (clone <a class="external" href="http://github.com/edavis10/redmine/tree/master">http://github.com/edavis10/redmine/tree/master</a>) for this patch like Eric Davis suggested <a href="http://www.redmine.org/issues/1671#note-11" class="external">here</a> - this will ease the integration of the patch to Redmine trunk. Please post the link here if you do.</p>
<p>Best regards,<br />Artem</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=81482009-03-19T12:36:45ZRiccardo Delpopolo
<ul><li><strong>File</strong> <a href="/attachments/1721">timelogging.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/1721/timelogging.patch">timelogging.patch</a> added</li></ul><p>Please, check it, because i've modified it by hand.<br />I've applied this patch to a fresh <a class="changeset" title="Fixes that user's last_login_on was not set when using registration with automatic activation." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/2594">r2594</a> with an old database.<br />I had only a problem patching two files, so i did the modify by hand.</p>
<p>I've also modificed the references to the deprecated lws function to l_hours</p>
<p>It's my first patch and very new to rubyonrails :)<br />So if you can, help me if i was wrong, and give me some resources where to learn</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=81702009-03-21T01:58:26ZRiccardo Delpopolo
<ul></ul><p>It should be ok :)<br />Try it and make me know if it's ok</p>
<p><a class="external" href="http://github.com/RDelpopolo/redmine/tree/tt">http://github.com/RDelpopolo/redmine/tree/tt</a></p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=83242009-03-30T02:04:41ZMischa The Evil
<ul><li><strong>File</strong> <a href="/attachments/1796">timelogging-r2641.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/1796/timelogging-r2641.patch">timelogging-r2641.patch</a> added</li></ul><p>Riccardo Delpopolo wrote:</p>
<blockquote>
<p>Try it and make me know if it's ok</p>
<p><a class="external" href="http://github.com/RDelpopolo/redmine/tree/tt">http://github.com/RDelpopolo/redmine/tree/tt</a></p>
</blockquote>
Riccardo, I've tested your patchfile (manually applied the changes) against Redmine trunk at <a class="changeset" title="Fixed: News summary field is not searchable (#2998)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/2641">r2641</a> but it seems it's becoming hopelessly broken by now:
<ul>
<li>Missing proper translations for Rails 2.2.2</li>
<li>Intersection links aren't caught any longer</li>
<li>Sorting on the start/end times is broken in the timelog report</li>
<li>Editing a timelog-entry leads to a 500-error</li>
<li>Still suffers the earlier noted bug when changing issue's status using the context-menu (though workaround still works), see <a class="external" href="http://www.redmine.org/boards/1/topics/show/2352#Proposed-workaroundfix">http://www.redmine.org/boards/1/topics/show/2352#Proposed-workaroundfix</a></li>
</ul>
<p>I'd like to raise a bounty for building a plugin which provides the exact same solutions of the initial iteration of this patch by Artem Vasiliev. That would be a great thing to have provided by a plugin instead of a core-hack...</p>
<p>Also interesting to follow in this context is the proposed solution by Eggplant Digital (see <a class="external" href="http://www.redmine.org/issues/440#note-15">http://www.redmine.org/issues/440#note-15</a>)...</p>
<p>Note: I'll attach the patch-file I've created (and used) against <a class="changeset" title="Fixed: News summary field is not searchable (#2998)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/2641">r2641</a>.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=632202015-04-19T16:22:42Z[ Desperados ]
<ul></ul><p>this is what I need, does it work with redmine 2.6.x ?</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=682942016-01-06T10:55:29ZAnonymous
<ul></ul><p>I need that too for redmine 2.6.5.stable.</p> Redmine - Patch #1650: Start/end time tracking for timelogginghttps://www.redmine.org/issues/1650?journal_id=904302019-02-26T07:29:48ZMichael A
<ul></ul><p>This looks fantastic! Please incorporate it.</p>