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

From Tal Nisan <tnisan@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@gmail.com> Signed-off-by: Phillip Bailey <phbailey@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@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review