153

When I save an entity with entity framework, I naturally assumed it would only try to save the specified entity. However, it is also trying to save that entity's child entities. This is causing all sorts of integrity problems. How do I force EF to only save the entity I want to save and therefore ignore all child objects?

If I manually set the properties to null, I get an error "The operation failed: The relationship could not be changed because one or more of the foreign-key properties is non-nullable." This is extremely counter-productive since I set the child object to null specifically so EF would leave it alone.

Why don't I want to save/insert the child objects?

Since this is being discussed back and forth in the comments, I'll give some justification of why I want my child objects left alone.

In the application I'm building, the EF object model is not being loaded from the database but used as data objects which I'm populating while parsing a flat file. In the case of the child objects, many of these refer to lookup tables defining various properties of the parent table. For example, the geographic location of the primary entity.

Since I've populated these objects myself, EF assumes these are new objects and need to be inserted along with the parent object. However, these definitions already exist and I don't want to create duplicates in the database. I only use the EF object to do a lookup and populate the foreign key in my main table entity.

Even with the child objects that are real data, I needs to save the parent first and get a primary key or EF just seems to make a mess of things. Hope this gives some explanation.

11
  • As far as I know you will have to null the child objects.
    – Johan
    Commented Aug 22, 2014 at 6:48
  • Hi Johan. Doesn't work. It throws errors if I null the collection. Depending on how I do it, it complains about keys being null or that I collection has been modified. Obviously, those things are true, but I did that on purpose so it would leave alone the objects it's not supposed to touch. Commented Aug 22, 2014 at 6:51
  • Euphoric, that is completely unhelpful. Commented Aug 22, 2014 at 6:51
  • 2
    C'mon guys, there must be some way to make EF behave sensibly. I've tried combinations of detaching child entities, cloning the main and creating new database connections, yet EF somehow hangs on to all its junk and complains. There must be some way to completely purge EF and force it to look only at what I give it. At this rate, I'm going to have to build my own set of objects and write SQL/linq to do the inserts I need, which is a huge step backwards. Commented Aug 22, 2014 at 8:39
  • 1
    @MarkyMark, this is EF behaving as EF is intended. It is behaving entirely responsibly. It sounds like you are trying to bend the framework into achieving your own unique objectives, in that situation a framework is not always the best way to go? You wouldn't use a hammer when you really needed a wrench would you. Commented Aug 22, 2014 at 9:04

15 Answers 15

80

Long story short: Use Foreign key and it will save your day.

Assume you have a School entity and a City entity, and this is a many-to-one relationship where a City has many Schools and a School belongs to a City. And assume the Cities are already existing in the lookup table so you do NOT want them to be inserted again when inserting a new school.

Initially you might define your entities like this:

public class City
{
    public int Id { get; set; }
    public string Name { get; set; }
}

public class School
{
    public int Id { get; set; }
    public string Name { get; set; }

    [Required]
    public City City { get; set; }
}

And you might do the School insertion like this (assume you already have City property assigned to the newItem):

public School Insert(School newItem)
{
    using (var context = new DatabaseContext())
    {
        context.Set<School>().Add(newItem);
        // use the following statement so that City won't be inserted
        context.Entry(newItem.City).State = EntityState.Unchanged;
        context.SaveChanges();
        return newItem;
    }
}

The above approach may work perfectly in this case; however, I do prefer the Foreign Key approach which to me is more clear and flexible. See the updated solution below:

public class City
{
    public int Id { get; set; }
    public string Name { get; set; }
}

public class School
{
    public int Id { get; set; }
    public string Name { get; set; }

    [ForeignKey("City_Id")]
    public City City { get; set; }

    [Required]
    public int City_Id { get; set; }
}

In this way, you explicitly define that the School has a foreign key City_Id and it refers to the City entity. So when it comes to the insertion of School, you can do:

    public School Insert(School newItem, int cityId)
    {
        if (cityId <= 0)
        {
            throw new Exception("City ID not provided");
        }

        newItem.City = null;
        newItem.City_Id = cityId;

        using (var context = new DatabaseContext())
        {
            context.Set<School>().Add(newItem);
            context.SaveChanges();
            return newItem;
        }
    }

In this case, you explicitly specify the City_Id of the new record and remove the City from the graph so that EF won't bother to add it to the context along with School.

Although at first glance the Foreign key approach seems more complicated, trust me this mentality will save you a lot of time when it comes to inserting a many-to-many relationship (imagine you have a School and Student relationship, and the Student has a City property) and so on.

Hope this is helpful to you.

