Working on #2078001: Modernize responsive_image.module forms I've found that this form class using global functions instead of one provided by FormBase class. We must updated it in order to correspond this change record https://drupal.org/node/2076011.

CommentFileSizeAuthor
#1 modernize_entityconfirmformbase-2095681-1.patch782 bytesRoSk0
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RoSk0’s picture

RoSk0
he/him
Primary language Russian
Location Wellington
CreditAttribution: RoSk0 commented
Status: Active » Needs review
FileSize
modernize_entityconfirmformbase-2095681-1.patch782 bytes

Initial patch.

plopesc’s picture

plopesc
he/him
Primary language Spanish
CreditAttribution: plopesc commented
Status: Needs review » Reviewed & tested by the community

Nice catch!!

It looks good to me.

RoSk0’s picture

RoSk0
he/him
Primary language Russian
Location Wellington
CreditAttribution: RoSk0 commented
Assigned: Unassigned » RoSk0
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 8b51c00 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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