Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: Extend AbstractSearchProcessor#postSearch(...) of the org.eclipse.help.searchProcessor extension point #1725

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

howlger
Copy link
Contributor

@howlger howlger commented Feb 4, 2025

The org.eclipse.help.searchProcessor extension point allows to modify the help search results. This is done by calling the postSearch(...) method of all implementations of org.eclipse.help.search.AbstractSearchProcessor specified by this extension point. In some cases, the search results to be modified and the query that are passed as parameters to the postSearch(...) method may not be enough. So this change adds another postSearch(...) method with additional parameters like the locale and the scopes if any. The simpler postSearch(...) is be kept for backward compatibility.

Additionally, if preSearch(...) returns a SearchProcessorInfo with an empty, non-null query (""), no search will be executed, resulting in no search results or in the search results that has been added via postSearch(...).

The `org.eclipse.help.searchProcessor` extension point allows to modify
the help search results. This is done by calling the `postSearch(...)`
method of all implementations of
`org.eclipse.help.search.AbstractSearchProcessor` specified by this
extension point. In some cases, the search results to be modified and
the query that are passed as parameters to the `postSearch(...)` method
may not be enough. So this change adds another `postSearch(...)` method
with additional parameters like the locale and the scopes if any. The
simpler `postSearch(...)` is be kept for backward compatibility.

Additionally, if `preSearch(...)` returns a `SearchProcessorInfo` with
an empty, non-`null` query (`""`), no search will be executed, resulting
in no search results or in the search results that has been added via
`postSearch(...)`.
@howlger howlger force-pushed the patch-searchProcessor branch from c74b88b to a202f0c Compare February 4, 2025 17:11
@howlger
Copy link
Contributor Author

howlger commented Feb 4, 2025

@iloveeclipse This is a work for Advantest and may be helpful for others too.

Copy link
Contributor

github-actions bot commented Feb 4, 2025

Test Results

 1 755 files  +12   1 755 suites  +12   1h 30m 9s ⏱️ -38s
 4 170 tests ± 0   4 147 ✅ + 1   23 💤 ±0  0 ❌  - 1 
13 107 runs  +88  12 940 ✅ +88  167 💤 +1  0 ❌  - 1 

Results for commit a202f0c. ± Comparison against base commit 31bb835.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant