Announcement

**Isomorphic** · 8 Jan 2011, 12:03

Can you show the stack trace from compiled mode, and some of the related code?

**jay.l.fisher** · 8 Jan 2011, 12:32

Unfortunately I have already reverted to the Jan 6 build and won't have time to retry until after next week. I'll try a new build at that point and see if it is still happening.

**Isomorphic** · 8 Jan 2011, 12:36

OK.. however note that this crash is as a result of refactoring to address ComboBoxItem limitations you wanted addressed, and we've got a large library of autotests that are all passing with the refactored code, so we really will need more information on this before we can finalize the new implementation.

**jay.l.fisher** · 8 Jan 2011, 13:29

In that case I've reinstalled the 1/8 build and taken another look. The error does not show up in compiled mode. I've traced it down a little more and it is happening after the ChangedHandler fires on the comboBox. Here is the code that creates the ComboBoxItem and adds the changed handler.

Code:

ComboBoxItem hven = getCodeNameComboBoxItem("HVEN", "VVEN", "VNAM", "IPMRVEN", false);
hven.addChangedHandler(new ChangedHandler() {
	@Override
	public void onChanged(ChangedEvent event) {
		setVendorDefaults();
	}
});

The setVendorDefaults() method executes without error and only afterwards does the js error appear. This is the code for the getCodeNameComboBoxItem() method.

Code:

private ComboBoxItem getCodeNameComboBoxItem(String field, final String idField, final String nameField, final String dsName, boolean addUnknownValues) {
	ComboBoxItem mySelector = new ComboBoxItem(field);
	mySelector.setOptionDataSource(DataSource.get(dsName));
	mySelector.setValueField(idField);
	mySelector.setDisplayField(nameField);
	ListGrid selectorGrid = new ListGrid();
	mySelector.setPickListFields(
			new ListGridField(idField),
			new ListGridField(nameField)
	);
	mySelector.setPickListProperties(selectorGrid);
	mySelector.setPickListWidth(200);
	mySelector.setSortField(idField);
	//		mySelector.setCompleteOnTab(true);
	//		mySelector.setAddUnknownValues(addUnknownValues);
	return mySelector;
}

**Isomorphic** · 10 Jan 2011, 11:08

OK, if you can isolate this to a test case, that's the quickest and easiest approach. Otherwise:

1. can you show setVendorDefaults()?

2. are you saying that if you remove the ChangedHandler entirely there's no error?

3. can you remove any specific part of what setVendorDefaults() does and cure the error?

Announcement

JavaScriptException in latest nightly

Comment

Comment

Comment

Comment

Comment