The request: selectors in the Facebook class now return a pointer to the instantiated FBRequest object (rather than void). This makes it easier to know in the delegate callback which request the callback maps to.
Updated the dialog() method to always include app_id. This will prevent an error if a dialog is requested and the application doesn't have an access token. App_id must now be specified in the Facebook constructor as this saves specifying it with every request. Xcode project has been updated to build with iOS SDK 4.3.
Summary: see https://github.com/facebook/facebook-ios-sdk/issues#issue/161 Test Plan: verified it was called only once DiffCamp Revision: 186903 Reviewed By: brent CC: brent Revert Plan: OK
Summary: Currently if the facebook app encounters a UIWebViewDelegate error, it passes it to the sdk in the error_code parameter. The SDK should know that if this parameter is passed, an error occurred, which implies the user *didn't* cancel. This fixes it. Test Plan: disconnected my laptop from the internet, went through the sso flow, verified the sdk set userDidCancel to NO before calling [self fbDialogNotLogin:userDidCancel]. DiffCamp Revision: 180310 Reviewed By: brent CC: brent Revert Plan: OK
Summary: Edits to the documentation for clarity and grammar. This brings documentation more in line with Android SDK for consistency. Reviewers: yariv CC: arunv, brent, lshepard Test Plan: All comments--no impact on code. DiffCamp Revision: 179248
Summary: It wasn't particularly useful, so it gets the rm -rf. Test Plan: Nothing depended on this, so we're good. DiffCamp Revision: 179177 Reviewed By: yariv CC: arunv, lshepard, brent, rmcelroy, arice, achao, yariv, bootcamp-diffs@lists Tasks: #431037: Go start-to-finish through iOS SDK and validate Revert Plan: OK
Summary: n/a Task ID: # Blame Rev: Reviewers: brent, jbrusstar, lshepard, arunv CC: Test Plan: verified login via SSO worked Revert Plan: Tags: - begin *PUBLIC* platform impact section - Bugzilla: # - end platform impact - DiffCamp Revision: 179100
Summary: This diff implements single sign-on for iOS. Test Plan: n/a DiffCamp Revision: 179082 Reviewed By: brent CC: brent, arice, yariv Revert Plan: OK
… the app. Summary: The app before showed all your friends regardless of if they were using this app. With this diff, only friends who are or have used the app will be shown in the list. Test Plan: Run the app with an app id which only some of your friends are using and only those friends should show in the friends list. DiffCamp Revision: 169803 Reviewed By: brent CC: brent Revert Plan: OK
summary: Fix some memory leak and dealloc issues The Facebook.m -logout: method calls this: [self requestWithMethodName:@"auth.expireSession" andParams:params andHttpMethod:@"GET" andDelegate:self]; By setting the Facebook object as the delegate, it gets retained by the FBRequest object, but in turn the -openUrl:params:httpMethod:delegate: method retains the request itself in the _request instance variable. This creates a retain cycle, and neither object will be freed. -in Session.m, there is a missing retain on the uid in -restore -in Session.m there is no dealloc method — one should be added to free facebook and uid -in UserInfo.m, the dealloc needs to release the userInfoDelegate that was retained -in UserRequestResult.m there is no dealloc — one should be added to free userRequestDelegate test plan: theRunAroundDemoApp, open the app login and turn off the app without log out. bring the app up again. make sure we are logged in and all the session info bring back correctly. Run the unittest
…ser cancel and address several other issue