views:

623

answers:

2

I want to loop through a collection of objects and add them all to a table. The destination table has an auto-increment field. If I add a single object there is no problem. If I add two objects both with the primary key of zero, the entity framework fails. I can manually specify primary keys but the whole point of trying the EF was to make life easier not more complicated. Here is the code and the exception received follows.

foreach (Contact contact in contacts)
{               
  Instructor instructor = InstructorFromContact(contact);               
  context.AddToInstructors(instructor);             
}

try
{                   
  context.SaveChanges();                    
}
catch (Exception ex)
{
  Console.WriteLine(ex.ToString());
}

System.InvalidOperationException: The changes to the database were committed successfully, but an error occurred while updating the object context. The ObjectContext might be in an inconsistent state. Inner exception message: AcceptChanges cannot continue because the object's key values conflict with another object in the ObjectStateManager. Make sure that the key values are unique before calling AcceptChanges.
at System.Data.Objects.ObjectContext.SaveChanges(SaveOptions options)
at System.Data.Objects.ObjectContext.SaveChanges()
at DataMigration.Program.CopyInstructors() in C:\Projects\DataMigration\Program.cs:line 52

+5  A: 

Set the StoreGeneratedPattern attribute to "Identity" in your SSDL for the autoincrement field. It should help.

Devart
Note that you can't just set this property in the designer, you have to edit the SSDL section by hand. I read past this answer and thought: "I've already done that.", and I kept looking, I could have save hours by noticing the *SSDL* This is a know bug in the designer. http://geeksharp.com/2010/05/27/ef4-bug-in-storegeneratedpattern-ssdl/ and http://social.msdn.microsoft.com/Forums/en-US/adodotnetentityframework/thread/404d3017-01b7-4129-8e05-f4aa48f15f08
DanO
+1  A: 

This happens because despite the auto-generated value of the column was created in the Database the EF never knew about it.

So, in order to inform EF that the DB will handle the generated value you have to to open your edmx file (I always use the XML editor of VS to do this) and in the store schema definition language (SSDL) area, add the attribute StoreGeneratedPattern="Identity" to the column that needs the generated pattern. In this way EF reads the value generated in the DB and stores it in memory cache.

Your entity type definition will look more or less like this:

 <EntityType Name="INVOICE">
          <Key>
            <PropertyRef Name="CODE" />
          </Key>
          <Property Name="CODE" Type="varchar" Nullable="false"
              MaxLength="10" StoreGeneratedPattern="Identity"/>                 
 </EntityType>

Be aware that if you happen to update your model all these changes will be lost and you'll have to repeat all the entire process.

This works for EF 1.0, I'm not sure if in EF4 all these issues are already fixed.

cepriego
I think where I went wrong is in that I initially created the model from the database and forgot to set the autonumber field. LaterI edited my database and set this property followed by the "update model from database" command within visual studio 2010. The update feature seems to add new properties but not exactly sure if it 'fixes' existing modified properties correctly. By deleting my model and recreating from scratch, the autonumber feature worked as expected. However had I known better, I would have tried your idea.
dcompiled
Currently (but hopefully soon!) I'm not using EF4, however I read something in this page that talks about this: http://geeksharp.com/2010/05/27/ef4-bug-in-storegeneratedpattern-ssdl/
cepriego