Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RMPlus Custom_menu plugin conflict #4

Open
dominch opened this issue Jul 14, 2015 · 4 comments
Open

RMPlus Custom_menu plugin conflict #4

dominch opened this issue Jul 14, 2015 · 4 comments

Comments

@dominch
Copy link

dominch commented Jul 14, 2015

hello,
I decided to try rm plus custom menu plugin. There is some conflict and top of site is damaged,
in error log there is:

^[[1;32mDeface: [ERROR]^[[0m The original source for 'add-maintenance-message-to-bodytop' has changed, this override should be reviewed to ensure it's still valid.
^[[1;32mDeface: [ERROR]^[[0m The original source for 'add-sudo-toggle-to-layout' has changed, this override should be reviewed to ensure it's still valid.

Error seems to come from base deface plugin (I'm using redmine_sudo too).

Both plugins works when only one is present.

@nanego
Copy link
Collaborator

nanego commented Jul 15, 2015

Hi
It may be a conflict between the redmine_sudo plugin and the rm_plus_custom_menu plugin, as they both modify the header and they have never been tested together.

I don't think the issue really concerns this redmine_base_deface plugin. These error messages only suggest that the overrides should be reviewed in the plugins where they are defined.

@jbbarth
Copy link
Owner

jbbarth commented Jul 15, 2015

To make it clear, those errors in the logs just suggest a review but deface still tries to perform its overrides and everything may work in the end.

And agreed with @nanego for the rest :-)

Thanks for reporting!

Jean-Baptiste

Le 15 juil. 2015 à 11:40, vrobert notifications@github.com a écrit :

Hi
It may be a conflict between the redmine_sudo plugin and the rm_plus_custom_menu plugin, as they both modify the header and they have never been tested together.

I don't think the issue really concerns this redmine_base_deface plugin. These error messages only suggest that the overrides should be reviewed in the plugins where they are defined.


Reply to this email directly or view it on GitHub.

@dominch
Copy link
Author

dominch commented Jul 15, 2015

Thanks for fast replies :) Sure, I expected that deface + redmine_sudo performs very simple change in header, custom menu makes much more changes there. Both cause that top of page is corrupted and not usable any more. I like redmine_sudo plugin so for now I was forced to remove custom menu plugin. RMPlus don't have good place for issue reporting for now (as well as any git repo with this plugin).

@rc5hack
Copy link

rc5hack commented Jun 8, 2016

Hello! I facing this issue:

The original source for 'add-maintenance-message-to-bodytop' has changed, this override should be reviewed to ensure it's still valid.

but there are no redmine_sudo and no rm_plus_custom_menu plugins installed in my redmine 2.6.10. All my plugins are:

a_common_libs 2.1.8
redmine_base_deface 0.0.1
redmine_checklists 3.1.3
redmine_dashboard 2.7.1
redmine_maintenance_mode 2.0.1
rm_user_mentions 1.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants