Regression: 6.x-2.x does not seem to work any more with Menu Admin per Menu [#1373002]

Hi
I am the maintainer of Menu Admin per Menu (http://drupal.org/project/usage/menu_admin_per_menu).
MAPM used to work well with CTM 6.x-1.x.
But I have just noticed that it does not seem any more since 6.x-2.x: when enabling both modules, the menu settings fieldset shows up when it should not.
Did you change CTM's behaviour or something?

Comments

Comment #1

seworthi commented 24 January 2012 at 18:30

I found the difference. In "ctm_form_alter", under the Menu Settings for Node Edit (Form), in version 1.x, if no menu settings, in has "unset($form['menu']);" in line 34, where in version 2.x, the same process (line 54) has "$form['menu']['#access'] = FALSE;"

I changed it to the way and it works, but I have not tested enough to know what other issues might arise.

Update: I changed the module weight of ctm to 1, and it worked without changing any code in version 2.x.

Hope this helps.

Comment #2

seworthi commented 7 August 2012 at 17:28

Patch for ctm.install

/**
 * Update 6201: Fix module weight
 */
function ctm_update_6201() {
  // Make sure this module loads after other menu alter modules.
  $ret = array();
  $ret[] = update_sql("UPDATE {system} SET weight = 9 WHERE name = 'ctm'");
  return $ret;
}

UPDATE: I had same issue as #3, but setting to 9 worked, as 11 did not.