PDA

View Full Version : [FIXED][3.0] r3589 regression



stever
7 Apr 2009, 9:59 PM
Assuming nothing else says otherwise, validateValue should return true. This bug came in changeset 3589 and took me a while to figure out!!!

mjlecomte
8 Apr 2009, 4:37 AM
Looks like it could use a line at the end:


validateValue : function(value){
if(Ext.isFunction(this.validator)){
var msg = this.validator(value);
if(msg !== true){
this.markInvalid(msg);
return false;
}
}
...
if(value.length > this.maxLength){
...
return false;
}
return true
},

aconran
8 Apr 2009, 7:10 AM
Thanks, Fixed in revision 3613