ASP.NET MVC BeginCollectionItem issue

Question

I am having an issue with the HTML helper BeginCollectionItem. It seems to be binding the item to the view but and changes are not being propagated.

I have a partial view and the model that is bound to it is an IEnumerable. Below is a snippet.

<tbody>
    @foreach (var entry in Model) {
        <tr>
            @using (Html.BeginCollectionItem("EditedEntries")) {
                <td>@entry.Storeid</td>
                <td>@entry.district</td>
                <td>@Html.EditorFor(x => entry.AdjHrs)</td>
            }
        </tr>
    }
</tbody>

If I remove the the foreach it works, however I need to use a foreach because a collection is returned to the partial view from the Ajax call along with the table and its members.

Have a look here stackoverflow.com/questions/8568160/…

lyz
– lyz

2016-01-15 23:27:12 +00:00
Commented Jan 15, 2016 at 23:27 — lyz
– lyz, Commented Jan 15, 2016 at 23:27

user3559349 · Accepted Answer · 2016-01-16 00:14:46Z

3

The BeginCollectionItem is designed to work with a partial view. Create one for your model (I'll assume its named MyModel and you name the partial "_MyModel.cshtml")

@model MyModel
<tr>
    @using (Html.BeginCollectionItem("EditedEntries"))
    {
        <td>@Html.DisplayFor(m => m.Storeid)</td>
        <td>@Html.DisplayFor(m => m.district)</td>
        <td>@Html.EditorFor(m => m.AdjHrs)</td>
    }
</tr>

and then in your other partial, replace the foreach loop with

<tbody>
    @foreach (var entry in Model)
    {
        @Html.Partial("_MyModel", entry)
    }
</tbody>

answered Jan 16, 2016 at 0:14

user3559349

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

3 Comments

Devin Wall Over a year ago

Yeah, I cant do that though as the the list is not populated until that partial is loaded and rendered. Hence the reason for the foreach being in the partial view.

user3559349 Over a year ago

You need to read my answer again. Your current partial contains a collection and the foreach loop calls the _MyModel.cshtml partial

Devin Wall Over a year ago

Sorry, It has been a long day and I misread. Thank you for your help.

Devin Wall · Accepted Answer · 2016-01-15 22:42:11Z

So I managed to find the cause of the issue. The HTML helper trying to be smart and auto generate the ID and Name of the hidden fields associated with the data.

What was generated

<tr role="row" class="odd">
<input type="hidden" name="EditedEntries.index" autocomplete="off" value="a2a18da0-528f-4b10-92c1-4a8ba7038dde">
    <td class="sorting_1">1</td>
    <td>1</td>
    <td>
        <input data-val="true" data-val-number="The field AdjHrs must be a number." data-val-required="The AdjHrs field is required." 
        id="EditedEntries_a2a18da0-528f-4b10-92c1-4a8ba7038dde__entry_AdjHrs" name="EditedEntries[a2a18da0-528f-4b10-92c1-4a8ba7038dde].entry.AdjHrs" type="text" value="0">
    </td>

What it needed to be

<tr role="row" class="odd">
<input type="hidden" name="EditedEntries.index" autocomplete="off" value="a2a18da0-528f-4b10-92c1-4a8ba7038dde">
    <td class="sorting_1">1</td>
    <td>1</td>
    <td>
        <input data-val="true" data-val-number="The field AdjHrs must be a number." data-val-required="The AdjHrs field is required." 
        id="EditedEntries_a2a18da0-528f-4b10-92c1-4a8ba7038dde_AdjHrs" name="EditedEntries[a2a18da0-528f-4b10-92c1-4a8ba7038dde].AdjHrs" type="text" value="0">
    </td>

So it prefixed the identifier with the variable name of the 'foreach' I may have to just write my own helper. I will post back when I do.

Collectives™ on Stack Overflow

ASP.NET MVC BeginCollectionItem issue

2 Answers 2

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related