-
-
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
[Admin][Taxon] Taxon image unique code validation #6183
[Admin][Taxon] Taxon image unique code validation #6183
Conversation
GSadee
commented
Sep 22, 2016
•
edited
Loading
edited
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Related tickets | |
License | MIT |
7b2282d
to
d47f42d
Compare
d47f42d
to
06d8069
Compare
@@ -67,6 +76,8 @@ public function iWantToCreateANewTaxon() | |||
*/ | |||
public function iWantToModifyATaxon(TaxonInterface $taxon) | |||
{ | |||
$this->sharedStorage->set('taxon', $taxon); |
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.
Do we need to set it?
<option name="fields"> | ||
<value>code</value> | ||
<value>owner</value> | ||
</option> |
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.
We should also add this constraint on the database level.
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.
And we already have this constraint: https://github.com/Sylius/Sylius/blob/master/src/Sylius/Bundle/CoreBundle/Resources/config/doctrine/model/TaxonImage.orm.xml#L21
06d8069
to
d342079
Compare
d342079
to
4fa0b74
Compare
Thanks Grzesiu! |
[Admin][Taxon] Taxon image unique code validation