PDA

View Full Version : [2.0a1][SOLVED] Combo.js SVN r. 1079


JTM
10-03-2007, 08:23 AM
Good morning,

After updating to r. 1079 this morning, it looks like some errors were introduced:

From Combo.js:

if(this.mode == 'local'){
if(config.queryDelay === undefined){
this.queryDelay = 10;
}
if(config.minChars === undefined){
this.minChars = 0;
}
}

After the refactoring config is not defined.

mystix
10-03-2007, 11:01 AM
as per 13985, pls post your test case + other required details.

stever
10-03-2007, 02:47 PM
Jack started making the components only use initComponent() rather than the constructor (thanks for that! Tree and stuff yet to go!). The bug is a copy & paste error since the constructor had config as an argument, and initComponent does not. The base constructor applys the config to the object, so it is likely supposed to use "this" instead of "config" in those locations.

jsakalos
10-03-2007, 02:57 PM
Thank you for the bug report. We will investigate and fix it soon.

jack.slocum
10-04-2007, 04:29 AM
Thanks, fixed.

JTM
10-04-2007, 08:27 AM
Thanks, Jack