Applies to both 1.x and 2.x

Problem/Motivation

Facets \Drupal\facets\Plugin\facets\facet_source\SearchApiDisplay class currently typehints the default ModuleHandler class instead of the interface. This prevents custom overrides of ModuleHandler, for example by https://drupal.org/project/hux

Steps to reproduce

Install Hux and Facets.
Navigate to a page which triggers SearchApiDisplay.
Result: Uncaught WSOD/Exception.

Proposed resolution

Typehint the property with interface.

Remaining tasks

Review and commit.

User interface changes

Nil.

API changes

Nil.

Data model changes

Nil.

CommentFileSizeAuthor
#4 3264045-02.patch2.49 KBuditrawat

Issue fork facets-3264045

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dpi created an issue. See original summary.

dpi opened merge request !48

dpi’s picture

dpi
Location Perth, Australia
commented
Status: Active » Needs review
uditrawat’s picture

uditrawat
Primary language English
commented
StatusFileSize
new 3264045-02.patch2.49 KB

Uploading MR#48 as a patch.
Tested with "2.0.5", and with "2.0.x-dev"

dpi’s picture

dpi
Location Perth, Australia
commented

This is going to be more of an issue as sites using Hook Event Dispatcher are beginning to the same failures as it has also implemented a decorator.

randalv’s picture

randalv commented
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community

The patch in #4 works here on 2.0.5, it would be nice to get this committed.
As @dpi said, due to hook_event_dispatcher adding a decorator this will appear on every site using both modules, making this quite urgent. I'm changing this to major.

dpi’s picture

dpi
Location Perth, Australia
commented
Related issues: +#3277301: Adapt to centralised hook dispatcher in Drupal 9.4

Grouping together

idebr’s picture

idebr commented
Status: Reviewed & tested by the community » Closed (duplicate)
Related issues: +#3308343: Use the ModuleHandlerInterface instead of the ModuleHandler concrete implementation

This was already fixed in #3308343: Use the ModuleHandlerInterface instead of the ModuleHandler concrete implementation

This issue targets 8.x-1.x, but this branch is no longer supported.