-
-
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
[Behat][Promotion] Removed softdeletable from Promotion #4401
Conversation
$this->itemQuantityModifier->modify($item, 1); | ||
|
||
$promotion->setUsed(1); | ||
$order->addPromotion($promotion); |
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.
The promotion should be attached by orderRecalculator
ac4b91f
to
988f1de
Compare
This should fix #1799. |
Don't think so as I don't even touch that constraint. Seems more like #4047 will fix it by not letting you to delete it if the constraint is present. |
988f1de
to
db6e6c1
Compare
/** | ||
* Auto-generated Migration: Please modify to your needs! | ||
*/ | ||
class Version20160305115221 extends AbstractMigration |
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.
Why do you have two migrations? Can you make only one?
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.
One's from the base branch => #4400
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.
👍
@NeverResponse I missed the fact that this scenario is only implemented at the domain level :) |
b9c6552
to
3669bc7
Compare
3669bc7
to
5a86e3d
Compare
public function iShouldBeNotifiedOfFailure() | ||
{ | ||
expect($this->sharedStorage->get('last_exception')) | ||
->toThrow(ForeignKeyConstraintViolationException::class) |
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.
toBeInstanceOf
?
Rebase needed :) |
5a86e3d
to
6d81f4a
Compare
6d81f4a
to
4b43865
Compare
[Behat][Promotion] Removed softdeletable from Promotion
Thanks Jasiek! 👍 |
Based on #4400