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-23969: Refactor: Slots should be shared by all Search Engines #1182
Conversation
* | ||
* @copyright Copyright (C) eZ Systems AS. All rights reserved. | ||
* @license For full copyright and license information view LICENSE file distributed with this source code. | ||
* @version //autogentag// | ||
*/ | ||
|
||
namespace eZ\Publish\Core\Search\Solr; | ||
namespace eZ\Publish\Core\Search; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May I suggest we create a Common
namespace or something?
If we do this we redo the mistake done in Persistence having several folders which are not implementation of a engine at root, messes up the convention of <API/SPI-part>/Implementation organization in Core.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it makes sense. I'll update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to Core\Search\Common
in 3b5c3f3.
Now ready for review, ping @bdunogier, @lolautruche, @glye, @yannickroger. |
+1 |
1 similar comment
+1 |
Thanks! |
…-extract-slots Fix EZP-23969: Refactor: Slots should be shared by all Search Engines
This PR resolves https://jira.ez.no/browse/EZP-23969, sub-task of https://jira.ez.no/browse/EZP-23940
This moves Slots into a common namespace and adapts the configuration, making them shared by all search engines (bar Legacy).