MVC with TextBoxFor having same id within loop

Question

I would like to have a table within a form, with an update button. Ideally each row would be a form, but that doesn't fit html spec.

my view looks like this

@model IEnumerable<DabTrial.Models.ManageFilesModel>

@foreach(var item in Model) {
<tr>
    <td>
        @Html.TextBoxFor(modelItem=>item.Name)
        ....

when I look at the markup, the inputs within each row for the same property have the same id

<input id="item_Name" type="text" value="ParentGuardianPMH.pdf" name="item.Name">
...
<input id="item_Name" type="text" value="ParentGuardianMMH.pdf" name="item.Name">

which is obviously invalid - how should I be doing this. Thank you

You can explicitly set the ID using one of the TextBoxFor<>() overloads (assuming everything else is the way you want it). You may want to also consider an editor template which accounts for one level of nested objects (it doesn't look like the name elements will match up with what the default model binder expects). — Tim M.
– Tim M., Commented Jun 6, 2013 at 6:35

xurca · Accepted Answer · 2013-06-06 07:12:22Z

4

you need such thing

@for(int i=0; i< Model.Count(); i++)
{
  @Html.TextBoxFor(item => item[i].Name)
}

read this post about model binding to a list http://haacked.com/archive/2008/10/23/model-binding-to-a-list.aspx

edited Jun 6, 2013 at 7:12

answered Jun 6, 2013 at 6:50

xurca

2,4363 gold badges25 silver badges32 bronze badges

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

Comments

Raphaël Althaus · Accepted Answer · 2013-06-06 06:37:02Z

2

use a for loop, not a foreach, and you'll have distinct id.

answered Jun 6, 2013 at 6:37

Raphaël Althaus

60.6k6 gold badges106 silver badges127 bronze badges

Collectives™ on Stack Overflow

MVC with TextBoxFor having same id within loop

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related