When SMA enabled it is impossible to set any other translation mode than the default "No multilingual options for menu items".

One can choose one of the radio buttons, but after saving it resets the original state.

(I'm not sure if it is the bug of SMA or the bug of i18n, but it's clear that they incompatible.

CommentFileSizeAuthor
#13 simplified_menu_admin-form-hierarchy-not-preserved-1256700-13.patch1.98 KBDavid_Rothstein
#10 simplified_menu_admin-form-hierarchy-not-preserved-1256700-10.patch1.62 KBb-prod
#6 Screenshot from 2013-03-26 17:53:52.png21.37 KBmrf
#6 Screenshot from 2013-03-26 17:53:18.png27.66 KBmrf

Comments

robertgarrigos’s picture

robertgarrigos commented

I can confirm this. They are incompatible :-(

sylus’s picture

sylus commented

Confirming as well.

ericmulder1980’s picture

ericmulder1980 commented

I can confirm this as well.

mrfelton’s picture

mrfelton commented

also confirming ;)

mariomc’s picture

mariomc commented

Confirm here as well

mrf’s picture

mrf commented
StatusFileSize
new Screenshot from 2013-03-26 17:53:18.png27.66 KB
new Screenshot from 2013-03-26 17:53:52.png21.37 KB

Still seeing this issue.

I've attached some before and after screenshots to show how the 'Multilingual' settings box UI changes when this module is enabled.

before

after

Anonymous’s picture

Anonymous (not verified) commented

Confirming as well.

petterw03’s picture

petterw03 commented
Issue summary: View changes

There is a patch for i18n here that apparently fixes the issue: https://www.drupal.org/node/2221189#comment-8590281

yareckon’s picture

yareckon commented

So is SMA hanging it's settings in a wrong place that causes issues for other modules, or is i18n not doing something right? The patch is about i18n needing to explicitly declare parents for form fields because the values are weirdly nested otherwise. is SMA doing something that would cause this?

Would be easier to expect a fix from SMA if you have a handful of open issues rather than i18n that has over 300.

b-prod’s picture

b-prod commented
Version: 7.x-1.0-beta2 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new simplified_menu_admin-form-hierarchy-not-preserved-1256700-10.patch1.62 KB

The patch fixes this issue. The problem is that the overview form is hierarchical and the edit form not. So this is caused by the Simplified Menu Administration module which does not takes care of that. Creating a patch against another contributed module (like i18n menu) is not relevant.

Cogax’s picture

Cogax commented

#10 works for me! I came over from https://www.drupal.org/node/2221189 and this patch solves my problems. Thanks to @B-Prod !

b-prod’s picture

b-prod commented
Status: Needs review » Reviewed & tested by the community

Patch used on several live sites.

Reviewed by Cogax (see comment above)

David_Rothstein’s picture

David_Rothstein commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new simplified_menu_admin-form-hierarchy-not-preserved-1256700-13.patch1.98 KB

But what happens if the second form (the one being merged in) has #tree set to TRUE (at the top level of the form) itself? This would break that.

Here's a version that should work in both cases. I tested with i18n_menu and it seemed to work.

b-prod’s picture

b-prod commented

@David_Rothstein: Thanks, good point for you!

I will be able to review your patch next week.

b-prod’s picture

b-prod commented
Status: Needs review » Reviewed & tested by the community

The patch in #13 works well (tested compatibility with i18n_menu).

  • a32f579 committed on 7.x-1.x 
    Fix issue #1256700 by thamas, David_Rothstein: SMA incompatible with...
b-prod’s picture

b-prod commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.