Itamar Heim has submitted this change and it was merged.
Change subject: core: Fixing I18N validation to allow empty strings
......................................................................
core: Fixing I18N validation to allow empty strings
The I18N validation in the validation framework was set to pass validation
an empty string, this behavior is incorrect since the I18N validation should
only care for extisting characters within the string and since an empty
string contains no illegal characters, validation should pass, for
validating a strings length the size validation should be used.
The regular expression was changed to pass validation on an empty string
and the places which used this validation without minimum string length
validation were changed to validate a minimum string length of 1 in order
not to have regressions.
This solves a validation problem when creating a VM disk via REST and
not specifying an alias, validation should not fail in that case and an
alias should be generated by the add disk command
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1065727
Change-Id: I6d6cf47a0ccaf1d8af3d46e5dee9c264e9a29bb3
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmTemplateParameters.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Tags.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/ValidationUtils.java
3 files changed, 3 insertions(+), 3 deletions(-)
Approvals:
Tal Nisan: Verified
Maor Lipchuk: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/24652
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6d6cf47a0ccaf1d8af3d46e5dee9c264e9a29bb3
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server