hi,

this issue drove me nuts for a while until I figured out that there is an extra (hardcoded) fieldset in inline_entity_form/includes/commerce_product.inline_entity_form.inc.

the trouble with this: if you use field_groups and reorder the field, the hardcoded fieldset will still render and botch up the display.

greets

CommentFileSizeAuthor
#2 ief_minor-remove-hardcoded-fieldset-from-ief-rendering_2597942-1.patch2.6 KBattisan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

attisan created an issue. See original summary.

attisan’s picture

attisan
Location Vienna
CreditAttribution: attisan as a volunteer commented
FileSize
ief_minor-remove-hardcoded-fieldset-from-ief-rendering_2597942-1.patch2.6 KB
hoff331’s picture

hoff331 CreditAttribution: hoff331 as a volunteer commented

@attisan, great find! This fieldset was driving me crazy too!

mihai_brb’s picture

mihai_brb CreditAttribution: mihai_brb commented

Can also be done via form alter. Assuming this is a field_product reference:

if (isset($form['field_product'])) {
    $form['field_product']['und']['#type'] = 'container';
    $form['field_product']['und']['form']['product_details']['#type'] = 'container';
}
SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: ief_minor-remove-hardcoded-fieldset-from-ief-rendering_2597942-1.patch, failed testing.

dcam’s picture

dcam CreditAttribution: dcam as a volunteer commented
Status: Needs work » Closed (won't fix)

Due to limited developer time old feature requests for IEF are being closed. If you still have a need for this functionality, then you may feel free to reopen the issue. But please note that if you are still using Drupal 7 and would like for this functionality to be added to the D7-compatible version of IEF, then our limited resources and lack of interest may prevent that from ever happening.