This is a follow up to #2252763: Views exposed filter form causes enormous form state cache entries

all changes of the form

- $errors = $this->getExecutable()->validate();
+ $errors = $executable->validate();

patch is just the patch renamed from

https://www.drupal.org/node/2252763#comment-9701299

Suggested commit message

git commit -m 'Issue #2449079 by damiankloip | martin107: ViewsUI executable cleanup'

CommentFileSizeAuthor
execute-0.patch1.44 KBmartin107

Comments

fabianx’s picture

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

RTBC

martin107’s picture

martin107 commented

I did not work on this patch, please credit damiankloip instead.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Issue tags: +Quick fix

This is a follow up task for a critical so we can commit it.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Added damiankloip to the commit message.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Oh I guess that is not editable like that... sorry. Please add him:)

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Issue summary: View changes

Added suggested commit message to IS.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, execute-0.patch, failed testing.

Status: Needs work » Needs review

Fabianx queued execute-0.patch for re-testing.

fabianx’s picture

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

  • alexpott committed d611ac4 on 8.0.x 
    Issue #2449079 by martin107, damiankloip: ViewsUI executable cleanup
martin107’s picture

martin107 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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