Asp.net MVC постбэк

Question

У меня есть контроллер со следующими двумя методами редактирования. Форма редактирования отображается правильно со всеми дополнительными выпадающими списками из FormViewModel. Однако, когда я изменил некоторые значения полей и отправил форму. Ни одно из измененных полей не было сохранено. Поля в коллекции postbask имеют значения по умолчанию или нулевые значения. У меня есть другая форма редактирования, которая обновляет другую таблицу. При отправке измененные значения сохраняются. Кто-нибудь знает почему?

// GET: /Transfers/Edit/5
public ActionResult Edit(int id)
{
    Transfer transfer = myRepository.GetTransfer(id);

    if (transfer == null)
        return View("NotFound");

    return View(new TransferFormViewModel(transfer));

} 

//
// POST: /Transfers/Edit/5

[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Edit(int id, Transfer collection)
{

    Transfer transfer = vetsRepository.GetTransfer(id);

    if (transfer == null)
        return View("NotFound");
    else
    {
        try
        {
            UpdateModel(transfer);
            vetsRepository.Save();

            return RedirectToAction("Details", new { id = transfer.TransfersID });
        }
        catch
        {
            ModelState.AddModelErrors(transfer.GetRuleViolations());

            return View(new TransferFormViewModel(transfer));
        }
    }
}

---

Вот Edit.aspx и пользовательский элемент управления ASCX:

<%@ Page Title="" Language="C#" MasterPageFile="~/Views/Shared/Site.Master" 
Inherits="System.Web.Mvc.ViewPage<VETS.Controllers.TransferFormViewModel>" %>

<asp:Content ID="Content1" ContentPlaceHolderID="TitleContent" runat="server">
    Edit
</asp:Content>

<asp:Content ID="Content2" ContentPlaceHolderID="MainContent" runat="server">

    <% Html.RenderPartial("TransferForm"); %>

    <div>
        <%=Html.ActionLink("Back to List", "Index") %>
    </div>

</asp:Content>

=============================================== ====================================

<%@ Control Language="C#" Inherits="System.Web.Mvc.ViewUserControl<VETS.Controllers.TransferFormViewModel>" %>
    <% 
        var thisTransfer = Model.Transfer;
        var thisVeteran = thisTransfer.Veteran;
    %>

    <%= Html.ValidationSummary("Please correct the errors and try again.") %>

    <% using (Html.BeginForm()) { %>

        <fieldset>
            <legend><%= (thisTransfer == null) ? string.Empty : (thisTransfer.OffList ? "Off List" : " On List")%></legend>
            <div class="sideBySideDiv">
            <p>
                <label for="LastName">Last Name:</label>
                <%= thisVeteran.LastName %>
            </p>
            <p>
                <label for="FirstName">First Name:</label>
                <%= thisVeteran.FirstName %>
            </p>
            <p>
                <label for="LastFourSSN">SSN:</label>
                <%= thisVeteran.LastFourSSN%>
            </p>
            <p>
                <label for="DOB">DOB:</label>
                <%= String.Format("{0:d}", thisVeteran.DOB)%>
            </p>
            <p>
                <label for="Hospital">Hospital:</label><br />
                <%= Html.DropDownList("Hospital", Model.NonVaHospitals)%>
            </p>
            <p>
                <label for="AdmitDX">Admit DX:</label>
                <%= Html.TextBox("AdmitDX", thisTransfer.AdmitDX)%>
            </p>
            <p>
                <label for="BedType">Bed Type:</label>
                <%= Html.DropDownList("BedType", Model.BedTypes)%>
            </p>
            <p>
                <label for="AdmitDate">Admit Date:</label>
                <%=Html.TextBox("AdmitDate", string.Format("{0:d}", thisTransfer.AdmitDate))%>
            </p>
            <p>
                <label for="_SC">%SC:</label>
                <%= thisVeteran.PercentSC._SC%>
            </p>
            <p>
                <label for="PCP">PCP:</label>
                <%= thisVeteran.PCP %>
            </p>
            <p>
                <label for="ListStatus">List Status</label>
                <%= Html.DropDownList("ListStatus", Model.StatusTypes)%>
            </p>
            <p>
                <label for="Ward">Ward:</label>
                <%= Html.TextBox("Ward", thisTransfer.Ward)%>
            </p>
            <p>
                <label for="WardPhone">Ward Phone</label>
                <%= Html.TextBox("WardPhone", thisTransfer.WardPhone)%>
            </p>
            <p>
                <label for="MDName">Referred MD:</label>
                <%= Html.TextBox("MDName",thisTransfer.MDName)%>
           </p>
            <p>
                <label for="MDPhone">MD Phone:</label>
                <%= Html.TextBox("MDPhone", thisTransfer.MDPhone)%>
            </p>
            <p>
                <label for="MDPager">MD Pager:</label>
                <%= Html.TextBox("MDPager", thisTransfer.MDPager)%>
            </p>
            <p>
                <label for="AdmitTime">Admit Time:</label>
                <%= Html.TextBox("AdmitTime", string.Format("{0:t}",thisTransfer.AdmitTime))%>
            </p>
            <p>
                <label for="CallerName">Caller Name:</label>
                <%=Html.TextBox("CallerName", thisTransfer.Caller_Name)%>
            </p>
            <p>
                <label for="CallerPhone">Caller Number:</label>
                <%= Html.TextBox("CallerPhone", thisTransfer.CallerNumber)%>
            </p>
            <p><label for="TransportElig">Eligibility:</label>
                <%= thisVeteran.TransportElig ? "Yes" : "No" %>
            </p>
            </div>

            <div class="sideBySideDiv">
            <p>
                <label for="CaseMgr">Case Mgr:</label>
                <%= Html.TextBox("CaseMgr", thisTransfer.CaseMgrName)%>
            </p>
            <p>
                <label for="MgrPhone">Mgr Phone:</label>
                <%= Html.TextBox("MgrPhone", thisTransfer.CaseMgrPhone)%>
            </p>      
            <p>
                <label for="CPRSDate">CPRS Date:</label>
                <%= Html.TextBox("CPRSDate", string.Format("{0:d}",thisTransfer.CPRSDate))%>
                <%= Html.ValidationMessage("CPRSDate", "*") %>
            </p> 
            <p>
                <label for="CPRSTime">CPRS Time:</label>
                <%= Html.TextBox("CPRSTime", string.Format("{0:t}",thisTransfer.CPRSTime))%>
                <%= Html.ValidationMessage("CPRSTime", "*") %>
            </p>    
            <p>
                <label for="TxPriority">TXFR Priority:</label>
                <%= Html.TextBox("TxPriority", thisTransfer.TXFRPriority)%>
                <%= Html.ValidationMessage("TxPriority", "*") %>
            </p>  
            <p>
                <label for="AcceptBy">Accepted By:</label>
                <%= Html.TextBox("PreferMD", thisTransfer.AcceptBy)%> 
                <%= Html.ValidationMessage("AcceptBy", "*") %>
            </p> 
            <p>
                <label for="OffListReason">Off List Reason:</label>
                <%= Html.DropDownList("OffListReason", Model.OffReasonTypes)%>
                <%= Html.ValidationMessage("OffListReason", "*") %>
            </p> 
            <p>
                <label for="InfectionType">Infection Type:</label>
                <%= Html.DropDownList("InfectionType", Model.InfectionTypes)%>
                <%= Html.ValidationMessage("InfectionType", "*") %>
            </p>                                                                     
            <p>
                <label for="COJ">COJ:</label>
                <% if (thisVeteran.VAHospital != null) %>
                <%= thisVeteran.VAHospital.VAHospital1 %>
            </p>           
            <p>
                <label for="FeePay">Fee Pay:</label>
                <%= Html.DropDownList("FeePay", Model.FeePayReasons) %>
                <%= Html.ValidationMessage("FeePay", "*") %>
            </p> 
            <p>
                <label for="Insurance">Insurance:</label>
                <%= thisVeteran.Insurance %>
            </p>
            <p>
                <label for="StatusReport">Status Report:</label>
                <%= Html.TextArea("StatusReport", thisTransfer.StatusReport)%>
                <%= Html.ValidationMessage("StatusReport", "*") %>
            </p>                                 
            </div>

            <div class="sideBySideDiv">
            <p>
                <label for="TxSource">Trnasfer Out Source:</label><br />
                <input type="radio" name="TxSource" id="radEd" title="ED"/>Ed
                <input type="radio" name="TxSource" id="radEdPsych" title="EdPsych"/>EdPsych
                <%= Html.ValidationMessage("TxOutSource", "*") %>
            </p>
            <p>
                <label for="InpatientUnit">Inpatient Unit:</label>
                <%=Html.TextBox("InpatientUnit", thisTransfer.InpatientUnit)%>
                <%= Html.ValidationMessage("InpatientUnit", "*") %>
            </p>   
            <p>
                <label for="CBOC">CBOC:</label>
                <%= Html.TextBox("CBOC", thisTransfer.CBOC)%>
                <%= Html.ValidationMessage("CBOC", "*") %>
            </p> 
            <p>
                <label for="CLC">CLC (ASIH):</label>
                <%= Html.TextBox("CLC_ASIH", thisTransfer.CLC_ASIH)%>
                <%= Html.ValidationMessage("CLC_ASIH", "*") %>
            </p>
            <p>
                <label for="TxReason">Treansfer Out Reason:</label><br />
                <input type="radio" id="radEDdivert" name="txReason" title="EDdivert" />Ed Divertc<br />
                <input type="radio" id="radTrauma" name="txReason" title="Trauma" />Trauma<br />
                <input type="radio" id="radOBGYN" name="txReason" title="OBGYN" />OBGYN<br />
                <input type="radio" id="radBedNA" name="txReason" title="Bed Type Not Available" />Bed Type Not Available<br />
                <input type="radio" id="radNoVet" name="txReason" title="Non_Veteran" />Non Veteran<br />
                <input type="radio" id="radNotEligible" name="txReason" title="Not_Eligible" />Not_Eligible<br />
                <input type="radio" id="radOtherServiceNA" name="txReason" title="Other Services Not Available" />Other Services Not Available<br />
                <input type="radio" id="radTxBackHome" name="txReason"title="Transfer Back to Home VA" />Transfer Back to Home VA<br />
                <%= Html.ValidationMessage("TxOutReason", "*") %>
            </p> 
            <p>
                <label for="OtherServiceNA">Service Not Available:</label>
                <%= Html.TextBox("OtherServiceNA", thisTransfer.OtherServicesNA)%>
                <%= Html.ValidationMessage("OtherServiceNA", "*") %>
            </p>
            <p>
                <label for="BackToHomeVA">TXFR to Home VA:</label>
                <%= Html.TextBox("BackToHomeVA",thisTransfer.BackToHomeVA) %>
                <%= Html.ValidationMessage("BackToHomeVA", "*") %>
            </p>   
            <p>
                <label for="TxID">Transfer Record ID:</label>
                <% %>
                <%= Html.ValidationMessage("TxID", "*") %>
            </p>                                                                                     
             <p>
                <label for="ListDate">Added to List Date:</label>
                <%= Html.TextBox("ListDate", string.Format("{0:d}", thisTransfer.AddedToListDate))%>
                <%= Html.ValidationMessage("ListDate", "*") %>
            </p>           
            </div>

        </fieldset>
        <p>
            <input type="submit" value="Save" />
        </p>
    <% } %><!-- close using Html.BeginForm()-->

Answer 1

Вы не привязываете значения обратно к модели, когда делаете

@Html.DropDownList("Hospital", Model.NonVaHospitals)
@Html.TextBox("TxPriority", thisTransfer.TXFRPriority)
@Html.TextArea("StatusReport", thisTransfer.StatusReport)

Вместо этого вы должны использовать это

@Html.DropDownListFor(model=>model.SelectedNonVaHospitals, Model.NonVaHospitals)
@Html.TextBoxFor(model=>model.TXFRPriority)
@Html.TextAreaFor(model=> model.StatusReport)

Answer 2

Мне кажется, у вас есть объект Transfer, у которого есть свойство Veteran, и это то, что вы пытаетесь редактировать? Поправь меня, если я все понял неправильно

Во-первых, я бы строго ввел ваши представления и элементы управления.

<%@ Page Title="" Language="C#" MasterPageFile="~/Views/Shared/Site.Master" 
    Inherits="System.Web.Mvc.ViewPage<Transfer>" %>

Я предполагаю, что тип Ветерана - Ветеран, это может быть человек или что-то еще, чего я не знаю. После этого вы можете удалить var Model.Transfer и т. Д.

<%@ Control Language="C#" Inherits="System.Web.Mvc.ViewUserControl<Veteran>" %> <% var thisTransfer = Model.Transfer; var thisVeteran = thisTransfer.Veteran; %>

Тогда я бы изменил вашу страницу редактирования на

<% Html.RenderPartial("TransferForm", Model.Veteran); %>

Поскольку вы привязались к объекту Transfer, но форма возвращает данные для объекта-ветерана, ваши имена в пользовательском элементе управления должны будут иметь имя

<%= Html.TextBox("collection.Veteran.AdmitDX", Model.AdmitDX)%>

Для правильного связывания я считаю, что вы можете использовать атрибут Bind(Prefix="") для переопределения префикса связующего, который по умолчанию смотрит на имя свойства в вашем методе действия, в данном случае «collection».

Вы также можете оставить имена без префикса «Ветеран» и вместо этого напрямую связать их с объектом «Ветеран».

[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Edit(int id,[Bind(Prefix="")]Veteran veteran)
{

Надеюсь, это поможет.

Answer 3

Для работы привязки модели по умолчанию имена переменных формы должны соответствовать именам свойств вашей модели Это может решить вашу проблему.