search

ollieread / multiauth

Laravel multi auth
441 stars 109 forks source link

Method access level should be public #3

Closed itsmko closed 10 years ago

itsmko commented 10 years ago

i'm getting this error on laravel 4.1.11 when calling Auth::xxxxx()->check();

Symfony \ Component \ Debug \ Exception \ FatalErrorException

Access level to Ollieread\Multiauth\AuthManager::getDefaultDriver() must be public (as in class Illuminate\Auth\AuthManager)

changing getDefaultDriver() method to public resolves the issue

ollieread commented 10 years ago

Ahh the new stable release changes the visibility, will take a look.

GrahamCampbell commented 10 years ago

Yep - https://github.com/laravel/framework/commit/5ce92ec37012782df309f415ef88902ee82e6244

ollieread commented 10 years ago

How very strange, I'm running 4.1.11 and I'm not getting the issue at all.

http://laravel.ollieread.com/ account@test.com:password testuser:password

itsmko commented 10 years ago

i'm using "minimum-stability": "dev"

not sure if this makes any difference

GrahamCampbell commented 10 years ago

@imlion It will do. Using dev means you get the code that will be in the next 4.1.x release. If you want, you can specify 4.1.*@stable on laravel/framework to make sure you only get tagged releases.

ollieread commented 10 years ago

This is now fixed by the new merge thanks to Graham.