Question

I am trying to delete a record being displayed in a table on my ManageUser view using a sub-form, thus

<table cellpadding="2" cellspacing="0" border="1" summary="User Grid" style="text-align: left">
    <tr style="background-color: #ABC3CB;">
        <th align="center">User Name</th>
        <th align="center">Approved</th>
        <th align="center">&nbsp;</th>
        <th align="center">&nbsp;</th>
    </tr>
    <% foreach(MembershipUser membershipUser in ViewData.Model) { %>
        <tr>
            <td><%: membershipUser.UserName %></td>
            <td align="center"><%: Html.CheckBox(" ", true, membershipUser.IsApproved )  %></td>
            <td align="center">
                <% using (Html.BeginForm( "DeleteItem", "Admin", new { id = membershipUser.UserName } )) { %>
                    <input type="image" src="<%: Url.Content( "~/Content/Images/Delete.jpg" ) %>" />
                <% } %>
            </td>
        </tr>
    <% } %>
</table>

The ManageUser view is displayed by the following code in the AdminController, thus

    public ViewResult ManageUser( string searchType, string searchInput )
{
    List<SelectListItem> searchOptionList = new List<SelectListItem>()
    {
        new SelectListItem() {Value="UserName", Text = "UserName"},
        new SelectListItem() {Value="Email", Text = "Email"},
    };

    ViewData["searchOptionList"] = new SelectList( searchOptionList, "Value", "Text", searchType ?? "UserName" );
    ViewData["searchInput"] = searchInput ?? string.Empty;
    ViewData["searchType"] = searchType;

    MembershipUserCollection viewData;

    if (String.IsNullOrEmpty( searchInput ))
        viewData = Membership.GetAllUsers();
    else if (searchType == "Email")
        viewData = Membership.FindUsersByEmail( searchInput );
    else
        viewData = Membership.FindUsersByName( searchInput );

    ViewData["PageTitle"] = "Account Management";
    return View( viewData );
    }

When I display the page, and select the Delete choice, I expect it to run Admin/DeleteItem in the AdminController, thus

    public RedirectToRouteResult DeleteItem( string id )
{
    Membership.DeleteUser( id );
    return RedirectToAction( "ManageUser" );
}

but instead, it is returning directly to the Admin/ManagerUser view, thus displaying my original set of records again.

I have obviously missed something but I cannot see what. Anybody help?

Answer 1

HTML <form> elements cannot be nested. Nesting them results in unexpected behavior which could vary between different browsers. Quote from the specification:

Every form must be enclosed within a
FORM element. There can be several
forms in a single document, but the
FORM element can’t be nested.

So you might need to remove the outer form or find another way of organizing your markup.