views:

666

answers:

2

I've tried many different ways to pass the selected items to the multiselect list with no luck. Finally, I tried this, which I think should display all the items as selected and still nothing in the list is selected.

public MultiSelectList Companies { get; private set; }

Companies = MulitSelectList(subcontractRepository.SubcontractCompanies(Subcontract.subcontract_id), "Value", "Text");

in SubcontractRepository.cs:

public IEnumerable<SelectListItem> SubcontractCompanies(Guid id)
{
     return c in db.companies
     select new SelectListItem
     {
          Text = c.company_name,
          Value = c.company_id.ToString(),
          Selected = true
     }
}

in View:

<p>
    <label for="Companies">Company:</label>
    <%= Html.ListBox("Companies", Model.Companies) %>
    <%= Html.ValidationMessage("Companies", "*") %>
</p>
A: 

MultiSelectList constructor has fourth parameter - selected items. Use it:

http://msdn.microsoft.com/en-us/library/system.web.mvc.multiselectlist.multiselectlist.aspx

Use this code:

public class SelectCompanyItem
{
    public string Name { get; set; }
    public Guid Id { get; set; }
}

public IEnumerable<SelectCompanyItem> SubcontractCompanies(Guid id)
{
     return c in db.companies
     select new SelectCompanyItem
     {
          Name = c.company_name,
          Id = c.company_id
     }
}

var companiesList = subcontractRepository.SubcontractCompanies(Subcontract.subcontract_id);
Companies = new MultiSelectList(companiesList , "Id", "Name", companiesList.Select(a => a.Id));

Does it work?

LukLed
That was the first way that I attempted to do it. I could not get that to work either. I was hoping that if I figured out what was wrong above, that I could discover what I was doing wrong.
RememberME
So you did something wrong. Don't convert `c.company_id` to string. It is not needed.
LukLed
I know I've done something wrong. I am attempting to learn what.When I do not have the .ToString() it complains "Cannot implicitly convert type 'System.Guid' to 'string'"
RememberME
@RememberME : This code works for me. I have ListBox with all items selected.
LukLed
No, nothing is selected for me.
RememberME
@RememberME : Please, show your view code.
LukLed
Added View code to original question
RememberME
@RememberME : View looks ok. Do you do the rest as I wrote in my response?
LukLed
Yes, I just double checked and everything is exactly like it is in your code.
RememberME
@LukLed - Thanks for the help. I discovered the issue. See my answer
RememberME
A: 

Discovered the issue here. The MultiSelectList must have a different name from the ListBox. Made that change and now both versions of the code work.

RememberME