Yaniv Bronhaim has uploaded a new change for review. Change subject: webadmin: Fixing validation of Auth Ssh Port ......................................................................
webadmin: Fixing validation of Auth Ssh Port The field's validation was set but without using the getIsValid as part of the general tab validation. Clicking OK forwarded the user to PM page instead of failing the operation. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=987910 Change-Id: Ie5bc99cfeaebd2c76a09ac2255557051626acef2 Signed-off-by: Yaniv Bronhaim <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/59/18359/1 diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java index be944e1..5ed0624 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java @@ -1582,7 +1582,6 @@ new LengthValidation(255), new HostAddressValidation() }); - getPort().validateEntity(new IValidation[] {new NotEmptyValidation(), new IntegerValidation(1, 65535)}); getAuthSshPort().validateEntity(new IValidation[] {new NotEmptyValidation(), new IntegerValidation(1, 65535)}); if ((Boolean) getConsoleAddressEnabled().getEntity()) { @@ -1623,8 +1622,9 @@ setIsGeneralTabValid(getName().getIsValid() && getComment().getIsValid() && getHost().getIsValid() - && getPort().getIsValid() - && getCluster().getIsValid()); + && getAuthSshPort().getIsValid() + && getCluster().getIsValid() + ); setIsPowerManagementTabValid(getManagementIp().getIsValid() && getPmUserName().getIsValid() -- To view, visit http://gerrit.ovirt.org/18359 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie5bc99cfeaebd2c76a09ac2255557051626acef2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
