The flawed code caused the [display_settings][format] recursively getting appended to itself on form submittal, some crazy s***t.. wow this was difficult to track down. while doing it i changed a few var names to be more descriptive, that patch is the second one. Now good night, its UNI time again in few hours.

CommentFileSizeAuthor
#1 multigroup-display-fields-fix-v2.patch1.8 KBeMPee584
multigroup-vars-more-descriptive.patch1.71 KBeMPee584
multigroup-display-fields-fix.patch879 byteseMPee584
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eMPee584’s picture

eMPee584 CreditAttribution: eMPee584 commented
FileSize
multigroup-display-fields-fix-v2.patch1.8 KB

ok that change messed up the display of the field label select box, so now here's a better one. The variable changes are included aswell but the relevant fix is only

+      $form[$form_name][$key]['exclude'] = array();

which lets the strange theming detection stay as it is.

eMPee584’s picture

eMPee584 CreditAttribution: eMPee584 commented
Status: Needs review » Fixed

Markus' current work on cck multiple values / groups includes this fix. Magic!

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented

Yep, the multigroup module had several bugs that I've been trying to fix as my goal was to get it into a state that can be considered as stable as possible so it can be distributed as any other module. :-D

Until the multigroup module is officially released, I would keep all related issues here: #119102: Combo field - group different fields into one

Status: Fixed » Closed (fixed)

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