@openui5/sap.ui.core 1.97.1 → 1.99.1

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
@@ -191,15 +191,16 @@ sap.ui.define([
 	 * "5.1.2 System Query Option $expand" and "5.1.3 System Query Option $select" in specification
 	 * "OData Version 4.0 Part 2: URL Conventions".
 	 *
-	 * @param {object} mQueryOptions The query options to be merged
-	 * @param {string} sBaseMetaPath This binding's meta path
-	 * @param {boolean} bCacheImmutable Whether the cache of this binding is immutable
+	 * @param {object} mQueryOptions - The query options to be merged
+	 * @param {string} sBaseMetaPath - This binding's meta path
+	 * @param {boolean} bCacheImmutable - Whether the cache of this binding is immutable
+	 * @param {boolean} bIsProperty - Whether the child is a property binding
 	 * @returns {boolean} Whether the query options can be fulfilled by this binding
 	 *
 	 * @private
 	 */
 	ODataParentBinding.prototype.aggregateQueryOptions = function (mQueryOptions, sBaseMetaPath,
-			bCacheImmutable) {
+			bCacheImmutable, bIsProperty) {
 		var mAggregatedQueryOptionsClone = _Helper.merge({},
 				bCacheImmutable && this.mLateQueryOptions || this.mAggregatedQueryOptions),
 			bChanged = false,
@@ -256,8 +257,9 @@ sap.ui.define([
 
 			// Top-level all query options in the aggregate are OK, even if not repeated in the
 			// child. In a $expand the child must also have them (and the second loop checks that
-			// they're equal).
-			return (!bInsideExpand || Object.keys(mAggregatedQueryOptions).every(function (sName) {
+			// they're equal). Property bindings are an exception to this rule.
+			return (bIsProperty || !bInsideExpand
+				|| Object.keys(mAggregatedQueryOptions).every(function (sName) {
 					return sName in mQueryOptions0 || sName === "$count" || sName === "$expand"
 						|| sName === "$select";
 				}))
@@ -321,6 +323,8 @@ sap.ui.define([
 	 *   pending changes are ignored if all parameters are unchanged. Since 1.97.0, pending changes
 	 *   are ignored if they relate to a
 	 *   {@link sap.ui.model.odata.v4.Context#setKeepAlive kept-alive} context of this binding.
+	 *   Since 1.98.0, {@link sap.ui.model.odata.v4.Context#isTransient transient} contexts
+	 *   of a {@link #getRootBinding root binding} do not count as pending changes.
 	 *
 	 * @public
 	 * @since 1.45.0
@@ -413,7 +417,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
@@ -460,6 +464,9 @@ sap.ui.define([
 	 *   A (temporary) key predicate for the transient entity: "($uid=...)"
 	 * @param {object} [oInitialData={}]
 	 *   The initial data for the created entity
+	 * @param {boolean} bAtEndOfCreated
+	 *   Whether the newly created entity should be inserted after previously created entities, not
+	 *   before them
 	 * @param {function} fnErrorCallback
 	 *   A function which is called with an error object each time a POST request for the create
 	 *   fails
@@ -472,7 +479,8 @@ sap.ui.define([
 	 * @private
 	 */
 	ODataParentBinding.prototype.createInCache = function (oUpdateGroupLock, vCreatePath,
-			sCollectionPath, sTransientPredicate, oInitialData, fnErrorCallback, fnSubmitCallback) {
+			sCollectionPath, sTransientPredicate, oInitialData, bAtEndOfCreated, fnErrorCallback,
+			fnSubmitCallback) {
 		var that = this;
 
 		return this.oCachePromise.then(function (oCache) {
@@ -481,7 +489,8 @@ sap.ui.define([
 			if (oCache) {
 				sPathInCache = _Helper.getRelativePath(sCollectionPath, that.getResolvedPath());
 				return oCache.create(oUpdateGroupLock, vCreatePath, sPathInCache,
-					sTransientPredicate, oInitialData, fnErrorCallback, fnSubmitCallback
+					sTransientPredicate, oInitialData, bAtEndOfCreated, fnErrorCallback,
+					fnSubmitCallback
 				).then(function (oCreatedEntity) {
 					if (that.mCacheByResourcePath) {
 						// Ensure that cache containing non-transient created entity is recreated
@@ -492,8 +501,8 @@ sap.ui.define([
 				});
 			}
 			return that.oContext.getBinding().createInCache(oUpdateGroupLock, vCreatePath,
-				sCollectionPath, sTransientPredicate, oInitialData, fnErrorCallback,
-				fnSubmitCallback);
+				sCollectionPath, sTransientPredicate, oInitialData, bAtEndOfCreated,
+				fnErrorCallback, fnSubmitCallback);
 		});
 	};
 
@@ -565,52 +574,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);
 	};
 
 	/**
@@ -620,7 +618,7 @@ sap.ui.define([
 	 * @since 1.61
 	 */
 	ODataParentBinding.prototype.destroy = function () {
-//		this.mAggregatedQueryOptions = undefined;
+		// this.mAggregatedQueryOptions = undefined;
 		this.aChildCanUseCachePromises = [];
 		this.removeReadGroupLock();
 		this.oResumePromise = undefined;
@@ -649,15 +647,18 @@ sap.ui.define([
 	 *   The child binding's binding path relative to <code>oContext</code>
 	 * @param {object|sap.ui.base.SyncPromise} vChildQueryOptions
 	 *   The child binding's (aggregated) query options or a promise resolving with them
+	 * @param {boolean} bIsProperty
+	 *   Whether the child is a property binding
 	 * @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
 	 */
 	ODataParentBinding.prototype.fetchIfChildCanUseCache = function (oContext, sChildPath,
-			vChildQueryOptions) {
+			vChildQueryOptions, bIsProperty) {
 		// getBaseForPathReduction must be called early, because the (virtual) parent context may be
 		// lost again when the path is needed
 		var sBaseForPathReduction = this.getBaseForPathReduction(),
@@ -767,9 +768,9 @@ sap.ui.define([
 				that.bAggregatedQueryOptionsInitial = false;
 			}
 			if (bIsAdvertisement) {
-				mWrappedChildQueryOptions = {"$select" : [sReducedChildMetaPath.slice(1)]};
+				mWrappedChildQueryOptions = {$select : [sReducedChildMetaPath.slice(1)]};
 				return that.aggregateQueryOptions(mWrappedChildQueryOptions, sBaseMetaPath,
-						bCacheImmutable)
+						bCacheImmutable, bIsProperty)
 					? sReducedPath
 					: undefined;
 			}
@@ -781,7 +782,7 @@ sap.ui.define([
 					that.oModel.oInterface.fetchMetadata);
 				if (mWrappedChildQueryOptions) {
 					return that.aggregateQueryOptions(mWrappedChildQueryOptions, sBaseMetaPath,
-							bCacheImmutable)
+							bCacheImmutable, bIsProperty)
 						? sReducedPath
 						: undefined;
 				}
@@ -789,7 +790,7 @@ sap.ui.define([
 			}
 			if (sReducedChildMetaPath === "value") { // symbolic name for operation result
 				return that.aggregateQueryOptions(mChildQueryOptions, sBaseMetaPath,
-						bCacheImmutable)
+						bCacheImmutable, bIsProperty)
 					? sReducedPath
 					: undefined;
 			}
@@ -833,7 +834,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;
 	};
@@ -866,9 +867,13 @@ sap.ui.define([
 		sMetaPath = _Helper.getMetaPath(oModel.resolve(this.sPath, oContext));
 		mConvertedQueryOptions = Object.assign({}, mQueryOptions, {$select : []});
 		return SyncPromise.all(mQueryOptions.$select.map(function (sSelectPath) {
-			return _Helper.fetchPropertyAndType(
-				fnFetchMetadata, sMetaPath + "/" + sSelectPath
-			).then(function () {
+			var sMetaSelectPath = sMetaPath + "/" + sSelectPath;
+
+			if (sMetaSelectPath.endsWith(".*")) { // fetch metadata for namespace itself
+				sMetaSelectPath = sMetaSelectPath.slice(0, -1);
+			}
+
+			return _Helper.fetchPropertyAndType(fnFetchMetadata, sMetaSelectPath).then(function () {
 				var mWrappedQueryOptions = _Helper.wrapChildQueryOptions(
 						sMetaPath, sSelectPath, {}, fnFetchMetadata);
 
@@ -900,8 +905,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();
 			}
 		}
@@ -1000,17 +1004,24 @@ sap.ui.define([
 	 * @override
 	 * @see sap.ui.model.odata.v4.ODataBinding#hasPendingChangesInDependents
 	 */
-	ODataParentBinding.prototype.hasPendingChangesInDependents = function (bIgnoreKeptAlive) {
+	ODataParentBinding.prototype.hasPendingChangesInDependents = function (bIgnoreKeptAlive0) {
 		return this.getDependentBindings().some(function (oDependent) {
 			var oCache = oDependent.oCache,
-				bHasPendingChanges;
+				bHasPendingChanges,
+				bIgnoreKeptAlive = bIgnoreKeptAlive0; // new copy for this dependent only
 
-			if (bIgnoreKeptAlive && oDependent.oContext.isKeepAlive()) {
-				return false;
+			if (bIgnoreKeptAlive) {
+				if (oDependent.oContext.isKeepAlive()) {
+					return false; // changes can be safely ignored here
+				}
+				if (oDependent.oContext.getIndex() !== undefined) {
+					bIgnoreKeptAlive = false; // context of ODLB which is not kept alive: unsafe!
+				}
 			}
 			if (oCache !== undefined) {
 				// Pending changes for this cache are only possible when there is a cache already
-				if (oCache && oCache.hasPendingChangesForPath("")) {
+				if (oCache && oCache.hasPendingChangesForPath("", false,
+						bIgnoreKeptAlive && oDependent.mParameters.$$ownRequest)) {
 					return true;
 				}
 			} else if (oDependent.hasPendingChangesForPath("")) {
@@ -1029,7 +1040,7 @@ sap.ui.define([
 				}
 			}
 			// Ask dependents, they might have no cache, but pending changes in mCacheByResourcePath
-			return oDependent.hasPendingChangesInDependents();
+			return oDependent.hasPendingChangesInDependents(bIgnoreKeptAlive);
 		})
 		|| this.oModel.withUnresolvedBindings("hasPendingChangesInCaches",
 				this.getResolvedPath().slice(1));
@@ -1072,9 +1083,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 +1103,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 +1135,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.
 	 *
@@ -1337,7 +1336,8 @@ sap.ui.define([
 	 * data service requests. Call {@link #resume} to resume the binding. Before 1.53.0, this method
 	 * was not supported and threw an error. Since 1.97.0, pending changes are ignored if they
 	 * relate to a {@link sap.ui.model.odata.v4.Context#setKeepAlive kept-alive} context of this
-	 * binding.
+	 * binding. Since 1.98.0, {@link sap.ui.model.odata.v4.Context#isTransient transient} contexts
+	 * of a {@link #getRootBinding root binding} do not count as pending changes.
 	 *
 	 * @throws {Error}
 	 *   If this binding

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

@@ -29,7 +29,38 @@ sap.ui.define([
 			dataRequested : true,
 			DataStateChange : true
 		},
-		sVirtualPath = "/" + Context.VIRTUAL; // a snippet indicating a virtual path
+		sVirtualPath = "/" + Context.VIRTUAL, // a snippet indicating a virtual path
+		/**
+		 * @alias sap.ui.model.odata.v4.ODataPropertyBinding
+		 * @author SAP SE
+		 * @class Property binding for an OData V4 model.
+		 *   An event handler can only be attached to this binding for the following events:
+		 *   'AggregatedDataStateChange', 'change', 'dataReceived', 'dataRequested' and
+		 *   'DataStateChange'. For unsupported events, an error is thrown.
+		 * @extends sap.ui.model.PropertyBinding
+		 * @hideconstructor
+		 * @mixes sap.ui.model.odata.v4.ODataBinding
+		 * @public
+		 * @since 1.37.0
+		 * @version 1.99.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
+		 * @borrows sap.ui.model.odata.v4.ODataBinding#hasPendingChanges as #hasPendingChanges
+		 * @borrows sap.ui.model.odata.v4.ODataBinding#isInitial as #isInitial
+		 * @borrows sap.ui.model.odata.v4.ODataBinding#refresh as #refresh
+		 * @borrows sap.ui.model.odata.v4.ODataBinding#requestRefresh as #requestRefresh
+		 * @borrows sap.ui.model.odata.v4.ODataBinding#resetChanges as #resetChanges
+		 * @borrows sap.ui.model.odata.v4.ODataBinding#toString as #toString
+		 */
+		ODataPropertyBinding
+			= PropertyBinding.extend("sap.ui.model.odata.v4.ODataPropertyBinding", {
+				constructor : constructor
+			});
+
+	//*********************************************************************************************
+	// ODataPropertyBinding
+	//*********************************************************************************************
 
 	/**
 	 * Do <strong>NOT</strong> call this private constructor, but rather use
@@ -45,66 +76,37 @@ sap.ui.define([
 	 *   Map of binding parameters
 	 * @throws {Error}
 	 *   If disallowed binding parameters are provided
-	 *
-	 * @alias sap.ui.model.odata.v4.ODataPropertyBinding
-	 * @author SAP SE
-	 * @class Property binding for an OData V4 model.
-	 *   An event handler can only be attached to this binding for the following events:
-	 *   'AggregatedDataStateChange', 'change', 'dataReceived', 'dataRequested' and
-	 *   'DataStateChange'. For unsupported events, an error is thrown.
-	 * @extends sap.ui.model.PropertyBinding
-	 * @hideconstructor
-	 * @mixes sap.ui.model.odata.v4.ODataBinding
-	 * @public
-	 * @since 1.37.0
-	 * @version 1.97.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
-	 * @borrows sap.ui.model.odata.v4.ODataBinding#hasPendingChanges as #hasPendingChanges
-	 * @borrows sap.ui.model.odata.v4.ODataBinding#isInitial as #isInitial
-	 * @borrows sap.ui.model.odata.v4.ODataBinding#refresh as #refresh
-	 * @borrows sap.ui.model.odata.v4.ODataBinding#requestRefresh as #requestRefresh
-	 * @borrows sap.ui.model.odata.v4.ODataBinding#resetChanges as #resetChanges
-	 * @borrows sap.ui.model.odata.v4.ODataBinding#toString as #toString
-	 */
-	var ODataPropertyBinding
-		= PropertyBinding.extend("sap.ui.model.odata.v4.ODataPropertyBinding", {
-			constructor : function (oModel, sPath, oContext, mParameters) {
-
-				PropertyBinding.call(this, oModel, sPath);
-				// initialize mixin members
-				asODataBinding.call(this);
-
-				if (sPath.endsWith("/")) {
-					throw new Error("Invalid path: " + sPath);
-				}
-				if (mParameters) {
-					this.checkBindingParameters(mParameters,
-						["$$groupId", "$$ignoreMessages", "$$noPatch"]);
-					this.sGroupId = mParameters.$$groupId;
-					this.bNoPatch = mParameters.$$noPatch;
-					this.setIgnoreMessages(mParameters.$$ignoreMessages);
-				} else {
-					this.sGroupId = undefined;
-					this.bNoPatch = false;
-				}
-				this.oCheckUpdateCallToken = undefined;
-				this.oContext = oContext;
-				this.bHasDeclaredType = undefined; // whether the binding info declares a type
-				this.bInitial = true;
-				// Note: system query options supported at property binding only for ".../$count"
-				this.mQueryOptions = this.oModel.buildQueryOptions(_Helper.clone(mParameters),
-					/*bSystemQueryOptionsAllowed*/sPath.endsWith("$count"));
-				this.vValue = undefined;
-				// BEWARE: #doFetchQueryOptions uses #isRoot which relies on this.oContext!
-				this.fetchCache(oContext);
-				oModel.bindingCreated(this);
-			},
-			metadata : {
-				publicMethods : []
-			}
-		});
+	 */
+	function constructor(oModel, sPath, oContext, mParameters) {
+		PropertyBinding.call(this, oModel, sPath);
+		// initialize mixin members
+		asODataBinding.call(this);
+
+		if (sPath.endsWith("/")) {
+			throw new Error("Invalid path: " + sPath);
+		}
+		if (mParameters) {
+			this.checkBindingParameters(mParameters,
+				["$$groupId", "$$ignoreMessages", "$$noPatch"]);
+			this.sGroupId = mParameters.$$groupId;
+			this.bNoPatch = mParameters.$$noPatch;
+			this.setIgnoreMessages(mParameters.$$ignoreMessages);
+		} else {
+			this.sGroupId = undefined;
+			this.bNoPatch = false;
+		}
+		this.oCheckUpdateCallToken = undefined;
+		this.oContext = oContext;
+		this.bHasDeclaredType = undefined; // whether the binding info declares a type
+		this.bInitial = true;
+		// Note: system query options supported at property binding only for ".../$count"
+		this.mQueryOptions = this.oModel.buildQueryOptions(_Helper.clone(mParameters),
+			/*bSystemQueryOptionsAllowed*/sPath.endsWith("$count"));
+		this.vValue = undefined;
+		// BEWARE: #doFetchQueryOptions uses #isRoot which relies on this.oContext!
+		this.fetchCache(oContext);
+		oModel.bindingCreated(this);
+	}
 
 	asODataBinding(ODataPropertyBinding.prototype);
 
@@ -230,9 +232,9 @@ sap.ui.define([
 	 *   The new value obtained from the cache, see {@link #onChange}
 	 * @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). If the cache is no
-	 *   longer the active cache when the response arrives, that response is silently ignored and
-	 *   the value remains unchanged.
+	 *   case of an error. If the cache is no longer the active cache when the response arrives,
+	 *   that response is ignored almost silently (that is, with a canceled error) and the value
+	 *   remains unchanged.
 	 *
 	 * @private
 	 * @see sap.ui.model.PropertyBinding#checkDataState
@@ -310,20 +312,19 @@ sap.ui.define([
 						|| (that.sPath[that.sPath.lastIndexOf("/") + 1] === "#" && !bIsMeta))) {
 					if (bIsMeta) {
 						return vValue;
-					} else if (that.bRelative){
+					} else if (that.bRelative) {
 						return _Helper.publicClone(vValue);
 					}
 				}
 				Log.error("Accessed value is not primitive", sResolvedPath, sClassName);
 			}, function (oError) {
-				// do not rethrow, ManagedObject doesn't react on this either
-				// throwing an error would cause "Uncaught (in promise)" in Chrome
 				that.oModel.reportError("Failed to read path " + sResolvedPath, sClassName, oError);
 				if (oError.canceled) { // canceled -> value remains unchanged
 					oCallToken.forceUpdate = false;
 					return that.vValue;
 				}
 				mParametersForDataReceived = {error : oError};
+				// oError is re-thrown below
 			});
 			if (bForceUpdate && vValue.isFulfilled()) {
 				if (vType && vType.isFulfilled && vType.isFulfilled()) {
@@ -352,6 +353,9 @@ sap.ui.define([
 			if (bDataRequested) {
 				that.fireDataReceived(mParametersForDataReceived);
 			}
+			if (mParametersForDataReceived.error) {
+				throw mParametersForDataReceived.error;
+			}
 		});
 	};
 
@@ -493,7 +497,8 @@ sap.ui.define([
 	 * @private
 	 */
 	ODataPropertyBinding.prototype.onChange = function (vValue) {
-		this.checkUpdateInternal(undefined, undefined, undefined, vValue);
+		this.checkUpdateInternal(undefined, undefined, undefined, vValue)
+			.catch(this.oModel.getReporter());
 	};
 
 	/**
@@ -505,7 +510,7 @@ sap.ui.define([
 		var that = this;
 
 		if (this.isRootBindingSuspended()) {
-			this.sResumeChangeReason = ChangeReason.Refresh;
+			this.refreshSuspended(sGroupId);
 			return SyncPromise.resolve();
 		}
 		return this.oCachePromise.then(function () {
@@ -522,7 +527,7 @@ sap.ui.define([
 	 *
 	 * @returns {Promise}
 	 *   A promise resolving with the resulting value or <code>undefined</code> if it could not be
-	 *   determined
+	 *   determined, or rejecting in case of an error
 	 *
 	 * @public
 	 * @since 1.69
@@ -633,7 +638,8 @@ sap.ui.define([
 
 		this.fetchCache(this.oContext);
 		if (bCheckUpdate) {
-			this.checkUpdateInternal(bParentHasChanges ? undefined : false, sResumeChangeReason);
+			this.checkUpdateInternal(bParentHasChanges ? undefined : false, sResumeChangeReason)
+				.catch(this.oModel.getReporter());
 		}
 	};
 
@@ -660,7 +666,8 @@ sap.ui.define([
 			this.sResumeChangeReason = undefined;
 			if (this.bRelative) {
 				this.fetchCache(this.oContext);
-				this.checkUpdateInternal(this.bInitial || undefined, ChangeReason.Context);
+				this.checkUpdateInternal(this.bInitial || undefined, ChangeReason.Context)
+					.catch(this.oModel.getReporter());
 			}
 		}
 	};
@@ -705,15 +712,14 @@ sap.ui.define([
 	 *   this binding (or its relevant parent binding) is used, see {@link #getUpdateGroupId}.
 	 *   Valid values are <code>undefined</code>, '$auto', '$auto.*', '$direct' or application group
 	 *   IDs as specified in {@link sap.ui.model.odata.v4.ODataModel}.
-	 * @throws {Error}
-	 *   If one of the following situations occurs:
+	 * @throws {Error} If
 	 *   <ul>
-	 *     <li> The binding's root binding is suspended.
-	 *     <li> The new value is not primitive.
-	 *     <li> No value has been read before and the binding does not have the parameter
+	 *     <li> the binding's root binding is suspended.
+	 *     <li> the new value is not primitive.
+	 *     <li> no value has been read before and the binding does not have the parameter
 	 *       <code>$$noPatch</code>.
-	 *     <li> The binding is not relative to a {@link sap.ui.model.odata.v4.Context}.
-	 *     <li> The binding has the parameter <code>$$noPatch</code> and a group ID has been given.
+	 *     <li> the binding is not relative to a {@link sap.ui.model.odata.v4.Context}.
+	 *     <li> the binding has the parameter <code>$$noPatch</code> and a group ID has been given.
 	 *   </ul>
 	 *
 	 * @public

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

@@ -116,13 +116,13 @@ sap.ui.define([
 			 *   </ul>
 			 *   If the specified <code>sResponseBody</code> contains responses for change sets,
 			 *   then the corresponding response objects will be returned in a nested array.
-			 * @throws {Error}
+			 * @throws {Error} If
 			 *   <ul>
-			 *     <li> If the <code>sContentType</code> parameter does not represent a
+			 *     <li> the <code>sContentType</code> parameter does not represent a
 			 *       "multipart/mixed" media type with "boundary" parameter
-			 *     <li> If the "charset" parameter of the "Content-Type" header of a nested response
+			 *     <li> the "charset" parameter of the "Content-Type" header of a nested response
 			 *       has a value other than "UTF-8"
-			 *     <li> If there is no "Content-ID" header for a change set response or its value is
+			 *     <li> there is no "Content-ID" header for a change set response or its value is
 			 *       not a number
 			 *   </ul>
 			 *
@@ -241,7 +241,7 @@ sap.ui.define([
 						// "round" to millis: "HH:mm:ss.SSS"
 						sTimeOfDay = sTimeOfDay.slice(0, 12);
 					}
-					oTimeOfDay =  oTimeFormatter.parse(sTimeOfDay);
+					oTimeOfDay = oTimeFormatter.parse(sTimeOfDay);
 				}
 				if (!oTimeOfDay) {
 					throw new Error("Not a valid Edm.TimeOfDay value: " + sTimeOfDay);

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

@@ -39,5 +39,4 @@ sap.ui.define(function () {
 	};
 
 	return SubmitMode;
-
 }, /* bExport= */ true);