search

SammyK / LaravelFacebookSdk

Fully unit tested Facebook SDK v5 integration for Laravel & Lumen
MIT License
693 stars 201 forks source link

Fix Solved merge config error( Error: Uncaught ReflectionException: Class config does not exist in) #193

Open pouu69 opened 7 years ago

pouu69 commented 7 years ago

When using version 3.5, I found an error in the source that mergeConfigForm. If the source code is placed in the function register(){} the following error occurs.

Uncaught ReflectionException: Class config does not exist in .. blah blah

So I moved to the function boot(){} and modified the slightly fixed source. And it works well.

almost same code

pouu69 commented 7 years ago

Why is CI failing?

LukeTowers commented 7 years ago

Check the log, it's because of a style violation. You need to have a space after your else keyword, and a space after your closing brace. https://s3.amazonaws.com/archive.travis-ci.org/jobs/272386964/log.txt?X-Amz-Expires=30&X-Amz-Date=20170909T152213Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170909/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host&X-Amz-Signature=eb0b8c43dd2c1e294db9d1d384225d595942806fd17d1df6916169b1157dd0b9

pouu69 commented 7 years ago

thx