More cleaning, better UX

mfa/recovery.py

@@ -2,7 +2,7 @@ from django.shortcuts import render
 from django.views.decorators.cache import never_cache
 from django.template.context_processors import csrf
 from django.contrib.auth.hashers import make_password, PBKDF2PasswordHasher
-from django.http import HttpResponse,HttpResponseNotFound
+from django.http import HttpResponse
 from .Common import get_redirect_url
 from .models import *
 import simplejson
@@ -24,7 +24,6 @@ def delTokens(request):
 def randomGen(n):
     return ''.join(random.choice(string.ascii_uppercase + string.ascii_lowercase + string.digits) for _ in range(n))
 
-
 @never_cache
 def genTokens(request):
     #Delete old ones
@@ -58,9 +57,6 @@ def verify_login(request, username, token):
                 key.properties["secret_keys"] = secret_keys
                 key.save()
                 return [True, key.id, len(secret_keys) == 0]
-        if len(secret_keys) == 0:
-            #Show a message ?
-            return [False]
     return [False]
 
 def getTokenLeft(request):

mfa/templates/RECOVERY/Add.html

@@ -15,14 +15,6 @@
     .row{
         margin: 3px;
     }
-    .crossed{
-        text-decoration: line-through;
-    }
-    .tooltip {
-  position: relative;
-  display: inline-block;
-    }
-
     .toolbtn {
         border-radius: 7px;
     }

mfa/templates/RECOVERY/recheck.html

@@ -38,7 +38,7 @@
       {% csrf_token %}
           {%  if invalid %}
           <div class="alert alert-danger">
-              Sorry, The provided code is not valid.
+              Sorry, The provided code is not valid, or has already been used.
           </div>
           {% endif %}
       {% if quota %}