9
  • Great answer, helped me a lot! But would it not be better to state the minimum value of 1 for City_Id using [Range(1, int.MaxValue)] attribute?
    – Dan Rayson
    Commented Nov 28, 2017 at 16:00
  • Like a dream!! Thanks a Million!
    – CJH
    Commented Dec 24, 2018 at 9:52
  • 1
    This would remove the value from the School object in the caller. Does SaveChanges reload the value of null navigation properties? Otherwise the caller should reload the City object after calling the Insert() method if it needs that info. This is the pattern I have used often, but I am still open to better patterns if anyone has a good one. Commented May 19, 2019 at 2:38
  • 1
    This was really helpful to me. EntityState.Unchaged is exactly what I needed to assign an object that represents a lookup table foreign key in a large object graph that I was saving as a single transaction. EF Core throws a less than intuitive error message IMO. I cached my lookup tables that rarely change for performance reasons. You're assumption is that, because the PK of the lookup table object is the same as what's in the database already that it knows its unchanged and to just assign the FK to the existing item. Instead of tries to insert the lookup table object as new.
    – tnk479
    Commented Sep 19, 2019 at 20:51
  • 1
    @ienglei... 4 years has passes by and your suggestion works yet... I Prefer the first solution... i'm using Entity Framework Core 6... thanks for this great advice! Commented Feb 16, 2022 at 14:03
75

As far as I know, you have two options.

Option 1)

Null all the child objects, this will ensure EF not to add anything. It will also not delete anything from your database.

Option 2)

Set the child objects as detached from the context using the following code

 context.Entry(yourObject).State = EntityState.Detached

Note that you can not detach a List/Collection. You will have to loop over your list and detach each item in your list like so

