When you're using Rubik theme as admin, option "Mark group as required if it contains required fields." doesn't work for vertical tabs.

That caused by following code in base Tao theme:

/**
 * Implementation of preprocess_fieldset().
 */
function tao_preprocess_fieldset(&$vars) {
  // ...
  $vars['attributes']['class'][] = 'fieldset';
  // ...
}

It adds "fieldset" class for each fieldset on the page. Since vertical tabs content is being rendered as fieldset, function Drupal.FieldGroup.Effects.processTabs is never executed, because $('fieldset.fieldset') is processed by Drupal.FieldGroup.Effects.processFieldset. That happened because these 2 functions are using the same class for once() method.

I found an easy way how to fix that. We just should change class for once function.

CommentFileSizeAuthor
#2 field_group-group_required-2617048-2.patch722 bytessanchiz

Comments

sanchiz created an issue. See original summary.

sanchiz’s picture

StatusFileSize
new722 bytes

Initial patch.

sanchiz’s picture

Issue summary: View changes
sanchiz’s picture

Status: Active » Needs review
chris matthews’s picture

The 3 year old patch in #2 to field_group.js applied cleanly to the latest field_group 7.x-1.x-dev, but still needs review (if still applicable).

nils.destoop’s picture

Status: Needs review » Closed (won't fix)