Fields are not organized by fieldgroup. They just appear like normal fields with no grouping.
I have a lot of fields that can be altered through comments but in the actual node, they are optional and collapsed in a fieldgroup hidden from plain view.
It would be awesome if the fieldgroup worked. Or am I not doing it right?

CommentFileSizeAuthor
#1 comment_cck_fieldgroup.patch698 bytesmatt2000

Comments

matt2000’s picture

Component: User interface » Code
Status: Active » Needs review
StatusFileSize
new698 bytes

This is a bug in the D6 upgrade. Here's a patch to fix. It simply adds the last line shown below:

      // This will hold the form with only the fields we need
      $final_form = array();
      $final_form['#node'] = $node;

in function comment_cck_form_alter().

The new line comes after line 32 in my copy.

joshuajabbour’s picture

Status: Needs review » Reviewed & tested by the community

This works as described. Thanks!

joshuajabbour’s picture

Status: Reviewed & tested by the community » Needs work

Actually, this doesn't seem to work. It seems to cause any changed values to not be recorded if the field is in a fieldgroup.

joshuajabbour’s picture

I've written a new, unified patch that fixes this issue along with a few others. It's listed under #420092: Bring comment_cck up-to-date, combine patches and can (should) be applied in place of the patch above.

mitchell’s picture

Status: Needs work » Fixed

Committed. Thanks!

mitchell’s picture

Status: Fixed » Needs work

Actually, fieldgroups don't seem to have the configuration available.

mitchell’s picture

Status: Needs work » Fixed

Actually, this is fixed. I expected to be able to have an #458488: Option to enable commment cck per fieldgroup, so I opened that as a separate issue.

Status: Fixed » Closed (fixed)
Issue tags: -CCK, -alter, -comment, -fieldgroup

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