Arik Hadas has uploaded a new change for review. Change subject: webadmin: Allow to change version name of template ......................................................................
webadmin: Allow to change version name of template It is now possible to change the name of a template sub-version from the template's edit dialog. Note that it is only allowed for sub-versions, for base versions the version name is "base version" and the text box is disabled. Change-Id: I3c33a8b87f5df7e3dbc23bfcb97cc839b0b6b7bb Bug-Url: https://bugzilla.redhat.com/1058501 Signed-off-by: Arik Hadas <[email protected]> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmClonePopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmPopupWidget.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/builders/template/VersionNameUnitToVmBaseBuilder.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/TemplateVmModelBehavior.java 9 files changed, 33 insertions(+), 6 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/69/25769/1 diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java index f82dc0e..11b290b 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java @@ -158,6 +158,11 @@ public StringEntityModelTextBoxOnlyEditor nameEditor; @UiField(provided = true) + @Path(value = "templateVersionName.entity") + @WithElementId("templateVersionName") + public StringEntityModelTextBoxEditor templateVersionNameEditor; + + @UiField(provided = true) @Ignore public InfoIcon poolNameIcon; @@ -767,6 +772,7 @@ } private void initTextBoxEditors() { + templateVersionNameEditor = new StringEntityModelTextBoxEditor(new ModeSwitchingVisibilityRenderer()); descriptionEditor = new StringEntityModelTextBoxEditor(new ModeSwitchingVisibilityRenderer()); commentEditor = new StringEntityModelTextBoxEditor(new ModeSwitchingVisibilityRenderer()); totalvCPUsEditor = new StringEntityModelTextBoxOnlyEditor(new ModeSwitchingVisibilityRenderer()); @@ -1052,6 +1058,7 @@ dataCenterWithClusterEditor.setLabel(constants.hostClusterVmPopup()); quotaEditor.setLabel(constants.quotaVmPopup()); nameLabel.setText(constants.nameVmPopup()); + templateVersionNameEditor.setLabel(constants.templateVersionName()); descriptionEditor.setLabel(constants.descriptionVmPopup()); commentEditor.setLabel(constants.commentLabel()); baseTemplateEditor.setLabel(constants.basedOnTemplateVmPopup()); @@ -1489,6 +1496,7 @@ templateEditor.setTabIndex(nextTabIndex++); nameEditor.setTabIndex(nextTabIndex++); + templateVersionNameEditor.setTabIndex(nextTabIndex++); descriptionEditor.setTabIndex(nextTabIndex++); commentEditor.setTabIndex(nextTabIndex++); isStatelessEditor.setTabIndex(nextTabIndex++); diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml index 6f90af0..f96d2dd 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml @@ -236,7 +236,7 @@ width: 250px; } - .prestartedLabel, .nameLabel { + .prestartedLabel, .nameLabel, .templateVersionNameLabel { float: left; color: #333333; } @@ -374,6 +374,7 @@ </g:FlowPanel> <ge:EntityModelTextBoxOnlyEditor ui:field="nameEditor" addStyleNames="{style.nameEditor}" contentWidgetStyleName="{style.nameEditorContent}"/> </g:FlowPanel> + <ge:StringEntityModelTextBoxEditor ui:field="templateVersionNameEditor" /> <ge:StringEntityModelTextBoxEditor ui:field="descriptionEditor" /> <ge:StringEntityModelTextBoxEditor ui:field="commentEditor" /> diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java index d90ad5e..3b5f2fb 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java @@ -119,7 +119,8 @@ update(newPoolEditMaxAssignedVmsPerUserPanel, hiddenField()). update(editPoolEditVmsPanel, simpleField()). update(editPoolIncraseNumOfVmsPanel, simpleField()). - update(editPoolEditMaxAssignedVmsPerUserPanel, simpleField()); + update(editPoolEditMaxAssignedVmsPerUserPanel, simpleField()). + update(templateVersionNameEditor, hiddenField()); } } diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java index 3422905..4b30264 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java @@ -90,7 +90,8 @@ putOne(editPoolEditVmsPanel, hiddenField()). putOne(editPoolIncraseNumOfVmsPanel, hiddenField()). putOne(logicalNetworksEditorPanel, hiddenField()). - putOne(editPoolEditMaxAssignedVmsPerUserPanel, hiddenField()); + putOne(editPoolEditMaxAssignedVmsPerUserPanel, hiddenField()). + update(templateVersionNameEditor, hiddenField()); } } diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmClonePopupWidget.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmClonePopupWidget.java index d55e684..662f62c 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmClonePopupWidget.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmClonePopupWidget.java @@ -35,7 +35,7 @@ return super.createWidgetConfiguration(). putAll(poolSpecificFields(), hiddenField()). putOne(logicalNetworksEditorPanel, hiddenField()). - update(consoleTab, simpleField().visibleInAdvancedModeOnly()); - + update(consoleTab, simpleField().visibleInAdvancedModeOnly()). + update(templateVersionNameEditor, hiddenField()); } } diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmPopupWidget.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmPopupWidget.java index f1744d4..5345732 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmPopupWidget.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmPopupWidget.java @@ -46,6 +46,7 @@ @Override protected PopupWidgetConfigMap createWidgetConfiguration() { return super.createWidgetConfiguration(). - putAll(poolSpecificFields(), hiddenField()); + putAll(poolSpecificFields(), hiddenField()). + update(templateVersionNameEditor, hiddenField()); } } diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/builders/template/VersionNameUnitToVmBaseBuilder.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/builders/template/VersionNameUnitToVmBaseBuilder.java new file mode 100644 index 0000000..97433ba --- /dev/null +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/builders/template/VersionNameUnitToVmBaseBuilder.java @@ -0,0 +1,12 @@ +package org.ovirt.engine.ui.uicommonweb.builders.template; + +import org.ovirt.engine.core.common.businessentities.VmTemplate; +import org.ovirt.engine.ui.uicommonweb.builders.BaseSyncBuilder; +import org.ovirt.engine.ui.uicommonweb.models.vms.UnitVmModel; + +public class VersionNameUnitToVmBaseBuilder extends BaseSyncBuilder<UnitVmModel, VmTemplate> { + @Override + protected void build(UnitVmModel source, VmTemplate destination) { + destination.setTemplateVersionName(source.getTemplateVersionName().getEntity()); + } +} diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java index 904e4e3..dde05d7 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java @@ -513,6 +513,7 @@ template.setCreatedByUserId(selectedItem.getCreatedByUserId()); template.setSingleQxlPci(model.getIsSingleQxlEnabled().getEntity()); template.setMigrationDowntime(model.getSelectedMigrationDowntime()); + template.setTemplateVersionName(model.getTemplateVersionName().getEntity()); if (model.getQuota().getIsAvailable() && model.getQuota().getSelectedItem() != null) { template.setQuotaId(model.getQuota().getSelectedItem().getId()); diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/TemplateVmModelBehavior.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/TemplateVmModelBehavior.java index 385bf3d..238dbe5 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/TemplateVmModelBehavior.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/TemplateVmModelBehavior.java @@ -37,6 +37,7 @@ getModel().getStorageDomain().setIsChangable(false); getModel().getIsSoundcardEnabled().setIsChangable(true); getModel().getVmType().setIsChangable(true); + getModel().getTemplateVersionName().setIsChangable(!template.getId().equals(template.getBaseTemplateId())); if (template.getStoragePoolId() != null && !template.getStoragePoolId().equals(Guid.Empty)) { @@ -216,6 +217,7 @@ updateConsoleDevice(template.getId()); getModel().getVmInitEnabled().setEntity(template.getVmInit() != null); getModel().getVmInitModel().init(template); + getModel().getTemplateVersionName().setEntity(template.getTemplateVersionName()); initPriority(template.getPriority()); } -- To view, visit http://gerrit.ovirt.org/25769 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3c33a8b87f5df7e3dbc23bfcb97cc839b0b6b7bb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Arik Hadas <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
