@openui5/sap.ui.core 1.100.0 → 1.102.1

package/src/sap/ui/model/odata/v4/ODataListBinding.js

@@ -58,7 +58,7 @@ sap.ui.define([
 		 * @mixes sap.ui.model.odata.v4.ODataParentBinding
 		 * @public
 		 * @since 1.37.0
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 * @borrows sap.ui.model.odata.v4.ODataBinding#getGroupId as #getGroupId
 		 * @borrows sap.ui.model.odata.v4.ODataBinding#getRootBinding as #getRootBinding
 		 * @borrows sap.ui.model.odata.v4.ODataBinding#getUpdateGroupId as #getUpdateGroupId
@@ -143,6 +143,7 @@ sap.ui.define([
 		// A kept-alive context may be parked here for a longer time, with undefined index.
 		this.mPreviousContextsByPath = {};
 		this.aPreviousData = [];
+		this.bRefreshKeptElements = false; // refresh kept elements when resuming?
 		this.bSharedRequest = mParameters.$$sharedRequest || oModel.bSharedRequests;
 		this.aSorters = _Helper.toArray(vSorters);
 		this.sUpdateGroupId = mParameters.$$updateGroupId;
@@ -795,8 +796,7 @@ sap.ui.define([
 	 *   contexts within this list.
 	 * @param {boolean} [bInactive]
 	 *   Create an inactive context. Such a context will only be sent to the server after the first
-	 *   property update. From then on it behaves like any other created context. This parameter is
-	 *   experimental and its implementation may still change. Do not use it in productive code yet.
+	 *   property update. From then on it behaves like any other created context.
 	 *   Supported since 1.97.0
 	 *   <p>
 	 *   Since 1.98.0, when the first property updates happens, the context is no longer
@@ -845,7 +845,7 @@ sap.ui.define([
 			throw new Error(
 				"Must know the final length to create at the end. Consider setting $count");
 		}
-		if (this.bCreatedAtEnd && !bAtEnd) {
+		if (this.bFirstCreateAtEnd && !bAtEnd) {
 			throw new Error("Cannot create at the start after creation at end");
 		}
 		if (bInactive) {
@@ -857,12 +857,17 @@ sap.ui.define([
 			this.iActiveContexts += 1;
 		}
 
-		if (this.bCreatedAtEnd === undefined) {
-			this.bCreatedAtEnd = bAtEnd;
+		if (this.bFirstCreateAtEnd === undefined) {
+			this.bFirstCreateAtEnd = bAtEnd;
 		}
 
 		// only for createInCache
 		oGroupLock = this.lockGroup(sGroupId, true, true, function () {
+			if (!that.aContexts.includes(oContext)) { // #setContext changed the parent context
+				oContext.destroy();
+				return;
+			}
+
 			that.destroyCreated(oContext);
 			return Promise.resolve().then(function () {
 				// Fire the change asynchronously so that Cache#delete is finished and #getContexts
@@ -871,7 +876,7 @@ sap.ui.define([
 			});
 		});
 		oCreatePromise = this.createInCache(oGroupLock, oCreatePathPromise, sResolvedPath,
-			sTransientPredicate, oInitialData, this.bCreatedAtEnd !== bAtEnd,
+			sTransientPredicate, oInitialData, this.bFirstCreateAtEnd !== bAtEnd,
 			function (oError) { // error callback
 				that.oModel.reportError("POST on '" + oCreatePathPromise
 					+ "' failed; will be repeated automatically", sClassName, oError);
@@ -881,7 +886,7 @@ sap.ui.define([
 			function () { // submit callback
 				that.fireEvent("createSent", {context : oContext});
 			}
-		).then(function (oCreatedEntity) {
+		).then(function (oCreatedEntity) { // the entity was created on the server
 			var sGroupId, sPredicate;
 
 			if (!(oInitialData && oInitialData["@$ui5.keepTransientPath"])) {
@@ -898,7 +903,7 @@ sap.ui.define([
 				return oContext.refreshDependentBindings(oContext.getPath().slice(1), sGroupId,
 					/*bCheckUpdate*/true);
 			}
-			if (!that.oModel.isDirectGroup(sGroupId) && !that.oModel.isAutoGroup(sGroupId)) {
+			if (that.oModel.isApiGroup(sGroupId)) {
 				sGroupId = "$auto";
 			}
 
@@ -911,8 +916,15 @@ sap.ui.define([
 		this.iCreatedContexts += 1;
 		oContext = Context.create(this.oModel, this, sTransientPath, -this.iCreatedContexts,
 			oCreatePromise, bInactive);
+		// to make sure that #fetchValue does not overtake #createInCache, avoid bCached flag!
+		oContext.fetchValue().then(function (oElement) {
+			if (oElement) {
+				_Helper.setPrivateAnnotation(oElement, "context", oContext);
+				_Helper.setPrivateAnnotation(oElement, "firstCreateAtEnd", that.bFirstCreateAtEnd);
+			} // else: context already destroyed
+		});
 
-		if (this.bCreatedAtEnd !== bAtEnd) {
+		if (this.bFirstCreateAtEnd !== bAtEnd) {
 			this.aContexts.splice(this.iCreatedContexts - 1, 0, oContext);
 			for (i = this.iCreatedContexts - 1; i >= 0; i -= 1) {
 				this.aContexts[i].iIndex = i - this.iCreatedContexts;
@@ -985,8 +997,9 @@ sap.ui.define([
 				sPredicate = _Helper.getPrivateAnnotation(aResults[i], "predicate");
 				sContextPath = sPath + (sPredicate || "/" + i$skipIndex);
 				oContext = this.mPreviousContextsByPath[sContextPath];
-				if (oContext && !oContext.created()) {
-					// reuse the previous context, unless it is created (and persisted)
+				if (oContext && (!oContext.created() || oContext.isKeepAlive())) {
+					// reuse the previous context, unless it is created (and persisted), but not
+					// kept alive
 					delete this.mPreviousContextsByPath[sContextPath];
 					oContext.iIndex = i$skipIndex;
 					oContext.checkUpdate();
@@ -1083,7 +1096,7 @@ sap.ui.define([
 			this.aContexts[i].iIndex += 1;
 		}
 		if (!this.iCreatedContexts) {
-			this.bCreatedAtEnd = undefined;
+			this.bFirstCreateAtEnd = undefined;
 		}
 		this.aContexts.splice(iIndex, 1);
 		this.destroyLater(oContext);
@@ -1116,7 +1129,8 @@ sap.ui.define([
 	 *
 	 * @param {string[]} [aPathsToDelete]
 	 *   If given, only contexts with paths in this list except kept-alive ones are removed and
-	 *   destroyed; otherwise all contexts in the list are removed and destroyed
+	 *   destroyed (transient contexts are removed only); otherwise all contexts in the list are
+	 *   removed and destroyed
 	 *
 	 * @private
 	 */
@@ -1131,7 +1145,9 @@ sap.ui.define([
 					if (aPathsToDelete && oContext.isKeepAlive()) {
 						oContext.iIndex = undefined;
 					} else {
-						oContext.destroy();
+						if (!oContext.isTransient()) {
+							oContext.destroy();
+						}
 						delete mPreviousContextsByPath[sPath];
 					}
 				}
@@ -1144,10 +1160,9 @@ sap.ui.define([
 	 * @see sap.ui.model.odata.v4.ODataBinding#doCreateCache
 	 */
 	ODataListBinding.prototype.doCreateCache = function (sResourcePath, mQueryOptions, oContext,
-			sDeepResourcePath, bKeepCreated, oOldCache) {
+			sDeepResourcePath, sGroupId, oOldCache) {
 		var sBindingPath,
 			aKeptElementPaths,
-			oCache,
 			that = this;
 
 		if (oOldCache && oOldCache.getResourcePath() === sResourcePath
@@ -1157,10 +1172,10 @@ sap.ui.define([
 				return that.mPreviousContextsByPath[sPath].isKeepAlive();
 			});
 
-			if (this.iCreatedContexts || !bKeepCreated && aKeptElementPaths.length) {
+			if (this.iCreatedContexts || aKeptElementPaths.length) {
 				oOldCache.reset(aKeptElementPaths.map(function (sPath) {
 					return _Helper.getRelativePath(sPath, sBindingPath);
-				}), bKeepCreated);
+				}), sGroupId);
 				// Note: #inheritQueryOptions as called below should not matter in case of own
 				// requests, which are a precondition for kept-alive elements
 				oOldCache.setQueryOptions(mQueryOptions, true);
@@ -1170,21 +1185,11 @@ sap.ui.define([
 		}
 
 		mQueryOptions = this.inheritQueryOptions(mQueryOptions, oContext);
-		oCache = this.getCacheAndMoveKeepAliveContexts(sResourcePath, mQueryOptions)
+
+		return this.getCacheAndMoveKeepAliveContexts(sResourcePath, mQueryOptions)
 			|| _AggregationCache.create(this.oModel.oRequestor, sResourcePath, sDeepResourcePath,
 				this.mParameters.$$aggregation, mQueryOptions, this.oModel.bAutoExpandSelect,
 				this.bSharedRequest);
-
-		if (aKeptElementPaths && aKeptElementPaths.length) {
-			oCache.setLateQueryOptions(oOldCache.getLateQueryOptions());
-			aKeptElementPaths.forEach(function (sPath) {
-				oCache.addKeptElement(
-					oOldCache.getValue(_Helper.getRelativePath(sPath, sBindingPath)));
-				that.mPreviousContextsByPath[sPath].checkUpdate(); // add change listeners
-			});
-		}
-
-		return oCache;
 	};
 
 	/**
@@ -1359,7 +1364,7 @@ sap.ui.define([
 		var oPromise,
 			that = this;
 
-		if (this.bCreatedAtEnd) {
+		if (this.bFirstCreateAtEnd) {
 			// Note: We still have to read iLength rows in this case to get all entities from
 			// the server. The created entities then are placed behind using the calculated or
 			// estimated length.
@@ -1815,7 +1820,7 @@ sap.ui.define([
 
 		this.withCache(function (oCache, sPath) {
 				aElements = oCache.getAllElements(sPath);
-			}, "", /*bSync=*/true);
+			}, "", /*bSync*/true);
 
 		if (this.createContexts(0, aElements)) {
 			// In the case that a control has requested new data and the data request is already
@@ -1826,7 +1831,9 @@ sap.ui.define([
 
 		return this.aContexts.filter(function (oContext) {
 			return oContext;
-		}).concat(Object.values(this.mPreviousContextsByPath));
+		}).concat(Object.values(this.mPreviousContextsByPath).filter(function (oContext) {
+			return oContext.isKeepAlive();
+		}));
 	};
 
 	/**
@@ -2018,7 +2025,7 @@ sap.ui.define([
 		if (!this.oDiff) { // w/o E.C.D there won't be a diff
 			// make sure "refresh" is followed by async "change"
 			oPromise = this.fetchContexts(iStart, iLength, iMaximumPrefetchSize, oGroupLock,
-				/*bAsync=*/bRefreshEvent, function () {
+				/*bAsync*/bRefreshEvent, function () {
 					bDataRequested = true;
 					that.fireDataRequested();
 				});
@@ -2089,7 +2096,7 @@ sap.ui.define([
 	ODataListBinding.prototype.getContextsInViewOrder = function (iStart, iLength) {
 		var aContexts, iCount, i;
 
-		if (this.bCreatedAtEnd) {
+		if (this.bFirstCreateAtEnd) {
 			aContexts = [];
 			iCount = Math.min(iLength, this.getLength() - iStart);
 			for (i = 0; i < iCount; i += 1) {
@@ -2155,6 +2162,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @see sap.ui.model.ListBinding#getCurrentContexts
+	 * @see #getAllCurrentContexts
 	 * @since 1.39.0
 	 */
 	// @override sap.ui.model.ListBinding#getCurrentContexts
@@ -2360,7 +2368,7 @@ sap.ui.define([
 	 * @private
 	 */
 	ODataListBinding.prototype.getModelIndex = function (iViewIndex) {
-		if (!this.bCreatedAtEnd) {
+		if (!this.bFirstCreateAtEnd) {
 			return iViewIndex;
 		}
 		if (!this.bLengthFinal) { // created at end, but the read is pending and $count unknown yet
@@ -2384,14 +2392,14 @@ sap.ui.define([
 	 * @param {string} [sGroupId]
 	 *   The group ID used for read requests for the context's entity or its properties. If not
 	 *   given, the binding's {@link #getGroupId group ID} is used. Supported since 1.100.0
-	 * @returns {sap.ui.model.odata.v4.Context|undefined}
-	 *   The kept-alive context, or <code>undefined</code> if the binding's root binding is
-	 *   suspended and no context with the given path exists
+	 * @returns {sap.ui.model.odata.v4.Context}
+	 *   The kept-alive context
 	 * @throws {Error} If
 	 *   <ul>
 	 *     <li> the group ID is invalid,
 	 *     <li> the binding is unresolved,
 	 *     <li> the given context path does not match this binding,
+	 *     <li> the binding's root binding is suspended,
 	 *     <li> or {@link sap.ui.model.odata.v4.Context#setKeepAlive} fails
 	 *   </ul>
 	 *
@@ -2407,14 +2415,12 @@ sap.ui.define([
 			iPredicateIndex = this.oModel.getPredicateIndex(sPath),
 			sResolvedPath = this.getResolvedPath();
 
+		this.checkSuspended();
 		this.oModel.checkGroupId(sGroupId);
 		if (!oContext) {
 			if (!sResolvedPath) {
 				throw new Error("Binding is unresolved: " + this);
 			}
-			if (this.isRootBindingSuspended()) {
-				return undefined;
-			}
 			if (sPath.slice(0, iPredicateIndex) !== sResolvedPath) {
 				throw new Error(this + ": Not a valid context path: " + sPath);
 			}
@@ -2554,18 +2560,6 @@ sap.ui.define([
 	};
 
 	/**
-	 * Check whether this binding is an active $$getKeepAliveContext binding for the given path.
-	 *
-	 * @param {string} sPath - An absolute binding path
-	 * @returns {boolean} - Whether this binding matches
-	 *
-	 * @private
-	 */
-	ODataListBinding.prototype.isKeepAliveBindingFor = function (sPath) {
-		return this.mParameters.$$getKeepAliveContext && this.getResolvedPath() === sPath;
-	};
-
-		/**
 	 * Enhance the inherited query options by the given query options if this binding does not have
 	 * any binding parameters. If both have a '$orderby', the resulting '$orderby' is the
 	 * concatenation of both '$orderby' with the given one first. If both have a '$filter', the
@@ -2643,7 +2637,22 @@ sap.ui.define([
 	 * @since 1.99.0
 	 */
 	ODataListBinding.prototype.isFirstCreateAtEnd = function () {
-		return this.bCreatedAtEnd;
+		return this.bFirstCreateAtEnd;
+	};
+
+	/**
+	 * Check whether this binding is an active $$getKeepAliveContext binding for the given path.
+	 *
+	 * @param {string} sPath - An absolute binding path
+	 * @returns {boolean} - Whether this binding matches
+	 *
+	 * @private
+	 */
+	ODataListBinding.prototype.isKeepAliveBindingFor = function (sPath) {
+		// When suspended it matches if it already has contexts. Then its getKeepAliveContext fails.
+		return this.mParameters.$$getKeepAliveContext && this.getResolvedPath() === sPath
+			&& (!this.isRootBindingSuspended() || this.aContexts.length
+				|| Object.keys(this.mPreviousContextsByPath).length);
 	};
 
 	/**
@@ -2689,7 +2698,9 @@ sap.ui.define([
 		}
 
 		if (this.isRootBindingSuspended()) {
+			// Note: side-effects (incl. refresh) are forbidden while suspended
 			this.refreshSuspended(sGroupId);
+			this.bRefreshKeptElements = true;
 			return SyncPromise.all(refreshAll(that.getDependentBindings()));
 		}
 
@@ -2704,22 +2715,9 @@ sap.ui.define([
 
 			if (oCache && !oPromise) { // do not refresh twice
 				that.removeCachesAndMessages(sResourcePathPrefix);
-				that.fetchCache(that.oContext, false, /*bKeepQueryOptions*/true, bKeepCacheOnError);
-				oKeptElementsPromise = that.oCachePromise.then(function (oNewCache) {
-					return oNewCache.refreshKeptElements(that.lockGroup(sGroupId),
-						function onRemove(sPredicate, iIndex) {
-							if (iIndex === undefined) {
-								that.mPreviousContextsByPath[that.getResolvedPath() + sPredicate]
-									.resetKeepAlive();
-							} else { // Note: implies oContext.created()
-								that.destroyCreated(that.aContexts[iIndex]);
-							}
-						});
-				}).catch(function (oError) {
-					that.oModel.reportError("Failed to refresh kept-alive elements", sClassName,
-						oError);
-					throw oError;
-				});
+				that.fetchCache(that.oContext, false, /*bKeepQueryOptions*/true,
+					bKeepCacheOnError ? sGroupId : undefined);
+				oKeptElementsPromise = that.refreshKeptElements(sGroupId);
 				if (that.iCurrentEnd > 0) {
 					oPromise = that.createRefreshPromise().catch(function (oError) {
 						if (!bKeepCacheOnError || oError.canceled) {
@@ -2756,13 +2754,46 @@ sap.ui.define([
 			}
 			// Note: after reset the dependent bindings cannot be found any more
 			aDependentBindings = that.getDependentBindings();
-			 // this may reset that.oRefreshPromise
-			that.reset(ChangeReason.Refresh, bKeepCacheOnError ? false : undefined);
+			// this may reset that.oRefreshPromise
+			that.reset(ChangeReason.Refresh, bKeepCacheOnError ? false : undefined, sGroupId);
 			return SyncPromise.all(
-				refreshAll(aDependentBindings).concat(oPromise, oKeptElementsPromise,
-					// Update after refresh event, otherwise $count is fetched before the request
-					that.oHeaderContext.checkUpdate()) // this is NOT done by refreshAll!
-			);
+				refreshAll(aDependentBindings).concat(oPromise, oKeptElementsPromise)
+			).then(function () {
+				// Update after refresh event, otherwise $count is fetched before the request.
+				// Avoid update in case bKeepCacheOnError needs to roll back.
+				return that.oHeaderContext.checkUpdateInternal(); // this is NOT done by refreshAll!
+			});
+		});
+	};
+
+	/**
+	 * Refreshes the kept-alive elements. This needs to be called before the cache has filled the
+	 * collection.
+	 *
+	 * @param {string} sGroupId
+	 *   The effective group ID
+	 * @returns {sap.ui.base.SyncPromise}
+	 *   A promise resolving without a defined result, or rejecting with an error if the refresh
+	 *   fails.
+	 *
+	 * @private
+	 */
+	ODataListBinding.prototype.refreshKeptElements = function (sGroupId) {
+		var that = this;
+
+		return this.oCachePromise.then(function (oCache) {
+			return oCache.refreshKeptElements(that.lockGroup(sGroupId),
+				function onRemove(sPredicate, iIndex) {
+					if (iIndex === undefined) {
+						that.mPreviousContextsByPath[that.getResolvedPath() + sPredicate]
+							.resetKeepAlive();
+					} else { // Note: implies oContext.created()
+						that.destroyCreated(that.aContexts[iIndex]);
+					}
+				});
+		}).catch(function (oError) {
+			that.oModel.reportError("Failed to refresh kept-alive elements", sClassName, oError);
+			throw oError;
 		});
 	};
 
@@ -3150,15 +3181,19 @@ sap.ui.define([
 	 *   as long as the binding is still empty.
 	 * @param {boolean} [bDrop]
 	 *   By default, all created persisted contexts are dropped while transient ones are not.
-	 *   <code>true</code> also drops transient ones, and <code>false</code> keeps created persisted
-	 *   that started as inactive.
+	 *   <code>true</code> also drops transient ones, and <code>false</code> keeps inline creation
+	 *   rows only and transient ones where the POST is not within the same $batch as the GET for
+	 *   the side-effects refresh.
+	 * @param {string} [sGroupId]
+	 *   The group ID to be used for refresh; used only in case <code>bDrop === false</code>
 	 *
 	 * @private
 	 */
-	ODataListBinding.prototype.reset = function (sChangeReason, bDrop) {
+	ODataListBinding.prototype.reset = function (sChangeReason, bDrop, sGroupId) {
 		var oContext,
+			iCreated = 0, // index (and finally number) of created elements that we keep
 			bEmpty = this.iCurrentEnd === 0,
-			iTransient = 0,
+			bKeepTransient = sGroupId && sGroupId !== this.getUpdateGroupId(),
 			i,
 			that = this;
 
@@ -3172,27 +3207,29 @@ sap.ui.define([
 			});
 			for (i = 0; i < this.iCreatedContexts; i += 1) {
 				oContext = this.aContexts[i];
-				if (oContext.isTransient()
-						|| bDrop === false && oContext.isInactive() !== undefined) {
-					this.aContexts[iTransient] = oContext;
-					iTransient += 1;
-				} else { // Note: "created persisted" elements must be active - drop 'em
+				if (bDrop === false
+						? bKeepTransient && oContext.isTransient()
+							|| oContext.isInactive() !== undefined
+						: oContext.isTransient()) {
+					this.aContexts[iCreated] = oContext;
+					iCreated += 1;
+				} else { // Note: inactive elements are always kept
 					this.iActiveContexts -= 1;
 					this.mPreviousContextsByPath[oContext.getPath()] = oContext;
 				}
 			}
-			for (i = 0; i < iTransient; i += 1) {
-				this.aContexts[i].iIndex = i - iTransient;
+			for (i = 0; i < iCreated; i += 1) {
+				this.aContexts[i].iIndex = i - iCreated;
 			}
 			// Note: no strict need to keep the reference here
-			this.aContexts.length = this.iCreatedContexts = iTransient;
+			this.aContexts.length = this.iCreatedContexts = iCreated;
 		} else {
 			this.aContexts = [];
 		}
 		if (!this.iCreatedContexts) {
 			// true if contexts have been created at the end, false if contexts have been created at
 			// the start, undefined if there are no created contexts
-			this.bCreatedAtEnd = undefined;
+			this.bFirstCreateAtEnd = undefined;
 		}
 		// the range of array indices for getCurrentContexts
 		this.iCurrentBegin = this.iCurrentEnd = 0;
@@ -3230,6 +3267,28 @@ sap.ui.define([
 		this.aContexts.forEach(reset);
 	};
 
+	/**
+	 * Restores all created elements, the bFirstCreateAtEnd flag and the iCreatedContexts,
+	 * iActiveContexts counters from cache to this list binding.
+	 *
+	 * @private
+	 */
+	ODataListBinding.prototype.restoreCreated = function () {
+		var that = this;
+
+		this.withCache(function (oCache, sPath) {
+			oCache.getCreatedElements(sPath).forEach(function (oElement, i) {
+				that.aContexts[i] = _Helper.getPrivateAnnotation(oElement, "context");
+				that.bFirstCreateAtEnd
+					= _Helper.getPrivateAnnotation(oElement, "firstCreateAtEnd");
+				that.iCreatedContexts += 1;
+				if (!oElement["@$ui5.context.isInactive"]) {
+					that.iActiveContexts += 1;
+				}
+			});
+		}).catch(this.oModel.getReporter());
+	};
+
 	/**
 	 * @override
 	 * @see sap.ui.model.odata.v4.ODataParentBinding#resumeInternal
@@ -3237,7 +3296,8 @@ sap.ui.define([
 	ODataListBinding.prototype.resumeInternal = function (_bCheckUpdate, bParentHasChanges) {
 		var aBindings = this.getDependentBindings(),
 			sResumeChangeReason = this.sResumeChangeReason,
-			bRefresh = bParentHasChanges || sResumeChangeReason;
+			bRefresh = bParentHasChanges || sResumeChangeReason,
+			that = this;
 
 		this.sResumeChangeReason = undefined;
 
@@ -3247,6 +3307,11 @@ sap.ui.define([
 			// if the parent binding resumes but there are no changes in the parent binding
 			// ignore the parent cache and create an own cache
 			this.fetchCache(this.oContext, !bParentHasChanges);
+
+			if (this.bRefreshKeptElements) {
+				this.bRefreshKeptElements = false;
+				that.refreshKeptElements(that.getGroupId());
+			}
 		}
 		aBindings.forEach(function (oDependentBinding) {
 			// do not call checkUpdate in dependent property bindings if the cache of this
@@ -3400,35 +3465,25 @@ sap.ui.define([
 	 * @param {sap.ui.model.Context} oContext
 	 *   The context object
 	 * @throws {Error}
-	 *   If the binding's root binding is suspended, or for relative bindings containing transient
-	 *   entities
+	 *   If the binding's root binding is suspended
 	 *
 	 * @private
 	 */
 	// @override sap.ui.model.Binding#setContext
 	ODataListBinding.prototype.setContext = function (oContext) {
-		var sResolvedPath,
-			i,
-			that = this;
+		var sResolvedPath;
 
 		if (this.oContext !== oContext) {
 			if (this.bRelative) {
 				this.checkSuspended(true);
-				for (i = 0; i < that.iCreatedContexts; i += 1) {
-					if (that.aContexts[i].isTransient()) {
-						// to allow switching the context for new created entities (transient or
-						// not), we first have to implement a store/restore mechanism for them
-						throw new Error("setContext on relative binding is forbidden if a "
-							+ "transient entity exists: " + that);
-					}
-				}
 				// Keep the header context even if we lose the parent context, so that the header
 				// context remains unchanged if the parent context is temporarily dropped during a
 				// refresh.
-				this.reset();
+				this.reset(/*sChangeReason*/undefined, /*bDrop*/true);
 				this.resetKeepAlive(); // before fetchCache to avoid that it copies data
 				this.fetchCache(oContext);
 				if (oContext) {
+					this.restoreCreated();
 					sResolvedPath = this.oModel.resolve(this.sPath, oContext);
 					// Note: oHeaderContext is missing only if called from c'tor
 					if (this.oHeaderContext && this.oHeaderContext.getPath() !== sResolvedPath) {

package/src/sap/ui/model/odata/v4/ODataMetaModel.js

@@ -157,7 +157,7 @@ sap.ui.define([
 		 * @hideconstructor
 		 * @public
 		 * @since 1.37.0
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 */
 		ODataMetaModel = MetaModel.extend("sap.ui.model.odata.v4.ODataMetaModel", {
 				constructor : constructor
@@ -1405,10 +1405,17 @@ sap.ui.define([
 								sTarget = sName = sSchemaChildName
 									= sSchemaChildName || mScope.$EntityContainer;
 								vResult = oSchemaChild = oSchemaChild || mScope[sSchemaChildName];
-								if (Array.isArray(vResult) && isParameter(sSegment, vResult[0])) {
-									// path evaluation relative to an operation overload
-									// @see [OData-CSDL-JSON-v4.01] "14.4.1.2 Path Evaluation"
-									return true;
+								if (Array.isArray(vResult)) {
+									if (vBindingParameterType) {
+										vResult = vResult.filter(isRightOverload);
+									}
+									if (isParameter(sSegment, vResult[0])) {
+										// path evaluation relative to an operation overload
+										// @see [OData-CSDL-JSON-v4.01] "14.4.1.2 Path Evaluation"
+										// or ".../@$ui5.overload/0/$Parameter/<i>/$Name/$" to refer
+										// back to (overloaded) operation's parameter
+										return true;
+									}
 								}
 								if (sSegment && sSegment[0] !== "@"
 									&& !(sSegment in oSchemaChild)) {
@@ -1828,7 +1835,7 @@ sap.ui.define([
 					if (!oEntity) {
 						error("No instance to calculate key predicate at " + vSegment.path);
 					}
-					if (_Helper.hasPrivateAnnotation(oEntity, "transient")) {
+					if (oEntity["@$ui5.context.isTransient"]) {
 						bNoEditUrl = true;
 						return undefined;
 					}