Merge pull request jaliss#368 from chrisbeach/patch-1

Prevent errors caused by missing email in FacebookProvider

Author: jaliss

module-code/app/securesocial/core/providers/FacebookProvider.scala

@@ -70,21 +70,21 @@ class FacebookProvider(application: Application) extends OAuth2Provider(applicat
           )
           throw new AuthenticationException()
         case _ =>
-          val userId = ( me \ Id).as[String]
-          val name = ( me \ Name).as[String]
-          val firstName = ( me \ FirstName).as[String]
-          val lastName = ( me \ LastName).as[String]
-          val picture = (me \ Picture)
+          val userId = (me \ Id).as[String]
+          val name = (me \ Name).as[String]
+          val firstName = (me \ FirstName).as[String]
+          val lastName = (me \ LastName).as[String]
+          val picture = me \ Picture
           val avatarUrl = (picture \ Data \ Url).asOpt[String]
-          val email = ( me \ Email).as[String]
+          val email = (me \ Email).asOpt[String]
 
           user.copy(
             identityId = IdentityId(userId, id),
             firstName = firstName,
             lastName = lastName,
             fullName = name,
             avatarUrl = avatarUrl,
-            email = Some(email)
+            email = email
           )
       }
     } catch {