Tomas Jelinek has posted comments on this change.
Change subject: userportal,webadmin: unify new server/desktop to newVm
......................................................................
Patch Set 5: (11 inline comments)
....................................................
File
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmClonePopupWidget.java
Line 37
Line 38
Line 39
Line 40
Line 41
no. there is no hard difference between server/desktop anymore. You can have HA
desktop or stateless server with 4 monitors.
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
Line 608: new StoragePoolQueryParametersBase(dataCenterId),
Line 609: aQuery);
Line 610: }
Line 611:
Line 612: public static void isSoundcardEnabled(AsyncQuery aQuery, Guid
vmId) {
it is. Previously the only way how to configure this was to have serve or
desktop. Since we don't have this anymore, need to configure this explicitly.
Line 613: aQuery.converterCallback = new IAsyncConverter() {
Line 614: @Override
Line 615: public Object Convert(Object source, AsyncQuery
_asyncQuery) {
Line 616: if (source != null) {
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateGeneralModel.java
Line 449
Line 450
Line 451
Line 452
Line 453
good point - will replace it with
setIsHighlyAvailable(template.isAutoStartup());
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewTemplateVmModelBehavior.java
Line 76:
model.setDataCentersAndClusters(model,
Line 77:
Arrays.asList(dataCenter),
Line 78: clusters,
Line 79:
vm.getVdsGroupId().getValue());
Line 80:
AsyncDataProvider.isSoundcardEnabled(new AsyncQuery(getModel(),
- same - it is
- ok, I can make it parallel to getClusterListByService
Line 81: new
INewAsyncCallback() {
Line 82:
Line 83: @Override
Line 84: public void
onSuccess(Object model, Object returnValue) {
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
Line 1274: else if
(ev.matchesDefinition(ListModel.selectedItemChangedEventDefinition))
Line 1275: {
Line 1276: if (sender == getVmType()) {
Line 1277: vmTypeChanged();
Line 1278: } else if (sender == getDataCenterWithClustersList())
what is wrong with the format? In new code I try to use the java formatting and
occasionally change the old one.
Line 1279: {
Line 1280: dataCenterWithClusterSelectedItemChanged(sender,
args);
Line 1281: initUsbPolicy();
Line 1282: }
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGeneralModel.java
Line 603:
Line 604: translator = EnumTranslator.Create(OriginType.class);
Line 605: setOrigin(translator.get(vm.getOrigin()));
Line 606:
Line 607: setHasHighlyAvailable(true);
ok, will remove the flag
Line 608: setIsHighlyAvailable(vm.isAutoStartup());
Line 609:
Line 610: setHasPriority(true);
Line 611:
setPriority(AsyncDataProvider.priorityToString(vm.getPriority()));
Line 606:
Line 607: setHasHighlyAvailable(true);
Line 608: setIsHighlyAvailable(vm.isAutoStartup());
Line 609:
Line 610: setHasPriority(true);
ok, will remove the flag
Line 611:
setPriority(AsyncDataProvider.priorityToString(vm.getPriority()));
Line 612:
Line 613: setHasMonitorCount(true);
Line 614: setMonitorCount(vm.getNumOfMonitors());
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
Line 393:
Line 394: consoleModelsCache = new ConsoleModelsCache(this);
Line 395: setConsoleHelpers();
Line 396:
Line 397: setNewVmCommand(new UICommand("NewVM", this)); //$NON-NLS-1$
oops :)
Line 398: setNewVmCommand(new UICommand("NewVm", this)); //$NON-NLS-1$
Line 399: setEditCommand(new UICommand("Edit", this)); //$NON-NLS-1$
Line 400: setRemoveCommand(new UICommand("Remove", this)); //$NON-NLS-1$
Line 401: setRunCommand(new UICommand("Run", this, true)); //$NON-NLS-1$
Line 699:
Line 700: UnitVmModel model = new UnitVmModel(new NewVmModelBehavior());
Line 701: model.setTitle(ConstantsManager.getInstance()
Line 702: .getMessages()
Line 703:
.newVmTitle(ConstantsManager.getInstance().getConstants().vm()));
yes, will do that
Line 704: model.setHashName("new_vm"); //$NON-NLS-1$
Line 705: model.setIsNew(true);
Line 706: model.getVmType().setSelectedItem(VmType.Server);
Line 707:
model.setCustomPropertiesKeysList(getCustomPropertiesKeysList());
Line 770: model.setVmAttachedToPool(vm.getVmPoolId() != null);
Line 771: setWindow(model);
Line 772: model.setTitle(ConstantsManager.getInstance()
Line 773: .getMessages()
Line 774:
.editVmTitle(ConstantsManager.getInstance().getConstants().vm()));
yes, will do that
Line 775: model.setHashName("edit_vm"); //$NON-NLS-1$
Line 776:
model.setCustomPropertiesKeysList(getCustomPropertiesKeysList());
Line 777:
Line 778: model.initialize(this.getSystemTreeSelectedItem());
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
Line 1310:
Line 1311: @DefaultStringValue("Uptime")
Line 1312: String uptimeVm();
Line 1313:
Line 1314: @DefaultStringValue("New Vm")
ok
Line 1315: String newVm();
Line 1316:
Line 1317: @DefaultStringValue("Edit")
Line 1318: String editVm();
--
To view, visit http://gerrit.ovirt.org/15048
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibcb7065e2968a84fef30951e8a45019ea1095520
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
