From 35f5ea03f8cf558e07dce6f9ca507a1271978220 Mon Sep 17 00:00:00 2001
From: Fabien Meghazi <fme@openerp.com>
Date: Wed, 26 Sep 2012 11:43:00 +0200
Subject: [PATCH] [FIX] oauth provider: do not return email key if user has no
 email

bzr revid: fme@openerp.com-20120926094300-09qcpak6qg5eokeh
---
 addons/auth_oauth_provider/res_users.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/addons/auth_oauth_provider/res_users.py b/addons/auth_oauth_provider/res_users.py
index 20d9e9c7ade..833286b5642 100644
--- a/addons/auth_oauth_provider/res_users.py
+++ b/addons/auth_oauth_provider/res_users.py
@@ -46,14 +46,15 @@ class res_users(osv.osv):
             }
         user = self.browse(cr, uid, user_id[0], context=context)
         if access_token == user.last_oauth_token:
-            return {
+            r  = {
                 "user_id": user.id,
                 "scope": user.last_oauth_token_scope,
-                "email": user.partner_id.email or '', # TODO: should deliver only according to scopes
-                "scope": user.last_oauth_token_scope,
                 #"audience": "8819981768.apps.googleusercontent.com",
                 #"expires_in": 436
             }
+            if user.partner_id.email: # TODO: should deliver only according to scopes
+                r['email'] = user.partner_id.email
+            return r
         else:
             return {
                 "error": "invalid_token"