kennylerma / facebook-actionscript-api

Automatically exported from code.google.com/p/facebook-actionscript-api
0 stars 0 forks source link

Mobile login broken #304

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. run the mobile connect example
2. on success a blank page with success remains
3. on error a a blank page with success remains
4. if app id is wrong a blank page with a json error remains

What is the expected output? What do you see instead?
The login success or error handlers should be called

What version of the product are you using? On what operating system?
Using 1.6 mobile swc Flex 4.5 Android

Please provide any additional information below.
I debugged this and the error lies in line 134 and 142 of MobileLoginWindow. 
These check for success urls being the index of 0 but the new urls append 
http://www.facebook.com/connect/uiserver.php? to the success response. this 
indexOf code needs to be updated. Since the login redirect urls could possible 
have the success url fragment and/or the error fragment there is no simple 
indefOf operation that will solve this. You will need to read the query params 
and check for the existence of both the success fragment, the error description 
and the error. If this was in github I would just give you a pull request ;)

Furthermore if you intentionally use an invalid app id no paths of the current 
code are able to catch it. The page redirects to a json error and no location 
change event is fired by the StageWebView. In this case I had to actually set a 
timer to check on the web view.

Original issue reported on code.google.com by kristofe...@gmail.com on 15 May 2011 at 9:51

GoogleCodeExporter commented 9 years ago
A fix for this is made in v1.6.1.

Original comment by rovertn...@gmail.com on 25 May 2011 at 3:19