Skip to content

Conversation

@AntePrkacin
Copy link

In bundle/QueryType/SearchQueryType file, changed the criterion for location visibility from using standard Ibexa Visibility criterion to using Visible criterion from 'ibexa-search-extra' bundle.

@AntePrkacin AntePrkacin requested a review from pspanja October 1, 2025 12:38
@AntePrkacin AntePrkacin self-assigned this Oct 1, 2025
@emodric
Copy link
Member

emodric commented Oct 2, 2025

Can you explain a bit why is this needed?

@AntePrkacin
Copy link
Author

@emodric
Before, when using the Visibility criterion from Ibexa, it wouldn't check that the Content is hidden AND that the location is it on is hidden. When searching for some content, this resulted in content being shown in the search results, even though it was hidden - the content's location was visible and that's why it was shown.
Now, using the Visible criterion from 'ibexa-search-extra' bundle, it is being checked that the content AND its location are both visible (or hidden).

@AntePrkacin
Copy link
Author

ae5d21e Added "ibexa-search-extra" bundle as dependency

@emodric emodric merged commit 28cd825 into master Oct 2, 2025
1 of 2 checks passed
@emodric
Copy link
Member

emodric commented Oct 2, 2025

Thanks @AntePrkacin !

@emodric emodric deleted the NGSTACK-826-use-Visible-criterion-for-Location-Query-from-search-extra branch October 2, 2025 13:59
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.

3 participants