There's a typo in webform_update_8087()

"wizard_comfirmation_label" instead of "wizard_confirmation_label" gets set

CommentFileSizeAuthor
#2 typo_in_update_8087-2920762-2.patch1.09 KBjrockowitz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agoradesign created an issue. See original summary.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Active » Needs review
FileSize
typo_in_update_8087-2920762-2.patch1.09 KB

The attached patch repairs the mistake.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs review » Fixed

I committed the attached patch. Please download the latest dev release to review.

agoradesign’s picture

agoradesign CreditAttribution: agoradesign commented

Why are you unsetting different keys?

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented

Thanks for catching that second mistake.

http://cgit.drupalcode.org/webform/commit/?id=8c815b4

agoradesign’s picture

agoradesign CreditAttribution: agoradesign commented

looks better :)

  • jrockowitz committed c0ddfb6 on 8.x-5.x 
    Issue #2920762 by jrockowitz: Typo in update 8087: "...

  • jrockowitz committed 8c815b4 on 8.x-5.x 
    Issue #2920762 by jrockowitz: Typo in update 8087: "...

Status: Fixed » Closed (fixed)

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