@openui5/sap.ui.integration 1.103.1 → 1.104.0

package/src/sap/ui/integration/editor/Editor.js

@@ -144,7 +144,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @constructor
 	 * @since 1.94
 	 * @private
@@ -892,7 +892,7 @@ sap.ui.define([
 		 * @experimental since 1.94
 		 * @public
 		 * @author SAP SE
-		 * @version 1.103.1
+		 * @version 1.104.0
 		 * @borrows sap.ui.integration.editor.Editor#getParameters as getParameters
 		 * @borrows sap.ui.integration.editor.Editor#resolveDestination as resolveDestination
 		 * @borrows sap.ui.integration.editor.Editor#request as request
@@ -1161,7 +1161,10 @@ sap.ui.define([
 			this._oDestinations.setHost(oHostInstance);
 		} else {
 			var sConfigurationPath = this.getConfigurationPath();
-			this._oDestinations = new Destinations(oHostInstance, this._manifestModel.getProperty(sConfigurationPath + "/destinations"));
+			this._oDestinations = new Destinations({
+				host: oHostInstance,
+				manifestConfig: this._manifestModel.getProperty(sConfigurationPath + "/destinations")
+			});
 		}
 	};
 
@@ -1491,8 +1494,22 @@ sap.ui.define([
 									break;
 								case "object":
 								case "object[]":
-									if (oItem.value && oItem.value !== "") {
-										mResult[oItem.manifestpath] = oItem.value;
+									if (Array.isArray(oItem.value)) {
+										var aValue = deepClone(oItem.value, 500);
+										// sort the value list according by the position value
+										aValue = aValue.sort(function (a, b) {
+											// if _position property not exists, do nothing
+											if (!a._dt || !a._dt._position || !b._dt || !b._dt._position) {
+												return 0;
+											}
+											return a._dt._position - b._dt._position;
+										});
+										// recount the position value
+										for (var i = 0; i < aValue.length; i++) {
+											var oValue = aValue[i];
+											oValue._dt._position = i + 1;
+										}
+										mResult[oItem.manifestpath] = aValue;
 									}
 									break;
 								default:
@@ -2043,7 +2060,6 @@ sap.ui.define([
 		}
 		oField._oDataProviderFactory = this._oDataProviderFactory;
 		oField.setAssociation("_messageStrip", MessageStripId);
-		oField._previewPosition = this.getPreviewPosition();
 		return oField;
 	};
 
@@ -3223,26 +3239,28 @@ sap.ui.define([
 			this._addItem(oItem);
 		}
 		// customize the size of card editor, define the size in dt.js
-		var cardHeight = this._settingsModel.getProperty("/form/height") !== undefined ? this._settingsModel.getProperty("/form/height") : "350px",
-		cardWidth = this._settingsModel.getProperty("/form/width") !== undefined ? this._settingsModel.getProperty("/form/width") : "100%";
+		var editorHeight = this._settingsModel.getProperty("/form/height") !== undefined ? this._settingsModel.getProperty("/form/height") : "350px",
+		editorWidth = this._settingsModel.getProperty("/form/width") !== undefined ? this._settingsModel.getProperty("/form/width") : "100%";
 		if (this.getProperty("height") === "") {
-			this.setProperty("height", cardHeight);
-			document.body.style.setProperty("--sapUiIntegrationEditorFormHeight", cardHeight);
+			this.setProperty("height", editorHeight);
+			document.body.style.setProperty("--sapUiIntegrationEditorFormHeight", editorHeight);
 		}
 		if (this.getProperty("width") === "") {
-			this.setProperty("width", cardWidth);
-			document.body.style.setProperty("--sapUiIntegrationEditorFormWidth", cardWidth);
+			this.setProperty("width", editorWidth);
+			document.body.style.setProperty("--sapUiIntegrationEditorFormWidth", editorWidth);
 		}
 		//add additional content
 		if (this.getMode() !== "translation") {
-			Promise.resolve(this._initPreview()).then(function() {
+			this._initPreview().then(function() {
 				Promise.all(this._aFieldReadyPromise).then(function () {
+					this.setPreviewPositionForFields();
 					this._ready = true;
 					this.fireReady();
 				}.bind(this));
 			}.bind(this));
 		} else {
 			Promise.all(this._aFieldReadyPromise).then(function () {
+				this.setPreviewPositionForFields();
 				this._ready = true;
 				this.fireReady();
 			}.bind(this));
@@ -3263,6 +3281,14 @@ sap.ui.define([
 		}
 	};
 
+	Editor.prototype.setPreviewPositionForFields = function() {
+		var sPosition = this.hasPreview() ? this.getPreviewPosition() : "";
+		var oSettings = this._settingsModel.getData();
+		oSettings.preview = oSettings.preview || {};
+		oSettings.preview.position = sPosition;
+		this._settingsModel.setData(oSettings);
+	};
+
 	/**
 	 * Destroy the editor and the internal instances that it created
 	 */
@@ -3474,7 +3500,7 @@ sap.ui.define([
 			var oItems = oSettings.form.items,
 				oHost = this.getHostInstance();
 			Object.keys(oConfiguration.destinations).forEach(function (n) {
-				oItems[n + ".destinaton"] = merge({
+				oItems[n + ".destination"] = merge({
 					manifestpath: sBasePath + "/" + n + "/name", //destination points to name not value
 					visible: true,
 					type: "destination",
@@ -3483,15 +3509,15 @@ sap.ui.define([
 					allowSettings: false,
 					value: oConfiguration.destinations[n].name,
 					defaultValue: oConfiguration.destinations[n].defaultUrl,
-					_settingspath: "/form/items/" + [n + ".destinaton"],
+					_settingspath: "/form/items/" + [n + ".destination"],
 					_values: [],
 					_destinationName: n
 				}, oConfiguration.destinations[n]);
-				if (typeof oItems[n + ".destinaton"].label === "undefined") {
-					oItems[n + ".destinaton"].label = n;
+				if (typeof oItems[n + ".destination"].label === "undefined") {
+					oItems[n + ".destination"].label = n;
 				}
 				if (oHost) {
-					oItems[n + ".destinaton"]._loading = true;
+					oItems[n + ".destination"]._loading = true;
 				}
 			});
 			var getDestinationsDone = false;
@@ -3499,8 +3525,8 @@ sap.ui.define([
 				this.getHostInstance().getDestinations().then(function (a) {
 					getDestinationsDone = true;
 					Object.keys(oConfiguration.destinations).forEach(function (n) {
-						oItems[n + ".destinaton"]._values = a;
-						oItems[n + ".destinaton"]._loading = false;
+						oItems[n + ".destination"]._values = a;
+						oItems[n + ".destination"]._loading = false;
 						this._settingsModel.checkUpdate(true);
 					}.bind(this));
 				}.bind(this)).catch(function () {
@@ -3511,13 +3537,13 @@ sap.ui.define([
 						return;
 					}
 					Object.keys(oConfiguration.destinations).forEach(function (n) {
-						oItems[n + ".destinaton"]._values = b;
-						oItems[n + ".destinaton"]._loading = false;
+						oItems[n + ".destination"]._values = b;
+						oItems[n + ".destination"]._loading = false;
 						this._settingsModel.checkUpdate(true);
 					}.bind(this));
 				}.bind(this)).catch(function (e) {
 					Object.keys(oConfiguration.destinations).forEach(function (n) {
-						oItems[n + ".destinaton"]._loading = false;
+						oItems[n + ".destination"]._loading = false;
 						this._settingsModel.checkUpdate(true);
 					}.bind(this));
 					Log.error("Can not get destinations list from '" + oHost.getId() + "'.");

package/src/sap/ui/integration/editor/EditorResourceBundles.js

@@ -16,7 +16,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.EditorResourceBundles
 	 * @author SAP SE
 	 * @since 1.94.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.94.0
 	 * @ui5-restricted

package/src/sap/ui/integration/editor/Extension.js

@@ -4,12 +4,10 @@
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
-	"sap/ui/integration/library",
-	"sap/base/Log",
-	"sap/ui/base/ManagedObject"
-], function (library,
-			 Log,
-			 ManagedObject) {
+	"sap/ui/integration/Extension",
+	"sap/base/Log"
+], function (BaseExtension,
+			 Log) {
 	"use strict";
 
 	/**
@@ -21,10 +19,10 @@ sap.ui.define([
 	 * @class
 	 * Brings JavaScript capabilities for an {@link sap.ui.integration.editor.Editor} where custom logic can be implemented.
 	 *
-	 * @extends sap.ui.base.ManagedObject
+	 * @extends sap.ui.integration.Extension
 	 *
 	 * @author SAP SE
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 *
 	 * @constructor
 	 * @public
@@ -32,27 +30,16 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.Extension
 	 * @ui5-metamodel This control/element will also be described in the UI5 (legacy) designtime metamodel
 	 */
-	var Extension = ManagedObject.extend("sap.ui.integration.editor.Extension", {
-		metadata: {
-			library: "sap.ui.integration",
-			properties: {
-				/**
-				 * The formatters, which can be used in the manifest.
-				 * @experimental since 1.94
-				 */
-				formatters: {
-					type: "object"
-				}
-			}
-		}
-	});
+	var Extension = BaseExtension.extend("sap.ui.integration.editor.Extension");
 
 	Extension.prototype.init = function () {
+		BaseExtension.prototype.init.apply(this, arguments);
 		this._oEditorInterface = null;
 		this._oEditor = null;
 	};
 
 	Extension.prototype.exit = function () {
+		BaseExtension.prototype.exit.apply(this, arguments);
 		this._oEditorInterface = null;
 		this._oEditor = null;
 	};
@@ -61,8 +48,7 @@ sap.ui.define([
 	 * See generated JSDoc
 	 */
 	Extension.prototype.setFormatters = function (aFormatters) {
-		this.setProperty("formatters", aFormatters);
-
+		BaseExtension.prototype.setFormatters.apply(this, arguments);
 		if (!this._oEditor) {
 			return;
 		}

package/src/sap/ui/integration/editor/Manifest.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @extends sap.ui.integration.util.Manifest
 	 *
 	 * @author SAP SE
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 *
 	 * @constructor
 	 * @since 1.94

package/src/sap/ui/integration/editor/Settings.js

@@ -66,7 +66,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.Settings
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted
@@ -131,7 +131,7 @@ sap.ui.define([
 			var iOffsetHeight = (!oPreview || oPreview.getDomRef() === null || oPreview.getDomRef().offsetHeight === 0) ? 350 : oPreview.getDomRef().offsetHeight;
 			oPopover.setContentWidth(iOffsetWidth + "px");
 			oPopover.setContentHeight((iOffsetHeight - 50) + "px");
-			if (oPreview && oPreview.getSettings().preview.position === "right") {
+			if (oParent.getPreviewPosition() === "right") {
 				oPopover.setPlacement("Right");
 			} else {
 				oPopover.setPlacement("Left");

package/src/sap/ui/integration/editor/fields/BaseField.js

@@ -34,7 +34,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.BaseField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @ui5-restricted
 	 * @experimental since 1.83.0
@@ -697,7 +697,7 @@ sap.ui.define([
 					this._triggerValidation(value);
 				}.bind(this));
 			}*/
-			var oBinding = this.getModel("currentSettings").bindProperty("value", this.getBindingContext("currentSettings"));
+			var oBinding = this._settingsModel.bindProperty("value", this.getBindingContext("currentSettings"));
 			oBinding.attachChange(function () {
 				this._triggerValidation(oConfig.value);
 			}.bind(this));
@@ -803,12 +803,12 @@ sap.ui.define([
 	BaseField.prototype._setCurrentProperty = function (sProperty, vValue) {
 		//avoid fire binding changes in the model
 		if (this._getCurrentProperty(sProperty) !== vValue) {
-			this.getModel("currentSettings").setProperty(sProperty, vValue, this.getBindingContext("currentSettings"));
+			this._settingsModel.setProperty(sProperty, vValue, this.getBindingContext("currentSettings"));
 		}
 	};
 
 	BaseField.prototype._getCurrentProperty = function (sProperty) {
-		return this.getModel("currentSettings").getProperty(sProperty, this.getBindingContext("currentSettings"));
+		return this._settingsModel.getProperty(sProperty, this.getBindingContext("currentSettings"));
 	};
 
 	BaseField.prototype._applySettings = function (oData) {
@@ -912,5 +912,9 @@ sap.ui.define([
 		return oItem;
 	};
 
+	BaseField.prototype.getPreviewPosition = function () {
+		return this._settingsModel.getProperty("/preview/position");
+	};
+
 	return BaseField;
 });

package/src/sap/ui/integration/editor/fields/BooleanField.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.BooleanField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted

package/src/sap/ui/integration/editor/fields/DateField.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.DateField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted

package/src/sap/ui/integration/editor/fields/DateTimeField.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.DateTimeField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted

package/src/sap/ui/integration/editor/fields/DestinationField.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.DestinationField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted
@@ -46,7 +46,6 @@ sap.ui.define([
 							text: "{currentSettings>name}",
 							key: "{currentSettings>name}"
 						})
-
 					}
 				}
 			};

package/src/sap/ui/integration/editor/fields/IntegerField.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.IntegerField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted

package/src/sap/ui/integration/editor/fields/NumberField.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.NumberField
 	 * @author SAP SE
 	 * @since 1.83.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.83.0
 	 * @ui5-restricted

package/src/sap/ui/integration/editor/fields/ObjectField.js

@@ -84,7 +84,7 @@ sap.ui.define([
 	 * @alias sap.ui.integration.editor.fields.ObjectField
 	 * @author SAP SE
 	 * @since 1.100.0
-	 * @version 1.103.1
+	 * @version 1.104.0
 	 * @private
 	 * @experimental since 1.100.0
 	 * @ui5-restricted
@@ -179,7 +179,6 @@ sap.ui.define([
 		var oVisualization = {
 			type: Table,
 			settings: {
-				rows: "{" + sPath + "}",
 				visibleRowCount: 5,
 				busy: "{currentSettings>_loading}",
 				columns: columns,
@@ -189,6 +188,12 @@ sap.ui.define([
 				filter: that.onFilter.bind(that)
 			}
 		};
+		if (oConfig.type === "object") {
+			oVisualization.settings.rows = "{" + sPath + "}";
+		} else {
+			// for object list parameter, allow sort
+			oVisualization.settings.rows = "{path: '" + sPath + "', sorter: {path: '_dt/_position', descending: false}}";
+		}
 		return oVisualization;
 	};
 
@@ -353,6 +358,8 @@ sap.ui.define([
 			var oSelectionColumn = new Column({
 				width: "3.2rem",
 				hAlign: "Center",
+				// hide selection column for object list field with properties defined only
+				visible: typeof oConfig.values === "undefined" ? false : true,
 				multiLabels: [
 					oSelectionColumnLabels
 				],
@@ -527,50 +534,66 @@ sap.ui.define([
 		if (bAddButtonVisible && oConfig.values) {
 			bAddButtonVisible = oConfig.values.allowAdd === true;
 		}
-		return new OverflowToolbar({
-			content: [
-				new ToolbarSpacer(),
-				new Button({
-					icon: "sap-icon://add",
-					visible: bAddButtonVisible,
-					tooltip: sAddButtonTooltip,
-					press: that.addNewObject.bind(that)
-				}),
-				new Button({
-					icon: "sap-icon://edit",
-					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_EDIT_TOOLTIP"),
-					enabled: "{= !!${/_hasTableSelected}}",
-					press: that.onEditOrViewDetail.bind(that)
-				}),
+		var oContents = [
+			new ToolbarSpacer(),
+			new Button({
+				icon: "sap-icon://add",
+				visible: bAddButtonVisible,
+				tooltip: sAddButtonTooltip,
+				press: that.addNewObject.bind(that)
+			}),
+			new Button({
+				icon: "sap-icon://edit",
+				tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_EDIT_TOOLTIP"),
+				enabled: "{= !!${/_hasTableSelected}}",
+				press: that.onEditOrViewDetail.bind(that)
+			}),
+			new Button({
+				icon: "sap-icon://delete",
+				tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_DELETE"),
+				enabled: "{= !!${/_canDelete}}",
+				press: that.onDelete.bind(that)
+			}),
+			new Button({
+				icon: "sap-icon://clear-filter",
+				visible: bHasFilterDefined,
+				enabled: "{= !!${/_hasFilter}}",
+				tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_CLEAR_ALL_FILTERS_TOOLTIP"),
+				press: that.clearAllFilters.bind(that)
+			}),
+			new Button({
+				icon: "sap-icon://multiselect-all",
+				visible: false,
+				enabled: "{= !${/_hasTableAllSelected}}",
+				tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_SELECT_ALL_SELETIONS_TOOLTIP"),
+				press: that.selectAllTableSelections.bind(that)
+			}),
+			new Button({
+				icon: "sap-icon://multiselect-none",
+				visible: false,
+				enabled: "{= !!${/_hasTableSelected}}",
+				tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_CLEAR_ALL_SELETIONS_TOOLTIP"),
+				press: that.clearAllTableSelections.bind(that)
+			})
+		];
+		if (oConfig.type === "object[]") {
+			oContents = oContents.concat([
 				new Button({
-					icon: "sap-icon://delete",
-					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_DELETE"),
-					enabled: "{= !!${/_canDelete}}",
-					press: that.onDelete.bind(that)
-				}),
-				new Icon({
-					src: "sap-icon://vertical-grip"
-				}),
-				new Button({
-					icon: "sap-icon://clear-filter",
-					visible: bHasFilterDefined,
-					enabled: "{= !!${/_hasFilter}}",
-					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_CLEAR_ALL_FILTERS_TOOLTIP"),
-					press: that.clearAllFilters.bind(that)
-				}),
-				new Button({
-					icon: "sap-icon://multiselect-all",
-					enabled: "{= !${/_hasTableAllSelected}}",
-					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_SELECT_ALL_SELETIONS_TOOLTIP"),
-					press: that.selectAllTableSelections.bind(that)
+					icon: "sap-icon://navigation-up-arrow",
+					enabled: "{= !!${/_hasOnlyOneRowSelected}}",
+					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_MOVE_UP_TOOLTIP"),
+					press: that.moveRowUp.bind(that)
 				}),
 				new Button({
-					icon: "sap-icon://multiselect-none",
-					enabled: "{= !!${/_hasTableSelected}}",
-					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_CLEAR_ALL_SELETIONS_TOOLTIP"),
-					press: that.clearAllTableSelections.bind(that)
+					icon: "sap-icon://navigation-down-arrow",
+					enabled: "{= !!${/_hasOnlyOneRowSelected}}",
+					tooltip: oResourceBundle.getText("EDITOR_FIELD_OBJECT_TABLE_BUTTON_MOVE_DOWN_TOOLTIP"),
+					press: that.moveRowDown.bind(that)
 				})
-			]
+			]);
+		}
+		return new OverflowToolbar({
+			content: oContents
 		});
 	};
 
@@ -683,8 +706,14 @@ sap.ui.define([
 		var aSelectedIndices = oTable.getSelectedIndices();
 		if (aSelectedIndices.length > 0) {
 			oModel.setProperty("/_hasTableSelected", true);
+			if (aSelectedIndices.length === 1) {
+				oModel.setProperty("/_hasOnlyOneRowSelected", true);
+			} else {
+				oModel.setProperty("/_hasOnlyOneRowSelected", false);
+			}
 		} else {
 			oModel.setProperty("/_hasTableSelected", false);
+			oModel.setProperty("/_hasOnlyOneRowSelected", false);
 			oModel.setProperty("/_canDelete", false);
 			return;
 		}
@@ -722,10 +751,46 @@ sap.ui.define([
 		var oModel = oTable.getModel();
 		oTable.clearSelection();
 		oModel.setProperty("/_hasTableSelected", false);
+		oModel.setProperty("/_hasOnlyOneRowSelected", false);
 		oModel.setProperty("/_canDelete", false);
 		oModel.setProperty("/_hasTableAllSelected", false);
 	};
 
+	ObjectField.prototype.moveRowUp = function (oEvent) {
+		this.moveSelectedRow("Up");
+	};
+
+	ObjectField.prototype.moveRowDown = function (oEvent) {
+		this.moveSelectedRow("Down");
+	};
+
+	ObjectField.prototype.moveSelectedRow = function (sDirection) {
+		var that = this;
+		var oTable = that.getAggregation("_field");
+		var oModel = oTable.getModel();
+		var iSelectedRowIndex = oTable.getSelectedIndex();
+		var oSelectedRowContext = oTable.getContextByIndex(iSelectedRowIndex);
+
+		var iSiblingRowIndex = iSelectedRowIndex + (sDirection === "Up" ? -1 : 1);
+		var oSiblingRowContext = oTable.getContextByIndex(iSiblingRowIndex);
+		if (!oSiblingRowContext) {
+			return;
+		}
+
+		// swap the selected and the siblings rank
+		var iSiblingRowRank = oSiblingRowContext.getProperty("_dt/_position");
+		var iSelectedRowRank = oSelectedRowContext.getProperty("_dt/_position");
+		oModel.setProperty("_dt/_position", iSiblingRowRank, oSelectedRowContext);
+		oModel.setProperty("_dt/_position", iSelectedRowRank, oSiblingRowContext);
+		oModel.refresh(true);
+
+		// after move select the sibling
+		oTable.setSelectedIndex(iSiblingRowIndex);
+
+		// save change
+		that.refreshValue();
+	};
+
 	ObjectField.prototype.onSelectionChange = function (oEvent) {
 		var that = this;
 		var oControl = oEvent.getSource();
@@ -1200,7 +1265,7 @@ sap.ui.define([
 		var oResourceBundle = that.getResourceBundle();
 		var oItemCloned = deepClone(oItem, 500);
 		var oModel;
-		var sPlacement = sMode === "add" && that._previewPosition !== "right" ? "Left" : "Right";
+		var sPlacement = sMode === "add" && that.getPreviewPosition() !== "right" ? "Left" : "Right";
 		if (!that._oObjectDetailsPopover) {
 			var oAddButton = new Button({
 				text: oResourceBundle.getText("EDITOR_FIELD_OBJECT_DETAILS_POPOVER_BUTTON_ADD"),
@@ -1626,7 +1691,7 @@ sap.ui.define([
 		if (!that._oTranslationPopover) {
 			var oList = that.buildTranslationsList();
 			var oTranslationsFooter = that.buildTranslationsFooter(oList, true);
-			var sPlacement = that._previewPosition === "right" ? "Right" : "Left";
+			var sPlacement = that.getPreviewPosition() === "right" ? "Right" : "Left";
 			that._oTranslationPopover = new Popover({
 				placement: sPlacement,
 				contentWidth: "300px",
@@ -1685,6 +1750,7 @@ sap.ui.define([
 		oModel.setProperty("/_hasSelected", true);
 		oModel.setProperty("/_hasTableAllSelected", false);
 		oModel.setProperty("/_hasTableSelected", false);
+		oModel.setProperty("/_hasOnlyOneRowSelected", false);
 		oModel.checkUpdate();
 		that.refreshValue();
 		that.updateTable();