From 171dbd4b099558fd5c9e688cba261e65e4db9b03 Mon Sep 17 00:00:00 2001
From: Robin Ward <robin.ward@gmail.com>
Date: Thu, 9 Jun 2016 15:51:46 -0400
Subject: [PATCH] Allow redirects on Facebook Browser

---
 .../users/omniauth_callbacks/complete.html.erb      | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/app/views/users/omniauth_callbacks/complete.html.erb b/app/views/users/omniauth_callbacks/complete.html.erb
index 3d4f96cea..07bd70146 100644
--- a/app/views/users/omniauth_callbacks/complete.html.erb
+++ b/app/views/users/omniauth_callbacks/complete.html.erb
@@ -22,17 +22,14 @@
     <p><%=t "login.close_window" %></p>
 
     <script type="text/javascript">
-       if (window.opener) {
-         window.opener.Discourse.authenticationComplete(<%=@auth_result.to_client_hash.to_json.html_safe%>);
-       }
-
        // On facebook browser, just redirect and don't close
        var ua = navigator.userAgent || navigator.vendor || window.opera;
-       // if ((ua.indexOf("FBAN") > -1) || (ua.indexOf("FBAV") > -1)) {
-       //   window.location.href = '<%= Discourse.base_url.html_safe %>';
-       // } else {
+       if ((ua.indexOf("FBAN") > -1) || (ua.indexOf("FBAV") > -1)) {
+         window.location.href = '<%= Discourse.base_url.html_safe %>';
+       } else {
+         window.opener.Discourse.authenticationComplete(<%=@auth_result.to_client_hash.to_json.html_safe%>);
          window.close();
-       // }
+       }
     </script>
   </div>
 </body>