From 0450cdd5505b07c6ebf7ad6eece7e4e129aafbc7 Mon Sep 17 00:00:00 2001
From: Shannon <sdeminick@gmail.com>
Date: Tue, 12 Jul 2016 13:36:08 +0200
Subject: [PATCH] removes some old  notes

---
 src/Umbraco.Core/Security/AuthenticationExtensions.cs | 6 ++----
 src/Umbraco.Core/Security/BackOfficeSignInManager.cs  | 2 +-
 src/Umbraco.Core/Security/BackOfficeUserManager.cs    | 4 ++--
 3 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/Umbraco.Core/Security/AuthenticationExtensions.cs b/src/Umbraco.Core/Security/AuthenticationExtensions.cs
index 524663c9c5..9007c45946 100644
--- a/src/Umbraco.Core/Security/AuthenticationExtensions.cs
+++ b/src/Umbraco.Core/Security/AuthenticationExtensions.cs
@@ -307,11 +307,9 @@ namespace Umbraco.Core.Security
             }
             catch (Exception)
             {
-                //TODO: Do we need to do more here?? need to make sure that the forms cookie is gone, but is that
-                // taken care of in our custom middleware somehow?
                 ctx.Authentication.SignOut(
-                    Core.Constants.Security.BackOfficeAuthenticationType,
-                    Core.Constants.Security.BackOfficeExternalAuthenticationType);
+                    Constants.Security.BackOfficeAuthenticationType,
+                    Constants.Security.BackOfficeExternalAuthenticationType);
                 return null;
             }
         }
diff --git a/src/Umbraco.Core/Security/BackOfficeSignInManager.cs b/src/Umbraco.Core/Security/BackOfficeSignInManager.cs
index f1d18b9d0f..1707813433 100644
--- a/src/Umbraco.Core/Security/BackOfficeSignInManager.cs
+++ b/src/Umbraco.Core/Security/BackOfficeSignInManager.cs
@@ -46,7 +46,7 @@ namespace Umbraco.Core.Security
         /// </summary>
         /// <param name="userName"/><param name="password"/><param name="isPersistent"/><param name="shouldLockout"/>
         /// <returns/>
-        public async override Task<SignInStatus> PasswordSignInAsync(string userName, string password, bool isPersistent, bool shouldLockout)
+        public override async Task<SignInStatus> PasswordSignInAsync(string userName, string password, bool isPersistent, bool shouldLockout)
         {
             var result = await base.PasswordSignInAsync(userName, password, isPersistent, shouldLockout);
 
diff --git a/src/Umbraco.Core/Security/BackOfficeUserManager.cs b/src/Umbraco.Core/Security/BackOfficeUserManager.cs
index e48079c10b..d9c96e5967 100644
--- a/src/Umbraco.Core/Security/BackOfficeUserManager.cs
+++ b/src/Umbraco.Core/Security/BackOfficeUserManager.cs
@@ -119,7 +119,7 @@ namespace Umbraco.Core.Security
             manager.ClaimsIdentityFactory = new BackOfficeClaimsIdentityFactory();
 
             manager.EmailService = new EmailService();
-
+            
             //NOTE: Not implementing these, if people need custom 2 factor auth, they'll need to implement their own UserStore to suport it
 
             //// Register two factor authentication providers. This application uses Phone and Emails as a step of receiving a code for verifying the user
@@ -200,7 +200,7 @@ namespace Umbraco.Core.Security
         /// We've allowed this check to be overridden with a simple callback so that developers don't actually
         /// have to implement/override this class.
         /// </remarks>
-        public async override Task<bool> CheckPasswordAsync(T user, string password)
+        public override async Task<bool> CheckPasswordAsync(T user, string password)
         {
             if (BackOfficeUserPasswordChecker != null)
             {