-
Notifications
You must be signed in to change notification settings - Fork 166
\TwigBridge\Extension\Laravel\Session is broken #280
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
Comments
Can you submit a PR? And does it fix your problem? |
I guess same issue here |
@SergeyMiracle |
Same here, but I get |
Looks like @lbausch beat me to it. (Thank you!) That's pretty much the exact fix I did locally, so that should work perfectly. |
\TwigBridge\Extension\Laravel\Session
type-hints against\Illuminate\Session\SessionManager
but I'm pretty sure it should be\Illuminate\Session\Store
.What you had there was working fine until just recently when I updated my packages. I wish I could tell you what exactly broke it, but I couldn't get it working again even after I rolled all my packages back to what I had in my old
composer.lock
file.I upgraded from Laravel 5.2.5 to 5.2.45.
My log blows up with errors; here's just one of the exceptions I get:
If you check out
\Illuminate\Session\SessionManager
, it indeed does not have a method calledget
but\Illuminate\Session\Store
does, so I think this is right.I was only able to figure this out because of Otwell's blog post in which has this snippet:
The text was updated successfully, but these errors were encountered: