Description Richard Kulp 2005-08-24 16:09:58 EDT

The following code in ComboCellEditor:

    void applyEditorValueAndDeactivate() {
        //	must set the selection before getting value
        selection = comboBox.getSelectionIndex();
        Object newValue = doGetValue();
        markDirty();
        boolean isValid = isCorrect(newValue);
        setValueValid(isValid);
        if (!isValid) {
            // try to insert the current value into the error message.
            setErrorMessage(MessageFormat.format(getErrorMessage(),
                    new Object[] { items[selection] }));
        }
        fireApplyEditorValue();
        deactivate();
    }

Has a problem in the MessageFormat.format(...) with items[selection]. If nothing
is selected then the selection is -1, but -1 is not valid for the array lookup
so it crashes and throws an ArrayIndexOutOfBounds exception and blows things up.