From 0749124f4571bd89767f54d4620f534be57ebd71 Mon Sep 17 00:00:00 2001 From: Shannon Date: Tue, 18 Mar 2014 20:11:22 +1100 Subject: [PATCH] fixes bug with col name --- .../Persistence/Repositories/MemberRepository.cs | 5 ++++- src/umbraco.cms/businesslogic/member/Member.cs | 3 +-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs b/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs index c999d84243..4a206d1f61 100644 --- a/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs +++ b/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs @@ -311,7 +311,10 @@ namespace Umbraco.Core.Persistence.Repositories Database.Update(dto.ContentVersionDto); //Updates the cmsMember entry if it has changed + + //NOTE: these cols are the REAL column names in the db var changedCols = new List(); + if (dirtyEntity.IsPropertyDirty("Email")) { changedCols.Add("Email"); @@ -323,7 +326,7 @@ namespace Umbraco.Core.Persistence.Repositories // DO NOT update the password if it is null or empty if (dirtyEntity.IsPropertyDirty("RawPasswordValue") && entity.RawPasswordValue.IsNullOrWhiteSpace() == false) { - changedCols.Add("RawPasswordValue"); + changedCols.Add("Password"); } //only update the changed cols if (changedCols.Count > 0) diff --git a/src/umbraco.cms/businesslogic/member/Member.cs b/src/umbraco.cms/businesslogic/member/Member.cs index c2b19b9d8b..10419b8922 100644 --- a/src/umbraco.cms/businesslogic/member/Member.cs +++ b/src/umbraco.cms/businesslogic/member/Member.cs @@ -233,8 +233,7 @@ namespace umbraco.cms.businesslogic.member throw new Exception(string.Format("Duplicate User name! A member with the user name {0} already exists", loginName)); var model = ApplicationContext.Current.Services.MemberService.CreateMemberWithIdentity( - loginName, Email.ToLower(), mbt.MemberTypeItem); - model.Name = Name; + loginName, Email.ToLower(), Name, mbt.MemberTypeItem); //The content object will only have the 'WasCancelled' flag set to 'True' if the 'Saving' event has been cancelled, so we return null. if (((Entity)model).WasCancelled)