@openui5/sap.ui.core 1.140.0 → 1.141.1

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

@@ -215,7 +215,7 @@ sap.ui.define([
 									+ sForbidden);
 							}
 						});
-					// falls through
+					// fall through
 				case "$$canonicalPath":
 				case "$$clearSelectionOnFilter":
 				case "$$noPatch":
@@ -242,14 +242,16 @@ sap.ui.define([
 	 * one. The error has the property <code>canceled : true</code>
 	 *
 	 * @param {sap.ui.model.odata.v4.lib._Cache} oExpectedCache - The expected cache
+	 * @param {number} [iResetCount] - The cache's expected reset count
 	 * @throws {Error} If the current cache is not the expected one
 	 *
 	 * @private
 	 */
-	ODataBinding.prototype.checkSameCache = function (oExpectedCache) {
+	ODataBinding.prototype.checkSameCache = function (oExpectedCache, iResetCount) {
 		var oError;
 
-		if (this.oCache !== oExpectedCache) {
+		if (this.oCache !== oExpectedCache
+				|| iResetCount !== undefined && this.oCache.iResetCount !== iResetCount) {
 			oError = new Error(this + " is ignoring response from inactive cache: "
 				+ oExpectedCache);
 			oError.canceled = true;
@@ -318,7 +320,7 @@ sap.ui.define([
 	 *
 	 * @param {boolean} [bForceUpdate]
 	 *   Whether the change event is fired in any case (only allowed for property bindings)
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise which is resolved without a defined result when the check is finished, or
 	 *   rejected in case of an error
 	 * @throws {Error} If called with illegal parameters
@@ -456,7 +458,7 @@ sap.ui.define([
 	 *
 	 * @param {sap.ui.model.Context} [oContext]
 	 *   The context instance to be used for a relative binding
-	 * @returns {object|undefined|sap.ui.base.SyncPromise}
+	 * @returns {object|undefined|sap.ui.base.SyncPromise<object|undefined>}
 	 *   The binding's query options (if any) or a promise resolving with them
 	 *
 	 * @abstract
@@ -490,14 +492,20 @@ sap.ui.define([
 	 *   The group ID; mandatory if <code>bSideEffectsRefresh</code> is set
 	 * @param {boolean} [bSideEffectsRefresh]
 	 *   Whether to perform a side-effects refresh
-	 * @throws {Error}
-	 *   If auto-$expand/$select is still running and query options shall be kept (this case is just
-	 *   not yet implemented and should not be needed)
+	 * @param {boolean} [bSync]
+	 *   Whether to create and set the cache synchronously; <code>bKeepQueryOptions</code> must be
+	 *   <code>true</code>
+	 * @throws {Error} If
+	 *   <ul>
+	 *     <li> auto-$expand/$select is still running and query options shall be kept (this case is
+	 *       just not yet implemented and should not be needed),
+	 *     <li> bSync is set but bKeepQueryOptions is not.
+	 *   </ul>
 	 *
 	 * @private
 	 */
 	ODataBinding.prototype.fetchCache = function (oContext, bIgnoreParentCache, bKeepQueryOptions,
-			sGroupId, bSideEffectsRefresh) {
+			sGroupId, bSideEffectsRefresh, bSync) {
 		var oCache = this.oCache,
 			oCallToken = {
 				// propagate old cache from first call of fetchCache to the latest call
@@ -517,11 +525,15 @@ sap.ui.define([
 			return;
 		}
 
+		if (bSync && !bKeepQueryOptions) {
+			throw new Error("Unsupported bSync w/o bKeepQueryOptions");
+		}
+
 		this.oCache = undefined;
 		this.oFetchCacheCallToken = oCallToken;
 		if (bKeepQueryOptions) {
-			// asynchronously re-create an equivalent cache, but skip auto-$expand/$select
-			this.oCachePromise = SyncPromise.resolve(Promise.resolve()).then(function () {
+			// re-create an equivalent cache, but skip auto-$expand/$select
+			this.oCachePromise = SyncPromise.resolve(bSync || Promise.resolve()).then(function () {
 				return that.createAndSetCache(that.mCacheQueryOptions, oCache.getResourcePath(),
 					oContext, sGroupId, bSideEffectsRefresh, oCache);
 			});
@@ -588,7 +600,7 @@ sap.ui.define([
 	 * @param {boolean} [bIgnoreParentCache]
 	 *   Whether the query options of the parent cache shall be ignored and own query options are
 	 *   determined (see {@link #fetchCache})
-	 * @returns {object|sap.ui.base.SyncPromise}
+	 * @returns {object|sap.ui.base.SyncPromise<object>}
 	 *   An object having two properties (or a promise resolving with it):
 	 *   {object} mQueryOptions - The query options to create the cache for this binding or
 	 *     <code>undefined</code> if no cache is to be created
@@ -599,7 +611,7 @@ sap.ui.define([
 	ODataBinding.prototype.fetchOrGetQueryOptionsForOwnCache = function (oContext,
 			bIgnoreParentCache) {
 		var bHasNonSystemQueryOptions,
-			vQueryOptions, // {object|undefined|sap.ui.base.SyncPromise}
+			vQueryOptions, // {object|undefined|sap.ui.base.SyncPromise<object|undefined>}
 			sResolvedPath = this.oModel.resolve(this.sPath, oContext),
 			that = this;
 
@@ -634,7 +646,7 @@ sap.ui.define([
 		 *   Whether an empty query options object should be replaced by <code>undefined</code>
 		 * @param {string} [sReducedPath=sResolvedPath]
 		 *   The reduced path
-		 * @returns {object|sap.ui.base.SyncPromise}
+		 * @returns {object|sap.ui.base.SyncPromise<object>}
 		 *   A result for #fetchOrGetQueryOptionsForOwnCache
 		 */
 		function wrapQueryOptions(bDropEmptyObject, sReducedPath) {
@@ -725,9 +737,9 @@ sap.ui.define([
 	 *
 	 * @param {sap.ui.model.Context|sap.ui.model.odata.v4.Context} [oContext=this.oContext]
 	 *   A context; if omitted, the binding's context is used
-	 * @returns {sap.ui.base.SyncPromise} A promise resolving with the resource path or
-	 *   <code>undefined</code> for an unresolved binding. If computation of the canonical path
-	 *   fails, the promise is rejected.
+	 * @returns {sap.ui.base.SyncPromise<string|undefined>}
+	 *   A promise resolving with the resource path or <code>undefined</code> for an unresolved
+	 *   binding. If computation of the canonical path fails, the promise is rejected.
 	 *
 	 * @private
 	 */
@@ -893,7 +905,7 @@ sap.ui.define([
 	/**
 	 * Returns a promise which resolves as soon as this binding is resumed.
 	 *
-	 * @returns {sap.ui.base.SyncPromise|undefined}
+	 * @returns {sap.ui.base.SyncPromise<void>|undefined}
 	 *   This binding's current promise for {@link sap.ui.model.odata.v4.ODataParentBinding#resume},
 	 *   or <code>undefined</code> in case it is not currently suspended.
 	 *
@@ -931,7 +943,7 @@ sap.ui.define([
 	/**
 	 * Returns a promise which resolves as soon as this binding's root binding is resumed.
 	 *
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   The root binding's current promise for {@link #resume}, or
 	 *   <code>SyncPromise.resolve()</code> in case we have no root binding or it is not currently
 	 *   suspended.
@@ -1203,7 +1215,7 @@ sap.ui.define([
 	 * Returns a sync promise that is resolved when this binding is ready to be used (that is, when
 	 * its cache has been determined).
 	 *
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A sync promise that is resolved without a defined result when this binding is ready
 	 *
 	 * @private
@@ -1287,7 +1299,11 @@ sap.ui.define([
 	 *   If <code>true</code>, a property binding is expected to check for updates
 	 * @param {boolean} [bKeepCacheOnError]
 	 *   If <code>true</code>, the binding data remains unchanged if the refresh fails
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @param {boolean} [bSync]
+	 *   If <code>true</code>, the cache is created synchronously; cannot be combined with the
+	 *   ODataModel's <code>sharedRequests</code> constructor parameter or with the
+	 *   ODataListBinding's <code>$$sharedRequest</code> binding parameter
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise which is resolved without a defined result when the refresh is finished, or
 	 *   rejected when the refresh fails; the promise is resolved immediately on a suspended binding
 	 * @throws {Error}
@@ -1358,7 +1374,7 @@ sap.ui.define([
 	 *   The effective group ID
 	 * @param {string[]} aAbsolutePaths
 	 *   The absolute paths to request side effects for
-	 * @returns {sap.ui.base.SyncPromise|undefined}
+	 * @returns {sap.ui.base.SyncPromise<void>|undefined}
 	 *   A promise which is resolved without a defined result, or rejected with an error if loading
 	 *   of side effects fails, or <code>undefined</code> if there is nothing to do
 	 *
@@ -1453,7 +1469,7 @@ sap.ui.define([
 	 *
 	 * @param {string} sPath
 	 *   The path (absolute or relative to this binding)
-	 * @param {sap.ui.base.SyncPromise[]} aPromises
+	 * @param {Array<sap.ui.base.SyncPromise<void>>} aPromises
 	 *   List of promises which is extended for each call to {@link #resetChangesForPath}
 	 * @throws {Error}
 	 *   If there is a change of this binding which has been sent to the server and for which there
@@ -1470,7 +1486,7 @@ sap.ui.define([
 	/**
 	 * Resets pending changes in all dependent bindings.
 	 *
-	 * @param {sap.ui.base.SyncPromise[]} aPromises
+	 * @param {Array<sap.ui.base.SyncPromise<void>>} aPromises
 	 *   List of promises which is extended for each call to {@link #resetChangesInDependents}.
 	 * @param {boolean} [sPathPrefix]
 	 *   If supplied, only caches having a resource path starting with <code>sPathPrefix</code> are
@@ -1541,7 +1557,7 @@ sap.ui.define([
 	 *   Whether the application wants to skip the automatic refresh
 	 * @param {string} sGroupId
 	 *   The group ID for missing properties requests
-	 * @returns {sap.ui.base.SyncPromise|undefined}
+	 * @returns {sap.ui.base.SyncPromise<void>|undefined}
 	 *   A promise which is resolved without a defined result when the update is finished, or
 	 *   rejected with an error if something went wrong; or <code>undefined</code> if there is no
 	 *   need to wait
@@ -1592,9 +1608,9 @@ sap.ui.define([
 	 * @param {boolean} [bSync] Whether to use the synchronously available cache
 	 * @param {boolean} [bWithOrWithoutCache] Whether to call the processor even without a cache
 	 *   (currently implemented for operation bindings only)
-	 * @returns {sap.ui.base.SyncPromise} A sync promise that is resolved with either the result of
-	 *   the processor or <code>undefined</code> if there is no cache for this binding, or if the
-	 *   cache determination is not yet completed
+	 * @returns {sap.ui.base.SyncPromise<any>} A sync promise that is resolved with either the
+	 *   result of the processor or <code>undefined</code> if there is no cache for this binding, or
+	 *   if the cache determination is not yet completed
 	 *
 	 * @private
 	 */

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

@@ -74,7 +74,7 @@ sap.ui.define([
 		 * @mixes sap.ui.model.odata.v4.ODataParentBinding
 		 * @public
 		 * @since 1.37.0
-		 * @version 1.140.0
+		 * @version 1.141.1
 		 *
 		 * @borrows sap.ui.model.odata.v4.ODataBinding#getGroupId as #getGroupId
 		 * @borrows sap.ui.model.odata.v4.ODataBinding#getRootBinding as #getRootBinding
@@ -225,7 +225,7 @@ sap.ui.define([
 
 		/*
 		 * Fires a "change" event and refreshes dependent bindings.
-		 * @returns {sap.ui.base.SyncPromise} A promise resolving when the refresh is finished
+		 * @returns {sap.ui.base.SyncPromise<void>} A promise resolving when the refresh is finished
 		 */
 		function fireChangeAndRefreshDependentBindings() {
 			that._fireChange({reason : ChangeReason.Change});
@@ -428,7 +428,7 @@ sap.ui.define([
 	 * If a back-end request fails, the 'dataReceived' event provides an <code>Error</code> in the
 	 * 'error' event parameter.
 	 *
-	 * Since 1.106 this event is bubbled up to the model, unless a listener calls
+	 * Since 1.106, this event is bubbled up to the model, unless a listener calls
 	 * {@link sap.ui.base.Event#cancelBubble oEvent.cancelBubble()}.
 	 *
 	 * @param {sap.ui.base.Event} oEvent
@@ -456,7 +456,7 @@ sap.ui.define([
 	 * fired: Whatever should happen in the event handler attached to that event, can instead be
 	 * done before calling {@link #invoke}.
 	 *
-	 * Since 1.106 this event is bubbled up to the model, unless a listener calls
+	 * Since 1.106, this event is bubbled up to the model, unless a listener calls
 	 * {@link sap.ui.base.Event#cancelBubble oEvent.cancelBubble()}.
 	 *
 	 * @param {sap.ui.base.Event} oEvent
@@ -563,7 +563,7 @@ sap.ui.define([
 	 *   actions only
 	 * @param {function} [fnOnStrictHandlingFailed]
 	 *   Callback for strict handling; supported for actions only
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<any>}
 	 *   The request promise
 	 * @throws {Error} If
 	 *   <ul>
@@ -879,7 +879,7 @@ sap.ui.define([
 	 *   A property binding which registers itself as listener at the cache
 	 * @param {boolean} [bCached]
 	 *   Whether to return cached values only and not initiate a request
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<any>}
 	 *   A promise on the outcome of the cache's <code>fetchValue</code> call; it is rejected in
 	 *   case cached values are asked for, but not found, or if the cache is no longer the active
 	 *   cache when the response arrives
@@ -1358,12 +1358,13 @@ sap.ui.define([
 	 *   context only. Such a message can only be connected to a corresponding control if the
 	 *   control's property bindings use the return value context as binding context.
 	 *   <br>
-	 *   A return value context may also be provided if the parent context's path contains a maximum
-	 *   of one navigation property. In addition to the existing preconditions for a return value
-	 *   context, the metadata has to specify a partner attribute for the navigation property and
-	 *   the partner relationship has to be bi-directional. Also the navigation property binding has
-	 *   to be available in the entity set of the first segment in the parent context's path
-	 *   (@experimental as of version 1.119.0).
+	 *   Since 1.141.0, a return value context may also be provided if the parent context's path
+	 *   contains a maximum of one navigation property. In addition to the existing preconditions
+	 *   for a return value context, the metadata has to specify a partner attribute for the
+	 *   navigation property and the partner relationship has to be bi-directional. Also a
+	 *   navigation property binding has to be available for the entity set of the first segment in
+	 *   the parent context's path. <b>Note:</b> Ensure your service implementation returns all
+	 *   selected key properties; otherwise, no return value context is provided.
 	 * @throws {Error} If
 	 *   <ul>
 	 *     <li> the binding's root binding is suspended,
@@ -1501,7 +1502,7 @@ sap.ui.define([
 	/**
 	 * Returns a sync promise that tells whether we are ready to inherit $expand/$select.
 	 *
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A sync promise that resolves without a defined result as soon as we are ready to inherit
 	 *   $expand/$select
 	 *
@@ -1525,7 +1526,7 @@ sap.ui.define([
 	 *   If <code>true</code>, a property binding is expected to check for updates
 	 * @param {boolean} [bKeepCacheOnError]
 	 *   If <code>true</code>, the binding data remains unchanged if the refresh fails
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise resolving when all dependent bindings are refreshed; it is rejected
 	 *   when the refresh fails; the promise is resolved immediately on a suspended binding
 	 * @throws {Error}
@@ -1636,7 +1637,7 @@ sap.ui.define([
 	 *   The group ID for the refresh
 	 * @param {boolean} [bKeepCacheOnError]
 	 *   If <code>true</code>, the binding data remains unchanged if the refresh fails
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise which is resolved without a defined result when the refresh is finished and if
 	 *   the context is this binding's return value context; <code>null</code> otherwise
 	 *

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

@@ -60,7 +60,7 @@ sap.ui.define([
 		 * @mixes sap.ui.model.odata.v4.ODataParentBinding
 		 * @public
 		 * @since 1.37.0
-		 * @version 1.140.0
+		 * @version 1.141.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
@@ -448,7 +448,7 @@ sap.ui.define([
 	 * If a back-end request fails, the 'dataReceived' event provides an <code>Error</code> in the
 	 * 'error' event parameter.
 	 *
-	 * Since 1.106 this event is bubbled up to the model, unless a listener calls
+	 * Since 1.106, this event is bubbled up to the model, unless a listener calls
 	 * {@link sap.ui.base.Event#cancelBubble oEvent.cancelBubble()}.
 	 *
 	 * @param {sap.ui.base.Event} oEvent
@@ -474,7 +474,7 @@ sap.ui.define([
 	 * for example to switch on a busy indicator. Registered event handlers are called without
 	 * parameters.
 	 *
-	 * Since 1.106 this event is bubbled up to the model, unless a listener calls
+	 * Since 1.106, this event is bubbled up to the model, unless a listener calls
 	 * {@link sap.ui.base.Event#cancelBubble oEvent.cancelBubble()}.
 	 *
 	 * @param {sap.ui.base.Event} oEvent
@@ -839,12 +839,16 @@ sap.ui.define([
 	 * {@link sap.ui.model.odata.v4.Context#requestSideEffects} in the same $batch to refresh the
 	 * complete collection containing the newly created entity.
 	 *
-	 * Since 1.115.0 it is possible to create nested entities in a collection-valued navigation
+	 * Since 1.115.0, it is possible to create nested entities in a collection-valued navigation
 	 * property together with the entity (so-called "deep create"), for example a list of items for
 	 * an order. For this purpose, bind the list relative to a transient context. Calling this
 	 * method then adds a transient entity to the parent's navigation property, which is sent with
 	 * the payload of the parent entity. Such a nested context cannot be inactive.
 	 *
+	 * <b>Caution:</b> Only a single list must be bound to the same collection-valued navigation
+	 * property relative to a transient context. Created data cannot be shared between list
+	 * bindings.
+	 *
 	 * <b>Note:</b> After a successful creation of the main entity the context returned for a
 	 * nested entity is no longer valid. Do not use the
 	 * {@link sap.ui.model.odata.v4.Context#created created} promise of such a context! New contexts
@@ -854,7 +858,7 @@ sap.ui.define([
 	 * of <code>Error</code>, even if the deep create succeeds. This error always has the property
 	 * <code>canceled</code> with the value <code>true</code>.
 	 *
-	 * Since 1.118.0 deep create also supports single-valued navigation properties; no API call is
+	 * Since 1.118.0, deep create also supports single-valued navigation properties; no API call is
 	 * required in this case. Simply bind properties of the related entity relative to a transient
 	 * context. An update to the property adds it to the POST request of the parent entity, and by
 	 * this the create becomes deep.
@@ -908,7 +912,7 @@ sap.ui.define([
 	 *   on the client and requests only data that is missing.
 	 * @param {boolean} [bAtEnd]
 	 *   Whether the entity is inserted at the end of the list. Supported since 1.66.0.
-	 *   Since 1.99.0 the first insertion determines the overall position of created contexts
+	 *   Since 1.99.0, the first insertion determines the overall position of created contexts
 	 *   within the binding's context list. Every succeeding insertion is relative to the created
 	 *   contexts within this list.
 	 * @param {boolean} [bInactive]
@@ -1102,7 +1106,8 @@ sap.ui.define([
 		if (bRefresh) {
 			oCreatePromise = SyncPromise.all([
 				oCreatePromise,
-				this.requestSideEffects(sGroupId, [""])
+				// sGroupId is set -> side-effects refresh
+				this.refreshInternal("", sGroupId, false, true, true)
 			]);
 		}
 
@@ -1144,6 +1149,8 @@ sap.ui.define([
 	 * @returns {boolean}
 	 *   <code>true</code>, if contexts have been created or dropped or <code>isLengthFinal</code>
 	 *   has changed
+	 * @throws {Error}
+	 *   If a created context from a foreign binding is about to be reused
 	 *
 	 * @private
 	 */
@@ -1200,6 +1207,9 @@ sap.ui.define([
 					// created persisted contexts can be restored from their data, for example in
 					// case of Recursive Hierarchy maintenance
 					oContext = _Helper.getPrivateAnnotation(aResults[i], "context");
+					if (oContext.getBinding() !== this) {
+						throw new Error("Cannot share created data between list bindings");
+					}
 					oContext.iIndex = i$skipIndex;
 					// oContext.checkUpdate(); // Note: no changes expected here
 					sContextPath = oContext.getPath();
@@ -1728,7 +1738,7 @@ sap.ui.define([
 	 *   used to expand all levels
 	 * @param {boolean} [bSilent]
 	 *   Whether no ("change") events should be fired
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise that is resolved when the expand is successful and rejected when it fails
 	 * @throws {Error}
 	 *   If the binding's root binding is suspended, if the given context is not part of a
@@ -1794,7 +1804,7 @@ sap.ui.define([
 	 * @param {function} [fnDataRequested]
 	 *   The function is called just before a back-end request is sent.
 	 *   If no back-end request is needed, the function is not called.
-	 * @returns {sap.ui.base.SyncPromise|Promise}
+	 * @returns {sap.ui.base.SyncPromise<boolean>|Promise<boolean>}
 	 *   A promise that resolves with a boolean indicating whether the binding's contexts have been
 	 *   modified; it rejects when iStart or iLength are negative, or when the request fails, or
 	 *   if this binding is already destroyed when the response arrives
@@ -1858,7 +1868,7 @@ sap.ui.define([
 	 * @param {function} [fnDataRequested]
 	 *   The function is called just before a back-end request is sent.
 	 *   If no back-end request is needed, the function is not called.
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<object>}
 	 *   A promise to be resolved with the requested range as described in _Cache#read or with
 	 *   <code>undefined</code> if the context changed before reading; it is rejected to discard a
 	 *   response because the cache is no longer active, in this case the error has the property
@@ -1891,7 +1901,8 @@ sap.ui.define([
 				return oCache.read(iIndex, iLength, iMaximumPrefetchSize, oGroupLock,
 					fnDataRequested, undefined, that.fireSeparateReceived.bind(that)
 				).then(function (oResult) {
-					oResult.$checkStillValid = that.checkSameCache.bind(that, oCache);
+					oResult.$checkStillValid
+						= that.checkSameCache.bind(that, oCache, oResult["@$ui5.resetCount"]);
 
 					return oResult;
 				});
@@ -1967,7 +1978,7 @@ sap.ui.define([
 	 *   that the cache promise is already created when the events are fired.
 	 * @param {string} sStaticFilter
 	 *   The static filter value
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<Array<string|undefined>>}
 	 *   A promise which resolves with an array that consists of three filters where each can be
 	 *   <code>undefined</code>. The first one has to be applied after data aggregation. The second
 	 *   one can simply be applied before data aggregation (which improves performance) because it
@@ -2046,8 +2057,8 @@ sap.ui.define([
 		 * @param {object} mLambdaVariableToPath The map from lambda variable to full path
 		 * @param {boolean} [bWithinAnd] Whether the embedding filter is an 'and'
 		 * @param {boolean} bThese - Whether the special syntax "$these/aggregate(...)" is needed
-		 * @returns {sap.ui.base.SyncPromise} A promise which resolves with the $filter value or
-		 *   rejects with an error if the filter value uses an unknown operator
+		 * @returns {sap.ui.base.SyncPromise<string>} A promise which resolves with the $filter
+		 *   value or rejects with an error if the filter value uses an unknown operator
 		 */
 		function fetchFilter(oFilter, mLambdaVariableToPath, bWithinAnd, bThese) {
 			var sResolvedPath;
@@ -2175,7 +2186,8 @@ sap.ui.define([
 	 * @param {boolean} [bAllowRequest]
 	 *   Whether it is allowed to send a GET request to fetch the parent node's data
 	 * @returns {sap.ui.model.odata.v4.Context|null|undefined|
-	 *     Promise<sap.ui.model.odata.v4.Context>|sap.ui.base.SyncPromise}
+	 *     Promise<sap.ui.model.odata.v4.Context>|
+	 *     sap.ui.base.SyncPromise<sap.ui.model.odata.v4.Context>}
 	 *   <ul>
 	 *     <li> The parent node if already known,
 	 *     <li> <code>null</code> if the given node is a root node and thus has no parent,
@@ -2309,7 +2321,7 @@ sap.ui.define([
 	 *   A property binding which registers itself as listener at the cache
 	 * @param {boolean} [bCached]
 	 *   Whether to return cached values only and not initiate a request
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<any>}
 	 *   A promise on the outcome of the cache's <code>fetchValue</code> call; it is rejected in
 	 *   case cached values are asked for, but not found
 	 *
@@ -3160,8 +3172,8 @@ sap.ui.define([
 
 	/**
 	 * Calls {@link sap.ui.model.odata.v4.Context#setKeepAlive} at the context for the given path
-	 * and returns it. Since 1.100.0 the function always returns such a context. If none exists yet,
-	 * it is created without data and a request for its entity is sent.
+	 * and returns it. Since 1.100.0, the function always returns such a context. If none exists
+	 * yet, it is created without data and a request for its entity is sent.
 	 *
 	 * @param {string} sPath
 	 *   The path of the context to be kept alive
@@ -3938,7 +3950,7 @@ sap.ui.define([
 	 * @see sap.ui.model.odata.v4.ODataBinding#refreshInternal
 	 */
 	ODataListBinding.prototype.refreshInternal = function (sResourcePathPrefix, sGroupId,
-			_bCheckUpdate, bKeepCacheOnError) {
+			_bCheckUpdate, bKeepCacheOnError, bSync) {
 		var that = this;
 
 		// calls refreshInternal on all given bindings and returns an array of promises
@@ -3955,7 +3967,7 @@ sap.ui.define([
 				// another update request in createContexts, when the context for the row is
 				// reused.
 				return oBinding.refreshInternal(sResourcePathPrefix, sGroupId, false,
-					bKeepCacheOnError);
+					bKeepCacheOnError, bSync);
 			});
 		}
 
@@ -3986,7 +3998,7 @@ sap.ui.define([
 					oCache.reset([]);
 				} else {
 					that.fetchCache(that.oContext, false, /*bKeepQueryOptions*/true,
-						sGroupId, bKeepCacheOnError);
+						sGroupId, bKeepCacheOnError, bSync);
 					oKeptElementsPromise = that.refreshKeptElements(sGroupId,
 						/*bIgnorePendingChanges*/ bKeepCacheOnError);
 					if (that.iCurrentEnd > 0) {
@@ -4050,7 +4062,7 @@ sap.ui.define([
 	 *   The effective group ID
 	 * @param {boolean} bIgnorePendingChanges
 	 *   Whether kept elements are refreshed although there are pending changes.
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise which is resolved without a defined result, or rejected with an error if the
 	 *   refresh fails.
 	 *
@@ -4098,7 +4110,7 @@ sap.ui.define([
 	 *   (since 1.129.0) no dataRequested/dataReceived events are fired in the first place
 	 * @param {boolean} [bWithMessages]
 	 *   Whether the "@com.sap.vocabularies.Common.v1.Messages" path is treated specially
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise<void>}
 	 *   A promise which resolves without a defined value when the entity is updated in the cache,
 	 *   or rejects if the refresh failed.
 	 * @throws {Error}