From e578b2b8f74589a7a9c48a3a37adb879efc2cab7 Mon Sep 17 00:00:00 2001
From: Omar Roth <omarroth@hotmail.com>
Date: Thu, 26 Jul 2018 10:37:16 -0500
Subject: [PATCH] Change default account type to Invidious accounts

---
 src/invidious.cr              |  4 ++--
 src/invidious/views/login.ecr | 40 +++++++++++++++++------------------
 2 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/src/invidious.cr b/src/invidious.cr
index b16af9a4..19bdab78 100644
--- a/src/invidious.cr
+++ b/src/invidious.cr
@@ -1027,7 +1027,7 @@ get "/login" do |env|
   referer ||= "/feed/subscriptions"
 
   account_type = env.params.query["type"]?
-  account_type ||= "google"
+  account_type ||= "invidious"
 
   if account_type == "invidious"
     captcha = generate_captcha(HMAC_KEY)
@@ -1144,7 +1144,7 @@ post "/login" do |env|
           end
 
           if !tfa_code
-            next env.redirect "/login?tfa=true"
+            next env.redirect "/login?tfa=true&type=google"
           end
 
           tl = challenge_results[1][2]
diff --git a/src/invidious/views/login.ecr b/src/invidious/views/login.ecr
index 00420eb8..3f19ba53 100644
--- a/src/invidious/views/login.ecr
+++ b/src/invidious/views/login.ecr
@@ -8,31 +8,14 @@
         <div class="h-box">
             <div class="pure-g">
                 <div class="pure-u-1-2">
-                    <a class="pure-button <% if account_type == "google" %>pure-button-disabled<% end %>" href="/login">Login to Google</a>
+                    <a class="pure-button <% if account_type == "invidious" %>pure-button-disabled<% end %>" href="/login">Login/Register</a>
                 </div>
                 <div class="pure-u-1-2">
-                    <a class="pure-button <% if account_type == "invidious" %>pure-button-disabled<% end %>" href="/login?type=invidious">Login/Register</a>
+                    <a class="pure-button <% if account_type == "google" %>pure-button-disabled<% end %>" href="/login?type=google">Login to Google</a>
                 </div>
             </div>
             <hr>
-            <% if account_type == "google" %>
-            <form class="pure-form pure-form-stacked" action="/login?referer=<%= referer %>" method="post">
-                <fieldset>
-                    <label for="email">Email:</label>
-                    <input required class="pure-input-1" name="email" type="email" placeholder="Email">
-
-                    <label for="password">Password:</label>
-                    <input required class="pure-input-1" name="password" type="password" placeholder="Password">
-                    
-                    <% if tfa %>
-                    <label for="tfa">Google verification code:</label>
-                    <input required class="pure-input-1" name="tfa" type="text" placeholder="Google verification code">
-                    <% end %>
-
-                    <button type="submit" class="pure-button pure-button-primary">Sign in</button>
-                </fieldset>
-            </form>
-            <% elsif account_type == "invidious" %>
+            <% if account_type == "invidious" %>
             <form class="pure-form pure-form-stacked" action="/login?referer=<%= referer %>&type=invidious" method="post">
                 <fieldset>
                     <label for="email">User ID:</label>
@@ -50,6 +33,23 @@
                     <button type="submit" name="action" value="register" class="pure-button pure-button-primary">Register</button>
                 </fieldset>
             </form>
+            <% elsif account_type == "google" %>
+            <form class="pure-form pure-form-stacked" action="/login?referer=<%= referer %>" method="post">
+                <fieldset>
+                    <label for="email">Email:</label>
+                    <input required class="pure-input-1" name="email" type="email" placeholder="Email">
+
+                    <label for="password">Password:</label>
+                    <input required class="pure-input-1" name="password" type="password" placeholder="Password">
+                    
+                    <% if tfa %>
+                    <label for="tfa">Google verification code:</label>
+                    <input required class="pure-input-1" name="tfa" type="text" placeholder="Google verification code">
+                    <% end %>
+
+                    <button type="submit" class="pure-button pure-button-primary">Sign in</button>
+                </fieldset>
+            </form>
             <% end %>
         </div>
     </div>