Navigation Properties updated in incoming entity during UpdateGraph about graphdiff HOT 3 CLOSED

Hello @fdahlberg ,

Thank you for reporting,

We will investigate this issue very soon and give you an answer.

Best Regards,

Jonathan

Help us to keep this library free: Donate

from graphdiff.

Hello @fdahlberg ,

We have a lot investigated this issue to better understand why this is happening.

We find out that the issue is caused by this method: https://github.com/zzzprojects/GraphDiff/blob/master/GraphDiff/GraphDiff/Internal/Graph/GraphNode.cs#L106-L122

        protected static void AttachCyclicNavigationProperty(IObjectContextAdapter context, object parent, object child)
        {
            if (parent == null || child == null) return;


            var parentType = ObjectContext.GetObjectType(parent.GetType());
            var childType = ObjectContext.GetObjectType(child.GetType());


            var navigationProperties = context.GetNavigationPropertiesForType(childType);


            var parentNavigationProperty = navigationProperties
                    .Where(navigation => navigation.TypeUsage.EdmType.Name == parentType.Name)
                    .Select(navigation => childType.GetProperty(navigation.Name, BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public))
                    .FirstOrDefault();


            if (parentNavigationProperty != null)
                parentNavigationProperty.SetValue(child, parent, null);
        }

The child is an object of the original entity which gets assigned the parent of the new entity.

We tested some scenario which could make existing child entities work but unfortunately for new entities, there is nothing we have found that we can do for now.

Removing this code will lead to more issue since some navigation property will not be correctly copied.

We can leave this issue open but I don't think we will do something until we choose to re-write this library.

Let me know if you need a better explanation of the issue.

Best Regards,

Jonathan

Help us to keep this library free: Donate

from graphdiff.

Hi Jonathan,

Thanks for investigating. It's a shame it can't be fixed but I can live with this. As a workaround I create a clone of the incoming object before using UpdateGraph.

Thanks
Fred

from graphdiff.