search

Andreyco / Instagram-for-PHP

PHP SDK for Instagram API
BSD 2-Clause "Simplified" License
80 stars 33 forks source link

Setting scope with Laravel config not working #1

Closed grayxr closed 9 years ago

grayxr commented 9 years ago

I was tinkering with Client.php and it seems that the 'mergeScope' method is not working properly with the Laravel implementation. I try to set the scope array, but I keep getting the default 'basic' as the scope parameter when trying to authorize a user with the 'getLoginUrl' method.

Andreyco commented 9 years ago

Will take a look into this.

Andreyco commented 9 years ago

getLoginUrl method takes array of scopes. If this array is empty, defaul basic scope is set instead. Actualy any other scope is merged with this default scope. Everytime.

@jgrayillustrate could you post some example code and expectation? Thanks

grayxr commented 9 years ago

You can view the relevant code in my git repo, here and here. For right now I'm pulling in a forked repo to fix the issue temporarily. BTW, I changed the scopes array back to using only 'basic'. I tried using 'likes' to no avail without using my forked version.

Andreyco commented 9 years ago

Fixed.