Change in ovirt-engine[ovirt-engine-3.4]: core: Fixing I18N validation to allow empty strings

[iheim at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server