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

Replaced array_get() with Arr::get(), array_get() removed in Laravel 6.0 #590

Open
wants to merge 67 commits into
base: master
Choose a base branch
from

Conversation

webtamizhan
Copy link

This appears to be a breaking change in Laravel 6.0

6.0 - Uses the following

Arr::get($attributes, 'id', $this->id);

instead of

array_get($attributes, 'id', $this->id);

raakesh and others added 30 commits February 27, 2018 11:53
Fixing links for 
- Storing the cart
- Restoring the cart
in Database section
Fixes the rounding issue for tax.  See Crinsane#344
add save for later flag on cart item
added section about Buyable interface to README
Add Buyable interface path to documentation
deleteStoredCart() added
restore wont delete stored cart
made $session protected
support L5.8
Update event->fire to event->dispatch
hardevine and others added 30 commits November 28, 2019 21:30
L6 support update
Using use Illuminate\Support\Arr; instead of array_get
Database query also check for instance
toJson Method will return CartItem's model if available. so it can be used in Vue Component
toJson return with model if available
Fixed a bug where 0 is passed as the taxrate to the Cart:add facade resulting in it not evaluating right since '0' is not true in a if statement. Changed therefore the comparison to isset() so it rather checks if there is passed a taxrate of some kind. Also added in the if statement a simple check to see if the taxrate is numeric, and if not default to the one in the config
deleteStoredCart made 'public'
Fix MongoDB Driver issue Crinsane#478
Add support to Laravel 9
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.

None yet