Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Hide hosted engine tab when editing host

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Wed Jul 27 12:19:08 UTC 2016


Piotr Kliczewski has submitted this change and it was merged.

Change subject: webadmin: Hide hosted engine tab when editing host
......................................................................


webadmin: Hide hosted engine tab when editing host

The hosted engine tab in host editing or adding dialog
in the ui is only relevant when adding a new host.
This patch hides the tab when a host is edited.

To achieve this the initialization of the popup window
was moved after the host model initialization.
This allows setting the visibility for hostedEngineTab in
HostPopupView by using the getIsNew method (was already
implemented but didn't work since the relevant fields
weren't initialized).

Change-Id: I4746dda9fa0a06f0dec1d179639a20e5941e8ec9
Bug-Url: https://bugzilla.redhat.com/1350423
Signed-off-by: Jenny Tokar <jtokar at redhat.com>
(cherry picked from commit bb20845)
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, approved
  Jenny Tokar: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4746dda9fa0a06f0dec1d179639a20e5941e8ec9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list