See #653108: beta6 introducing incompatibilities. Simple Access breaks the node edit form when Vertical Tabs beta6 is activated:
Fatal error: Call to undefined function vertical_tabs_fieldsets() in /[...]/sites/all/modules/simple_access/simple_access.module on line 332
The function above seems to have been removed in Vertical Tabs. Since it is still in beta we should expect the code not to be stable.

A trivial fix is to check if vertical_tabs_fieldsets() exists. In simple_access.module on line 331 we could edit the line from:
if (!isset($conf['vertical_tabs_fieldsets_'. $form['type']['#value']])) {
to :
if (!isset($conf['vertical_tabs_fieldsets_'. $form['type']['#value']]) && function_exists('vertical_tabs_fieldsets')) {

Comments

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
Status: Active » Needs review
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Ok I see what the integration is doing here. Basically, you want to tabify your node fieldset by default. The good news is this happens by default with the beta6 version, so it's no longer necessary. You should just update the code to do:

  if (module_exists('vertical_tabs')) {
    drupal_add_js(drupal_get_path('module', 'simple_access') .'/simple_access.js');
  }

I'm possibly looking into backporting a version of the D7 attached form property so that you'd do the following (but TBD).
$form['element']['#attached']['js']['vertical_tabs'] = drupal_get_path('module', 'simple_access') .'/simple_access.js');

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Actually just committed the attached JS support, so now do:

$form['sa']['#attached']['js']['vertical_tabs'] = drupal_get_path('module', 'simple_access') . '/simple_access.js');

and that's it!

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Issue tags: +vertical tabs

Adding tag...

gordon’s picture

gordon commented
Status: Needs review » Fixed

Thanks I have committed this to test.

Status: Fixed » Closed (fixed)
Issue tags: -vertical tabs

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

Issue tags: +vertical tabs

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.