Change in ovirt-engine[ovirt-engine-4.1]: webadmin: Add specialized validation for Long values

[Code Review]

>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Add specialized validation for Long values
......................................................................


webadmin: Add specialized validation for Long values

This patch adds a LongValidation class to provide validation of Long
values.

Previously, the EditQuotaClusterModel and EditQuotaStorageModel classes
used the IntegerValidation class to validate some Long type values,
which resulted in mistakes such as those detailed in
https://bugzilla.redhat.com/1423468.

Change-Id: Ieddd09396be5bdb113bec430afa71913715130db
Bug-Url: https://bugzilla.redhat.com/1423468
Signed-off-by: Tony Wang <Sline798 at gmail.com>
Signed-off-by: Phillip Bailey <phbailey at redhat.com>
---
A backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/LongCompat.java
A backend/manager/modules/compat/src/test/java/org/ovirt/engine/core/compat/LongCompatTest.java
M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Compat.gwt.xml
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/quota/EditQuotaClusterModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/quota/EditQuotaStorageModel.java
A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/validation/LongValidation.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java
M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIMessages.properties
8 files changed, 140 insertions(+), 6 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/73421
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieddd09396be5bdb113bec430afa71913715130db
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>