copying from one select list to another

[J P Nelson]

I'm trying to copy a selected element from one select list to another. The code below works fine in Firefox but throws an error in IE.

Can someone tell my why its failing in IE?

The error is: Object doesn't support this property or method.

The error occurs on the line where I get newValue.

The last part of the code sorts the selected list.

    
/*
 * This function will move the predeccessor element from the full list to
 * the selected list.
 */
function addPredecessor()
{
    var index = document.EditEntityForm.sourcePredEntities.selectedIndex;
    var count = document.EditEntityForm.selectedPredEntities.length;
    
    var newValue = 
        document.EditEntityForm.sourcePredEntities.options[index].value;
    
    document.EditEntityForm.selectedPredEntities.options[count] = 
        new Option(newValue, newValue, true, true);
        
    // populate array with options
    var sortArray = new Array();
    for(var ii = 0; ii < count + 1; ii++) 
    {
        sortArray[sortArray.length] = 
            document.EditEntityForm.selectedPredEntities.options[ii].value;
    }
    // sort array
    sortArray = sortArray.sort();
 
    // write sorted options
    for(var ii = 0; ii < sortArray.length; ii++) 
    {
        document.EditEntityForm.selectedPredEntities.options[ii] = 
            new Option(sortArray[ii], sortArray[ii], true, true);
    }
    
    dataChangedEdit();
            
    return;
}

[ May 17, 2006: Message edited by: J P Nelson ]

[J P Nelson]

OK, the problem is actually coming when I get the selected index. It comes back as -1.

[Eric Pascarello]

Is this a single select or multiple select. Are you referencing the right field? Do you have them switched on the page?

Eric

[J P Nelson]

Eric,

I just tested the solution. It turns out that IE and Firefox seem to process the onclick event in slightly different ways. In IE the selection hadn't been registered when the function was called, so there was no selection.

I changed the event to onchange and things seem happy now.

Thanks anyway.

Joe