Problem
\n
If a user clicks \"Don't Allow\" when an app asks for additional\ninformation, evt.success and hasPermission will still return \"true\"\nfor this permission, when they should return false. This false\npositive only occurs in iOS, in my testing.
\n
Sample Code Demonstrating Problem
\n
The following sample code demonstrates the problem.
\n
Use the following steps to reproduce in sample code:
\n1. Click \"Connect with Facebook\"
\n2. Sign in
\n3. Click \"Don't Allow\"
\n
\n/**\n* Function that logs the user into Facebook, and makes sure they have\n* granted us the \"user_checkins\" permission.\n*/\nfunction checkIfUserIsLoggedIntoFacebook() {\n // if user hasn't logged in yet, don't do anything\n // (they need to click the \"Connect with Facebook\" button)\n if (Titanium.Facebook.isLoggedIn() === false) {\n }\n // if the user hasn't given us the permissions we want, request them.\n else if (!Titanium.Facebook.hasPermission(\"user_checkins\")) {\n Titanium.Facebook.requestPermission(\"user_checkins\", function (evt) {\n if (evt.success) {\n alert('Permission GRANTED');\n } else {\n alert(\"Permission DENIED\");\n }\n });\n }\n // otherwise, we should be good to go\n else {\n alert('Permission ALREADY GRANTED');\n }\n}\n\n/**\n* Add a login event listener so we can check if we have all the permissions\n* we want.\n*/\nTitanium.Facebook.addEventListener('login', function (evt) {\n if (evt.success) {\n checkIfUserIsLoggedIntoFacebook();\n }\n});\n\n// create our window\nvar win = Ti.UI.createWindow({ backgroundColor: '#fff' });\nwin.add(Ti.Facebook.createLoginButton({\n apikey: '645bf1d897d1fa8dc93ed45bc37edc04',\n secret: '49d146f85b8429438262380b85f2ebac',\n style: 'wide'\n}));\nwin.open();\n
\n
Tested On
\n
Tested on CI build 1.6.0 from January 8th, 2010.
\nBROKEN on iPhone Simulator, 4.2
\nWORKS on Android Simulator APIs 2.2
\n
Associated Helpdesk Ticket
\n
http://developer.appcelerator.com/helpdesk/view/63481
The easy fix for this (short of ripping out and replacing the\nFacebook module with new code, which is ideal) is to change one\nline in the FBConnect/FBDialog.m (line 391 in Mobile SDK\n1.5.1).
\n
The line should change FROM:
\n
\nif ([url.resourceSpecifier isEqualToString:@\"cancel\"]) {\n
\n
TO:
\n
\nif ([[url.resourceSpecifier substringToIndex:6] isEqualToString:@\"cancel\"]) {\n
\n
Attached is a patch file that accomplishes that. Again, ideally,\nthe new FBConnect module gets put in place which also includes new\npermissions and proper resposes to their new API. The current REST\nAPI is being deprecated.
Having this broken also affects\nTitanium.Facebook.requestPermission, and causes it to not pop up a\ndialog if a user previously declined it.
\n
Jesus H. Christ's patch above fixes that too, though! Thanks,\nJesus.