Yaniv Bronhaim has uploaded a new change for review.

Change subject: webadmin: check explicitly if isDiscovered is equal to true
......................................................................

webadmin: check explicitly if isDiscovered is equal to true

Currently if isDiscovered entity is equal to  null we get exception which
blocks addHost at all.

Change-Id: Ide820aa4d51bfc92414d2515d8d1eeb0551a18ca
Signed-off-by: Yaniv Bronhaim <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/68/32268/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
index 102d418..9d6fc79 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
@@ -992,7 +992,7 @@
                 
parameters.setNetworkMappings(model.getInterfaceMappings().getEntity());
             }
 
-            if (model.getIsDiscorveredHosts().getEntity()) {
+            if 
(Boolean.TRUE.equals(model.getIsDiscorveredHosts().getEntity())) {
                 Provider<?> provider = (Provider<?>) 
model.getProviders().getSelectedItem();
                 ExternalHostGroup hostGroup = (ExternalHostGroup) 
model.getExternalHostGroups().getSelectedItem();
                 ExternalComputeResource computeResource = 
(ExternalComputeResource) model.getExternalComputeResource().getSelectedItem();


-- 
To view, visit http://gerrit.ovirt.org/32268
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide820aa4d51bfc92414d2515d8d1eeb0551a18ca
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to