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

EZP-31226: Added MaxDepth parameter for query location criterion and set default max depth=2 for integration tests #165

Merged
merged 3 commits into from
Jan 31, 2020
Merged

EZP-31226: Added MaxDepth parameter for query location criterion and set default max depth=2 for integration tests #165

merged 3 commits into from
Jan 31, 2020

Conversation

michal-myszka
Copy link
Contributor

@michal-myszka michal-myszka commented Jan 22, 2020

Question Answer
JIRA issue EZP-31226
Bug/Improvement yes
New feature no
Target version 1.7 for bug fixes or improvements (on existing features)
BC breaks no
Tests pass yes
Doc needed no

Solution for reported bug:
https://jira.ez.no/browse/EZP-31226
Fixed missing "depth" parameters and added new test for fulltext search for embed content
Changes also in ezpublish-kernel: ezsystems/ezpublish-kernel#2922

TODO:

Copy link
Member

@alongosz alongosz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please adjust naming conventions as described in the Kernel PR.

composer.json Outdated Show resolved Hide resolved
lib/Resources/config/container/solr/criterion_visitors.yml Outdated Show resolved Hide resolved
@michal-myszka michal-myszka changed the title EZP-31226: Indexing depth doesn't work properly in Solr when Locatio… [DO NOT MERGE] EZP-31226: Indexing depth doesn't work properly in Solr when Locatio… Jan 22, 2020
@michal-myszka michal-myszka changed the title [DO NOT MERGE] EZP-31226: Indexing depth doesn't work properly in Solr when Locatio… [DO NOT MERGE] EZP-31226: Added MaxDepth parameter for query location criterion and set default max depth=2 for integration tests Jan 22, 2020
@michal-myszka michal-myszka changed the base branch from 2.0 to 1.7 January 27, 2020 10:46
Copy link
Member

@alongosz alongosz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, friendly reminder: CI is still failing :)

@michal-myszka
Copy link
Contributor Author

+1, friendly reminder: CI is still failing :)

after talk with the @adamwojs this CI fail is not connected with my PR

@adamwojs
Copy link
Member

after talk with the @adamwojs this CI fail is not connected with my PR

Yes, CI failure is not related to PR changes. I will take care of it.

@alongosz
Copy link
Member

Ok, you can move this to QA.

Copy link
Member

@micszo micszo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA Approved on eZ Platform EE v2.5.8 with diffs.

@micszo micszo removed their assignment Jan 31, 2020
Michał Myszka and others added 3 commits January 31, 2020 12:09
Symfony 4 no longer allows using private services
in DIC @=service expressions so we need to provide
factory instead.
@alongosz alongosz changed the title [DO NOT MERGE] EZP-31226: Added MaxDepth parameter for query location criterion and set default max depth=2 for integration tests EZP-31226: Added MaxDepth parameter for query location criterion and set default max depth=2 for integration tests Jan 31, 2020
@alongosz alongosz merged commit 290b393 into ezsystems:1.7 Jan 31, 2020
alongosz pushed a commit to ezsystems/ezpublish-kernel that referenced this pull request Jan 31, 2020
@alongosz
Copy link
Member

Merged up to 2.0 as a9b568b and up to master as bc2f8dc. We'll need a follow-up for Solr tests in Kernel as the scoring changed and I had to temporarily disable one of them on master via ezsystems/ezpublish-kernel@a2dd5ad2af.

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

Successfully merging this pull request may close these issues.

4 participants