moltin / laravel-cart

Laravel Facade and Service Provider for Moltin\Cart
276 stars 87 forks source link

this is not work on laravel 4 #1

Closed aliuygur closed 11 years ago

aliuygur commented 11 years ago

this is not work on laravel 4 i think laravel not allowed session set in __destroy() method

chrisnharvey commented 11 years ago

You're right. I'll update this to use Laravel's built in Session driver.

Sent from my iPhone

On 31 May 2013, at 10:17 PM, ali oygur notifications@github.com wrote:

this is not work on laravel 4 i think laravel not allowed session set in __destroy() method

— Reply to this email directly or view it on GitHubhttps://github.com/moltin/laravel-cart/issues/1 .

aliuygur commented 11 years ago

when ?

chrisnharvey commented 11 years ago

Updating it now :)

aliuygur commented 11 years ago

Okay thank you. i'm waiting

dcolumbus commented 11 years ago

Does this now persist throughout the Session? I posted an issue about this in our other Issue a few days ago.

chrisnharvey commented 11 years ago

This will be resolved today, sorry for the delay.

LensFettish commented 11 years ago

Hi, when will this be fixed, the session storage still doesn't persist...?

chrisnharvey commented 11 years ago

This should now be resolved as of my latest commits to moltin/cart and moltin/laravel-cart