From 21c5b6fe026119ec4a2dd79059e27118500223ef Mon Sep 17 00:00:00 2001 From: Shannon Date: Wed, 8 Jan 2014 18:20:18 +1100 Subject: [PATCH] fixes user lookup on legacy provider --- .../UsersMembershipProvider.cs | 20 +++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/src/umbraco.providers/UsersMembershipProvider.cs b/src/umbraco.providers/UsersMembershipProvider.cs index 8a8fd1e212..f5e9acf961 100644 --- a/src/umbraco.providers/UsersMembershipProvider.cs +++ b/src/umbraco.providers/UsersMembershipProvider.cs @@ -334,12 +334,24 @@ namespace umbraco.providers public override MembershipUser GetUser(string username, bool userIsOnline) { var userId = User.getUserId(username); + if (userId == -1) + { + return null; + } - var user = new User(userId); - //We need to log this since it's the only way we can determine the number of users online - Log.Add(LogTypes.Login, user, -1, "User " + username + " has logged in"); + try + { + var user = new User(userId); - return (userId != -1) ? ConvertToMembershipUser(user) : null; + //We need to log this since it's the only way we can determine the number of users online + Log.Add(LogTypes.Login, user, -1, "User " + username + " has logged in"); + + return (userId != -1) ? ConvertToMembershipUser(user) : null; + } + catch (Exception) + { + return null; + } } ///