foreach (var item in properties)
{
     db.Entry(item).State = EntityState.Detached;
}
12
  • Hi Johan, I tried detatching one of the collections and it threw the following error: The entity type HashSet`1 is not part of the model for the current context. Commented Aug 22, 2014 at 6:57
  • 29
    Unfortunately, even with a list of loops to detatch everything, EF seems to be still trying to insert into some of the related tables. At this point, I'm ready to rip EF out and switch back to SQL which at least behaves sensibly. What a pain. Commented Aug 22, 2014 at 8:29
  • 2
    Can I use context.Entry(yourObject).State before even adding it?
    – Tommehh
    Commented Mar 29, 2018 at 7:01
  • 1
    @mirind4 I did not use state. If I insert an object, I make sure all childs are null. On update I get the object first without the childs.
    – Tommehh
    Commented May 14, 2018 at 14:13
  • 1
    EntityState.Detached did not work for me but EntityState.Unchanged did work.
    – crichavin
    Commented Jul 13, 2020 at 19:31
37

If you just want to store changes to a parent object and avoid storing changes to any of its child objects, then why not just do the following:

using (var ctx = new MyContext())
{
    ctx.Parents.Attach(parent);
    ctx.Entry(parent).State = EntityState.Added;  // or EntityState.Modified
    ctx.SaveChanges();
}

The first line attaches the parent object and the whole graph of its dependent child objects to the context in Unchanged state.

The second line changes the state for the parent object only, leaving its children in the Unchanged state.

Note that I use a newly created context, so this avoids saving any other changes to the database.

5
  • 4
    This answer has the advantage that if someone comes along later and adds a child object, it won't break existing code. This is an "opt in" solution where the others require you to explicitly exclude child objects.
    – Jim
    Commented Feb 27, 2019 at 22:06
  • 1
    This no longer works in core. "Attach: Attaches every reachable entity, except where a reachable entity has a store-generated key and no key value is assigned; these will be marked as added." If children are new as well they will be Added.
    – mmix
    Commented Apr 30, 2020 at 9:49
  • easy and clean. no headache! Thanks.
    – FLICKER
    Commented Apr 13, 2021 at 22:22
  • Fantastic! Does what I need when dealing with existing employee entities that are loaded from a different instance of my db context. No fuss saving the primary keys to my main table. Commented Jun 11, 2021 at 16:53
  • Thank you. Worked, It saved lot of time. Simple solution.
    – iprashant
    Commented Nov 28, 2021 at 14:32
16

One of the suggested solutions is to assign the navigation property from the same database context. In this solution, the navigation property assigned from outside the database context would be replaced. Please, see following example for illustration.

class Company{
    public int Id{get;set;}
    public Virtual Department department{get; set;}
}
class Department{
    public int Id{get; set;}
    public String Name{get; set;}
}

Saving to database:

 Company company = new Company();
 company.department = new Department(){Id = 45}; 
 //an Department object with Id = 45 exists in database.    

 using(CompanyContext db = new CompanyContext()){
      Department department = db.Departments.Find(company.department.Id);
      company.department = department;
      db.Companies.Add(company);
      db.SaveChanges();
  }

Microsoft enlists this as a feature, however I find this annoying. If the department object associated with company object has Id that already exists in database, then why doesn't EF just associates company object with database object? Why should we need to take care of the association by ourselves? Taking care of the navigation property during adding new object is something like moving the database operations from SQL to C#, cumbersome to the developers.

2
  • 4
    I agree 100% it is ridiculous that EF attempts to create a new record when the ID is provided. If there was a way to fill select list options with the actual object we'd be in business.
    – T3.0
    Commented Jul 10, 2020 at 18:09
  • 1
    I also agree and why add the additional read on the database. reads generates locks and locks slow down stuff. not an issue when debugging bet catastrophic on a busy application. Commented Aug 19, 2022 at 7:38
16

First you need to know that there are two ways for updating entity in EF.

  • Attached objects

When you change the relationship of the objects attached to the object context by using one of the methods described above, the Entity Framework needs to keep foreign keys, references, and collections in sync.

  • Disconnected objects

If you are working with disconnected objects you must manually manage the synchronization.

In the application I'm building, the EF object model is not being loaded from the database but used as data objects which I'm populating while parsing a flat file.

That means you are working with disconnected object, but it's unclear whether you are using independent association or foreign key association.

  • Add

    When adding new entity with existing child object (object that exists in the database), if the child object is not tracked by EF, the child object will be re-inserted. Unless you manually attach the child object first.

      db.Entity(entity.ChildObject).State = EntityState.Modified;
      db.Entity(entity).State = EntityState.Added;
    
  • Update

    You can just mark the entity as modified, then all scalar properties will be updated and the navigation properties will simply be ignored.

      db.Entity(entity).State = EntityState.Modified;
    

Graph Diff

If you want to simplify the code when working with disconnected object, you can give a try to graph diff library.

Here is the introduction, Introducing GraphDiff for Entity Framework Code First - Allowing automated updates of a graph of detached entities.

Sample Code

  • Insert entity if it doesn't exist, otherwise update.

      db.UpdateGraph(entity);
    
  • Insert entity if it doesn't exist, otherwise update AND insert child object if it doesn't exist, otherwise update.

      db.UpdateGraph(entity, map => map.OwnedEntity(x => x.ChildObject));
    
3

Best way to do this is in by overriding the SaveChanges function in your datacontext.

    public override int SaveChanges()
    {
        var added = this.ChangeTracker.Entries().Where(e => e.State == System.Data.EntityState.Added);

        // Do your thing, like changing the state to detached
        return base.SaveChanges();
    }
2

This worked for me:

// temporarily 'detach' the child entity/collection to have EF not attempting to handle them
var temp = entity.ChildCollection;
entity.ChildCollection = new HashSet<collectionType>();

.... do other stuff

context.SaveChanges();

entity.ChildCollection = temp;
2

I've got the same problem when I trying to save profile, I already table salutation and new to create profile. When I insert profile it also insert into salutation. So I tried like this before savechanges().

db.Entry(Profile.Salutation).State = EntityState.Unchanged;

0

What we have done is before adding the parent to the dbset, disconnect the child collections from the parent, making sure to push the existing collections to other variables to allow working with them later, and then replacing the current child collections with new empty collections. Setting the child collections to null/nothing seemed to fail for us. After doing that then add the parent to the dbset. This way the children are not added until you want them to.

0

I had a similar challenge using Entity Framework Core 3.1.0, my repo logic is quite generic.

This worked for me:

builder.Entity<ChildEntity>().HasOne(c => c.ParentEntity).WithMany(l =>
       l.ChildEntity).HasForeignKey("ParentEntityId");

Please note "ParentEntityId" is the foreign key column name on the child entity. I added the above mentioned line of code on this method:

protected override void OnModelCreating(ModelBuilder builder)...
0

If you want to set specific object without their relations do track. Use this option.

public void Insert(School newItem)
{

    using (var context = new DatabaseContext())
    {
        context.Entry(newItem).State = EntityState.Added;
        context.SaveChanges();
    }
}

Read more for a better Knolage at EF Core Documentation about Tracking

0

For those looking for a solution for Entity Framework Core, there is an Ignore function:

    public void Configure(EntityTypeBuilder<Parent> entity)
    {
        entity.Property(e => e.ChildId);

        entity.Ignore(e => e.Child);
    }
0

I have managed to make it work with EF Core like this, when adding new entities:

// Update children state:
targetDbContext.Entry(parent.Child1).State = EntityState.Detached;
targetDbContext.Entry(parent.Child2).State = EntityState.Detached;
foreach(var chi in parent.Children)
    targetDbContext.Entry(chi).State = EntityState.Detached;

// Update parent state:
targetDbContext.Entry(parent).State = EntityState.Added;

// Add & save:
await destinationSet.Add(parent);
await targetDbContext.SaveChanges();

You first mark all children as Detached, and then you mark the parent as Added. This will only try to insert the parent. I have no idea why this works, but it does for me.

Microsoft should have a broader perception of how their libraries could be used. Their implementations are too short-sighted, too limiting, and too unintuitive. Not to mention, that I have seen EF Core behave like having a memory leak and taking all the available RAM, when you do anything diverging from the mainstream usage.

6
  • Not useful because it's not clear why you had a problem, why these navigation properties were populated in the first place and in which state. Usually, a line like targetDbContext.Entry(parent).State = EntityState.Added; only sets the state of this single entry, not the navigation, but you decided to Add it on top of that. Also, the rant against EF is pretty premature and unsubstantiated. It's not unreasonable to expect a minimum perception of how libraries should be used. Commented May 24 at 11:58
  • @GertArnold I thought this was the point of the question, wasn't it? It is not just me that have the problem. My entities have navigation properties but not the ID of those navigation properties (e.g. I have a User property, but not a UserId property). Why? Because that's what I want. So, nulling out the navigation property will try to insert NULL into the FK column, and I can't have that. For my rant, I am sure MS has its reasons for their design, but I am tired of trying to use MS libraries to do my job, only to discover that it might have been easier to roll out my own library instead. Commented May 24 at 12:22
  • And how should a reader know all this seeing your code? Maybe you should attach the child entities as Unchanged, who knows. It's impossible to see what would have been the best solution to your unknown problem. Commented May 24 at 13:13
  • @GertArnold Well, if the EF Core programming interface and its internals was designed correctly, there would be no problem. Not for me, neither for the person that posted the original question, neither for the countless people that tried (over the span of a decade or more) to answer the question and failed, and even more for the person reading the solution to a problem like that... In any case, my problem is the same as the original question and it is far from "unknown". And it persists for God knows how many iterations of EF. Commented May 24 at 16:04
  • @GertArnold All in all, my solution works. I have migrated an entire DB using it. If someone finds that it doesn't work for his own case, I am interested to know about it. The fact that you don't understand the problem (which is not anything remotely distinct from the original question that got over 150 upvotes) is not very relevant. Commented May 24 at 16:14
-1

I know it's old post however if you are using code-first approach you may achieve the desired result by using following code in your mapping file.

Ignore(parentObject => parentObject.ChildObjectOrCollection);

This will basically tell EF to exclude "ChildObjectOrCollection" property from model so that it is not mapped to database.

1
  • 1
    What context is "Ignore" being used in? It doesn't appear to exist in the presumed context.
    – T3.0
    Commented Jul 10, 2020 at 17:46
-1

This is the working code of Entity Framework Core .net 5 that saves just in parent table with no errors. I am saving the list, you can just save the object as well.

Changing to this, starts giving me another error: The entity type <typename> was not found. Ensure that the entity type has been added to the model

note: YouObject is the entity

error code: 
_context.yourObject.AttachRange(yourObjectList);
_context.Entry(_context.yourObject).State = EntityState.Added;
await _context.SaveChangesAsync().ConfigureAwait(false);

So, this is what I did: (Working Code below)

Step 1: I made my child components as null.

yourObjectList.ForEach(req => { req.ChildObject = null; });

Step 2: Changed the above code to this:

working code:

_context.yourObject.UpdateRange(yourObjectList);
await _context.SaveChangesAsync().ConfigureAwait(false); 
4
  • This is a totally different situation than in the question. And your "fix" would be a very bad idea in that situation. Commented Mar 19, 2022 at 10:01
  • what is the problem with this code? @GertArnold
    – KushalSeth
    Commented Mar 19, 2022 at 10:26
  • You sever a relationship and then save the entities. Think of what woud happen if the child objects were known entities (as in the question). Apart from that, it's not clear why you have unknown entities in your model and how the objects get there. Commented Mar 19, 2022 at 10:30
  • In short, your answer only tells how you solved your own, totally different, problem. Commented Mar 19, 2022 at 10:32

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.