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

[DB] Allow orders to be deleted directly through DB with correct onDelete=CASCADEs #3648

Merged
merged 1 commit into from
Dec 14, 2015

Conversation

peteward
Copy link

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets
License MIT
Doc PR

Setup onDelete=CASCADE foreign keys for Order and OrderItem relations so that orders can be deleted directly in the DB along with their relevant relations:

  • Order => OrderItem
  • Order => Payment
  • Order => Shipment
  • Order => InventoryUnit
  • Order => Adjustment
  • Order => OrderPromotion (m2m)
  • OrderItem => OrderItemPromotion (m2m)
  • OrderItem => InventoryUnit
  • OrderItem => Adjustment

… so that orders can be deleted directly in the DB along with their relevant relations:

Order => OrderItem
Order => Payment
Order => Shipment
Order => InventoryUnit
Order => Adjustment
Order => OrderPromotion (m2m)
OrderItem => OrderItemPromotion (m2m)
OrderItem => InventoryUnit
OrderItem => Adjustment
@pjedrzejewski pjedrzejewski added this to the v0.16.0 milestone Nov 29, 2015
@PWalkow
Copy link
Contributor

PWalkow commented Dec 1, 2015

👍

@pjedrzejewski pjedrzejewski removed this from the v0.16.0 milestone Dec 14, 2015
pjedrzejewski pushed a commit that referenced this pull request Dec 14, 2015
[DB] Allow orders to be deleted directly through DB with correct onDelete=CASCADEs
@pjedrzejewski pjedrzejewski merged commit 421809a into Sylius:master Dec 14, 2015
@pjedrzejewski
Copy link
Member

Thanks Peter!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants