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

Fix EZP-23975: Move Solr Search Engine implementation to new namespace #1178

Merged
merged 15 commits into from Feb 6, 2015

Conversation

pspanja
Copy link
Contributor

@pspanja pspanja commented Feb 6, 2015

This PR resolves https://jira.ez.no/browse/EZP-23975, a sub-task for https://jira.ez.no/browse/EZP-23940

Followup: https://jira.ez.no/browse/EZP-23974

This moves Solr Search Engine implementation to eZ\Publish\Core\Search and updates container configuration.

MVC integration tested manually.

@pspanja
Copy link
Contributor Author

pspanja commented Feb 6, 2015

This is #1161 reopened against master.

Ping @andrerom, @bdunogier, @glye, @lolautruche, @yannickroger.

@andrerom
Copy link
Contributor

andrerom commented Feb 6, 2015

+1, but make sure you test that web application works ;) (we will need to add a bdd test run at some point to kernel, but right now is not a good time, to many changing things)

@pspanja
Copy link
Contributor Author

pspanja commented Feb 6, 2015

MVC integration tested manually.

I did, it works :)

@yannickroger
Copy link
Contributor

+1

@pspanja
Copy link
Contributor Author

pspanja commented Feb 6, 2015

Travis config needed an update, now it passes. Thanks for the review!

pspanja added a commit that referenced this pull request Feb 6, 2015
Fix EZP-23975: Move Solr Search Engine implementation to new namespace
@pspanja pspanja merged commit f513a40 into master Feb 6, 2015
@pspanja pspanja deleted the fix-EZP-23975-move-solr-search branch February 6, 2015 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants