webtechnick / CakePHP-Facebook-Plugin

CakePHP Facebook Plugin
http://facebook.webtechnick.com
445 stars 138 forks source link

remove try-catch from FB proxy #85

Open guitcastro opened 12 years ago

guitcastro commented 12 years ago

I think you should remove all try-catch from FB.php and FacebookApi.php, and give the developers the possibilite to handle exceptions.It's more easily to debug erros in graph-api or avoid black-holes. For example if you have an error you will get from exception :

                [message] => (#352) Video file format is not supported
                [type] => OAuthException
                [code] => 352

using exception it's easy to handle this , and print an error msg for the final user.

without exception i only have a boolean "false" =/

It's just a suggestion =)