Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Webform 6.x should support PHP 8.1.
Steps to reproduce
Switch to PHP 8.1 and run webform's automated tests.
Proposed resolution
Fix all PHP 8.1 tickets in one MR.
Below are the outstanding PHP 8.1 tickets which may be merged into one MR.
- #3270344: [PHP 8.1] Compatibility issue with PHP version equal to or greater than 8.1
- #3281958: Error with PHP 8.1
Remaining tasks
Research how Drupal core handler PHP 8.1 deprecations.
User interface changes
N/A
API changes
N/A
Data model changes
N/A
Issue fork webform-3283614
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #5
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedComment #6
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedFor now, I am going to fix all PHP 8.1 notices via the broken tests.
@see https://www.drupal.org/pift-ci-job/2390910
Comment #9
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedComment #10
Liam MorlandFixed typo
Comment #11
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedHere is a zinger
PHP 8.1: HTML entity en/decode functions process single quotes and substitute by default
https://php.watch/versions/8.1/html-entity-default-value-changes
Comment #13
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedComment #17
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot for Vardot commentedThank you so much for have time to work on this issue.
It was released on the
6.2.x
branch, But it's still betaHoping for a release on the
6.1.x
stable branch.Thank you, Jacob for the Webform module.
Comment #18
aaronpinero CreditAttribution: aaronpinero as a volunteer commentedI second Rajah's request. This error with preg_match is holding up my move from PHP 7.3 to PHP 8.1 because we depend on a webform that uses regex in the conditional logic.
Comment #19
gregglesComing here to say the same thing as comments 17 and 18.
We're also working to upgrade to 6.2.x branch, but that will be a little bit longer before we can do that and we'd like to be on php 8.1 ASAP.
Comment #20
aaronpinero CreditAttribution: aaronpinero as a volunteer commentedI was able to composer require 'drupal/webform: 6.1.4-beta1'. That version also has the PHP 8.1 fixes. I might just move forward with that one while waiting for an officially supported release. So far in my testing this beta is okay.
Comment #21
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot for Vardot commentedBetter not to move from
Stable
release to aBeta
release on a live production site.I'm using
"drupal/webform": "~6.0",
in my template project. To be sure that developers are only using a stable version on production.At this time having the following commit patch file
with
"drupal/webform": "6.1.4",
Hoping for a release on the
6.1.x
stable branch