Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

refactor remove onvalidate from TextInput and Select

  • Loading branch information...
commit 49a4f992632f692a3da3acb463fdef0975554aae 1 parent 9e76e14
@jakub-g authored
View
4 src/aria/widgets/CfgBeans.js
@@ -254,10 +254,6 @@ Aria.beanDefinitions({
$description : "Contains the groups to validate."
}
},
- "onvalidate" : {
- $type : "common:Callback",
- $description : "Function to be called for custom validation."
- },
"validationEvent" : {
$type : "json:Enum",
$enumValues : ["onFocus", "onBlur", "none"],
View
16 src/aria/widgets/form/Select.js
@@ -201,9 +201,6 @@ Aria.classDefinition({
aria.utils.Data.validateValue(dataholder, name, null, groups, 'onblur');
}
}
- this._onvalidate({
- popup : false
- });
}
}
this._updateValue(true);
@@ -225,19 +222,6 @@ Aria.classDefinition({
this._updateState();
},
- _onvalidate : function (arg) {
- if (this._cfg.onvalidate) {
- this.$logWarn(this.WIDGET_PROPERTY_DEPRECATION, ["onvalidate"]);
- }
- this.evalCallback(this._cfg.onvalidate);
- if (arg.popup && (this._cfg.error)) {// show errortip onfocus
- this._validationPopupShow();
- } else { // dispose of error tip
- this._validationPopupHide();
-
- }
- },
-
/**
* Internal method to handle the mousedown event
* @protected
View
21 src/aria/widgets/form/TextInput.js
@@ -789,24 +789,6 @@ Aria.classDefinition({
},
/**
- * The method called when calling custom validation
- * @param {Object} arg is Popup needed
- * @private
- */
- _onvalidate : function (arg) {
- if (this._cfg.onvalidate) {
- this.$logWarn(this.WIDGET_PROPERTY_DEPRECATION, ["onvalidate"]);
- this.evalCallback(this._cfg.onvalidate);
- }
-
- if (arg.popup && (this._cfg.error)) {
- this._validationPopupShow();
- } else {
- this._validationPopupHide();
- }
- },
-
- /**
* Internal method to handle the onkeydown event.
* @param {DOMEvent or aria.DomEvent} e event object (on W3C browsers) or wrapper on it
* @protected
@@ -958,9 +940,6 @@ Aria.classDefinition({
}
}
}
- this._onvalidate({
- popup : false
- });
}
}
Please sign in to comment.
Something went wrong with that request. Please try again.