Oved Ourfali has submitted this change and it was merged.
Change subject: ui: fixing addHost form issues when playing with using host provider
......................................................................
ui: fixing addHost form issues when playing with using host provider
Apparently hiding the host provider form was buggy. In this patch we fix
the fields cleaning and the visibility of the provider parameters.
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1165670
Change-Id: I700ec281a1bd1617d0168d59c8daef5b5bcf4f6b
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java
M
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java
3 files changed, 12 insertions(+), 18 deletions(-)
Approvals:
Yaniv Bronhaim: Verified
Oved Ourfali: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/35239
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I700ec281a1bd1617d0168d59c8daef5b5bcf4f6b
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server