Problem/Motivation

Part of #3220021: [meta] Ensure compatibility of Drupal 9 with PHP 8.1 (as it evolves)

Steps to reproduce

Proposed resolution

Form builder should get 0 instead of NULL when counter initially requested.
Also first submission could use 0 as default too.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3239558-2.patch1.25 KBandypost
#2 interdiff.txt1.25 KBandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Assigned: Unassigned » alexpott
Status: Active » Needs review
FileSize
interdiff.txt1.25 KB
3239558-2.patch1.25 KB

@alexpott interdiff against META

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Needs review » Reviewed & tested by the community

Looks good, just adding a default argument.

alexpott’s picture

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

Committed 2178a98 and pushed to 9.3.x. Thanks!

  • alexpott committed 2178a98 on 9.3.x 
    Issue #3239558 by andypost: Properly use state counter in \Drupal\...

Status: Fixed » Closed (fixed)

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