From 338e92417f7fed4dec163090b47b71cde05f911b Mon Sep 17 00:00:00 2001 From: Cameron Ball Date: Wed, 14 Jan 2015 09:47:01 +0800 Subject: [PATCH] Facebook stuf --- Controllers/UserAuthController.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Controllers/UserAuthController.php b/Controllers/UserAuthController.php index ec9e8d8..af257b1 100644 --- a/Controllers/UserAuthController.php +++ b/Controllers/UserAuthController.php @@ -7,13 +7,11 @@ use Services\Http\IHttpResponse; use Services\Http\IHttpRequest; use Services\IFacebookSessionFactory; use Domain\Entities\IUserStepByStepBuilder; -use Domain\Entities\IUser; use DataAccess\IUserRepository; use Facebook\FacebookSession; use Facebook\FacebookRequest; use Facebook\GraphUser; use Facebook\GraphLocation; -use Facebook\FacebookRequestException; class UserAuthController implements IDivineController { @@ -46,6 +44,9 @@ class UserAuthController implements IDivineController $token = $this->validateAuthRequest(); $facebookSession = $this->_facebookSessionFactory->createInstance($token); + //If it fails to validate the exception will deal with it. + $facebookSession->validate(); + $this->_facebookSession = $this->isSessionLongLived($facebookSession) ? $facebookSession->getLongLivedSession() : $facebookSession; $this->_facebookRequest = (new FacebookRequest($this->_facebookSession, 'GET', '/me?fields=hometown,first_name,last_name'))->execute(); @@ -66,6 +67,8 @@ class UserAuthController implements IDivineController if(!isset($request['token'])) { + //TODO: Perhaps I should be using the status reporter class here (and above). + //Will require changes to front end though, but at least it would be more consistent. $response->setBody(json_encode(array('result' => 'error', 'message' => 'missing auth token'))) ->sendResponse(); die(); -- 2.11.0