Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Related to #1287924: There should be an alter hook to manipulate search keywords.
If the related issue is incorporated into core, a similar patch would be necessary for page manager's search task.
Without a patch here, the other hooks fire, but if you've got a view that receives args from the search panel page, those args wouldn't have been altered.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | issue_1287962-implement_search_alter_hooks-1.patch | 616 bytes | willvincent |
| #2 | issue_1287962-implement_search_alter_hooks-1-p0.patch | 491 bytes | willvincent |
| #1 | issue_1287962-implement_search_alter_hooks.patch | 615 bytes | willvincent |
| #1 | issue_1287962-implement_search_alter_hooks-p0.patch | 490 bytes | willvincent |
Comments
Comment #1
willvincent CreditAttribution: willvincent commentedThis patch seems to take care of the issue for me.
Ideally one should apply the patch from the related core issue as well as this patch, though with drupal_alter that doesn't appear necessary.
Attached is a version based on the current git 7.x-1.x dev branch, and an older p0 format version that can be easily applied with drush make.
Comment #2
willvincent CreditAttribution: willvincent commentedApparently I'm full of fail and syntax errors today.
Here's a syntactically correct version of the above.
Comment #3
merlinofchaos CreditAttribution: merlinofchaos commentedCommitted #2. I figure there's no problem committing ahead of core.
If somehow core commits a different alter hook name, we'll have to adjust.