- Issue created by @ericgsmith
- 🇳🇿New Zealand ericgsmith
If this idea is something that could be accepted I can work on a patch - otherwise we will likely implement this by extending the processor to add the query tag check in a custom module - but we prefer to avoid custom code and upstream the solution if we can.
- Merge request !245Issue #3532193: Add ability to skip Highlight processor using query tag → (Merged) created by ericgsmith
- 🇳🇿New Zealand ericgsmith
Easier to show what I mean with an MR - setting to needs review.
- Status changed to Needs review
about 1 month ago 1:51pm 29 August 2025 - 🇦🇹Austria drunken monkey Vienna, Austria
drunken monkey → made their first commit to this issue’s fork.
- 🇦🇹Austria drunken monkey Vienna, Austria
Thanks a lot for posting this suggestion, and sorry it took me so long to get back to you.
Your use case definitely makes sense. And while my first instinct was to come up with some architecture for skipping or even adding specific processors in a generic way ( #1720348: Add the concept of query extenders → would be something similar, but I think there was also some issue specifically for overriding the used processors on a per-query basis), you are probably right that just skipping the “Highlight” processor is the primary use case and before adding yet another complicated and rarely-used component to the framework, just doing this with a query tag seems like a great and simple option.A query option would probably be a bit cleaner, but then you couldn’t just set it using the existing Views UI and we’d either have to add UI for it after all or people would need to implement an alter hook just for this. I think I can live with this just being a magic tag for now, we can see whether there is any interest in changing this or we notice any disadvantages.
I just added a small paragraph about this to the README.md file so people have a better chance of finding out about it. Please review and tell me what you think, otherwise I think this is RTBC.And thanks again, especially also for already including test coverage!
- 🇳🇿New Zealand ericgsmith
Thanks Thomas - the update to the README looks good to me - thank you for the review!
- 🇦🇹Austria drunken monkey Vienna, Austria
Great to hear, thanks for reporting back!
Merged.
Thanks again! Now that this issue is closed, please review the contribution record.
As a contributor, attribute any organization helped you, or if you volunteered your own time.
Maintainers, please credit people who helped resolve this issue.
-
drunken monkey →
committed 4ac90c7a on 8.x-1.x authored by
ericgsmith →
[#3532193] feat: Added the ability to skip the Highlight processor for...
-
drunken monkey →
committed 4ac90c7a on 8.x-1.x authored by
ericgsmith →