In relation to the issue Make "search id" a query property instead of option the "search id" options was converted to the property $searchID.

This effects two lines in the module

src/Plugin/facets/facet_source/SearchApiPage.php:82:
'search id' => 'search_api_page:' . $search_api_page->id(),

src/Controller/SearchApiPageController.php:62:
'search id' => 'search_api_page:' . $search_api_page->id(),

The first instance of this bug causes Facets on search API pages not to render, since an auto generated search id is created and thus the results are not retrieved properly.

CommentFileSizeAuthor
#2 2832565-2.patch1.52 KBrichgerdes
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

richgerdes created an issue. See original summary.

richgerdes’s picture

richgerdes
he/him
Primary language English
Location New Jersey, USA
CreditAttribution: richgerdes commented
Issue summary: View changes
FileSize
2832565-2.patch1.52 KB

Here is the patch to fix these two instances.

richgerdes’s picture

richgerdes
he/him
Primary language English
Location New Jersey, USA
CreditAttribution: richgerdes commented
Status: Active » Needs review
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Needs review » Reviewed & tested by the community

  • swentel committed 4727584 on 8.x-1.x authored by richgerdes 
    Issue #2832565 by richgerdes: Search ID was moved to a property instead...
swentel’s picture

swentel CreditAttribution: swentel commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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