Problem/Motivation

Working on removal in #3097428-2: Remove node.module BC layers I found that constant still used in one test

Proposed resolution

Remove the usage and backport to 8.8.x branch

Remaining tasks

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

no

CommentFileSizeAuthor
#2 3097447-2.patch1.04 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
Status: Active » Needs review
FileSize
3097447-2.patch1.04 KB
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Needs review » Reviewed & tested by the community

Better than my fix in #3097428: Remove node.module BC layers where I used the class instead of the interface :)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Version: 8.9.x-dev » 8.8.x-dev
Status: Reviewed & tested by the community » Fixed

Backported to 8.8.x since this is a test-only fix.

Committed and pushed 5814aee96a to 9.0.x and 10214c5c16 to 8.9.x and 4a11d6a3ae to 8.8.x. Thanks!

  • alexpott committed 5814aee on 9.0.x 
    Issue #3097447 by andypost, longwave: Remove remaining usage of...

  • alexpott committed 10214c5 on 8.9.x 
    Issue #3097447 by andypost, longwave: Remove remaining usage of...

  • alexpott committed 4a11d6a on 8.8.x 
    Issue #3097447 by andypost, longwave: Remove remaining usage of...

Status: Fixed » Closed (fixed)

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