#2453175: Remove EntityFormInterface::validate() and stop using button-level validation by default in entity forms follow-up. We need to replace calls to EntityFormInterface::validate() with calls to FormInterface::validateForm().

Change record: http://drupal.org/node/2524364

CommentFileSizeAuthor
#2 2528998_1.patch1.67 KBslashrsm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

slashrsm’s picture

slashrsm CreditAttribution: slashrsm at Examiner.com commented
Issue summary: View changes
slashrsm’s picture

slashrsm CreditAttribution: slashrsm at Examiner.com commented
Status: Active » Needs review
FileSize
2528998_1.patch1.67 KB
Primsi’s picture

Primsi CreditAttribution: Primsi at Examiner.com for Examiner.com commented
Status: Needs review » Reviewed & tested by the community

Looks ok to me. After an IRC chat with slashrsm tests locally pass.

webflo’s picture

webflo CreditAttribution: webflo at UEBERBIT GmbH commented
Status: Reviewed & tested by the community » Fixed

Thanks!

  • webflo committed f0760ab on 8.x-1.x 
    Issue #2528998 by slashrsm: Chase core: Validation of inline entities

Status: Fixed » Needs work

The last submitted patch, 2: 2528998_1.patch, failed testing.

Status: Needs work » Needs review

isntall queued 2: 2528998_1.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: 2528998_1.patch, failed testing.

webflo’s picture

webflo CreditAttribution: webflo at UEBERBIT GmbH commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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