Problem/Motivation

The ExceptionController class has a static method that creates the exception listener. As noted in the phpdoc, this code does not belong to this file and anyway, this can be done in the DIC quite easily.

Proposed resolution

The patch removes this factory and moves the configuration directly into the DIC, where it belongs.

CommentFileSizeAuthor
#1 exception-listener-factory.patch2.49 KBfabpot
exception-listener-factory.patch3.24 KBfabpot
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fabpot’s picture

fabpot CreditAttribution: fabpot commented
FileSize
exception-listener-factory.patch2.49 KB

Re-uploaded the right patch.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

It is amazing to see the flexibility of the DIC.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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