https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292021-08-25T13:41:55ZRedmineRedmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039342021-08-25T13:41:55ZMischa The Evil
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/103934/diff?detail_id=84545">diff</a>)</li><li><strong>Status</strong> changed from <i>New</i> to <i>Needs feedback</i></li><li><strong>Assignee</strong> set to <i>Thomas Zauner</i></li><li><strong>Affected version</strong> changed from <i>4.2.2</i> to <i>4.2.1</i></li></ul><p>That looks weird. What theme are you using? Is it one that's developed by a third-party? Then it might be incompatible with Redmine 4.2.x.<br />Furthermore, are you using any plugins? They can cause issues like these too.</p>
<p>So, please test this without running plugins and custom themes to see if it can be reproduced still and let us know the result.</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039462021-08-26T06:24:37ZThomas Zauner
<ul></ul><p>Thanks for the hint. I just tried a fresh docker redmine:4.2.2 and it works there.<br />We just have 3 themes shipped with redmine (Standard, Alternate, Classic)</p>
<p>It was an old plugin we used in redmine 3.x already. <br />I was not aware of this, but the old plugin broke the context menu (css styling, missing entries). <br />Without the plugin (redmine_archive_issue_categories), it looks like fine now. Thanks!</p>
<p>This issue can be closed.</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039482021-08-26T07:38:08ZMischa The Evil
<ul><li><strong>Status</strong> changed from <i>Needs feedback</i> to <i>Closed</i></li><li><strong>Assignee</strong> deleted (<del><i>Thomas Zauner</i></del>)</li><li><strong>Resolution</strong> set to <i>Invalid</i></li></ul><p>Thomas Zauner wrote:</p>
<blockquote>
<p>[...]</p>
<p>It was an old plugin we used in redmine 3.x already. <br />I was not aware of this, but the old plugin broke the context menu (css styling, missing entries). <br />Without the plugin (redmine_archive_issue_categories), it looks like fine now. Thanks!</p>
<p>This issue can be closed.</p>
</blockquote>
<p>Thanks for the feedback. I'll close this issue accordingly.</p>
<p>About the plugin: this is an example of one that needs to follow core changes closely because it includes overrides for essential issue views. The latest changes to (a fork of) the original plugin were for Redmine 3.3.8 and 4.0.x.<br />Note that uninstalling the plugin includes a db migration (roll-back). Besides that, after uninstalling it, all formerly-archived issue categories will become usable as regular again by your users (conform the core behavior).</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039502021-08-26T08:12:07ZThomas Zauner
<ul></ul><p>Yup, thanks for the explanation. I have never checked, how the plugin really worked.<br />Looks like the plugin simple overrides the whole file (eg. issues.html.erb) and of course any changes made in the original redmine version got lost until someone updates the changes.</p>
<p>Maybe I will create a fork and readd the functionality for 4.2.2, but all in all, this feature would be nice to have in redmine itself instead of a plugin.<br />Issue <a class="issue tracker-2 status-1 priority-5 priority-high2" title="Feature: Mark project categories as "inactive" (New)" href="https://www.redmine.org/issues/16188">#16188</a> already exists a while.</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039522021-08-26T09:23:27ZMischa The Evil
<ul></ul><p>Thomas Zauner wrote:</p>
<blockquote>
<p>Looks like the plugin simple overrides the whole file (eg. issues.html.erb) and of course any changes made in the original redmine version got lost until someone updates the changes.</p>
</blockquote>
<p>Indeed. Unless a plugin uses something like <a href="https://github.com/spree/deface" class="external">Deface</a> for their core view changes, that's the reality.</p>
<blockquote>
<p>Maybe I will create a fork and readd the functionality for 4.2.2, [...]</p>
</blockquote>
<p>FWIW: It shouldn't be all too difficult. Though note that there seems to be a defect in the original upstream implementation of the feature too (which seems already fixed in the latest updated fork).</p>
<blockquote>
<p>[...] but all in all, this feature would be nice to have in redmine itself instead of a plugin.</p>
</blockquote>
<p>If you're really up to it, it should also not be very hard to port this plugin implementation of the feature into a fully-fledged core patch to propose for upstream integration yourself... As long as it provides test coverage it might get reviewed and considered.</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039542021-08-26T10:00:46ZThomas Zauner
<ul></ul><p>Mischa The Evil wrote:</p>
<blockquote>
<p>If you're really up to it, it should also not be very hard to port this plugin implementation of the feature into a fully-fledged core patch to propose for upstream integration yourself... As long as it provides test coverage it might get reviewed and considered.</p>
</blockquote>
<p>I will have a look into it I guess.</p>
<p>Havent developed any ruby application yet. I think porting the plugin (which seem to be several categories.reject(if archived)) wont be to much effort.<br />But I will have to check how to run/debug the ruby redmine application and then check, how to test this. And how other tests are written.</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039572021-08-26T20:12:53ZThomas Zauner
<ul></ul><p>I've looked into the contribution page and the forum. The ruby code is checked out (via github).<br />Could you provide me with further information, how to get this ruby rails project running to test my changes? I'm somewhat lost right now... :)</p> Redmine - Defect #35813: Repeating context menu iconshttps://www.redmine.org/issues/35813?journal_id=1039772021-08-29T03:40:05ZMischa The Evil
<ul></ul><p><a class="user active" href="https://www.redmine.org/users/573">Thomas Eccard</a>: at first: you shouldn't post to closed issues. The chances that your post is read or that you'll get an actual reply are very slim.</p>
<p>Despite the above, let me make clear that you shouldn't embark on a project like this unless you have at least a solid basic understanding of Ruby, Ruby on Rails, Redmine and their interplay. Especially in this case where the original plugin implementation has defects and is incomplete to begin with.</p>
<p>If you decide to carry-on nevertheless and run into specific problems or questions, you should post into the <a href="/projects/redmine/boards">forums</a> instead as we don't provide such support in the issues which are solely meant for core issue tracking.</p>