0

День добрый.

Сделал аналогично примеру Many to many Generic update method EF 6

Но в моем случае значения из свойства-коллекции заново создаются в "БД".

public void Update(T updateItem)
{
    using (var databaseContext = new TelesystemContext())
    {

        T foundDal = databaseContext.Set<T>().Find(updateItem.Id);
        var entry = databaseContext.Entry(foundDal);
        entry.CurrentValues.SetValues(updateItem);

        var collection = entry.Collection("Permissions");
        collection.Load();
        collection.CurrentValue = typeof(T).GetProperty("Permissions").GetValue(updateItem);


        databaseContext.SaveChanges();
    }
}

Т.е. Permissions каждый раз при сохранении объекта updateItem создаются заново, а должны были просто добавиться в таблицу - связку "многие-ко-многим".

6
  • То что используется строковый литерал "Permissions" это только для теста. В общем случае, естественно будет заменено на параметр. Commented 30 авг. 2016 в 12:29
  • 1
    У вас как-то слишком сложно написано. Достаточно было databaseContext.Entry(updateItem).State = EntityState.Modified; Commented 31 авг. 2016 в 5:39
  • Это generic реализация. Но все равно элементы коллекции Permissions создаются в БД снова. Т.е. имееют состояние Added Commented 31 авг. 2016 в 6:07
  • Александр, какая разница что там за реализация? Приведенный вам код делает то же самое что и ваш. А ответ я вам написал ниже. Commented 31 авг. 2016 в 6:08
  • Ссори, почему то вначале я видел только комментарий. Commented 31 авг. 2016 в 6:32

1 ответ 1

1

Ваша проблема в том, что добавленные вами сущности Permissions не относятся к контексту - и в итоге оказываются в него добавлены. Вам надо явно пройтись по этим разрешениям - и указать что они находятся в статусе Unchanged:

foreach (var p in updateItem.Permissions)
    databaseContext.Entry(p).State = EntityState.Unchanged;

PS поскольку свойство Permissions недоступно напрямую - вам придется придумать как получить к нему доступ. Я намеренно не писал кода для получения этого свойства - потому что он не имеет отношения к задаче.

2
  • Присвоить EntityState.Unchanged тоже не правильно. А если они действительно добавились. Как правильно сделать обобщенное решение с отсоединенными сущностями? Commented 31 авг. 2016 в 6:34
  • @Александр а это уже вы должны придумать как отслеживать. В общем случае задача решения не имеет. Commented 31 авг. 2016 в 6:36

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.