[INTERNAL] sap.ui.model.odata.v4.ODataModel: make synchronizationMode optional

PS1: downport

Change-Id: I9cda5c7094a0591bffe568b535b11c8682ed6e62
CR-Id: 002075125900004730852025
SNOW: CS20250011008073

Author: larskissel

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

@@ -133,7 +133,7 @@ sap.ui.define([
 	 *   Whether <code>&lt;edmx:Reference></code> and <code>&lt;edmx:Include></code> directives are
 	 *   supported in order to load schemas on demand from other $metadata documents and include
 	 *   them into the current service ("cross-service references").
-	 * @param {string} mParameters.synchronizationMode
+	 * @param {string} [mParameters.synchronizationMode]
 	 *   Controls synchronization between different bindings which refer to the same data for the
 	 *   case data changes in one binding. Must be set to 'None' which means bindings are not
 	 *   synchronized at all; all other values are not supported and lead to an error.
@@ -205,7 +205,9 @@ sap.ui.define([
 					// do not pass any parameters to Model
 					Model.apply(this);
 
-					if (!mParameters || mParameters.synchronizationMode !== "None") {
+					mParameters = mParameters || {};
+					if ("synchronizationMode" in mParameters
+							&& mParameters.synchronizationMode !== "None") {
 						throw new Error("Synchronization mode must be 'None'");
 					}
 					sODataVersion = mParameters.odataVersion || "4.0";

src/sap.ui.core/test/sap/ui/core/qunit/odata/v4/ODataModel.qunit.js

@@ -76,8 +76,7 @@ sap.ui.define([
 	 */
 	function createModel(sQuery, mParameters) {
 		mParameters = jQuery.extend({}, mParameters, {
-			serviceUrl : getServiceUrl() + (sQuery || ""),
-			synchronizationMode : "None"
+			serviceUrl : getServiceUrl() + (sQuery || "")
 		});
 		return new ODataModel(mParameters);
 	}
@@ -120,20 +119,22 @@ sap.ui.define([
 			mModelOptions = {};
 
 		assert.throws(function () {
-			return new ODataModel();
+			return new ODataModel({synchronizationMode : undefined});
+		}, new Error("Synchronization mode must be 'None'"));
+		assert.throws(function () {
+			return new ODataModel({synchronizationMode : "Nope"});
 		}, new Error("Synchronization mode must be 'None'"));
 		assert.throws(function () {
-			return new ODataModel({synchronizationMode : "None"});
+			return new ODataModel();
 		}, new Error("Missing service root URL"));
 		assert.throws(function () {
-			return new ODataModel({serviceUrl : "/foo", synchronizationMode : "None"});
+			return new ODataModel({serviceUrl : "/foo"});
 		}, new Error("Service root URL must end with '/'"));
 		assert.throws(function () {
-			return new ODataModel({synchronizationMode : "None", useBatch : true});
+			return new ODataModel({useBatch : true});
 		}, new Error("Unsupported parameter: useBatch"));
 		assert.throws(function () {
-			return new ODataModel({operationMode : OperationMode.Auto, serviceUrl : "/foo/",
-				synchronizationMode : "None"});
+			return new ODataModel({operationMode : OperationMode.Auto, serviceUrl : "/foo/"});
 		}, new Error("Unsupported operation mode: Auto"), "Unsupported OperationMode");
 		oModelPrototypeMock.expects("initializeSecurityToken").never();