#1208824: use 'views_ui_pre_render_add_fieldset_markup' for plugins added the helpful #fieldset option. But it's missing from most plugins:

grep -nr "parent::options_form" plugins | wc -l
17
ls -l plugins | wc -l
49

Plus, it's missing from views_plugin_style.inc itself.

Patch to come.

Comments

tim.plunkett’s picture

Status: Active » Needs review
StatusFileSize
new10.8 KB

My grep counts weren't exact, not every plugin used options_form, and one of them was the original options_form(){}.

dawehner’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Needs review » Patch (to be ported)

Thanks for providing that patch!

Committed to 7.x-3.x
This probably makes sense to add to 6.x-3.x as well.

tim.plunkett’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new13 KB

Here's the D6 version. But now that I look, this patch adds to the plugins in the /modules directory, I might have missed some for D7? I'll look into that tomorrow.

mustanggb’s picture

Issue summary: View changes
Status: Needs review » Closed (won't fix)