Lior Vernia has posted comments on this change.
Change subject: webadmin: Network Provider main tab
......................................................................
Patch Set 18: (10 inline comments)
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/CommonModel.java
Line 813: list.add(networkList);
Line 814:
Line 815: providerList = new ProviderListModel();
Line 816: list.add(providerList);
Line 817:
Added support in succeeding patch.
Line 818: setItems(list);
Line 819:
Line 820: roleListModel = new RoleListModel();
Line 821: systemPermissionListModel = new SystemPermissionListModel();
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderGeneralModel.java
Line 19: protected void onEntityChanged() {
Line 20: super.onEntityChanged();
Line 21:
Line 22: if (super.getEntity() != null)
Line 23: {
Both fixed in succeeding clean-up patch.
Line 24: updateProperties();
Line 25: }
Line 26: }
Line 27:
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderListModel.java
Line 20: import org.ovirt.engine.ui.uicompat.ObservableCollection;
Line 21:
Line 22: public class ProviderListModel extends ListWithDetailsModel implements
ISupportSystemTreeContext
Line 23: {
Line 24: private UICommand addCommand;
Same, don't see any point in changing.
Line 25: private UICommand removeCommand;
Line 26:
Line 27: private SystemTreeItemModel systemTreeSelectedItem;
Line 28:
Line 89: protected void asyncSearch() {
Line 90: // super.AsyncSearch();
Line 91: //
Line 92: // setAsyncResult(Frontend.RegisterSearch(getSearchString(),
SearchType.Provider, getSearchPageSize()));
Line 93: // setItems(getAsyncResult().getData());
Removed in succeeding patch as part of clean-up.
Line 94: }
Line 95:
Line 96: @Override
Line 97: protected void onSelectedItemChanged() {
Line 117: public void executeCommand(UICommand command) {
Line 118: super.executeCommand(command);
Line 119:
Line 120: if (command == getAddCommand())
Line 121: {
This is accepted by checkstyle, but fixed the formatting in succeeding patch as
well.
Line 122: add();
Line 123: }
Line 124: else if (command == getRemoveCommand())
Line 125: {
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/ProviderModule.java
Line 34: UICommand lastExecutedCommand, Model windowModel) {
Line 35:
Line 36: // if (lastExecutedCommand ==
getModel().getAddCommand()) {
Line 37: // return editNetworkPopupProvider.get();
Line 38: // } else {
Left here to remind us of work to be done, removed in succeeding patch.
Line 39: return super.getModelPopup(source,
lastExecutedCommand, windowModel);
Line 40: // }
Line 41: }
Line 42:
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabProviderView.java
Line 59: @Override
Line 60: public String getValue(Provider object) {
Line 61: return object.getUrl();
Line 62: }
Line 63: };
Done
Line 64:
Line 65: getTable().addColumn(urlColumn, constants.urlProvider());
Line 66:
Line 67: getTable().addActionButton(new
WebAdminButtonDefinition<Provider>(constants.addProvider()) {
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/provider/SubTabProviderGeneralView.java
Line 28: ViewUiBinder uiBinder = GWT.create(ViewUiBinder.class);
Line 29: }
Line 30:
Line 31: interface Driver extends
SimpleBeanEditorDriver<ProviderGeneralModel, SubTabProviderGeneralView> {
Line 32: Driver driver = GWT.create(Driver.class);
Done
Line 33: }
Line 34:
Line 35: private final ApplicationConstants constants =
ClientGinjectorProvider.instance().getApplicationConstants();
Line 36:
Line 48: // Init formPanel
Line 49: formPanel = new GeneralFormPanel();
Line 50:
Line 51: initWidget(ViewUiBinder.uiBinder.createAndBindUi(this));
Line 52: Driver.driver.initialize(this);
Done
Line 53:
Line 54: // Build a form using the FormBuilder
Line 55: formBuilder = new FormBuilder(formPanel, 1, 1);
Line 56:
Line 58: }
Line 59:
Line 60: @Override
Line 61: public void setMainTabSelectedItem(Provider selectedItem) {
Line 62: Driver.driver.edit(getDetailModel());
Done
Line 63: formBuilder.update(getDetailModel());
Line 64: }
Line 65:
--
To view, visit http://gerrit.ovirt.org/11114
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ide66dd42695293f66d1a0c6ca4a067fff5e81a1d
Gerrit-PatchSet: 18
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
