It would be cool if you wouldn't have to set '#process' => array('ctools_dependent_process') on every field.

Therefore this could be added to every form element.

CommentFileSizeAuthor
#3 1062972-depedenct-automatic-process.patch4.93 KBdawehner
#1 1062972-depedenct-automatic-process.patch1.2 KBdawehner

Comments

dawehner’s picture

dawehner
Primary language German
commented
StatusFileSize
new 1062972-depedenct-automatic-process.patch1.2 KB

Here is the patch.

merlinofchaos’s picture

merlinofchaos commented

We don't actually need it in #process, we need it in #pre_render. In D7 I moved this to prevent it from adding javascript even if the form isn't rendered. I wish I'd known about pre_Render back when we wrote this the first time. :)

dawehner’s picture

dawehner
Primary language German
commented
StatusFileSize
new 1062972-depedenct-automatic-process.patch4.93 KB

So here is patch with this changes.

Not sure whether it's there the right place to set "Deprecated".

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Had to fix a merge conflict with the textarea stuff, but fixed that and pushed.

We need a followup to remove all the #process designations everywhere.

Plus, we need some more solid documentation on using this with things like fieldsets; I think it'll work more nicely now.

Status: Fixed » Closed (fixed)

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