Now you can define forms in areas using views_form in the handlers, but the submit callback is not called because "views_form_views_form_submit" only uses fields.
I'm attaching a patch to invoke submit forms from areas too

CommentFileSizeAuthor
#3 1225322-area_submit-4.patch1.13 KBpcambra
#1 1225322-area_submit-2.patch671 bytespcambra

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new 1225322-area_submit-2.patch671 bytes
bojanz’s picture

bojanz commented

Do we even support views forms from "empty" handlers? I thought we only supported header / footer (my memory is fuzzy on this).
The change itself sounds fine, but we should then to the same with views_form_validate(), no?

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
StatusFileSize
new 1225322-area_submit-4.patch1.13 KB

Yeah, I was doubting about empty as well, but you're right, there's no support for forms in empty area

From views_view_has_form_elements function:

$area_handlers = array_merge(array_values($view->header), array_values($view->footer));

Here's the patch w/o empty area and validate support as well.

bojanz’s picture

bojanz commented
Status: Needs review » Reviewed & tested by the community

Looks good.

dawehner’s picture

dawehner
Primary language German
commented
Status: Reviewed & tested by the community » Fixed

This looks fine. Thanks!

Commited to 7.x-3.x and 6.x-3.x

Status: Fixed » Closed (fixed)

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