Lior Vernia has uploaded a new change for review. Change subject: webadmin: Findbugs fix ......................................................................
webadmin: Findbugs fix Findbugs justfully point out that a field was used before it was initialized. Change-Id: Ic0be1bfbfbbad8db328296a93069f2b1ed679f1d Signed-off-by: Lior Vernia <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/18/16718/1 diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java index b3a35b0..f4f19c3 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java @@ -18,12 +18,13 @@ } public void discoverNetworks() { + getProviders().setItems(Collections.singletonList(provider)); getProviders().setSelectedItem(provider); } @Override protected void initProviderList() { - getProviders().setItems(Collections.singletonList(provider)); + // do nothing, already have the only provider to be displayed in the list } } -- To view, visit http://gerrit.ovirt.org/16718 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic0be1bfbfbbad8db328296a93069f2b1ed679f1d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
