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

Allow onUpdate in install #1728

Closed
michsk opened this issue Aug 22, 2015 · 4 comments
Closed

Allow onUpdate in install #1728

michsk opened this issue Aug 22, 2015 · 4 comments
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@michsk
Copy link

michsk commented Aug 22, 2015

I would like to suggest to add onUpdate parameter to Magento\Framework\DB\Ddl\Table::addForeignKey we now can only add onDelete

@vpelipenko vpelipenko added the PS label Aug 25, 2015
@mazhalai
Copy link
Contributor

@Michal-sk Thank you for your suggestion, we have created MAGETWO-42066 to track this request.

@mazhalai mazhalai added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Aug 25, 2015
@akaplya
Copy link
Contributor

akaplya commented Aug 25, 2015

Hi, this is deliberate restriction, to achieve better compliance with other possible database engines.
But you can implement actions like "ON UPDATE" on application level. In this case your code will be compatible with Magento SPI ( events, interceptors, e.t.c).

@piotrekkaminski
Copy link
Contributor

Hello. We decided against this one and also remove existing cascades for performance reasons wherever we could.

@michsk
Copy link
Author

michsk commented Oct 8, 2015

No problem, im actually removing it from other projects as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

5 participants