Change in ovirt-engine[master]: webadmin: Simplify mapping in Setup Networks initialization

lvernia at redhat.com lvernia at redhat.com
Wed Jan 15 08:33:26 UTC 2014


Lior Vernia has submitted this change and it was merged.

Change subject: webadmin: Simplify mapping in Setup Networks initialization
......................................................................


webadmin: Simplify mapping in Setup Networks initialization

Some mapping was performed in two steps, first from one entity's name
to another entity's name, and then from the second entity's name to
the entity itself, so I removed the intermediary phase. Another map
was actually used as a collection, so changed it to a list.

Change-Id: Ia29636b437262301996a459f017d2260680e708f
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java
1 file changed, 31 insertions(+), 43 deletions(-)

Approvals:
  Lior Vernia: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia29636b437262301996a459f017d2260680e708f
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list