How to update multiple models at the same time in ASP.NET MVC 5 Entity Framework

Question

I am trying to update Two models at the same time.

Models:

Page
Fields

One page has multiple fields, I want them to update at the same time.

public class PageEditViewModel
{
    public Page mPage { get; set; }

    public IEnumerable<Field> Fields { get; set; }
}

Here is my View:

<div class="row">
                            <div class="col-md-12">
                                <h3>Fields</h3>
                                @foreach (var field in Model.ContentFields)
                                {
                                    @Html.HiddenFor(m => field.Id)

                                    switch (field.FieldType)
                                    {
                                        case "TextBox":

                                            <div class="form-group">
                                                <label class="control-label">
                                                     @field.FieldName<span class="required"> * </span>

                                                </label>
                                                @Html.TextBoxFor(m => field.Content , new { @class = "form-control" })
                                            </div>
                                            break;

                                        case "TextArea":
                                            <div class="form-group">
                                                <label class="control-label">
                                                    @field.FieldName<span class="required"> * </span>
                                                </label>
                                                @Html.TextAreaFor(m => field.Content, new { @class = "form-control" })
                                            </div>
                                            break;

                                        case "Image":
                                            <div class="form-group">
                                                <label class="control-label">
                                                    @field.FieldName<span class="required"> * </span>
                                                </label>
                                                <input type="file" name="contentImage" id="cImage" class="form-control" accept="image/*" />
                                            </div>
                                            break;
                                    }
                                }
                            </div>

                        </div>

And Controller:

public ActionResult Update(PageEditViewModel viewModel)
{
    if (!ModelState.IsValid)
    {
        var page = _context.MenuPages.Single(s => s.Id == viewModel.mPage.Id);
        var contentFields = _context.ContentFields.Where(c => c.MenuPageId == page.Id);

        var viewM = new PageEditViewModel
        {
            DashboardHeading = "Edit a Page",
            mPage = page,
            ContentFields = contentFields
        };
        return View("EditPage", viewM);
    }


    var pageEdit = _context.MenuPages.SingleOrDefault(p => p.Id == viewModel.mPage.Id);

    pageEdit.Name = viewModel.mPage.Name;
    pageEdit.IsActive = viewModel.mPage.IsActive;
    pageEdit.IsShowInMenu = viewModel.mPage.IsShowInMenu;

    //            _context.SaveChanges();

    foreach (var field in viewModel.ContentFields)
    {
        var cfield = _context.ContentFields.SingleOrDefault(f => f.Id == field.Id);

        cfield.Content = field.Content;

    }
    _context.SaveChanges();

When I Send the data from View to Controller, I get the data for Pages but Null for the Fields Model (Object Reference not set to an instance...).

I am looking forward to any guide from members here.

Thanks.

There's quite similar type of question found here: stackoverflow.com/questions/18237945/… — jyrkim
– jyrkim, Commented Oct 10, 2017 at 12:16
@GertArnold I have updated the question, Please have a look. — Aleem
– Aleem, Commented Oct 10, 2017 at 12:25

Emanuele Buccelli · Accepted Answer · 2017-10-10 13:01:12Z

1

I think Umesh answer is correct. After changing your loop, how are you setting your Html.TextBoxFor, HiddenFor and so on?

It should be:

@Html.HiddenFor(m => m.ContentFields[i].Id)

answered Oct 10, 2017 at 13:01

Emanuele Buccelli

263 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Emanuele Buccelli Over a year ago

If you like, I have a post about this in my blog :) debuxing.com/model-binding-to-a-list-of-objects

Emanuele Buccelli Over a year ago

Thanks a lot @Aleem glad that you like it and that it worked.

Umesh Dubal · Accepted Answer · 2017-10-10 12:36:37Z

0

You need to replace @foreach loop on view

 @foreach (var field in Model.ContentFields)
                            {

By for loop

@for (var i = 0; i < Model.ContentFields.Count; i++)
            {

This will bind your list to model, while sending data to controller

answered Oct 10, 2017 at 12:36

Umesh Dubal

314 bronze badges

Collectives™ on Stack Overflow

How to update multiple models at the same time in ASP.NET MVC 5 Entity Framework

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related