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

[SF4] Implemented factory for IndexingDepthProvider #141

Merged
merged 1 commit into from
Jun 25, 2019

Conversation

alongosz
Copy link
Member

@alongosz alongosz commented Jun 17, 2019

Bug yes
Target version v2.0 (master) for eZ Platform v3.0
Required by ezpublish-kernel, ezplatform-richtext to unblock failing Solr integration tests.

Symfony 4 no longer allows using private services in DIC @=service expressions so we need to provide
factory instead.

TODO:

  • Confirm Travis agrees
  • Check code style

@alongosz alongosz requested review from adamwojs and webhdx June 17, 2019 18:43
@alongosz alongosz force-pushed the sf4-fix-indexing-depth-provider branch 2 times, most recently from 73877f5 to 06240f9 Compare June 18, 2019 14:02
Symfony 4 no longer allows using private services
in DIC @=service expressions so we need to provide
factory instead.
@alongosz alongosz force-pushed the sf4-fix-indexing-depth-provider branch from 06240f9 to afa4c21 Compare June 18, 2019 14:14
@alongosz
Copy link
Member Author

Merging to unblock failing RichText integration tests

@alongosz alongosz merged commit 680677d into ezsystems:master Jun 25, 2019
@alongosz alongosz deleted the sf4-fix-indexing-depth-provider branch June 25, 2019 08:36
alongosz added a commit that referenced this pull request Jan 31, 2020
* Added MaxDepth parameter for Content and Location FullText Criteria

* Set default max depth=2 for integration tests

* [SF4 forward compatibility] Implemented factory for IndexingDepthProvider (back-ported from #141)

* Refactored max-depth parameter providing

Co-authored-by: Andrew Longosz <alongosz@users.noreply.github.com>
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.

3 participants