-
-
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
[Taxation] Scalar types in Taxes #8451
Conversation
Zales0123
commented
Aug 24, 2017
Q | A |
---|---|
Bug fix? | no |
New feature? | no |
BC breaks? | yes |
Related tickets | #8425 |
License | MIT |
@@ -3,16 +3,16 @@ | |||
<div class="divider"> / </div> | |||
{% if product.mainTaxon is not null %} | |||
{% set taxon = product.mainTaxon %} | |||
{% set parents = taxon.parents|reverse %} | |||
{#{% set parents = taxon.parents|reverse %}#} |
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.
#8456 fixes it.
*/ | ||
public function setName($name); | ||
public function setName(string $name): 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.
?string
|
||
* The following methods does not longer have a default null argument and requires one to be explicitly passed: | ||
|
||
* `TaxRateInterface::setTranslatable` |
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.
Also TaxRateInterface::setCategory
3e5ee6b
to
897805c
Compare
@@ -53,7 +53,7 @@ public function getAmount(): float; | |||
public function getAmountAsPercentage(): float; | |||
|
|||
/** | |||
* @param float|null $amount | |||
* @param float $amount |
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.
Should not be changed.
@@ -175,9 +175,9 @@ public function setCalculator(?string $calculator): void | |||
} | |||
|
|||
/** | |||
* @return string|null | |||
* @return string |
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.
Should not be changed.
897805c
to
7af58bb
Compare
Thank you Zaleś! 🥇 |