Skip to content
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.

Request::hasSessionStore() has been removed #154

Merged
merged 1 commit into from
Feb 10, 2014

Conversation

shrikantmeena
Copy link
Contributor

Users of laravel 4.1 branch will get an error when the validator fires.

Due to the commit laravel/framework@5a1829f made to 4.1 framework branch in which hasSessionStore() method has been removed.

Users of laravel 4.1 branch will get an error when the validator fires.

Due to the commit 5a1829f7658a5cf752b97c354fa89386ee60d5dc made to 4.1 framework branch in which hasSessionStore() method has been removed.
@scarlac
Copy link

scarlac commented Jan 21, 2014

This patch seems to be working...

@benharold
Copy link

How do I tell composer to use this patch before it gets pulled into the main repo? Or is that possible?

@scarlac
Copy link

scarlac commented Jan 28, 2014

In my experience, you don't. I've applied several patches on Laravel itself for quite a few bugs and they are always lost upon update/upgrade without warning.

@benharold
Copy link

So just patch it in my vendor directory until a composer update overwrites it?

@warksit
Copy link
Contributor

warksit commented Jan 29, 2014

Instructions on how to use a branch with the fix:

#133 (comment)

Or just limit laravel to 4.1.16 in composer.json

@benharold
Copy link

Thanks @awestrope!

igorsantos07 added a commit that referenced this pull request Feb 10, 2014
Request::hasSessionStore() has been removed from Laravel.
@igorsantos07 igorsantos07 merged commit 1fce9b0 into laravel-ardent:master Feb 10, 2014
@igorsantos07
Copy link
Member

Btw, thanks @awestrope for the great tip for the others (:

softnix11 added a commit to softnix11/entrust that referenced this pull request Feb 19, 2022
…solved]

Per laravel-ardent/ardent#154 Ardent updated and stable. No need for workaround. Should resolve #91 and #94. See Also comments on Zizaco/entrust@d030af2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants