Change in ovirt-engine[master]: ui: fixing addHost form issues when playing with using host ...

oourfali at redhat.com oourfali at redhat.com
Thu Nov 20 08:40:21 UTC 2014


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 at 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 at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list