Problem/Motivation

From #2901418-85: Add hook_post_config_import_NAME after config import
The reference to event name has wrong namespace

Proposed resolution

Fix and backport to 8.9

Remaining tasks

review/commit

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

no

CommentFileSizeAuthor
#3 3156123-3.patch1.18 KBandypost
#3 interdiff.txt636 bytesandypost
#2 3156123-2.patch574 bytesandypost
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
3156123-2.patch574 bytes

Fix and another follow-up #3156126: Make MissingContentEvent extend non-deprecated event class

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
FileSize
interdiff.txt636 bytes
3156123-3.patch1.18 KB

One more clean-up from the patch

johnwebdev’s picture

johnwebdev CreditAttribution: johnwebdev commented
Status: Needs review » Reviewed & tested by the community

Yes, looks about right

alexpott’s picture

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

Committed and pushed 50e58a9ff5 to 9.1.x and d6cd19407a to 9.0.x and 6cd0b99d80 to 8.9.x. Thanks!

  • alexpott committed 50e58a9 on 9.1.x 
    Issue #3156123 by andypost: Fix MissingContentEvent see reference

  • alexpott committed d6cd194 on 9.0.x 
    Issue #3156123 by andypost: Fix MissingContentEvent see reference

(...

  • alexpott committed 6cd0b99 on 8.9.x 
    Issue #3156123 by andypost: Fix MissingContentEvent see reference

(...

Status: Fixed » Closed (fixed)

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