Use the ModuleHandlerInterface instead of the ModuleHandler concrete implementation, because other modules might decorate this class and it would lead to some fatal error.

CommentFileSizeAuthor
#2 Use-the-ModuleHandlerInterface-instead-of-the-ModuleHandler-concrete-implementation-3308849-2.patch1.36 KBauth

Comments

auth created an issue. See original summary.

auth’s picture

auth commented
Status: Active » Needs review
StatusFileSize
new Use-the-ModuleHandlerInterface-instead-of-the-ModuleHandler-concrete-implementation-3308849-2.patch1.36 KB
murilomleandro’s picture

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

I can confirm, the proposed patch #2 fixed the issue.

aspilicious’s picture

aspilicious commented

I can confirm this works, we get fatal errors without this patch because hook event subscriber decorates the module handler.

aspilicious’s picture

aspilicious commented
Priority: Normal » Major

  • Berdir committed f7745c4 on 8.x-1.x 
    Issue #3308849 by auth: Use the ModuleHandlerInterface instead of the...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

This was committed, forgot to update the status.

Status: Fixed » Closed (fixed)

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