From a74feab9eb19f2745d3c46f82e35339596ac77db Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Thu, 1 Sep 2016 15:15:47 +0200 Subject: [PATCH] U4-8923 Adding new property to media breaks ability to save If you add a new property to a member or media type and then try to update an existing media item/member ContentRepository didn't have this problem because it was already using the additional if statement that has now also been added to MemberRepository and MediaRepository --- src/Umbraco.Core/Persistence/Repositories/MediaRepository.cs | 2 ++ src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs | 2 ++ 2 files changed, 4 insertions(+) diff --git a/src/Umbraco.Core/Persistence/Repositories/MediaRepository.cs b/src/Umbraco.Core/Persistence/Repositories/MediaRepository.cs index 97981849bc..8c9bec71d4 100644 --- a/src/Umbraco.Core/Persistence/Repositories/MediaRepository.cs +++ b/src/Umbraco.Core/Persistence/Repositories/MediaRepository.cs @@ -413,6 +413,8 @@ namespace Umbraco.Core.Persistence.Repositories { foreach (var property in entity.Properties) { + if (keyDictionary.ContainsKey(property.PropertyTypeId) == false) continue; + property.Id = keyDictionary[property.PropertyTypeId]; } } diff --git a/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs b/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs index f14d66691b..0f0e797f17 100644 --- a/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs +++ b/src/Umbraco.Core/Persistence/Repositories/MemberRepository.cs @@ -365,6 +365,8 @@ namespace Umbraco.Core.Persistence.Repositories { foreach (var property in ((Member)entity).Properties) { + if (keyDictionary.ContainsKey(property.PropertyTypeId) == false) continue; + property.Id = keyDictionary[property.PropertyTypeId]; } }