DevForce Community Forum : Bug in using AttachEntity and Navigation Property. http://www.ideablade.com/forum/ This is an XML content feed of; DevForce Community Forum : DevForce 2010 : Bug in using AttachEntity and Navigation Property. Wed, 13 May 2026 05:44:40 -700 Tue, 22 May 2012 00:56:44 -700 http://blogs.law.harvard.edu/tech/rss Web Wiz Forums 9.69 360 www.ideablade.com/forum/RSS_post_feed.asp?TID=3417 DevForce Community Forum http://www.ideablade.com/forum/forum_images/IdeaBlade_logo_tm.png http://www.ideablade.com/forum/ Bug in using AttachEntity and Navigation Property. : Nice,Thanks for the feedback. http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13626#13626 Author: Walid
Subject: 3417
Posted: 22-May-2012 at 12:56am

Nice,

Thanks for the feedback.
]]> Tue, 22 May 2012 00:56:44 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13626#13626 Bug in using AttachEntity and Navigation Property. : Hi Walid,Just want to let you... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13612#13612 Author: DenisK
Subject: 3417
Posted: 21-May-2012 at 12:29pm

Hi Walid,

Just want to let you know that the AttachEntity bug as you reported above has been fixed. It'll be included in the next release.
]]> Mon, 21 May 2012 12:29:43 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13612#13612 Bug in using AttachEntity and Navigation Property. : Hi Walid,Thanks for pointing out... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13490#13490 Author: DenisK
Subject: 3417
Posted: 03-May-2012 at 4:20pm

Hi Walid,

Thanks for pointing out that possible bug. Let me find out whether it's a real bug and file a report as necessary.

For now, I'm suggesting to use the first workaround with the addition of calling AcceptChanges on the entity.

     var manager = new TestEntities();

      var profession = new Profession() {
        Id = 1,
        Title = "Boss",
      };
      if (attachProfession)
        manager.AttachEntity(profession);

      var user = new User {
        Id = 2,
        Name = "User test with Profession",
      };

      manager.AttachEntity(user);
      user.Profession = profession;

      //Call AcceptChanges to make the user's state change from Modified to Unchanged
      EntityAspect.Wrap(user).AcceptChanges();

Please let me know if this works for you or not.
]]> Thu, 03 May 2012 16:20:31 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13490#13490 Bug in using AttachEntity and Navigation Property. : There is another problem. I made... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13472#13472 Author: Walid
Subject: 3417
Posted: 02-May-2012 at 3:36am

There is another problem. I made a function which  first create all the entities and set all the navigation properties, then attach them to the manager.

The Navigation properties are correctly set but all the foreignkey are set to 0. 
Note : AddEntities does correctly update the foreignKey;


        private static void Test4()        {            var manager = new TestEntities();             var professions = new List<Profession>()                                {                                    new Profession()                                        {                                            Id = 1,                                            Title = "Profession 1",                                        },                                        new Profession()                                            {                                                Id = 2,                                                Title = "Profession 2",                                            },                               };             var users = new List<User>()                            {                                new User()                                    {                                        Id = 1,                                        Name = "User 1",                                        Profession = professions[0],                                    },                                new User()                                    {                                        Id = 2,                                        Name = "User 2",                                        Profession = professions[1],                                    }                            };              manager.AttachEntities(professions);            manager.AttachEntities(users);             foreach (var user in users)            {                // Navigation property is correctly set                Debug.Assert(!EntityAspect.Wrap(user.Profession).IsNullOrPendingEntity);                // The foreignkey isn't set                Debug.Assert(user.ProfessionId == 0);            }        }



]]> Wed, 02 May 2012 03:36:01 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13472#13472 Bug in using AttachEntity and Navigation Property. : Hi Denis, Thanks but only one... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13471#13471 Author: Walid
Subject: 3417
Posted: 02-May-2012 at 12:48am

Hi Denis,

Thanks but only one of those workaround works and it isn't always applicable.
The workaround where we attach first then set the navigation property change the entitystate of the entity which is not what we want (or we would have use AddEntity).
 
In fact, what you said earlier is what is happening. Setting a navigation property to an already attached/added entity will add the entity to the EM and set the auto generated Id. Attaching isn't the real cause.
 
The workaround where I attach all the entities in last (after all the settings is done) might not be applicable in some case. Sometimes, the entity I want to use for my navigation property been attached "somewhere else".
How can I deal with this issue in an elegant way ? Attaching, changing back the Id, updating the entitystate doesn't sound like a solution to me when you have to deal with hundreds of entities.
 
 
 
