While working on #2881567: Fields removed from index remain in sort I noticed that the \Drupal\search_api\Display\DisplayPluginManager is no longer implementing \Drupal\search_api\Display\DisplayPluginManagerInterface.

This change was introduced by #3023704: Convert hooks to events.

CommentFileSizeAuthor
#2 3118224-2.patch584 bytesJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JeroenT created an issue. See original summary.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Active » Needs review
Related issues: +#3023704: Convert hooks to events
FileSize
3118224-2.patch584 bytes

Patch attached re-adds the interface.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Related issues: +#2881567: Fields removed from index remain in sort
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Component: General code » Plugins
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Status: Needs review » Fixed

Thanks a lot for reporting and already providing the patch!
Not sure how that change slipped in, but you’re right, this doesn’t make any sense and the solution is obvious.

Committed. Thanks again!

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Component: Plugins » Framework

  • drunken monkey committed 53114f5 on 8.x-1.x authored by JeroenT 
    Issue #3118224 by JeroenT, drunken monkey: Fixed DisplayPluginManager no...

Status: Fixed » Closed (fixed)

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