Edit action does not update the database details ASP.NET MVC

Question

"CODE 01: Data update form does the validation correctly but it does not update the records in the table. "

"There's similar code which I have used as an example, which is "CODE 02" that code work but the other does not .... "

"I did accordingly to the CODE 02, but CODE 01 does not work"

@* CODE 01 *@

 public ActionResult Edit(Film F)
        {

            if (ModelState.IsValid) {
                var item3 = db.Films.Where(x => x.Id == F.Id).First();
                item3.Title = F.Title;
                item3.Director = F.Director;
                item3.Actor = F.Actor;
                item3.Year = F.Year;
                item3.Budget = F.Budget;
                ModelState.Clear();
                db.SaveChanges();
                ViewBag.alert = "Your data is updated";
            }
                return View(F);
        }

@* CODE 02 *@

public ActionResult Edit(Film E) {

            var item3 = db.Films.Where(e=> e.Id==E.Id).First();
            item3.Title = E.Title;
            item3.Director = E.Director;
            ViewBag.alert = "Data is success";
            db.SaveChanges();
            return View();
        }

CODE 01 doesnt update the data but CODE 02 does upadate it

try putting db.Entry(item3).State = EntityState.Modified before db.SaveChanges(). — Ajay, Sep 30 '19 at 13:08
Have you tried debugging to check if your `ModelState.IsValid` returns true? — Rahul Sharma, Sep 30 '19 at 13:12
I think validation fails. I dont understand why .... jQuery validation works properly. — silvachathura, Sep 30 '19 at 13:17
Check `ModelState.ModelErrors`, it will tell you the problem. — mxmissile, Sep 30 '19 at 13:52

score 1 · Accepted Answer · answered Sep 30 '19 at 13:48

1

It seems as though you are not fulling defining the "Film" class that you are passing to the edit function. If your model is not being defined as valid then that means that you have something in your class that is either defined wrong or undefined.

An example of this may be you have a budget that you have submitted as a double (2000.30) when your class defines this as an integer or a title or actor is not defined when it is not of a nullable type OR you have data annotations in your class restricting it further and those criteria are not being met.

refer to ModelState.IsValid question here

answered Sep 30 '19 at 13:48

Jeremy Buentello

133
9

1

ID was not added as input hidden form. That's why the model wasnt validated. Now its working ... thnk u ! – silvachathura Oct 01 '19 at 08:34
That's great! No problem, if you don't mind validating the answer to show others that it helped, it would be appreciated :) – Jeremy Buentello Oct 03 '19 at 17:19

Edit action does not update the database details ASP.NET MVC

1 Answers1