MT-17 cleanup

KVA/Migration.Toolkit.Source/Mappers/ContentItemMapper.cs

@@ -817,7 +817,6 @@ ICmsClass nodeClass
                 {
                     if (value is string pageReferenceJson)
                     {
-#warning [PATCHED] - [VERIFY] "NodeGuid may not be unique, use other means of searching for node!"
                         var parsed = JObject.Parse(pageReferenceJson);
                         foreach (var jToken in parsed.DescendantsAndSelf())
                         {

KVA/Migration.Toolkit.Source/Mappers/PageTemplateConfigurationMapper.cs

@@ -194,7 +194,6 @@ private void WalkProperties(int siteId, JObject properties, List<EditingFormCont
                         {
                             if (value?.ToObject<List<Migration.Toolkit.Source.Services.Model.PageSelectorItem>>() is { Count: > 0 } items)
                             {
-#warning [PATCHED] - [CHECK] "NodeGuid may not be unique, use other means of searching for node!"
                                 properties[key] = JToken.FromObject(items.Select(x => new WebPageRelatedItem
                                 {
                                     WebPageGuid = spoiledGuidContext.EnsureNodeGuid(x.NodeGuid, siteId)

KVA/Migration.Toolkit.Source/Services/PrimaryKeyLocatorService.cs

@@ -82,27 +82,6 @@ public IEnumerable<SourceTargetKeyMapping> SelectAll<T>(Expression<Func<T, objec
             yield break;
         }
 
-//         if (sourceType == typeof(ICmsTree) && memberName == nameof(ICmsTree.NodeID))
-//         {
-// #error "NodeGuid may not be unique, use other means of searching for node!"
-//             var source = modelFacade.SelectAll<ICmsTree>().Select(x => new { x.NodeID, x.NodeGUID, x.NodeSiteID }).ToList();
-//             var target = kxpContext.CmsChannels.Select(x => new { x.ChannelId, x.ChannelGuid }).ToList();
-//
-//             var result = source.Join(target,
-// #error "NodeGuid may not be unique, use other means of searching for node!"
-//                 a => a.NodeGUID,
-//                 b => b.ChannelGuid,
-//                 (a, b) => new SourceTargetKeyMapping(a.NodeID, b.ChannelId)
-//             );
-//
-//             foreach (var resultingMapping in result)
-//             {
-//                 yield return resultingMapping;
-//             }
-//
-//             yield break;
-//         }
-
         if (sourceType == typeof(ICmsState) && memberName == nameof(ICmsState.StateID))
         {
             var source = modelFacade.SelectAll<ICmsState>().Select(x => new { x.StateID, x.StateName }).ToList();
@@ -216,14 +195,6 @@ public bool TryLocate<T>(Expression<Func<T, object>> keyNameSelector, int source
                 targetId = kxpContext.OmContacts.Where(x => x.ContactGuid == sourceGuid).Select(x => x.ContactId).Single();
                 return true;
             }
-
-//             if (sourceType == typeof(ICmsTree))
-//             {
-// #error "NodeGuid may not be unique, use other means of searching for node!"
-//                 var sourceGuid = modelFacade.SelectById<ICmsTree>(sourceId)?.NodeGUID;
-//                 targetId = kxpContext.CmsChannels.Where(x => x.ChannelGuid == sourceGuid).Select(x => x.ChannelId).Single();
-//                 return true;
-//             }
         }
         catch (InvalidOperationException ioex)
         {