Problem/Motivation

Part of #2959269: [meta] Core should not trigger deprecated code except in tests and during updates

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 interdiff.txt1.59 KBMile23
#4 3003298_4.patch3.14 KBMile23
#2 3003298_2.patch1.55 KBMile23
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mile23 created an issue. See original summary.

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Status: Active » Needs review
FileSize
3003298_2.patch1.55 KB

Just removing the error from the deprecation listener to see what fails.

Status: Needs review » Needs work

The last submitted patch, 2: 3003298_2.patch, failed testing. View results

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Status: Needs work » Needs review
FileSize
3003298_4.patch3.14 KB
interdiff.txt1.59 KB

Just needed to fix the legacy test here.

Updated the CR with some relevant issues.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

Looks good to go

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 2a383d0 and pushed to 8.7.x. Thanks!

  • alexpott committed 2a383d0 on 8.7.x 
    Issue #3003298 by Mile23: Fix "Adding or retrieving messages prior to...

Status: Fixed » Closed (fixed)

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