Problem/Motivation

Webform will throw errors when using php 7.1+, see #2811063: Fatal error: Cannot use lexical variable $value as a parameter name in components/select.inc on line 765

Proposed resolution

Update to version 4.15 (coming from 4.14).

Remaining tasks

Update makefile, check patches, test.

User interface changes

None, according to release notes.
https://www.drupal.org/project/webform/releases/7.x-4.15

API changes

Minor: #2851753: API change: Provide $export_info to exporter::download()

Data model changes

None.

CommentFileSizeAuthor
#8 webform_update-2914128-8.patch698 bytesjoshahubbers
#4 webform_update-2914128-4.patch625 bytesjoshahubbers

Comments

askibinski created an issue. See original summary.

askibinski’s picture

askibinski commented
Status: Active » Needs review

Tested, patches still apply.

Wrong commit message, so here is the manual link:
http://drupalcode.org/dvg/commit/?id=3b4417d

ruudvanoijen’s picture

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

Approved. This needs to be in the 1.2 release

joshahubbers’s picture

joshahubbers commented
StatusFileSize
new webform_update-2914128-4.patch625 bytes

Update to Webform 4.16

  • askibinski committed bc49bf0 on 7.x-1.x 
    Issue #2914128 by askibinski, JoshaHubbers: Update webform 4.14 -> 4.16...
askibinski’s picture

askibinski commented
Status: Reviewed & tested by the community » Fixed
joshahubbers’s picture

joshahubbers commented
Title: Update webform 4.14 -> 4.15 (php 7.1+ compatibility) » Update webform 4.14 -> 4.16 (php 7.1+ compatibility)
joshahubbers’s picture

joshahubbers commented
StatusFileSize
new webform_update-2914128-8.patch698 bytes

In addition to the update of webform.
Updated one of the patches for webform (https://www.drupal.org/node/2704757) to a newer version...

  • askibinski committed c4217df on 7.x-1.x 
    Issue #2914128 by JoshaHubbers: Update webform patch to newer version
askibinski’s picture

askibinski commented

Status: Fixed » Closed (fixed)

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