-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[VariationBundle] Added option repository #4856
Conversation
GSadee
commented
Apr 23, 2016
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Related tickets | |
License | MIT |
@@ -115,7 +116,7 @@ private function addClassesSection(ArrayNodeDefinition $node) | |||
->scalarNode('model')->isRequired()->cannotBeEmpty()->end() | |||
->scalarNode('interface')->isRequired()->cannotBeEmpty()->end() | |||
->scalarNode('controller')->defaultValue(ResourceController::class)->cannotBeEmpty()->end() | |||
->scalarNode('repository')->cannotBeEmpty()->end() | |||
->scalarNode('repository')->defaultValue(OptionRepository::class)->cannotBeEmpty()->end() |
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.
This couples it to Doctrine ORM, we do not specify driver specific repositories in Configuration class. You should load a driver file, which will contain the repo. With VariationBundle it can be a bit more complex, because we generate the services, but I am pretty sure @lchrusciel will be able to help here because we have figured out this with ArchetypeBundle.
e8cc406
to
4212574
Compare
/** | ||
* @author Grzegorz Sadowski <grzegorz.sadowski@lakion.com> | ||
*/ | ||
interface OptionRepositoryInterface |
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.
Should extend Sylius\Component\Resource\Repository\RepositoryInterface
4212574
to
41356e9
Compare
Thank you Grzesiu! |