В EF Core не сохраняется подчиненная сущность?

Question

[Роман]

На контроллер приходить сущность

public class IdentityUser
    {
        public int Id { get; set; }
        public string UserName { get; set; }
        public string Login { get; set; }
        public string PasswordHash { get; set; }
        public int RoleId { get; set; }

        public virtual List<UserDivision> UserDivisions { get; set; }
  }

    public class UserDivision
    {
        public int IdentityUserId { get; set; }
        [JsonIgnore]
        public IdentityUser IdentityUser { get; set; }

        public int DivisionId { get; set; }
        [JsonIgnore]
        public Division Division { get; set; }
    }

код контроллера

[HttpPut("{id}")]
        public async Task<IActionResult> PutIdentityUser([FromRoute] int id, [FromBody] IdentityUser identityUser)
        {
            if (!ModelState.IsValid)
            {
                return BadRequest(ModelState);
            }

            if (id != identityUser.Id)
            {
                return BadRequest();
            }
            _context.Entry(identityUser).State = EntityState.Modified;
            try
            {
                await _context.SaveChangesAsync();
            }
            catch (DbUpdateConcurrencyException)
            {
                if (!IdentityUserExists(id))
                {
                    return NotFound();
                }
                else
                {
                    throw;
                }
            }

            return NoContent();
        }

сущность UserDivisions, не сохраняется в БД.

Comments

[Толстый Лорри]

Где и как у вас создается контекст? Как он определен?
Подчиненные сущности вообще приходят в экшн?

[Роман]

Толстый Лорри, через DI внедряется в конструктор контроллера

[Толстый Лорри]

Роман,

Как он определен?
Подчиненные сущности вообще приходят в экшн?

[eRKa]

Роман, да, в первую очередь надо смотреть на то, что контекст у Вас один или, что Вы помечаете модифицированным нужный контекст. И потом надо смотреть, а приходят ли данные в запросе. Поэтому Толстый Лорри правильно вопрос задает. Надо смотреть, какие данные вы в запрос отправляете и сравнить с тем, что пришло в контроллер.
Лично меня смущает, что UserDivisions помечен как virtual, значит на него будет создан прокси, а вот IdentityUser и Division не помечены, значит при запросе без Include они будут null.

[Роман]

eRKa, да в контроллер все приходит, я думаю, что проблема в том, что EF автоматом не помечает подчиненные сущности как EntityState.Modified или EntityState.Added и трекер просто не видит изменений и в бд они не попадают.

[eRKa]

Роман, проверьте это. Пройдитесь по списку и каждый отметьте EntityState.Modified. Поможет или нет.

[Роман]

eRKa, Тут немного в другом проблема, записи из UserDivisions могут уже быть в базе и пометка модифицированными вызовет ошибку, если записи нет в базе. Сделал проще, удаляю все записи из сущности UsersDivisions и потом добавляю из пришедшей сущности. Видать это проблема при работе с many-to-many.

[eRKa]

Роман, т.е. в этом списке есть и модифицированные и новые? Мне кажется должно быть более адекватное решение. Буду дома, обязательно попробую эту ситуацию.

[Роман]

eRKa, С задачи объясню, есть пользователь, это его редактирование, UserDivisions это список подразделений к которому он имеет доступ, те можно добавить или удалить их, со SPA приходит выбранный список, те или надо удалить которые были уже или добавить.

[eRKa]

Роман, понял от куда многие ко многим. Значит с клиента приходит в месте с редактируемой моделью пользователя и измененный список подразделений, я так понимаю там просто новый список айдишников, ведь сами подразделения там не редактируются?

[Роман]

eRKa, Да. сами подразделения хранятся в другой таблице и естественно меняются в другом месте SPA. При добавлении, все норм, подчиненные сущности сохраняются, дело при изменении, ну тут проблема, EF должен знать, что добавить, что удалить, видать проблема в этом. Сделал велосипед, в полном удалении данных и потом добавлении нужных, типа

var ud = new List<UserDivision>();
            ud.AddRange(identityUser.UserDivisions);
            _context.Entry(identityUser).State = EntityState.Modified;
            try
            {
                _context.UserDivisions.RemoveRange(_context.UserDivisions.Where(w => w.IdentityUserId == identityUser.Id));
                await _context.SaveChangesAsync();
                _context.UserDivisions.AddRange(ud);
                await _context.SaveChangesAsync();
            }

но два SaveChanges смущают.

[eRKa]

Роман, SaveChangesAsync это единица работы (UoW) и если первый убрать, то по идее ничего не должно поменяться.
А если попробовать как то так?

var newDivisions = _context.UserDivisions.Where(w => IdentityUser.UserDivisions.Any(x => w.Id == x.Id ))
IdentityUser.UserDivisions = newDivisions;
_context.Entry(identityUser).State = EntityState.Modified;

[Роман]

eRKa, если первый убрать, то в базе останутся записи, которые могут быть в новом UserDivisions, и тогда вылетает исключение. И Ваш способ тоже пробывал. Я часа два, копался (тут вместо копался ненормативное слово), прежде чем написать. И в принципе ничего интересного в инете не нашел. Я так понял, это проблема в том, что трекером не помечаются изменения в подчиненной сущности, ибо он должен знать, что добавить, что удалить.

[eRKa]

Роман, ну ладно, без студии под рукой я просто гадаю, завтра руками попробую. Просто я второй раз на Вас обращаю внимание, потому что схожие задачи и даже в рабочем чате проверяю, а нет ли у нас Романа))

[Роман]

eRKa, Да я больше отвечаю, чем задаю, просто после долгих различных изменений кода и гугления, задал тут вопрос.