Gilad Chaplik has posted comments on this change.
Change subject: webadmin: Default host for RunOnce dialog enabled
......................................................................
Patch Set 1: (2 inline comments)
still reviewing
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java
Line 719: model.getRunAsStateless().setEntity(vm.isStateless());
Line 720: model.getRunAndPause().setEntity(false);
Line 721: model.setHwAcceleration(true);
Line 722: // disable Host tab
Line 723: model.setIsHostTabValid(false);
avail or valid?
Line 724: model.getIsAutoAssign().setEntity(true);
Line 725:
Line 726: fillIsoList(vm);
Line 727: fillFloppyImages(vm);
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
Line 1554:
Line 1555: // disable host tab when no active host is available
Line 1556:
model.setIsHostTabValid(!model.getDefaultHost().getIsEmpty());
Line 1557: }
Line 1558:
lately i get the impression that run once dialog == new vm dialog+run. we need
to think about it.
Line 1559: private void OnRunOnce()
Line 1560: {
Line 1561: VM vm = (VM) getSelectedItem();
Line 1562: if (vm == null)
--
To view, visit http://gerrit.ovirt.org/10654
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1906caf40c91e12a56523ddda794c72690947d4c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Libor Spevak <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
