Closed (fixed)
Project:
Search API block
Version:
1.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
10 Nov 2022 at 17:40 UTC
Updated:
22 Feb 2023 at 16:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
droath commentedComment #3
droath commentedComment #4
phjouGood job droath! Nice feature.
I found two things:
- The patch was crashing when the block was placed on a page that is not a node. So we need ", required = FALSE" in the contextdefinition
- Why not applying the token replacement on the input name as well, that gives more power to content editors
I will make these adjustments and merge.
Comment #5
phjouComment #7
phjouComment #8
fjgarlin commentedThis introduced a regression if the "node" module is not enabled. I know it's not very common on Drupal sites but totally possible.
Before the update, it all works, after the update:
Running a trace, the error is in "SearchBlock.php":
I'll open a follow up issue with all the above info.