yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fixes logic in membershiphelper updatemember

Browse Source
This commit is contained in:
Shannon
2014-02-20 13:55:56 +11:00
parent a206652a0d
commit 606b5aa60a
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/Umbraco.Web/Security/MembershipHelper.cs
View File

@@ -576,37 +576,37 @@ namespace Umbraco.Web.Security
DateTime? lastActivityDate = null,
string comment = null)
{
var needsUpdating = false;
var needsUpdating = new List<bool>();
//set the writable properties
if (email != null)
{
needsUpdating = member.Email != email;
needsUpdating.Add(member.Email != email);
member.Email = email;
}
if (isApproved.HasValue)
{
needsUpdating = member.IsApproved != isApproved.Value;
needsUpdating.Add(member.IsApproved != isApproved.Value);
member.IsApproved = isApproved.Value;
}
if (lastLoginDate.HasValue)
{
needsUpdating = member.LastLoginDate != lastLoginDate.Value;
needsUpdating.Add(member.LastLoginDate != lastLoginDate.Value);
member.LastLoginDate = lastLoginDate.Value;
}
if (lastActivityDate.HasValue)
{
needsUpdating = member.LastActivityDate != lastActivityDate.Value;
needsUpdating.Add(member.LastActivityDate != lastActivityDate.Value);
member.LastActivityDate = lastActivityDate.Value;
}
if (comment != null)
{
needsUpdating = member.Comment != comment;
needsUpdating.Add(member.Comment != comment);
member.Comment = comment;
}
//Don't persist anything if nothing has changed
if (needsUpdating)
if (needsUpdating.Any(x => x == true))
{
provider.UpdateUser(member);
return Attempt<MembershipUser>.Succeed(member);