https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292011-01-26T19:06:28ZRedmineRedmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245232011-01-26T19:06:28ZEtienne Massip
<ul><li><strong>File</strong> deleted (<del><i>gantt_filters.patch</i></del>)</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245242011-01-26T19:16:09ZEtienne Massip
<ul><li><strong>File</strong> <i>gantt_filters.htm</i> added</li></ul><p>My bad, missed broken tests.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245252011-01-26T19:16:19ZEtienne Massip
<ul><li><strong>File</strong> deleted (<del><i>gantt_filters.htm</i></del>)</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245262011-01-26T19:16:53ZEtienne Massip
<ul><li><strong>File</strong> <i>gantt_filters.patch</i> added</li></ul><p>This is the good one.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245292011-01-26T20:18:50ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>That's a big patch that is supposed to fix several defects and no tests are included.<br />I think it should be split into smaller ones that fix a specific issue, with each of them including test(s) for the fix.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245402011-01-26T22:23:35ZEtienne Massip
<ul><li><strong>File</strong> <i>gantt_filters_render.patch</i> added</li><li><strong>File</strong> <i>gantt_filters_issue_compare.patch</i> added</li></ul><p>I think I can just split it into 2 patches : the first one one for the rendering algorithm which is about 95% of modified code and the second one for issue sorting which is a very tiny one. Both are attached.</p>
<p>It may look a big patch but just a few methods are actually modified ; that is mainly the <code>render()</code> method and its callees.</p>
<p>About the tests, you're right but there are Gantt tests already and I'm not used to writing ruby tests so it'll take a bit more time (and there is quite a bunch of work to go).</p>
<p>That's also why I posted the patch : to get some feedback to be sure I'm not going straight to the wrong direction before writing the tests.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245462011-01-27T08:24:56ZEtienne Massip
<ul><li><strong>File</strong> deleted (<del><i>gantt_filters_issue_compare.patch</i></del>)</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=245472011-01-27T08:25:08ZEtienne Massip
<ul><li><strong>File</strong> <i>gantt_filters_issue_compare.patch</i> added</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=248322011-02-04T13:53:08ZDmit nitkin
<ul><li><strong>File</strong> <a href="/attachments/5280">gantt.jpg</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5280/gantt.jpg">gantt.jpg</a> added</li></ul><p>Great!<br />I couldn't apply gantt_filters_render.patch and then apply gantt_filters_issue_compare.patch (I used tortoise svn and get conflict on second patch) so I applied gantt_filters.patch (I guess it combines both this patches)<br />I found an error showed in attached file. "test2" is subproject of "testpr" and there are two test2 projects. One with versioned issue and other with unversioned issue. Also I think version "testprVersion" should be before subprojects because it belongs to testpr</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=248332011-02-04T14:05:58ZEtienne Massip
<ul><li><strong>File</strong> <a href="/attachments/5281">gantt_filters.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5281/gantt_filters.patch">gantt_filters.patch</a> added</li></ul><p>Glad I have someone to help !</p>
<p>I knew this bug, this is the only one remaining AFAIK ; I thought I would upload the final version with associated tests I'm actually <em>trying</em> to write.</p>
<p>Anyway, here's the fixed .patch file (just fixed the .patch file itself directly from memory, hope it will be Ok).</p>
<p>Thanks for your feedback !</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=250772011-02-10T13:49:29ZAlcnedlor Nuratar
<ul></ul><p>I tried the new .patch but there is a conflict with query.rb</p>
<p>Since my redmine is in production, I couldn't try to understand the issue and reverted<br />I'll let you know if I find the cause</p>
<p>I have a question though, don't you think it would be best to display only projects that have tickets displayed in them ? For example, if I select an Assignee, all my projects are listed even if the assignee is not part of it.</p>
<p>Regards</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=250782011-02-10T14:17:01ZEtienne Massip
<ul></ul><p>Alcnedlor Nuratar wrote:</p>
<blockquote>
<p>I tried the new .patch but there is a conflict with query.rb</p>
<p>Since my redmine is in production, I couldn't try to understand the issue and reverted<br />I'll let you know if I find the cause</p>
</blockquote>
<p>I just tried on trunk and had no conflict ; since <code>query.rb</code> hasn't been modified since 1.1.1 was released, it should work with stable too.</p>
<blockquote>
<p>I have a question though, don't you think it would be best to display only projects that have tickets displayed in them ? For example, if I select an Assignee, all my projects are listed even if the assignee is not part of it.</p>
</blockquote>
<p>Well, the patch actually change this behavior : only projects and versions with issues not filtered out are displayed.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=250792011-02-10T14:33:08ZAlcnedlor Nuratar
<ul></ul><p>Thank you for the quick answer and I must apologize, I should have looked more deeply</p>
<p>I have a conflict, that's the truth... but I don't really know why<br />The line<br /><code>(filters_clauses << project_statement(Issue.table_name)).join(' AND ')</code><br />was not updated correctly</p>
<p>Except for this little trick, the patch works great</p>
<p>Thank you</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=250822011-02-10T15:07:32ZEtienne Massip
<ul></ul><p>Alcnedlor Nuratar wrote:</p>
<blockquote>
<p>Thank you for the quick answer and I must apologize, I should have looked more deeply</p>
<p>I have a conflict, that's the truth... but I don't really know why<br />The line<br /><code>(filters_clauses << project_statement(Issue.table_name)).join(' AND ')</code><br />was not updated correctly</p>
</blockquote>
<p>What's your redMine version ? 1.1.1 ? How did you apply the patch ?</p>
<blockquote>
<p>Except for this little trick, the patch works great</p>
</blockquote>
<p>If you could give me some feedback, that would be great !</p>
<p>Thanks !</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=250832011-02-10T15:18:22ZAlcnedlor Nuratar
<ul></ul><p>Hum,</p>
<p>I have currently Redmine 1.1.1 (2011-01-30)<br />To apply the patch, I used<br /><code>patch -p0 < gantt_filters.patch</code></p>
<p>Otherwise, I don't know what to report :-)</p>
<ul>
<li>It is now possible to filter by projects</li>
<li>Only projects that contains non-filtered tasks are displayed</li>
<li>Issues are correctly sorted by started date (older first)</li>
<li>also, if I clear every filters, I think that more issues are displayed (but it's just an impression, nothing scientific in the test)</li>
</ul>
<p>My DB is MySQL also, so this seems be DB independent</p>
<p>That's all for now</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=250842011-02-10T15:43:58ZEtienne Massip
<ul></ul><p>Perfect, thanks a lot !</p>
<p>I now know that it works with MySQL too =)</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255102011-02-23T09:05:45ZChris Darts
<ul></ul><p>I've just tried to run the gantt_filters.patch (v2) on v1.1.1 and it failed with the following error message:</p>
<pre>
(Stripping trailing CRs from patch.)
patching file app/models/issue.rb
(Stripping trailing CRs from patch.)
patching file app/models/query.rb
(Stripping trailing CRs from patch.)
patching file lib/redmine/helpers/gantt.rb
Hunk #3 succeeded at 99 (offset 1 line).
Hunk #4 FAILED at 115.
Hunk #5 succeeded at 301 (offset 22 lines).
Hunk #6 FAILED at 299.
Hunk #7 succeeded at 364 with fuzz 1 (offset 38 lines).
Hunk #8 succeeded at 402 (offset 41 lines).
Hunk #9 succeeded at 434 (offset 41 lines).
Hunk #10 succeeded at 466 (offset 41 lines).
Hunk #11 succeeded at 483 (offset 41 lines).
Hunk #12 succeeded at 491 (offset 41 lines).
Hunk #13 succeeded at 503 (offset 41 lines).
Hunk #14 succeeded at 521 (offset 41 lines).
Hunk #15 succeeded at 537 (offset 41 lines).
Hunk #16 succeeded at 557 (offset 41 lines).
Hunk #17 succeeded at 583 (offset 41 lines).
Hunk #18 succeeded at 600 (offset 41 lines).
Hunk #19 FAILED at 580.
Hunk #20 succeeded at 723 (offset 123 lines).
Hunk #21 succeeded at 733 (offset 123 lines).
Hunk #22 succeeded at 746 (offset 123 lines).
Hunk #23 succeeded at 754 (offset 123 lines).
Hunk #24 succeeded at 794 (offset 123 lines).
Hunk #25 succeeded at 809 with fuzz 2 (offset 123 lines).
Hunk #26 succeeded at 817 (offset 123 lines).
Hunk #27 succeeded at 892 (offset 140 lines).
3 out of 27 hunks FAILED -- saving rejects to file lib/redmine/helpers/gantt.rb.rej
(Stripping trailing CRs from patch.)
patching file test/unit/lib/redmine/helpers/gantt_test.rb
</pre>
<p>Any help would be much appreciated.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255112011-02-23T09:07:56ZEtienne Massip
<ul></ul><p>Are you using 1.1.1 stable or trunk ?</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255132011-02-23T09:22:24ZEtienne Massip
<ul></ul><p>I guess you are trying to patch an already patched <code>lib/redmine/helpers/gantt.rb</code> with <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: gantt chart editing (New)" href="https://www.redmine.org/issues/2024">#2024</a>.</p>
<p>That is a bit touchy as this patch (gantt_filters) is quite a bit destructive, you'll have to patch some part by hand.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255142011-02-23T09:56:37ZChris Darts
<ul></ul><p>Yes, sorry I forgot to mention that I had applied the patch under <a class="wiki-page" href="#2024">#2024</a> beforehand.</p>
<p>Ideally I would like to use both patches as they seem to offer the perfect combination of what I need to achieve using the gantt chart. I guess it isn't that easy at the moment?</p>
<p>I'm fairly new to Redmine and even more so to patches, so I'm not sure how comfortable I would be to do some of the patching by hand. Any help or advise would be greatly received.</p>
<p>I'm currently setting up a separate instance of Redmine to try your patch on it's own, without the gantt chart editing in issue <a class="wiki-page" href="#2024">#2024</a>. I'll let you know how that goes.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255882011-02-24T14:33:32ZChris Darts
<ul></ul><p>I've just tried this patch on a fresh install of Redmine 1.1.1 (taken from the svn repository) and it seems to work fine now. Looks like the conflict was indeed with another gantt chart patch <a class="wiki-page" href="#2024">#2024</a>.</p>
<p>My only question now is that the gantt chart doesn't seem to appear any different from how it looked pre-patch. I have only run the 'gantt_filters.patch' (v2), as I'm not sure if the 'gantt_filters_render.patch' and 'gantt_filters_issue_compare.patch' are still relevant.</p>
<p>I'm just wondering if most of the issues fixed with this patch are now in the core system, which I why I'm not seeing any difference.</p>
<p>Can anyone highlight the main issues this patch is intended to resolve and if it's still necessary with 1.1.1. Should I also be applying the other 2 patches as listed above?</p>
<p>Thanks.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255892011-02-24T14:35:01ZEtienne Massip
<ul><li><strong>File</strong> deleted (<del><i>gantt_filters_issue_compare.patch</i></del>)</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255902011-02-24T14:35:04ZEtienne Massip
<ul><li><strong>File</strong> deleted (<del><i>gantt_filters.patch</i></del>)</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255912011-02-24T14:35:08ZEtienne Massip
<ul><li><strong>File</strong> deleted (<del><i>gantt_filters_render.patch</i></del>)</li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255932011-02-24T14:42:43ZEtienne Massip
<ul></ul><p>I just removed remaining irrelevant patches.</p>
<p>As for what it does, I thought that the list in the description was pretty clear :(</p>
<p>Mostly, projects and versions which have no issue related are not displayed anymore, query filters are all taken into account and issue sorting is a bit more precise.</p>
<p>And no, it has not been incorporated to core.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=255942011-02-24T14:51:17ZChris Darts
<ul></ul><p>Thanks Etienne, I appreciate your quick response.</p>
<p>The list is pretty clear, but it just made me wonder as I didn't see any difference on my system. It's probably because I was only viewing one test project and didn't look at it across all projects. Now I look at all projects and issues I can see the improvements. Sorry, this was my fault.</p>
<p>Thanks for removing the redundant files, this will be a big help for anyone new to this thread.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=256602011-02-26T16:40:28ZEtienne Massip
<ul><li><strong>Target version</strong> set to <i>Candidate for next major release</i></li></ul> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=259182011-03-04T14:34:16ZB Brey
<ul></ul><p>I tried to patch 1.1.1.stable.5000 but to no avail.</p>
<pre>
patch -p0 < gantt_filters.patch
</pre>
<p>Failed with on all points, what am I doing wrong?</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=259242011-03-04T14:55:23ZEtienne Massip
<ul></ul><p>You may have already applied another patch on your gantt ?</p>
<p>If not, try with <code>-p1</code>.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260132011-03-07T08:25:13ZB Brey
<ul></ul><p>Etienne Massip wrote:</p>
<blockquote>
<p>You may have already applied another patch on your gantt ?</p>
<p>If not, try with <code>-p1</code>.</p>
</blockquote>
<p>No but tried it anyway and it didn't work, any chance on this being fixed soon in a release? Because this breaks the gantt charting :(</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260162011-03-07T09:08:09ZEtienne Massip
<ul></ul><p>Just done it on <a class="changeset" title="scm: mercurial: add path encoding tests in functional test (#2664, #4050). TODO: This test fails..." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5000">r5000</a> without any problem, and this patch is still applicable on trunk.</p>
<p>What failures were you talking about, exactly ?</p>
<p>Please revert your changes on <code>lib/redmine/helpers/gantt.rb</code>, <code>app/models/issue.rb</code> and <code>app/models/query.rb</code> before patching.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260252011-03-07T11:29:50ZB Brey
<ul></ul><p>Etienne Massip wrote:</p>
<blockquote>
<p>Just done it on <a class="changeset" title="scm: mercurial: add path encoding tests in functional test (#2664, #4050). TODO: This test fails..." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5000">r5000</a> without any problem, and this patch is still applicable on trunk.</p>
<p>What failures were you talking about, exactly ?</p>
<p>Please revert your changes on <code>lib/redmine/helpers/gantt.rb</code>, <code>app/models/issue.rb</code> and <code>app/models/query.rb</code> before patching.</p>
</blockquote>
<p>I'm at revision 5022 btw. Really don't know what i am doing wrong.</p>
<pre>
patch -p0 < gantt_filters.patch
Hmm... Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|Index: app/models/issue.rb
|===================================================================
|--- app/models/issue.rb (revision 4761)
|+++ app/models/issue.rb (working copy)
--------------------------
Patching file app/models/issue.rb using Plan A...
Hunk #1 failed at 17.
Hunk #2 failed at 68.
Hunk #3 failed at 75.
3 out of 3 hunks failed--saving rejects to app/models/issue.rb.rej
Hmm... The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|Index: app/models/query.rb
|===================================================================
|--- app/models/query.rb (revision 4761)
|+++ app/models/query.rb (working copy)
--------------------------
Patching file app/models/query.rb using Plan A...
Hunk #1 failed at 388.
Hunk #2 failed at 434.
Hunk #3 failed at 443.
Hunk #4 failed at 496.
Hunk #5 failed at 508.
Hunk #6 failed at 537.
Hunk #7 failed at 569.
7 out of 7 hunks failed--saving rejects to app/models/query.rb.rej
Hmm... The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|Index: lib/redmine/helpers/gantt.rb
|===================================================================
|--- lib/redmine/helpers/gantt.rb (revision 4761)
|+++ lib/redmine/helpers/gantt.rb (working copy)
--------------------------
Patching file lib/redmine/helpers/gantt.rb using Plan A...
Hunk #1 failed at 38.
Hunk #2 failed at 53.
Hunk #3 failed at 98.
Hunk #4 failed at 115.
Hunk #5 failed at 429.
Hunk #6 failed at 449.
Hunk #7 failed at 471.
Hunk #8 failed at 506.
Hunk #9 failed at 538.
Hunk #10 failed at 570.
Hunk #11 failed at 587.
Hunk #12 failed at 595.
Hunk #13 failed at 607.
Hunk #14 failed at 625.
Hunk #15 failed at 641.
Hunk #16 failed at 661.
Hunk #17 failed at 687.
Hunk #18 failed at 704.
Hunk #19 failed at 725.
Hunk #20 failed at 745.
Hunk #21 failed at 755.
Hunk #22 failed at 768.
Hunk #23 failed at 776.
Hunk #24 failed at 816.
Hunk #25 failed at 831.
Hunk #26 failed at 839.
Hunk #27 failed at 897.
27 out of 27 hunks failed--saving rejects to lib/redmine/helpers/gantt.rb.rej
Hmm... The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|Index: test/unit/lib/redmine/helpers/gantt_test.rb
|===================================================================
|--- test/unit/lib/redmine/helpers/gantt_test.rb (revision 4761)
|+++ test/unit/lib/redmine/helpers/gantt_test.rb (working copy)
--------------------------
Patching file test/unit/lib/redmine/helpers/gantt_test.rb using Plan A...
Hunk #1 failed at 91.
1 out of 1 hunks failed--saving rejects to test/unit/lib/redmine/helpers/gantt_test.rb.rej
</pre> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260272011-03-07T13:10:46ZEtienne Massip
<ul><li><strong>File</strong> <a href="/attachments/5521">patch.log</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5521/patch.log">patch.log</a> added</li></ul><p>Here's my <code>patch -p0 --verbose < gantt_filters.patch</code> results on <a class="changeset" title="Fixed: error on JournalsController#index when custom fields are present (#7795)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5021">r5021</a> (windows).</p>
<p>Sorry, I've no clue why yours just fail. I suppose that your <code>cwd</code> is redmine ?</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260362011-03-07T14:49:42ZB Brey
<ul></ul><p>Etienne Massip wrote:</p>
<blockquote>
<p>Here's my <code>patch -p0 --verbose < gantt_filters.patch</code> results on <a class="changeset" title="Fixed: error on JournalsController#index when custom fields are present (#7795)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5021">r5021</a> (windows).</p>
<p>Sorry, I've no clue why yours just fail. I suppose that your <code>cwd</code> is redmine ?</p>
</blockquote>
<p>Yes it's the current working directory, /home/www/redmine-1.1/</p>
<p>I'm running FreeBSD by the way.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260372011-03-07T15:05:49ZEtienne Massip
<ul><li><strong>File</strong> <a href="/attachments/5522">patched.zip</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5522/patched.zip">patched.zip</a> added</li></ul><p>Here are the files once patched, could you create a patch and compare both patches which each other ?</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260672011-03-08T08:48:16ZB Brey
<ul><li><strong>File</strong> <a href="/attachments/5525">patch_unified_diff.zip</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5525/patch_unified_diff.zip">patch_unified_diff.zip</a> added</li></ul><p>Etienne Massip wrote:</p>
<blockquote>
<p>Here are the files once patched, could you create a patch and compare both patches which each other ?</p>
</blockquote>
<p>Very weird, this patch is very different. I deleted the 3 files and got them fresh from the 1.1 repository<br />I included both patches + diff file in the zip.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260762011-03-08T10:52:30ZEtienne Massip
<ul><li><strong>File</strong> <a href="/attachments/5527">patched_1.1.0.zip</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5527/patched_1.1.0.zip">patched_1.1.0.zip</a> added</li><li><strong>File</strong> <a href="/attachments/5528">patched_1.1.1.zip</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5528/patched_1.1.1.zip">patched_1.1.1.zip</a> added</li></ul><p>I guess that's mostly because patched files I sent to you where based on <a class="changeset" title="Fixed: error on JournalsController#index when custom fields are present (#7795)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5021">r5021</a>.</p>
<p>Could you please do the same thing again, this time with the files matching your redmine (I uploaded 2 archives : one for <a class="version" href="https://www.redmine.org/versions/30">1.1.1</a> / <a class="changeset" title="tagged version 1.1.1" href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4786">r4786</a> and one for <a class="version" href="https://www.redmine.org/versions/20">1.1.0</a> / <a class="changeset" title="tagged version 1.1.0" href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4672">r4672</a>) ?</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=260792011-03-08T11:09:48ZB Brey
<ul><li><strong>File</strong> <a href="/attachments/5529">patch_unified_diff_1.1.zip</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5529/patch_unified_diff_1.1.zip">patch_unified_diff_1.1.zip</a> added</li></ul><p>Etienne Massip wrote:</p>
<blockquote>
<p>I guess that's mostly because patched files I sent to you where based on <a class="changeset" title="Fixed: error on JournalsController#index when custom fields are present (#7795)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5021">r5021</a>.</p>
<p>Could you please do the same thing again, this time with the files matching your redmine (I uploaded 2 archives : one for <a class="version" href="https://www.redmine.org/versions/30">1.1.1</a> / <a class="changeset" title="tagged version 1.1.1" href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4786">r4786</a> and one for <a class="version" href="https://www.redmine.org/versions/20">1.1.0</a> / <a class="changeset" title="tagged version 1.1.0" href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4672">r4672</a>) ?</p>
</blockquote>
<p>Done!</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=261972011-03-10T18:19:27ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>I've just committed some changes to the gantt chart rendering in <a class="changeset" title="Gantt: do not ignore project filter (#7000, #7352), do not display empty projects/versions, and d..." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5077">r5077</a>.<br />It should basically implements the changes proposed in the patch, except the issue sort change.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=261992011-03-10T19:16:06ZEtienne Massip
<ul></ul><p>You're a damn good coder, nice clean code !</p>
<p>I think you've fixed <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Provide simple way of hiding completed versions from a gantt chart (New)" href="https://www.redmine.org/issues/6605">#6605</a> too, didn't you ?</p>
<p>I'll run some tests tomorrow, the few ones I wrote til now all passe =)</p>
<p>Guess you can remove the <code>Issue#for_gantt</code> scope too ?<br />And maybe the <code>GanttHelper#number_of_issues_on_versions</code>, I can't find any use of it in the project ?</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=262002011-03-10T19:37:20ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>Thanks :-)</p>
<p>Etienne Massip wrote:</p>
<blockquote>
<p>Guess you can remove the <code>Issue#for_gantt</code> scope too ?<br />And maybe the <code>GanttHelper#number_of_issues_on_versions</code>, I can't find any use of it in the project ?</p>
</blockquote>
<p>Cleanup done. There's maybe more to do.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=262202011-03-11T09:36:00ZEtienne Massip
<ul></ul><p>There's still at least 1 glitch :</p>
<p>If a subproject issue is targeted to the parent project shared version, its path is rendered as :<br /><pre>parent -> subproject -> shared version -> shared version issues (including parent issues)</pre><br />It should be :<br /><pre>parent -> shared version -> subproject -> shared version issues (including parent issues)</pre></p>
<p>Here, <code>parent</code> is what I named "top_version_project" in the patch, this is the project the version should be rendered aside.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=262962011-03-11T18:36:34ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>The current implementation actually gives what I was expecting (and what is expected in <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Subproject's issues are not shown in the subproject's gantt (Closed)" href="https://www.redmine.org/issues/6476">#6476</a> note-2).<br />I think it makes sense to display the shared version under the subproject, not the opposite.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=262972011-03-11T18:57:41ZEtienne Massip
<ul></ul><p>I disagree : Gantt is a planning tool and you're planning your issue around a version and then, or if there is no version, around a project.</p>
<p>What note-2 is proposing is to follow the project first and your version second which is wrong to me (you split your version in several Gantt branches).</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=262982011-03-11T19:05:38ZEtienne Massip
<ul></ul><p>About <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Subproject's issues are not shown in the subproject's gantt (Closed)" href="https://www.redmine.org/issues/6476">#6476</a>, I think that rendering should be different in function of context project.</p>
<p>Version's project Gantt :<br /><pre>
parent -> shared version -> subproject -> shared version issues (including parent issues)
</pre></p>
<p>Subproject Gantt :<br /><pre>
subproject -> shared version -> shared version issues (excluding parent issues)
</pre></p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=295852011-06-06T06:36:34ZChris Darts
<ul></ul><p>Jean-Philippe Lang wrote:</p>
<blockquote>
<p>I've just committed some changes to the gantt chart rendering in <a class="changeset" title="Gantt: do not ignore project filter (#7000, #7352), do not display empty projects/versions, and d..." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/5077">r5077</a>.<br />It should basically implements the changes proposed in the patch, except the issue sort change.</p>
</blockquote>
<p>I'm interested to find out why the issue sort change was not included in this update.</p>
<p>I've just tested 1.2.0 and there are a number of improvements on the Gantt chart, some or most of which I assuming originated from this patch. However, it seems there is no way to influence the sort order issues in the Gantt chart, with the default being the issue id. It would be great if the Gantt could take the sorting order from the issue list or after selecting a specific custom query from the sidebar.</p>
<p>Are there any plans to incorporate the missing aspects from this patch, in particular the sort issue, into Redmine core in the near future?</p>
<p>Thanks.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=362182012-02-21T13:10:22ZMartin Wall
<ul><li><strong>Assignee</strong> set to <i>Etienne Massip</i></li></ul><p>Can you add a ticket related to this gant chart improvement?<br /><a class="external" href="http://www.redmine.org/boards/4/topics/14801?r=20375#message-20375">http://www.redmine.org/boards/4/topics/14801?r=20375#message-20375</a></p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=362202012-02-21T14:10:26ZEtienne Massip
<ul><li><strong>Assignee</strong> deleted (<del><i>Etienne Massip</i></del>)</li></ul><p>Please don't assign issues.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=437872013-01-05T04:28:18ZJonathan Vargas
<ul></ul><p>I am using redmine 2.2.0 and the sort by start_date issue is a feature I would appreciate, as reported on several issues.</p>
<p>Current sorting by issue_id doesn't provides usable sorting at all, since most of time the issues are not created sequentially in the same order that they should be executed.</p>
<p>The fix for this is very easy, I applied it quickly on 2.2.0. Is there a known reason to postpone it?</p>
<p>Thanks.</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=438042013-01-05T11:34:06ZDaniel Felix
<ul></ul><p>Here are some other patches which solves some gantt sort issues too: <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Gantt Version Ordering (Closed)" href="https://www.redmine.org/issues/8366">#8366</a></p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=515582013-09-03T14:40:35ZArtem Kondratyev
<ul></ul><p>Is there a possibility to add sort by due_date too? It's much more needed when talking about Network planning and design</p> Redmine - Patch #7456: Gantt filtershttps://www.redmine.org/issues/7456?journal_id=980462020-06-03T06:45:08ZAlessandro Zucchi
<ul></ul><p>+1</p>