This exemple show AttachEntity isn"'t the real problem. 
        var profession = new Profession()        {            Id = 1,            Title = "Boss",        };        manager.AttachEntity(profession); 
         var user = new User        {            Id = 2,            Name = "User test with Profession",        };         user.Profession = profession;        Debug.Assert(user.Id == 2, string.Format("Id expected = 2 - Id current value = {0} ", user.Id));
 
]]> Wed, 02 May 2012 00:48:27 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13471#13471 Bug in using AttachEntity and Navigation Property. : Hi Walid,Not sure what happened... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13469#13469 Author: DenisK
Subject: 3417
Posted: 01-May-2012 at 6:39pm

Hi Walid,

Not sure what happened but I can finally see the assert failed. I've modified the solution and added some comments to illustrate the correct way to achieve what you want.

uploads/912/ConsoleApplication3_Modified.zip
]]> Tue, 01 May 2012 18:39:45 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13469#13469 Bug in using AttachEntity and Navigation Property. : Strange, the assert fail each... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13463#13463 Author: Walid
Subject: 3417
Posted: 01-May-2012 at 1:32am

Strange, the assert fail each time for me. 
i am using Devforce 6.1.6 and EntityFramework 4.3.1

 




Edited by Walid - 01-May-2012 at 1:49am]]> Tue, 01 May 2012 01:32:01 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13463#13463 Bug in using AttachEntity and Navigation Property. : Hi Walid,My apologies, but you&#039;re... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13461#13461 Author: DenisK
Subject: 3417
Posted: 30-Apr-2012 at 5:36pm

Hi Walid,

My apologies, but you're correct. AttachEntity is NOT the same as AddEntity. Our DRC says,

"theEntity’s EntityKey (“the key”) must be preset prior to the attach operation (which will not touch the key)."

which confirms what you said above.

Now with this said, I just ran your sample and I don't see user.Id changes to -100. It stayed at 2 in both cases.
]]> Mon, 30 Apr 2012 17:36:23 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13461#13461 Bug in using AttachEntity and Navigation Property. : hi Denis,thanks, i thought attach... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13460#13460 Author: Walid
Subject: 3417
Posted: 30-Apr-2012 at 4:47pm

hi Denis,

thanks, i thought attach doesn't touch the key, but only add it in the unchanged state.

so the "best practice" is to attach an entity right after it has been created and to set the navigation properties in last.
]]> Mon, 30 Apr 2012 16:47:47 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13460#13460 Bug in using AttachEntity and Navigation Property. : Hi Walid,By default, User.Id and... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13459#13459 Author: DenisK
Subject: 3417
Posted: 30-Apr-2012 at 4:35pm

Hi Walid,

By default, User.Id and Profession.Id are Identity columns when doing Code First unless you specify it otherwise by marking it as such.

[DatabaseGenerated(System.ComponentModel.DataAnnotations.DatabaseGeneratedOption.None)]

An identity column/id will be assigned a new temporary id as soon as it is attached or added to the EntityManager. In your "what doesn't work" scenario, when you create a User with Id 1, it is still unattached. But as soon as you're setting its User.Profession property to an already attached/added Profession entity, the unattached User will be attached and its Id will automatically be set to a temp Id overwriting whatever Id it was set to.
]]> Mon, 30 Apr 2012 16:35:58 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13459#13459 Bug in using AttachEntity and Navigation Property. : Hi,Currently working with AttachEntity... http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13456#13456 Author: Walid
Subject: 3417
Posted: 30-Apr-2012 at 10:41am

Hi,

Currently working with AttachEntity for the Faking (faking isn't relevant in this case), I have a strange behavior when I set a navigation property.

Considering those 2 entities :

    public class User : BaseEntity    {        [Key]        public int Id { get; set; }         public string Name { get; set; }         [ForeignKey("Profession")]        public int ProfessionId { get; set; }        public Profession Profession { get; set; }    }     public class Profession : BaseEntity    {        [Key]        public int Id { get; set; }         [Required]        public string Title { get; set; }    }

What works :
1 - I create a Profession with Id 1 and DOESN'T attach it to the manager.
2 - I create a user with Id 1 and set the Profession to the User
=>  user.Id == 1

What doesn't work :
1 - I create a Profession with Id 1 and attach it to the manager.
2 - I create a user with Id 1 and set the Profession to the user
=> user.Id == -100


Here is a test  application uploads/998/ConsoleApplication3.zip


]]> Mon, 30 Apr 2012 10:41:08 -700 http://www.ideablade.com/forum/forum_posts.asp?TID=3417&PID=13456#13456