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

hasTax() for price #126

Merged
merged 4 commits into from
Jun 13, 2021
Merged

hasTax() for price #126

merged 4 commits into from
Jun 13, 2021

Conversation

Larsklopstra
Copy link
Contributor

Easily check if the price has tax:

$price->price()->hasTax();

In some countries sales tax is not required, this allows for easier conditionals.

@taylorotwell taylorotwell merged commit 108ce67 into laravel:master Jun 13, 2021
driesvints pushed a commit that referenced this pull request Jan 6, 2022
* Update Price.php

* Update ProductPriceTest.php

* Update Price.php

* Update Price.php

Co-authored-by: Taylor Otwell <[email protected]>
@driesvints driesvints mentioned this pull request Jan 6, 2022
@driesvints
Copy link
Member

I noticed this was sent to master so I cherry picked this to master: #149

taylorotwell added a commit that referenced this pull request Jan 6, 2022
* `hasTax()` for price (#126)

* Update Price.php

* Update ProductPriceTest.php

* Update Price.php

* Update Price.php

Co-authored-by: Taylor Otwell <[email protected]>

* Apply fixes from StyleCI

Co-authored-by: Lars Klopstra <[email protected]>
Co-authored-by: Taylor Otwell <[email protected]>
Co-authored-by: StyleCI Bot <[email protected]>
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