Mike Kolesnik has posted comments on this change.

Change subject: webadmin: Ability to add external subnet
......................................................................


Patch Set 1:

(4 comments)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/NewExternalSubnetModel.java
Line 24: public class NewExternalSubnetModel extends Model {
Line 25: 
Line 26:     private EntityModel<String> name;
Line 27:     private EntityModel<String> cidr;
Line 28:     private ListModel<NetworkView> network;
Changed to EntityModel
Line 29:     private ListModel<IpVersion> ipVersion;
Line 30:     private final EntityModel sourceModel;
Line 31:     private ExternalSubnet subnet = null;
Line 32: 


Line 38:         setNetwork(new ListModel<NetworkView>());
Line 39:         setIpVersion(new ListModel<IpVersion>());
Line 40:         
getIpVersion().setItems(AsyncDataProvider.getExternalSubnetIpVerionList());
Line 41: 
Line 42:         
setTitle(ConstantsManager.getInstance().getConstants().externalSubnetTitle());
Done
Line 43:         setHashName("new_external_subnet"); //$NON-NLS-1$
Line 44: 
Line 45:         initCommands();
Line 46:     }


....................................................
File 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
Line 463:     @DefaultStringValue("VNIC Profiles")
Line 464:     String vnicProfilesTitle();
Line 465: 
Line 466:     @DefaultStringValue("External Subnet")
Line 467:     String externalSubnetTitle();
Done
Line 468: 
Line 469:     @DefaultStringValue("External Subnets")
Line 470:     String externalSubnetsTitle();
Line 471: 


....................................................
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ExternalSubnetPopupView.java
Line 51:     protected ListModelListBoxEditor<IpVersion> ipVersionEditor;
Line 52: 
Line 53:     @UiField(provided = true)
Line 54:     @Path("network.selectedItem")
Line 55:     ListModelListBoxEditor<Object> networkEditor;
Done
Line 56: 
Line 57:     private final Driver driver = GWT.create(Driver.class);
Line 58: 
Line 59:     @Inject


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7315f9ffe29f72d06deb56dcd637c53fba47b116
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to