Alona Kaplan has posted comments on this change.

Change subject: webadmin: Network as Main Tab- (Phase 3)
......................................................................


Patch Set 4: (4 inline comments)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkClusterListModel.java
Line 18: 
Line 19: @SuppressWarnings("unused")
Line 20: public class NetworkClusterListModel extends ClusterListModel
Line 21: {
Line 22:     private UICommand privateManageCommand;
Will be fixed in a cleanup patch
Line 23: 
Line 24:     public UICommand getManageCommand()
Line 25:     {
Line 26:         return privateManageCommand;


Line 59:         networkManageModel.getCommands().add(0, okCommand);
Line 60: 
Line 61:     }
Line 62: 
Line 63: //    private ListModel createNetworkList(List<Network> dcNetworks) {
It was removed in the next patches. This patch is just the infrastructure.
Line 64: //        List<ClusterNetworkManageModel> networkList = new 
ArrayList<ClusterNetworkManageModel>();
Line 65: //        java.util.ArrayList<Network> clusterNetworks = 
Linq.<Network> Cast(getItems());
Line 66: //        for (Network network : dcNetworks) {
Line 67: //            ClusterNetworkManageModel networkManageModel = new 
ClusterNetworkManageModel(network);


Line 217:         }
Line 218: 
Line 219:         super.SyncSearch();
Line 220: 
Line 221:         AsyncQuery _asyncQuery = new AsyncQuery();
Will be fixed in a cleanup patch
Line 222:         _asyncQuery.setModel(this);
Line 223:         _asyncQuery.asyncCallback = new INewAsyncCallback() {
Line 224:             @Override
Line 225:             public void OnSuccess(Object model, Object ReturnValue)


....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkHostListModel.java
Line 60:         {
Line 61:             return;
Line 62:         }
Line 63: 
Line 64:         AsyncQuery _asyncQuery = new AsyncQuery();
Will be fixed in a cleanup patch
Line 65:         _asyncQuery.setModel(getViewFilterType());
Line 66:         _asyncQuery.asyncCallback = new INewAsyncCallback() {
Line 67:             @Override
Line 68:             public void OnSuccess(Object model, Object ReturnValue)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I538a4269315d4c3e005d6e4c0847d9d96d0f469e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Muli Salem <msa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to