Format settings not saved

Yes, I've just run into this problem with the latest version. The "Fieldset" widget has "fieldset collapsible" and "required_fields yes" as defaults. If I change them they appear to change (e.g. the ajax callback changes the widget values), but in reality they don't (e.g. reloading the page (or using the actual node form) shows that it's still the defaults)

Umm, sorry. So in my case at least, after actually looking at the code, it does work. It's not enough to just alter the widget, you then need to click Save on the Manage Fields page as well. This may be default Drupal behaviour, but it did throw me.

On my setup, with the latest 1.x-dev, saving the form revert to the default values, on both "Managed fields" and "Manage Display" screens. The AJAX callback is indeed working, but the save doesn't appear to work properly.

Would it be possible there's a conflict with another module?

Umm, double sorry. While it looked like it worked, now that I actually have fields in it, it doesn't. Whatever happens (update and additionally saving the manage fields page) it reverts back to the defaults.

OK. This is what I get for not drinking enough coffee.

The reason I thought it was working, was it actually did on an older version of the site. The problem is with the latest dev version.

The problem change is in field_group.field_ui.inc which in field_group_field_ui_overview_form_alter now calls field_group_load_field_group on line 329. This loads up a copy of the group with the original settings, which then seem to be used on the save.

To be honest, I'm not really sure what is/isn't needed.

Just changing 329 from

$group = field_group_load_field_group($group_name, $params->entity_type, $params->bundle, $params->mode);

to

$group_check = field_group_load_field_group($group_name, $params->entity_type, $params->bundle, $params->mode);

(and then using $group_check in the rest of the function) seems to work.

Patch attached

Seems to work for me. Thanks !

Works for me as well.

Bravo! ++ #6

This has been pushed to git. Thx for the patch!