Using Foreach To Save Data Not Save Courses Correctly

Discussion of open issues, suggestions and bugs regarding Entity Framework support in ADO.NET Data providers
Post Reply
Posts: 9
Joined: Mon 15 Aug 2016 10:17

Using Foreach To Save Data Not Save Courses Correctly

Post by ahmedsa » Sat 03 Sep 2016 23:29


save name in employee table and save all courses

in employee courses table when click submit button in edit [HTTPPOST]

Problem summary

submit button save changes only and delete data exist before

Problem details

when click submit button in edit post .courses will save

what i added or selected courses before click submit it will save

but courses that employee have before will delete


1- i add employee name MEDO have course c++

2-in edit view it show c++

3-if i add python in edit view then click submit it must have two courses

c++ and python

4- it show to me python only when open edit for this employee again

it save courses selected but courses exist before deleted

Image for problem

image found below show all details ... g_save.jpg

SAVE all courses in employeecourse table


Code: Select all

public class EditEmployeeVm
public int Id { set; get; }
public string Name { get; set; }
public List<SelectListItem> Courses { get; set; }
public int[] CourseIds { set; get; }
public List<CourseVm> ExistingCourses { set; get; }

public class CourseVm
public int Id { set; get; }
public string Name { set; get; }
in edit function get i pass data to edit view

public ActionResult Edit(int id)
var vm = new EditEmployeeVm { Id = id };
var emp = db.Employees.FirstOrDefault(f => f.Id == id);
vm.Name = emp.Name;
vm.ExistingCourses = db.EmployeeCourses
.Where(g => g.EmployeeId == id)
.Select(f => new CourseVm
Id = f.Id,
Name = f.Course.CourseName

vm.CourseIds = vm.ExistingCourses.Select(g => g.Id).ToArray();
vm.Courses = db.Courses.Select(f => new SelectListItem
Value = f.Id.ToString(),
Text = f.CourseName

return View(vm);

public ActionResult Edit(EditEmployeeVm model)
var emp = db.Employees.FirstOrDefault(f => f.Id == model.Id);
foreach (EmployeeCourse eec in emp.EmployeeCourses.ToList())
var ec = db.EmployeeCourses.Find(eec.Id);

foreach (var couseid in model.CourseIds)
db.EmployeeCourses.Add(new EmployeeCourse { CourseId = couseid, EmployeeId = emp.Id });

return View();

Site Admin
Posts: 8248
Joined: Thu 14 Aug 2008 12:44

Re: Using Foreach To Save Data Not Save Courses Correctly

Post by Shalex » Tue 06 Sep 2016 08:22

The problem is not clear. Please give us the following information:
a) send us a small test project with the corresponding DDL/DML script
b) specify the exact steps we should follow for reproducing
c) the actual result (error or incorrect behaviour) you are experiencing
d) the expected result

Post Reply