@openui5/sap.ui.core 1.97.1 → 1.98.0

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

@@ -49,7 +49,6 @@ sap.ui.define([
 		EdmDate, DateTimeOffset, Decimal, Double, Guid, Int16, Int32, Int64, Raw, SByte, Single,
 		Stream, String, TimeOfDay, URI) {
 	"use strict";
-	/*global Map */
 	/*eslint max-nested-callbacks: 0 */
 
 	var Any = ManagedObject.extend("sap.ui.model.odata.v4._any", {
@@ -81,7 +80,7 @@ sap.ui.define([
 			"Edm.Date" : {Type : EdmDate},
 			"Edm.DateTimeOffset" : {
 				constraints : {
-					"$Precision" : "precision"
+					$Precision : "precision"
 				},
 				Type : DateTimeOffset
 			},
@@ -93,8 +92,8 @@ sap.ui.define([
 					"@Org.OData.Validation.V1.Maximum/$Decimal" : "maximum",
 					"@Org.OData.Validation.V1.Maximum@Org.OData.Validation.V1.Exclusive" :
 						"maximumExclusive",
-					"$Precision" : "precision",
-					"$Scale" : "scale"
+					$Precision : "precision",
+					$Scale : "scale"
 				},
 				Type : Decimal
 			},
@@ -109,13 +108,13 @@ sap.ui.define([
 			"Edm.String" : {
 				constraints : {
 					"@com.sap.vocabularies.Common.v1.IsDigitSequence" : "isDigitSequence",
-					"$MaxLength" : "maxLength"
+					$MaxLength : "maxLength"
 				},
 				Type : String
 			},
 			"Edm.TimeOfDay" : {
 				constraints : {
-					"$Precision" : "precision"
+					$Precision : "precision"
 				},
 				Type : TimeOfDay
 			}
@@ -250,8 +249,8 @@ sap.ui.define([
 	 *   The term
 	 * @param {string} sExpectedTerm
 	 *   The expected term
-	 * @returns {string}
-	 *   The qualifier or undefined, if the term is not the expected term
+	 * @returns {string|undefined}
+	 *   The qualifier or <code>undefined</code>, if the term is not the expected term
 	 */
 	function getQualifier(sTerm, sExpectedTerm) {
 		if (sTerm === sExpectedTerm) {
@@ -268,8 +267,8 @@ sap.ui.define([
 	 *
 	 * @param {string} sTerm
 	 *   The term
-	 * @returns {string}
-	 *   The qualifier or undefined, if the term is not as expected
+	 * @returns {string|undefined}
+	 *   The qualifier or <code>undefined</code>, if the term is not as expected
 	 */
 	function getValueListQualifier(sTerm) {
 		var sQualifier = getQualifier(sTerm, sValueListMapping);
@@ -387,6 +386,7 @@ sap.ui.define([
 			// @see sap.ui.model.Binding#initialize
 			initialize : function () {
 				var oElementContext = this.oModel.createBindingContext(this.sPath, this.oContext);
+
 				this.bInitial = false; // initialize() has been called
 				if (oElementContext !== this.oElementContext) {
 					this.oElementContext = oElementContext;
@@ -481,7 +481,7 @@ sap.ui.define([
 				return Object.keys(oResult).filter(function (sKey) {
 					// always filter technical properties;
 					// filter annotations iff not iterating them
-					return sKey[0] !== "$" &&  bIterateAnnotations !== (sKey[0] !== "@");
+					return sKey[0] !== "$" && bIterateAnnotations !== (sKey[0] !== "@");
 				}).map(function (sKey) {
 					return new BaseContext(that.oModel, sResolvedPath + sKey);
 				});
@@ -658,7 +658,7 @@ sap.ui.define([
 	 * @hideconstructor
 	 * @public
 	 * @since 1.37.0
-	 * @version 1.97.1
+	 * @version 1.98.0
 	 */
 	var ODataMetaModel = MetaModel.extend("sap.ui.model.odata.v4.ODataMetaModel", {
 		/*
@@ -686,11 +686,11 @@ sap.ui.define([
 			//   "B." : {"/B/$metadata" : true} // namespace already read
 			// }
 			this.mSchema2MetadataUrl = {};
-			this.mSupportedBindingModes = {"OneTime" : true, "OneWay" : true};
+			this.mSupportedBindingModes = {OneTime : true, OneWay : true};
 			this.bSupportReferences = bSupportReferences !== false; // default is true
 			// ClientListBinding#filter calls checkFilterOperation on the model; ClientModel does
 			// not support "All" and "Any" filters
-			this.mUnsupportedFilterOperators = {"All" : true, "Any" : true};
+			this.mUnsupportedFilterOperators = {All : true, Any : true};
 			this.sUrl = sUrl;
 		}
 	});
@@ -1661,30 +1661,32 @@ sap.ui.define([
 			throw oError;
 		}
 
-		// First fetch the complete metapath to ensure that everything is in mScope
+		// First fetch the complete meta path to ensure that everything is in mScope
 		// This also ensures that the metadata is valid
 		return this.fetchObject(_Helper.getMetaPath(sResolvedPath)).then(function () {
 			// Then fetch mScope
 			return that.fetchEntityContainer();
 		}).then(function (mScope) {
-			var aEditUrl,        // The edit URL as array of segments (encoded)
+			var aEditUrl, // The edit URL as array of segments (encoded)
 				oEntityContainer = mScope[mScope.$EntityContainer],
-				sEntityPath,     // The absolute path to the entity for the PATCH (encoded)
-				oEntitySet,      // The entity set that starts the edit URL
-				sEntitySetName,  // The name of this entity set (decoded)
+				sEntityPath, // The absolute path to the entity for the PATCH (encoded)
+				oEntitySet, // The entity set that starts the edit URL
+				sEntitySetName, // The name of this entity set (decoded)
 				sFirstSegment,
 				bInsideAnnotation = false,
-				sInstancePath,   // The absolute path to the instance currently in evaluation
-								// (encoded; re-builds sResolvedPath)
+				// The absolute path to the instance currently in evaluation (encoded; re-builds
+				// sResolvedPath)
+				sInstancePath,
 				sNavigationPath, // The relative meta path starting from oEntitySet (decoded)
-				//sPropertyPath, // The relative path following sEntityPath (parameter re-used -
-								// encoded)
-				aSegments,       // The resource path split in segments (encoded)
-				oType;           // The type of the data at sInstancePath
+				// The relative path following sEntityPath (parameter re-used - encoded)
+				//sPropertyPath,
+				aSegments, // The resource path split in segments (encoded)
+				oType; // The type of the data at sInstancePath
 
 			// Determines the predicate from a segment (empty string if there is none)
 			function predicate(sSegment) {
 				var i = sSegment.indexOf("(");
+
 				return i >= 0 ? sSegment.slice(i) : "";
 			}
 
@@ -1697,6 +1699,7 @@ sap.ui.define([
 			// Strips off the predicate from a segment
 			function stripPredicate(sSegment) {
 				var i = sSegment.indexOf("(");
+
 				return i >= 0 ? sSegment.slice(0, i) : sSegment;
 			}
 
@@ -1850,7 +1853,7 @@ sap.ui.define([
 		 *
 		 * @returns {string} The annotation target
 		 */
-		function getOverloadTarget () {
+		function getOverloadTarget() {
 			var oOverload = aOverloads[0],
 				sSignature = "";
 
@@ -1909,8 +1912,8 @@ sap.ui.define([
 			});
 
 			if (!aTargets.length) {
-				throw new Error("No annotation '" + sValueList.slice(1) + "' in " +
-					oValueListModel.sServiceUrl);
+				throw new Error("No annotation '" + sValueList.slice(1) + "' in "
+					+ oValueListModel.sServiceUrl);
 			}
 
 			if (aTargets.length === 1) {
@@ -1934,8 +1937,8 @@ sap.ui.define([
 						}
 					});
 				} else if (!bValueListOnValueList) {
-					throw new Error("Unexpected annotation '" + sTerm.slice(1) +
-						"' for target '" + aTargets[0] + "' with namespace of data service in "
+					throw new Error("Unexpected annotation '" + sTerm.slice(1)
+						+ "' for target '" + aTargets[0] + "' with namespace of data service in "
 						+ oValueListModel.sServiceUrl);
 				}
 			});
@@ -2119,7 +2122,7 @@ sap.ui.define([
 	 *   The property
 	 * @param {string} sMetaPath
 	 *   The OData metadata model path corresponding to the given property
-	 * @returns {object}
+	 * @returns {object|undefined}
 	 *   The type constraints for the property or <code>undefined</code> if the property's type is
 	 *   not supported
 	 *
@@ -2165,7 +2168,7 @@ sap.ui.define([
 	 * Returns a snapshot of each $metadata or annotation file loaded so far, combined into a
 	 * single "JSON" object according to the streamlined OData V4 Metadata JSON Format.
 	 *
-	 * @returns {object}
+	 * @returns {object|undefined}
 	 *   The OData metadata as a "JSON" object, if it is already available, or
 	 *   <code>undefined</code>.
 	 *
@@ -2399,7 +2402,7 @@ sap.ui.define([
 	 *
 	 * @param {string} sPropertyPath
 	 *   An absolute path to an OData property within the OData data model
-	 * @returns {string}
+	 * @returns {string|undefined}
 	 *   The path of the property's unit or currency relative to the property's entity, or
 	 *   <code>undefined</code> in case the property has no associated unit or currency
 	 *
@@ -2871,9 +2874,9 @@ sap.ui.define([
 	 *     This way, "/EMPLOYEES" addresses the same object as "/$EntityContainer/EMPLOYEES", namely
 	 *     the "EMPLOYEES" child of the entity container.
 	 *   <li> Afterwards, if the current object is an array, it represents overloads for an
-	 *     operation. Annotations of an operation (since 1.71.0) or a parameter (since 1.66.0) can be
-	 *     immediately addressed, no matter if they apply for a single overload or for all overloads
-	 *     at the same time, for example "/TEAMS/acme.NewAction@" or
+	 *     operation. Annotations of an operation (since 1.71.0) or a parameter (since 1.66.0) can
+	 *     be immediately addressed, no matter if they apply for a single overload or for all
+	 *     overloads at the same time, for example "/TEAMS/acme.NewAction@" or
 	 *     "/TEAMS/acme.NewAction/Team_ID@". Annotations of an unbound operation overload can be
 	 *     addressed like "/OperationImport/@$ui5.overload@", while "/OperationImport/@" addresses
 	 *     annotations of the operation import itself. The special name "$ReturnType" can be used
@@ -3208,6 +3211,7 @@ sap.ui.define([
 					// insert the returned mappings into oValueListInfo in the order of aMappingUrls
 					aMappingUrls.forEach(function (sMappingUrl, i) {
 						var mvalueListMappingByQualifier = aResults[i].valueListMappingByQualifier;
+
 						Object.keys(mvalueListMappingByQualifier).forEach(function (sQualifier) {
 							addMapping(mvalueListMappingByQualifier[sQualifier], sQualifier,
 								sMappingUrl, aResults[i].$model);
@@ -3228,8 +3232,8 @@ sap.ui.define([
 				// Each reference must have contributed at least one qualifier. So if oValueListInfo
 				// is empty, there cannot have been a reference.
 				if (isEmptyObject(oValueListInfo)) {
-					throw new Error("No annotation '" + sValueListReferences.slice(1) + "' for " +
-						sPropertyPath);
+					throw new Error("No annotation '" + sValueListReferences.slice(1) + "' for "
+						+ sPropertyPath);
 				}
 
 				return aRelevantQualifiers && oContext && oContext.getBinding
@@ -3279,7 +3283,6 @@ sap.ui.define([
 	 */
 	ODataMetaModel.prototype.filterValueListRelevantQualifiers = function (mValueListByQualifier,
 		aRawRelevantQualifiers, sMetaPath, oContext) {
-
 		return this.requestValue4Annotation(aRawRelevantQualifiers, sMetaPath, oContext)
 			.then(function (aRelevantQualifiers) {
 				var mValueListByRelevantQualifier = {};
@@ -3334,7 +3337,7 @@ sap.ui.define([
 	 *   A relative or absolute path within the metadata model
 	 * @param {sap.ui.model.Context} [oContext]
 	 *   The context to be used as a starting point in case of a relative path
-	 * @returns {string}
+	 * @returns {string|undefined}
 	 *   Resolved path or <code>undefined</code>
 	 * @throws {Error}
 	 *   If relative path starts with a dot which is not followed by a forward slash

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

@@ -210,7 +210,7 @@ sap.ui.define([
 	 * @extends sap.ui.model.Model
 	 * @public
 	 * @since 1.37.0
-	 * @version 1.97.1
+	 * @version 1.98.0
 	 */
 	var ODataModel = Model.extend("sap.ui.model.odata.v4.ODataModel",
 			/** @lends sap.ui.model.odata.v4.ODataModel.prototype */
@@ -435,7 +435,8 @@ sap.ui.define([
 	 * @private
 	 */
 	ODataModel.prototype.addPrerenderingTask = function (fnPrerenderingTask, bFirst) {
-		var fnRunTasks, iTimeoutId, that = this;
+		var fnRunTasks, iTimeoutId,
+			that = this;
 
 		function runTasks(aTasks) {
 			clearTimeout(iTimeoutId);
@@ -852,7 +853,7 @@ sap.ui.define([
 		 * @param {string[]} aAllowed The allowed system query options
 		 * @throws {error} If the given query option name is not allowed
 		 */
-		function parseAndValidateSystemQueryOption (mOptions, sOptionName, aAllowed) {
+		function parseAndValidateSystemQueryOption(mOptions, sOptionName, aAllowed) {
 			var sExpandOptionName,
 				mExpandOptions,
 				sExpandPath,
@@ -879,7 +880,7 @@ sap.ui.define([
 					}
 				}
 			} else if (sOptionName === "$count") {
-				if (typeof vValue  === "boolean") {
+				if (typeof vValue === "boolean") {
 					if (!vValue) {
 						delete mOptions.$count;
 					}
@@ -1125,7 +1126,7 @@ sap.ui.define([
 				+ "' without context");
 		}
 
-		iSeparator = sResolvedPath.indexOf('#');
+		iSeparator = sResolvedPath.indexOf("#");
 		if (iSeparator >= 0) {
 			sDataPath = sResolvedPath.slice(0, iSeparator);
 			sMetaPath = sResolvedPath.slice(iSeparator + 1);
@@ -1553,12 +1554,25 @@ sap.ui.define([
 		this.oRequestor.refreshSecurityToken().catch(function () {});
 	};
 
+	/**
+	 * Determines whether the given group ID uses mode {@link sap.ui.model.odata.v4.SubmitMode.API}
+	 *
+	 * @param {string} sGroupId
+	 *   The group ID
+	 * @returns {boolean|undefined} Whether it is an API group
+	 *
+	 * @private
+	 */
+	ODataModel.prototype.isApiGroup = function (sGroupId) {
+		return this.getGroupProperty(sGroupId, "submit") === SubmitMode.API;
+	};
+
 	/**
 	 * Determines whether the given group ID uses mode {@link sap.ui.model.odata.v4.SubmitMode.Auto}
 	 *
 	 * @param {string} sGroupId
 	 *   The group ID
-	 * @returns {boolean|undefined} Whether it is an auto group
+	 * @returns {boolean} Whether it is an auto group
 	 *
 	 * @private
 	 */
@@ -1572,7 +1586,7 @@ sap.ui.define([
 	 *
 	 * @param {string} sGroupId
 	 *   The group ID
-	 * @returns {boolean|undefined} Whether it is a direct group
+	 * @returns {boolean} Whether it is a direct group
 	 *
 	 * @private
 	 */
@@ -1663,7 +1677,8 @@ sap.ui.define([
 		}
 		this.checkGroupId(sGroupId);
 
-		// Note: getBindings() returns an array that contains all bindings with change listeners (owned by Model)
+		// Note: getBindings() returns an array that contains all bindings with change listeners
+		// (owned by Model)
 		this.getBindings().forEach(function (oBinding) {
 			if (oBinding.isRoot()) {
 				// ignore the group ID for suspended bindings to avoid mismatches and errors; they
@@ -1748,7 +1763,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 */
-	 ODataModel.prototype.reportStateMessages = function (sResourcePath, mPathToODataMessages,
+	ODataModel.prototype.reportStateMessages = function (sResourcePath, mPathToODataMessages,
 			aCachePaths) {
 		var sDataBindingPath = "/" + sResourcePath,
 			aNewMessages = [],
@@ -1836,7 +1851,7 @@ sap.ui.define([
 	 * @param {string[]} aAbsolutePaths
 	 *   The absolute paths to request side effects for; each path must not start with the fully
 	 *   qualified container name.
-	 * @returns {sap.ui.base.SyncPromise}
+	 * @returns {sap.ui.base.SyncPromise|undefined}
 	 *   A promise resolving without a defined result, or rejecting with an error if loading of side
 	 *   effects fails, or <code>undefined</code> if there is nothing to do
 	 *
@@ -1906,7 +1921,7 @@ sap.ui.define([
 	 *   A relative or absolute path within the data model
 	 * @param {sap.ui.model.Context} [oContext]
 	 *   The context to be used as a starting point in case of a relative path
-	 * @returns {string}
+	 * @returns {string|undefined}
 	 *   Resolved path or <code>undefined</code>
 	 *
 	 * @private

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

@@ -9,12 +9,11 @@
 sap.ui.define([
 	"./Context",
 	"./ODataBinding",
-	"./SubmitMode",
 	"./lib/_Helper",
 	"sap/base/Log",
 	"sap/ui/base/SyncPromise",
 	"sap/ui/model/ChangeReason"
-], function (Context, asODataBinding, SubmitMode, _Helper, Log, SyncPromise, ChangeReason) {
+], function (Context, asODataBinding, _Helper, Log, SyncPromise, ChangeReason) {
 	"use strict";
 
 	/**
@@ -83,9 +82,10 @@ sap.ui.define([
 	 *   The new value which must be primitive
 	 * @param {sap.ui.model.odata.v4.lib._GroupLock} [oGroupLock]
 	 *   A lock for the group ID to be used for the PATCH request; without a lock, no PATCH is sent
-	 * @returns {sap.ui.base.SyncPromise} <code>undefined</code> for the general case which is
-	 *   handled generically by the caller {@link sap.ui.model.odata.v4.Context#doSetProperty}
-	 *   or a <code>SyncPromise</code> for the exceptional case
+	 * @returns {sap.ui.base.SyncPromise|undefined}
+	 *   <code>undefined</code> for the general case which is handled generically by the caller
+	 *   {@link sap.ui.model.odata.v4.Context#doSetProperty} or a <code>SyncPromise</code> for the
+	 *   exceptional case
 	 *
 	 * @abstract
 	 * @function
@@ -413,7 +413,7 @@ sap.ui.define([
 	 *
 	 * @returns {sap.ui.base.SyncPromise}
 	 *   A promise resolving without a defined result when the check is finished, or rejecting in
-	 *   case of an error (e.g. thrown by the change event handler of a control)
+	 *   case of an error
 	 * @throws {Error} If called with parameters
 	 *
 	 * @private
@@ -565,52 +565,41 @@ sap.ui.define([
 	 * Deletes the entity in the cache. If the binding doesn't have a cache, it forwards to the
 	 * parent binding adjusting the path.
 	 *
-	 * @param {sap.ui.model.odata.v4.lib._GroupLock} oGroupLock
-	 *   A lock for the group ID to be used for the DELETE request; if no group ID is specified, it
-	 *   defaults to <code>getUpdateGroupId()</code>()
+	 * @param {sap.ui.model.odata.v4.lib._GroupLock} [oGroupLock]
+	 *   A lock for the group ID to be used for the DELETE request; w/o a lock, no DELETE is sent.
+	 *   For a transient entity, the lock is ignored (use NULL)!
 	 * @param {string} sEditUrl
-	 *   The edit URL to be used for the DELETE request
+	 *   The entity's edit URL to be used for the DELETE request;  w/o a lock, this is mostly
+	 *   ignored.
 	 * @param {string} sPath
 	 *   The path of the entity relative to this binding
 	 * @param {object} [oETagEntity]
 	 *   An entity with the ETag of the binding for which the deletion was requested. This is
 	 *   provided if the deletion is delegated from a context binding with empty path to a list
-	 *   binding.
+	 *   binding. W/o a lock, this is ignored.
 	 * @param {boolean} [bDoNotRequestCount]
 	 *   Whether not to request the new count from the server; useful in case of
 	 *   {@link sap.ui.model.odata.v4.Context#replaceWith} where it is known that the count remains
-	 *   unchanged
+	 *   unchanged; w/o a lock this should be true
 	 * @param {function} fnCallback
-	 *   A function which is called after the entity has been deleted from the server and from the
-	 *   cache; the index of the entity is passed as parameter
+	 *   A function which is called after a transient entity has been deleted from the cache or
+	 *   after the entity has been deleted from the server and from the cache; the index of the
+	 *   entity and the entity list are both passed as parameter, or none of them
 	 * @returns {sap.ui.base.SyncPromise}
 	 *   A promise which is resolved without a result in case of success, or rejected with an
-	 *   instance of <code>Error</code> in case of failure
+	 *   instance of <code>Error</code> in case of failure; returns <code>undefined</code> if the
+	 *   cache promise for this binding is not yet fulfilled
 	 * @throws {Error}
-	 *   If the group ID has {@link sap.ui.model.odata.v4.SubmitMode.Auto} or if the cache promise
-	 *   for this binding is not yet fulfilled
+	 *   If the cache is shared
 	 *
 	 * @private
 	 */
 	ODataParentBinding.prototype.deleteFromCache = function (oGroupLock, sEditUrl, sPath,
 			oETagEntity, bDoNotRequestCount, fnCallback) {
-		var sGroupId;
-
-		if (this.oCache === undefined) {
-			throw new Error("DELETE request not allowed");
-		}
-
-		if (this.oCache) {
-			sGroupId = oGroupLock.getGroupId();
-			if (!this.oModel.isAutoGroup(sGroupId) && !this.oModel.isDirectGroup(sGroupId)) {
-				throw new Error("Illegal update group ID: " + sGroupId);
-			}
-			return this.oCache._delete(oGroupLock, sEditUrl, sPath, oETagEntity, bDoNotRequestCount,
+		return this.withCache(function (oCache, sCachePath) {
+			return oCache._delete(oGroupLock, sEditUrl, sCachePath, oETagEntity, bDoNotRequestCount,
 				fnCallback);
-		}
-		return this.oContext.getBinding().deleteFromCache(oGroupLock, sEditUrl,
-			_Helper.buildPath(this.oContext.iIndex, this.sPath, sPath), oETagEntity,
-			bDoNotRequestCount, fnCallback);
+		}, sPath, /*bSync*/true);
 	};
 
 	/**
@@ -651,7 +640,8 @@ sap.ui.define([
 	 *   The child binding's (aggregated) query options or a promise resolving with them
 	 * @returns {sap.ui.base.SyncPromise}
 	 *   A promise resolved with the reduced path for the child binding if the child binding can use
-	 *   this binding's or an ancestor binding's cache; <code>undefined</code> otherwise.
+	 *   this binding's or an ancestor binding's cache; resolved with <code>undefined</code>
+	 *   otherwise.
 	 *
 	 * @private
 	 * @see sap.ui.model.odata.v4.ODataMetaModel#getReducedPath
@@ -767,7 +757,7 @@ sap.ui.define([
 				that.bAggregatedQueryOptionsInitial = false;
 			}
 			if (bIsAdvertisement) {
-				mWrappedChildQueryOptions = {"$select" : [sReducedChildMetaPath.slice(1)]};
+				mWrappedChildQueryOptions = {$select : [sReducedChildMetaPath.slice(1)]};
 				return that.aggregateQueryOptions(mWrappedChildQueryOptions, sBaseMetaPath,
 						bCacheImmutable)
 					? sReducedPath
@@ -833,7 +823,7 @@ sap.ui.define([
 		// catch the error, but keep the rejected promise
 		this.oCachePromise.catch(function (oError) {
 			that.oModel.reportError(that + ": Failed to enhance query options for "
-				+ "auto-$expand/$select for child " + sChildPath,  sClassName, oError);
+				+ "auto-$expand/$select for child " + sChildPath, sClassName, oError);
 		});
 		return oCanUseCachePromise;
 	};
@@ -900,8 +890,7 @@ sap.ui.define([
 			oParentBinding = this.oContext.getBinding();
 			sParentUpdateGroupId = oParentBinding.getUpdateGroupId();
 			if (sParentUpdateGroupId === this.getUpdateGroupId()
-					|| this.oModel.getGroupProperty(sParentUpdateGroupId, "submit")
-						!== SubmitMode.API) {
+					|| !this.oModel.isApiGroup(sParentUpdateGroupId)) {
 				return oParentBinding.getBaseForPathReduction();
 			}
 		}
@@ -1072,9 +1061,11 @@ sap.ui.define([
 	 * @param {boolean} [bKeepCacheOnError]
 	 *   If <code>true</code>, the binding data remains unchanged if the refresh fails
 	 * @returns {sap.ui.base.SyncPromise}
-	 *   A promise resolving when all dependent bindings are refreshed; it is rejected if the
-	 *   binding's root binding is suspended and a group ID different from the binding's group ID is
-	 *   given
+	 *   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}
+	 *   If the binding's root binding is suspended and a group ID different from the binding's
+	 *   group ID is given
 	 *
 	 * @private
 	 */
@@ -1090,7 +1081,11 @@ sap.ui.define([
 	 * Recursively refreshes all dependent list bindings that have no own cache.
 	 *
 	 * @returns {sap.ui.base.SyncPromise}
-	 *   A promise resolving when all dependent list bindings without own cache are refreshed
+	 *   A promise resolving when all dependent list bindings without own cache are refreshed; it is
+	 *   rejected when the refresh fails; the promise is resolved immediately on a suspended binding
+	 * @throws {Error}
+	 *   If the binding's root binding is suspended and a group ID different from the binding's
+	 *   group ID is given
 	 *
 	 * @private
 	 */
@@ -1118,24 +1113,6 @@ sap.ui.define([
 		}
 	};
 
-	/**
-	 * Refreshes the binding; expects it to be suspended.
-	 *
-	 * @param {string} sGroupId
-	 *   The group ID to be used for the refresh
-	 * @throws {Error}
-	 *   If a group ID different from the binding's group ID is given
-
-	 * @private
-	 */
-	ODataParentBinding.prototype.refreshSuspended = function (sGroupId) {
-		if (sGroupId && sGroupId !== this.getGroupId()) {
-			throw new Error(this + ": Cannot refresh a suspended binding with group ID '"
-				+ sGroupId  + "' (own group ID is '" + this.getGroupId() + "')");
-		}
-		this.setResumeChangeReason(ChangeReason.Refresh);
-	};
-
 	/**
 	 * Loads side effects for the given context of this binding.
 	 *