Question

I have a registration view in my web app. However, account registration would be unsuccessful and my drop-down menu would be outlined in red.

Here is my drop down code:

<div class="editor-label">  
   @Html.LabelFor(model => model.CompanyId, "Company")  
</div>   
<div class="editor-field">  
   @Html.DropDownList("Companies", "<Select Company>")  
</div>

Here is my associated code in the view model:

[Display(Name = "Company")]    
public int? CompanyId { get; set; }

public IEnumerable<SelectListItem> Companies { get; set; }

Here is my associated code in the controller before I return the view:

ViewBag.Companies = new SelectList(db.Companies, "CompanyId", "CompanyName");

As I debug, I find that ModelState.IsValid returns false, and therefore re-returns the view (and re-sets ViewBag.Companies in my case).

How do I complete a successful registration? I do fill out each required field within parameters. It is not required to select company. When I do not select a company, ModelState.IsValid returns true and runs through the code. However, I would like the user to have the option to associate with his company.

Thanks.

Answer 1

Why would you need setting ViewBag.Companies, when you’ve got the better approach – having select list into the model? No need to use ugly ViewBag

You should have something like this

ViewModel

public class RegisterViewModel
{
     [Display(Name = "Company")]    
     public int? CompanyId { get; set; }

     public IEnumerable<SelectListItem> Companies { get; set; }
}

Controller

[HttpGet]
public ActionResult Register()
{
     RegisterViewModel viewModel = new RegisterViewModel();
     viewModel.Companies = new SelectList(db.Companies, "CompanyId", "CompanyName");

     return View(viewModel);
}

And in the view

<div class="editor-label">  
   @Html.LabelFor(model => model.CompanyId, "Company")  
</div>   
<div class="editor-field">  
   @Html.DropDownListFor(model => model.CompanyId, Model.Companies, "<Select Company>")  
</div>

This will bind your CompanyId on server correctly.