-
-
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
[Order] Use typehints introduced by PHP7+ #8473
Conversation
pamil
commented
Aug 28, 2017
Q | A |
---|---|
Bug fix? | no |
New feature? | no |
BC breaks? | yes |
Related tickets | part of #8425 |
License | MIT |
2eb30dc
to
1ae9875
Compare
|
||
/** | ||
* @param AdjustableInterface|null $adjustable | ||
*/ | ||
public function setAdjustable(AdjustableInterface $adjustable = null); | ||
public function setAdjustable(?AdjustableInterface $adjustable): void; |
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 mention this in UPGRADE.md
file
*/ | ||
public function setOrder(OrderInterface $order = null); | ||
public function setOrder(?OrderInterface $order): void; |
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 this.
*/ | ||
public function setCheckoutCompletedAt(\DateTimeInterface $checkoutCompletedAt = null); | ||
public function setCheckoutCompletedAt(?\DateTimeInterface $checkoutCompletedAt): void; |
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 this and everything else where we remove the default value from public method.
1ae9875
to
a861962
Compare
Thank you Kamil! 🥇 |
Thank you for this medal, it means so much for me! |
[Order] Use typehints introduced by PHP7+