Commit 4ff64bce authored by Rafael Ostertag's avatar Rafael Ostertag
Browse files

Removed todo and wrote comment why.

parent b4e85b74
......@@ -428,11 +428,16 @@ if (!QfqNS) {
/**
*
* @todo Move this function to Element.FormGroup
* @param formControlName
* @param text
*/
n.QfqForm.prototype.setHelpBlockValidationMessage = function (formControlName, text) {
/*
* Why is this method here and not in FormGroup? Adding this particluar method to FormGroup is easy, however
* QfqForm.clearAllValidationStates() would not find its proper place in FormGroup, since FormGroup operates
* on one element. We would end up having the responsibilities spread across several classes, which would be
* confusing.
*/
var $formGroup = this.getFormGroupByControlName(formControlName);
if (!$formGroup) {
return;
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment