From 4602a6b989bc511014f743217f822a9869b1da26 Mon Sep 17 00:00:00 2001 From: Shannon Date: Tue, 1 Oct 2013 14:46:16 +1000 Subject: [PATCH] Fixes an issue with the MemberTypeReadOnlyFactory --- .../Persistence/Factories/MemberTypeReadOnlyFactory.cs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/Umbraco.Core/Persistence/Factories/MemberTypeReadOnlyFactory.cs b/src/Umbraco.Core/Persistence/Factories/MemberTypeReadOnlyFactory.cs index f7b314cbff..3462cef894 100644 --- a/src/Umbraco.Core/Persistence/Factories/MemberTypeReadOnlyFactory.cs +++ b/src/Umbraco.Core/Persistence/Factories/MemberTypeReadOnlyFactory.cs @@ -59,8 +59,11 @@ namespace Umbraco.Core.Persistence.Factories foreach (var groupDto in dto.PropertyTypeGroups.Where(x => x.Id.HasValue)) { var group = new PropertyGroup(); + //Only assign an Id if the PropertyGroup belongs to this ContentType - if (groupDto.Id.HasValue && groupDto.Id == memberType.Id) + if (groupDto.Id.HasValue + /*SD: I've commented this out since it's never correct, why would the group id be the same as the type?? + * && groupDto.Id == memberType.Id*/) { group.Id = groupDto.Id.Value;