From 065c34d609667e11b105002d5221fb2c4ddd0d14 Mon Sep 17 00:00:00 2001
From: Xavier Guimard <xavier.guimard@laposte.net>
Date: Sat, 5 Oct 2013 12:59:55 +0000
Subject: [PATCH] Set user to Facebook username

---
 .../lib/Lemonldap/NG/Portal/AuthFacebook.pm            | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/AuthFacebook.pm b/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/AuthFacebook.pm
index 69bdf472f..f6961bde2 100644
--- a/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/AuthFacebook.pm
+++ b/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/AuthFacebook.pm
@@ -97,7 +97,12 @@ sub extractFormInfo {
             $self->{sessionInfo}->{_facebookToken} = $access_token;
 
             # TODO
-            $self->{user} = $access_token;
+            my $datas = $self->fb->get('https://graph.facebook.com/me',{fields=>'id,username'})->as_hash;
+            $self->{user} = $datas->{username} || $access_token;
+            $self->lmLog( 'Good Facebook authentication', 'debug' );
+
+            # Force redirection to avoid displaying Oauth datas
+            $self->{mustRedirect} = 1;
             return PE_OK;
         }
         return PE_BADCREDENTIALS;
@@ -105,7 +110,8 @@ sub extractFormInfo {
 
     # 2. Else redirect user to Facebook login page:
 
-    # Build Facbook redirection
+    # Build Facebook redirection
+    # TODO: use a param to use "publish_stream" or not
     my $check_url = $self->fb()->get_authorization_url(
         scope   => [ 'offline_access', 'publish_stream' ],
         display => 'page',