Problem/Motivation

Part of effort #1577902: [META] Remove all usages of drupal_static() & drupal_static_reset() for EntityReferenceFieldTranslatedReferenceViewTest.

Proposed resolution

Remove usage of drupal_static_reset() from EntityReferenceFieldTranslatedReferenceViewTest.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

CommentFileSizeAuthor
#2 3035349-2.patch984 bytesclaudiu.cristea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

claudiu.cristea created an issue. See original summary.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon commented
Status: Needs work » Needs review
FileSize
3035349-2.patch984 bytes

Patch.

catch’s picture

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

Straightforward. This could probably have been combined with the other 'remove drupal_static_reset() from tests' issues, but if there's really only three not sure that's worth doing after the fact.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon commented

@claudiu.cristea, I agree but I wrote tickets as I discovered the occurences. I realized later, but the patches were already posted...

catch’s picture

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

Committed be934e6 and pushed to 8.7.x. Thanks!

  • catch committed be934e6 on 8.7.x 
    Issue #3035349 by claudiu.cristea: Remove usage of drupal_static_reset...

Status: Fixed » Closed (fixed)

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