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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Fabianx’s picture

Status: Needs review » Reviewed & tested by the community

RTBC

martin107’s picture

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

jibran’s picture

Issue tags: +Quick fix

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

joelpittet’s picture

Added damiankloip to the commit message.

joelpittet’s picture

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

jibran’s picture

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

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

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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