Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Setting Error code in FacebookAPIException #27

Closed
wants to merge 1 commit into from

2 participants

@jsnuggle

Right now the error code is not being set properly on this object when the response returns as OAuth 2.0 Draft 00 style. Adding a line where, if the error code is set, it's passed properly to the Exception class on instantiation. If not it falls back to the previous default.

@lacker

Sorry for not getting to this before now. But, it does seem like this task got completed along the way. Thanks for your effort here - we will try to be more prompt in the future.

@lacker lacker closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jun 19, 2012
  1. @jsnuggle
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 0 deletions.
  1. +1 −0  src/base_facebook.php
View
1  src/base_facebook.php
@@ -50,6 +50,7 @@ public function __construct($result) {
} else if (isset($result['error']) && is_array($result['error'])) {
// OAuth 2.0 Draft 00 style
$msg = $result['error']['message'];
+ $code = isset($result['error']['code']) ? $result['error']['code'] : $code;
} else if (isset($result['error_msg'])) {
// Rest server style
$msg = $result['error_msg'];
Something went wrong with that request. Please try again.