philippec / PhFacebook

MacOSX Interface to Facebook graph API
http://developer.casgrain.com/?p=107
Other
178 stars 44 forks source link

getAccessTokenForPermissions: silently dies if user isn't logged in to FB. #3

Closed AndrewSchenk closed 13 years ago

AndrewSchenk commented 13 years ago

In both the sample code and my own code, the method

getAccessTokenForPermissions:

fails to return a proper error to the delegate method (tokenResult:) for when the user isn't logged into a FB account.

The console shows this:

didCommitLoadForFrame: {https://www.facebook.com/login.php?api_key=112800368794148&skip_api_login=1&display=popup&cancel_url=http%3A%2F%2Fwww.facebook.com%2Fconnect%2Flogin_success.html%3Ferror_reason%3Duser_denied%26error%3Daccess_denied%26error_description%3DThe%2Buser%2Bdenied%2Byour%2Brequest.&fbconnect=1&next=http%3A%2F%2Fwww.facebook.com%2Fconnect%2Fuiserver.php%3Fmethod%3Dpermissions.request%26app_id%3D112800368794148%26display%3Dpopup%26redirect_uri%3Dhttp%253A%252F%252Fwww.facebook.com%252Fconnect%252Flogin_success.html%26type%3Duser_agent%26fbconnect%3D1%26perms%3Dread_friendlists%252Cfriends_about_me%26from_login%3D1&rcount=1} didFinishLoadForFrame: {https://www.facebook.com/login.php?api_key=112800368794148&skip_api_login=1&display=popup&cancel_url=http%3A%2F%2Fwww.facebook.com%2Fconnect%2Flogin_success.html%3Ferror_reason%3Duser_denied%26error%3Daccess_denied%26error_description%3DThe%2Buser%2Bdenied%2Byour%2Brequest.&fbconnect=1&next=http%3A%2F%2Fwww.facebook.com%2Fconnect%2Fuiserver.php%3Fmethod%3Dpermissions.request%26app_id%3D112800368794148%26display%3Dpopup%26redirect_uri%3Dhttp%253A%252F%252Fwww.facebook.com%252Fconnect%252Flogin_success.html%26type%3Duser_agent%26fbconnect%3D1%26perms%3Dread_friendlists%252Cfriends_about_me%26from_login%3D1&rcount=1}

philippec commented 13 years ago

Should be fixed in latest commit. Please try!

AndrewSchenk commented 13 years ago

That seemed to fix it. It now pops up a login dialog. Thanks!

philippec commented 13 years ago

Awesome! If you can test it further, or if you run into other problems, let me know :-)