The simplify node form option seemed super useful at first, but I found it too zealous in its simplification of my node forms. Specifically I did not want CCK fieldsets that I created myself to be dumped into Advanced options. This patch adds an option to exclude CCK fieldsets from Advanced options. As best I can tell, all the CCK fieldsets start with "group_" so all this does is exclude any fieldsets with that in the name. It appears to work well for me. The patch is against the 6.x version in this thread: http://drupal.org/node/311333, as that is what I am using, but it appears to apply cleanly to the 5.x-1.0 version as well, though I haven't tested it with Drupal 5.x.

Comments

nedjo’s picture

Status: Needs review » Needs work

Makes sense and the patch looks good.

Patch should now be on new formfilter_ui module in HEAD. Other minor issues:

* Pass title and description through t() in $form['formfilter_simplify_node_cck']
* else clauses should be on separate line and should have a space before the bracket
* should use module_exists('content') to show the form and do the special handling only if if the content module is present.

I'd welcome a new patch. Thanks!

gilgabar’s picture

Status: Needs work » Needs review
StatusFileSize
new1.67 KB

Thanks for catching that stuff. Here is a revised patch that should resolve those issues.

gilgabar’s picture

StatusFileSize
new1.68 KB

This is a patch against the current 6.x-1.x-dev version.

nedjo’s picture

Status: Needs review » Fixed

Belatedly, thanks. I applied a simplified version--no config, just some explanation in the existing config that CCK fieldgroups aren't affected.

Status: Fixed » Closed (fixed)

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