@openui5/sap.ui.fl 1.115.1 → 1.117.0

package/src/sap/ui/fl/FakeLrepConnector.js

@@ -69,18 +69,18 @@ sap.ui.define([
 	};
 
 	FakeLrepConnector.forTesting = {
-		getNumberOfChanges: function (oConnector, sReference) {
+		getNumberOfChanges: function(oConnector, sReference) {
 			return oConnector.loadFlexData({reference: sReference})
-			.then(function (aResponses) {
-				return aResponses.reduce(function (iNumberOfChanges, oResponse) {
+			.then(function(aResponses) {
+				return aResponses.reduce(function(iNumberOfChanges, oResponse) {
 					return iNumberOfChanges + oResponse.changes.length;
 				}, 0);
 			});
 		},
-		spyMethod: function (sandbox, assert, oConnector, sMethod) {
+		spyMethod: function(sandbox, assert, oConnector, sMethod) {
 			var oSpy = sandbox.spy(oConnector, sMethod);
 
-			return function (iNumberOfExpectedObjects, iCallIndex) {
+			return function(iNumberOfExpectedObjects, iCallIndex) {
 				iCallIndex = iCallIndex || 0;
 				var iNumberOfObjects = oSpy.getCall(iCallIndex).args[0].flexObjects.length;
 				assert.equal(iNumberOfObjects, iNumberOfExpectedObjects, sMethod + " was called " + iNumberOfExpectedObjects + " times");
@@ -94,7 +94,7 @@ sap.ui.define([
 			oConnector.storage = oNewStorage;
 		},
 		synchronous: {
-			clearAll: function (oStorage) {
+			clearAll: function(oStorage) {
 				var fnRemoveItem = function(sKey) {
 					var bIsFlexObject = sKey.includes(FL_PREFIX);
 
@@ -107,7 +107,7 @@ sap.ui.define([
 
 				Object.keys(oStorage).map(fnRemoveItem);
 			},
-			store: function (oStorage, sKey, oItem) {
+			store: function(oStorage, sKey, oItem) {
 				var sFlexKey = ObjectStorageUtils.createFlexKey(sKey);
 				var sItem = JSON.stringify(oItem);
 				oStorage.setItem(sFlexKey, sItem);

package/src/sap/ui/fl/FakeLrepConnectorLocalStorage.js

@@ -20,7 +20,7 @@ function(
 	 * @namespace
 	 *
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 *
 	 * @private
 	 * @since 1.48
@@ -34,25 +34,25 @@ function(
 	 */
 
 	return {
-		enableFakeConnector: function (mPropertyBag) {
+		enableFakeConnector: function(mPropertyBag) {
 			var sJsonPath = mPropertyBag ? mPropertyBag.sInitialComponentJsonPath : undefined;
 			FakeLrepConnector.setFlexibilityServicesAndClearCache("LocalStorageConnector", sJsonPath);
 		},
-		disableFakeConnector: function () {
+		disableFakeConnector: function() {
 			FakeLrepConnector.disableFakeConnector();
 		},
 		forTesting: {
-			spyWrite: function (sandbox, assert) {
+			spyWrite: function(sandbox, assert) {
 				return FakeLrepConnector.forTesting.spyMethod(sandbox, assert, LocalStorageConnector, "write");
 			},
-			getNumberOfChanges: function (sReference) {
+			getNumberOfChanges: function(sReference) {
 				return FakeLrepConnector.forTesting.getNumberOfChanges(LocalStorageConnector, sReference);
 			},
 			synchronous: {
-				clearAll: function () {
+				clearAll: function() {
 					FakeLrepConnector.forTesting.synchronous.clearAll(window.localStorage);
 				},
-				store: function (sKey, oItem) {
+				store: function(sKey, oItem) {
 					FakeLrepConnector.forTesting.synchronous.store(window.localStorage, sKey, oItem);
 				}
 			}

package/src/sap/ui/fl/FakeLrepConnectorSessionStorage.js

@@ -20,7 +20,7 @@ function(
 	 * @namespace
 	 *
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 *
 	 * @private
 	 * @since 1.58
@@ -34,7 +34,7 @@ function(
 	 */
 
 	return {
-		enableFakeConnector: function (mPropertyBag) {
+		enableFakeConnector: function(mPropertyBag) {
 			var sJsonPath = mPropertyBag ? mPropertyBag.sInitialComponentJsonPath : undefined;
 			FakeLrepConnector.setFlexibilityServicesAndClearCache("SessionStorageConnector", sJsonPath);
 		},
@@ -42,10 +42,10 @@ function(
 			FakeLrepConnector.disableFakeConnector();
 		},
 		forTesting: {
-			spyWrite: function (sandbox, assert) {
+			spyWrite: function(sandbox, assert) {
 				return FakeLrepConnector.forTesting.spyMethod(sandbox, assert, SessionStorageConnector, "write");
 			},
-			getNumberOfChanges: function (sReference) {
+			getNumberOfChanges: function(sReference) {
 				return FakeLrepConnector.forTesting.getNumberOfChanges(SessionStorageConnector, sReference);
 			},
 			clear: function(mPropertyBag) {
@@ -55,7 +55,7 @@ function(
 				FakeLrepConnector.forTesting.setStorage(SessionStorageConnector, oNewStorage);
 			},
 			synchronous: {
-				clearAll: function () {
+				clearAll: function() {
 					FakeLrepConnector.forTesting.synchronous.clearAll(window.sessionStorage);
 				},
 				getNumberOfChanges: function(sReference) {

package/src/sap/ui/fl/FakeLrepLocalStorage.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @namespace
 	 *
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 *
 	 * @private
 	 * @since 1.48

package/src/sap/ui/fl/FlexController.js

@@ -41,35 +41,36 @@ sap.ui.define([
 
 	function revertChangesAndUpdateVariantModel(oComponent, bSkipUrlUpdate, aChanges) {
 		return Promise.resolve()
-			.then(function() {
-				if (aChanges.length !== 0) {
-					// Always revert changes in reverse order
-					aChanges.reverse();
-					return Reverter.revertMultipleChanges(aChanges, {
-						appComponent: oComponent,
-						modifier: JsControlTreeModifier,
-						flexController: this
-					});
-				}
-			}.bind(this))
-			.then(function() {
-				if (oComponent) {
-					var oModel = oComponent.getModel(ControlVariantApplyAPI.getVariantModelName());
-					if (oModel) {
-						// Temporary fix, parameters generally should not be removed
-						if (!bSkipUrlUpdate) {
-							URLHandler.update({
-								parameters: [],
-								updateURL: true,
-								updateHashEntry: true,
-								model: oModel
-							});
-						}
+		.then(function() {
+			if (aChanges.length !== 0) {
+				// Always revert changes in reverse order
+				aChanges.reverse();
+				return Reverter.revertMultipleChanges(aChanges, {
+					appComponent: oComponent,
+					modifier: JsControlTreeModifier,
+					flexController: this
+				});
+			}
+			return undefined;
+		}.bind(this))
+		.then(function() {
+			if (oComponent) {
+				var oModel = oComponent.getModel(ControlVariantApplyAPI.getVariantModelName());
+				if (oModel) {
+					// Temporary fix, parameters generally should not be removed
+					if (!bSkipUrlUpdate) {
+						URLHandler.update({
+							parameters: [],
+							updateURL: true,
+							updateHashEntry: true,
+							model: oModel
+						});
 					}
 				}
+			}
 
-				return aChanges;
-			});
+			return aChanges;
+		});
 	}
 
 	/**
@@ -81,7 +82,7 @@ sap.ui.define([
 	 * @alias sap.ui.fl.FlexController
 	 * @experimental Since 1.27.0
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 */
 	var FlexController = function(sComponentName) {
 		this._oChangePersistence = undefined;
@@ -122,157 +123,6 @@ sap.ui.define([
 		return this._oVariantSwitchPromise;
 	};
 
-	/**
-	 * Base function for creation of a change
-	 *
-	 * @param {object} oChangeSpecificData - Property bag holding the change information
-	 * The property "oPropertyBag.packageName" is set to $TMP and internally since flex changes are always local when they are created.
-	 * @param {sap.ui.core.Component} oAppComponent - Application Component of the control at runtime in case a map has been used
-	 * @returns {sap.ui.fl.apply._internal.flexObjects.FlexObject} the created change
-	 * @public
-	 */
-	FlexController.prototype.createBaseChange = function(oChangeSpecificData, oAppComponent) {
-		if (!oAppComponent) {
-			throw new Error("No application component found. To offer flexibility a valid relation to its owning component must be present.");
-		}
-
-		oChangeSpecificData.reference = this.getComponentName(); //in this case the component name can also be the value of sap-app-id
-		oChangeSpecificData.packageName = "$TMP"; // first a flex change is always local, until all changes of a component are made transportable
-
-		var oChange = FlexObjectFactory.createUIChange(oChangeSpecificData);
-
-		if (oChangeSpecificData.variantReference) {
-			oChange.setVariantReference(oChangeSpecificData.variantReference);
-		}
-
-		return oChange;
-	};
-
-	FlexController.prototype._createChange = function(oChangeSpecificData, oAppComponent, oControl) {
-		// for getting the change handler the change type and potentially the control type are needed
-		var sControlType = oControl && (oControl.controlType || Utils.getControlType(oControl));
-
-		var oChange = this.createBaseChange(oChangeSpecificData, oAppComponent);
-
-		return ChangeHandlerStorage.getChangeHandler(oChange.getChangeType(), sControlType, oControl, JsControlTreeModifier, oChange.getLayer())
-			.then(function(oChangeHandler) {
-				if (oChangeHandler) {
-					return oChangeHandler.completeChangeContent(oChange, oChangeSpecificData, {
-						modifier: JsControlTreeModifier,
-						appComponent: oAppComponent,
-						view: Utils.getViewForControl(oControl)
-					});
-				}
-				throw new Error("Change handler could not be retrieved for change " + JSON.stringify(oChangeSpecificData) + ".");
-			})
-			.then(function () {
-				// completeChangeContent changes the content and might make it dirty
-				oChange.setState(States.LifecycleState.NEW);
-				return oChange;
-			})
-			.catch(function(oError) {
-				return Promise.reject(oError);
-			});
-	};
-
-	/**
-	 * Create a change with an ExtensionPoint as selector.
-	 *
-	 * @param {object} oChangeSpecificData - Property bag (nvp) holding the change information
-	 * The property "oPropertyBag.packageName" is set to $TMP and internally since flex changes are always local when they are created.
-	 * @param {object} mExtensionPointReference - Reference map for extension point
-	 * @param {string} mExtensionPointReference.name - Name of the extension point
-	 * @param {sap.ui.core.Component} mExtensionPointReference.view - View including the extension point
-	 * @returns {Promise.<sap.ui.fl.apply._internal.flexObjects.FlexObject>} Created change wrapped in a promise
-	 * @public
-
-	 */
-	FlexController.prototype.createChangeWithExtensionPointSelector = function(oChangeSpecificData, mExtensionPointReference) {
-		return Promise.resolve()
-			.then(function() {
-				if (!mExtensionPointReference) {
-					throw new Error("A flexibility change on extension point cannot be created without a valid extension point reference.");
-				}
-				var oView = mExtensionPointReference.view;
-				var oAppComponent = Utils.getAppComponentForControl(oView);
-				oChangeSpecificData.selector = {
-					name: mExtensionPointReference.name,
-					viewSelector: JsControlTreeModifier.getSelector(oView.getId(), oAppComponent)
-				};
-				return oAppComponent;
-			})
-			.then(function(oAppComponent) {
-				return this._createChange(oChangeSpecificData, oAppComponent);
-			}.bind(this));
-	};
-
-	/**
-	 * Create a change
-	 *
-	 * @param {object} oChangeSpecificData - Property bag (nvp) holding the change information
-	 * The property "oPropertyBag.packageName" is set to $TMP and internally since flex changes are always local when they are created.
-	 * @param {sap.ui.core.Control | map} oControl - Control for which the change will be added
-	 * @param {string} oControl.id - ID of the control in case a map has been used to specify the control
-	 * @param {sap.ui.core.Component} [oControl.appComponent] - Application component of the control at runtime in case a map has been used
-	 * @param {string} oControl.controlType - Control type of the control in case a map has been used
-	 * @returns {Promise.<sap.ui.fl.apply._internal.flexObjects.FlexObject>} Created change wrapped in a promise
-	 * @public
-	 */
-	FlexController.prototype.createChangeWithControlSelector = function(oChangeSpecificData, oControl) {
-		var oAppComponent;
-		return new Utils.FakePromise()
-			.then(function() {
-				if (!oControl) {
-					throw new Error("A flexibility change cannot be created without a targeted control.");
-				}
-
-				var sControlId = oControl.id || oControl.getId();
-
-				if (!oChangeSpecificData.selector) {
-					oChangeSpecificData.selector = {};
-				}
-				oAppComponent = oControl.appComponent || Utils.getAppComponentForControl(oControl);
-				if (!oAppComponent) {
-					throw new Error("No application component found. To offer flexibility, the control with the ID '"
-						+ sControlId + "' has to have a valid relation to its owning application component.");
-				}
-
-				// differentiate between controls containing the component id as a prefix and others
-				// get local Id for control at root component and use it as selector id
-				Object.assign(oChangeSpecificData.selector, JsControlTreeModifier.getSelector(sControlId, oAppComponent));
-				return oAppComponent;
-			})
-			.then(function(oAppComponent) {
-				return this._createChange(oChangeSpecificData, oAppComponent, oControl);
-			}.bind(this));
-	};
-
-	/**
-	 * Adds a change to the flex persistence (not yet saved). Will be saved with #saveAll.
-	 *
-	 * @param {object} oChangeSpecificData property bag (nvp) holding the change information
-	 * The property "oPropertyBag.packageName" is set to $TMP internally since flex changes are always local when they are created.
-	 * @param {sap.ui.core.Control} oControl control for which the change will be added
-	 * @returns {Promise.<sap.ui.fl.apply._internal.flexObjects.FlexObject>} the created change
-	 * @public
-	 */
-	FlexController.prototype.addChange = function(oChangeSpecificData, oControl) {
-		return this.createChangeWithControlSelector(oChangeSpecificData, oControl)
-			.then(function(oChange) {
-				var oAppComponent = Utils.getAppComponentForControl(oControl);
-				// adding a change to the persistence will trigger the propagation listener which would try to apply the change
-				// but in this scenario the change is applied in .createAndApplyChange and no dependencies are added,
-				// so the propagation listener should ignore this change once
-				oChange._ignoreOnce = true;
-				this.addPreparedChange(oChange, oAppComponent);
-
-				// TODO this should go into an API that applied multiple changes, that does not yet exist
-				// all changes should be queued to apply before the first change gets applied
-				oChange.setQueuedForApply();
-				return oChange;
-			}.bind(this));
-	};
-
 	/**
 	 * Adds an already prepared change to the flex persistence (not yet saved). This method will not call
 	 * createChange again, but expects a fully computed and appliable change.
@@ -300,7 +150,6 @@ sap.ui.define([
 	 * Otherwise it will be marked for deletion.
 	 *
 	 * @param {sap.ui.fl.apply._internal.flexObjects.FlexObject} oChange - the change to be deleted
-	 * @param {sap.ui.core.Component} oAppComponent - Application component instance
 	 */
 	FlexController.prototype.deleteChange = function(oChange) {
 		this._oChangePersistence.deleteChange(oChange);
@@ -375,10 +224,10 @@ sap.ui.define([
 			return this._waitForChangesToBeApplied(mSelector);
 		}.bind(this));
 		return Promise.all(aPromises)
-			.then(function() {
-				// the return value is not important in this function, only that it resolves
-				return undefined;
-			});
+		.then(function() {
+			// the return value is not important in this function, only that it resolves
+			return undefined;
+		});
 	};
 
 	/**
@@ -450,7 +299,14 @@ sap.ui.define([
 	 * @returns {Promise} resolving with an array of responses or rejecting with the first error
 	 * @public
 	 */
-	FlexController.prototype.saveAll = function(oAppComponent, bSkipUpdateCache, bDraft, sLayer, bRemoveOtherLayerChanges, bCondenseAnyLayer) {
+	FlexController.prototype.saveAll = function(
+		oAppComponent,
+		bSkipUpdateCache,
+		bDraft,
+		sLayer,
+		bRemoveOtherLayerChanges,
+		bCondenseAnyLayer
+	) {
 		var sParentVersion;
 		var aDraftFilenames;
 		if (bDraft) {
@@ -462,26 +318,35 @@ sap.ui.define([
 			aDraftFilenames = oVersionModel.getProperty("/draftFilenames");
 		}
 		return this._removeOtherLayerChanges(oAppComponent, sLayer, bRemoveOtherLayerChanges)
-			.then(this._oChangePersistence.saveDirtyChanges.bind(this._oChangePersistence, oAppComponent, bSkipUpdateCache, undefined, sParentVersion, aDraftFilenames, bCondenseAnyLayer, sLayer))
-			.then(function(oResult) {
-				if (bDraft && oResult && oResult.response) {
-					var vChangeDefinition = oResult.response;
-					var aDraftFilenames = [];
-					if (Array.isArray(vChangeDefinition)) {
-						vChangeDefinition.forEach(function(change) {
-							aDraftFilenames.push(change.fileName);
-						});
-						// the reference and layer of all items are the same
-						vChangeDefinition = vChangeDefinition[0];
-					}
-					Versions.onAllChangesSaved({
-						reference: vChangeDefinition.reference,
-						layer: vChangeDefinition.layer,
-						draftFilenames: aDraftFilenames
+		.then(this._oChangePersistence.saveDirtyChanges.bind(
+			this._oChangePersistence,
+			oAppComponent,
+			bSkipUpdateCache,
+			undefined,
+			sParentVersion,
+			aDraftFilenames,
+			bCondenseAnyLayer,
+			sLayer
+		))
+		.then(function(oResult) {
+			if (bDraft && oResult && oResult.response) {
+				var vChangeDefinition = oResult.response;
+				var aDraftFilenames = [];
+				if (Array.isArray(vChangeDefinition)) {
+					vChangeDefinition.forEach(function(change) {
+						aDraftFilenames.push(change.fileName);
 					});
+					// the reference and layer of all items are the same
+					vChangeDefinition = vChangeDefinition[0];
 				}
-				return oResult;
-			});
+				Versions.onAllChangesSaved({
+					reference: vChangeDefinition.reference,
+					layer: vChangeDefinition.layer,
+					draftFilenames: aDraftFilenames
+				});
+			}
+			return oResult;
+		});
 	};
 
 	/**
@@ -503,8 +368,8 @@ sap.ui.define([
 		mPropertyBag.view = oView;
 
 		return this._oChangePersistence.getChangesForView(mPropertyBag)
-			.then(Applier.applyAllChangesForXMLView.bind(Applier, mPropertyBag))
-			.catch(handleXMLApplyError.bind(this, mPropertyBag.view));
+		.then(Applier.applyAllChangesForXMLView.bind(Applier, mPropertyBag))
+		.catch(handleXMLApplyError.bind(this, mPropertyBag.view));
 	};
 
 	function handleXMLApplyError(oView, oError) {
@@ -517,7 +382,7 @@ sap.ui.define([
 	 *
 	 * also used by OVP!
 	 *
-	 * @param {map} mPropertyBag - (optional) contains additional data that are needed for reading of changes
+	 * @param {object} mPropertyBag - (optional) contains additional data that are needed for reading of changes
 	 * @param {object} [mPropertyBag.appDescriptor] Manifest that belongs to the current running component
 	 * @param {string} [mPropertyBag.siteId] ID of the site belonging to the current running component
 	 * @param {boolean} bInvalidateCache - (optional) should the cache be invalidated
@@ -554,7 +419,7 @@ sap.ui.define([
 	 */
 	FlexController.prototype.resetChanges = function(sLayer, sGenerator, oComponent, aSelectorIds, aChangeTypes) {
 		return this._oChangePersistence.resetChanges(sLayer, sGenerator, aSelectorIds, aChangeTypes)
-			.then(revertChangesAndUpdateVariantModel.bind(this, oComponent, undefined));
+		.then(revertChangesAndUpdateVariantModel.bind(this, oComponent, undefined));
 	};
 
 	/**
@@ -596,21 +461,28 @@ sap.ui.define([
 					return Applier.applyChangeOnControl(oChange, oControl, mPropertyBag);
 				}
 				Log.error("A flexibility change tries to change a nonexistent control.");
+				return undefined;
 			}.bind(this));
-		}.bind(this), new Utils.FakePromise());
+		}.bind(this), (Utils.FakePromise ? new Utils.FakePromise() : Promise.resolve()));
 	};
 
 	/**
 	 * Saves changes sequentially on the associated change persistence instance;
-	 * This API must be only used in scnarios without draft (like personalization).
+	 * This API must be only used in scenarios without draft (like personalization).
 	 *
 	 * @param {sap.ui.fl.apply._internal.flexObjects.FlexObject[]} aDirtyChanges Array of dirty changes to be saved
 	 * @param {sap.ui.core.UIComponent} [oAppComponent] - AppComponent instance
-	 * @returns {Promise} A Promise which resolves when all changes have been saved
+	 * @returns {Promise} A Promise which resolves when all changes have been saved with the backend response
 	 * @public
 	 */
 	FlexController.prototype.saveSequenceOfDirtyChanges = function(aDirtyChanges, oAppComponent) {
-		return this._oChangePersistence.saveDirtyChanges(oAppComponent, false, aDirtyChanges);
+		return this._oChangePersistence.saveDirtyChanges(oAppComponent, false, aDirtyChanges)
+		.then(function(oResponse) {
+			aDirtyChanges.forEach(function(oDirtyChange) {
+				oDirtyChange.setState(States.LifecycleState.PERSISTED);
+			});
+			return oResponse;
+		});
 	};
 
 	return FlexController;

package/src/sap/ui/fl/FlexControllerFactory.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 * @alias sap.ui.fl.FlexControllerFactory
 	 * @experimental Since 1.27.0
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.fl
@@ -67,5 +67,16 @@ sap.ui.define([
 		}
 	};
 
+	/**
+	 * Creates or returns an instance of the FlexController for the specified selector.
+	 *
+	 * @param {sap.ui.fl.Selector} oSelector - Selector object
+	 * @returns {sap.ui.fl.FlexController} instance
+	 */
+	FlexControllerFactory.createForSelector = function(oSelector) {
+		var sComponentName = ManifestUtils.getFlexReferenceForSelector(oSelector);
+		return FlexControllerFactory.create(sComponentName);
+	};
+
 	return FlexControllerFactory;
 }, true);

package/src/sap/ui/fl/Layer.js

@@ -4,7 +4,7 @@
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
-sap.ui.define([], function () {
+sap.ui.define([], function() {
 	"use strict";
 
 	/**

package/src/sap/ui/fl/LayerUtils.js

@@ -15,7 +15,7 @@ sap.ui.define([
 ) {
 	"use strict";
 
-	//Stack of layers in the layered repository
+	// Stack of layers in the layered repository
 	var aLayers = [
 		Layer.BASE,
 		Layer.VENDOR,
@@ -26,7 +26,7 @@ sap.ui.define([
 		Layer.USER
 	];
 
-	//Precalculates index of layers
+	// Precalculates index of layers
 	var mLayersIndex = {};
 	aLayers.forEach(function(sLayer, iIndex) {
 		mLayersIndex[sLayer] = iIndex;
@@ -41,7 +41,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.LayerUtils
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 */
 	var LayerUtils = {
 		_mLayersIndex: mLayersIndex,
@@ -55,8 +55,8 @@ sap.ui.define([
 		 * @returns {boolean} <code>true</code> if the layer is valid
 		 * @public
 		 */
-		isValidLayer: function (sLayer) {
-			return Object.keys(Layer).some(function (sExistingLayer) {
+		isValidLayer: function(sLayer) {
+			return Object.keys(Layer).some(function(sExistingLayer) {
 				return sExistingLayer === sLayer;
 			});
 		},
@@ -67,7 +67,7 @@ sap.ui.define([
 		 * @returns {boolean} true if it's an application variant
 		 * @public
 		 */
-		isVendorLayer: function () {
+		isVendorLayer: function() {
 			return this.getCurrentLayer() === Layer.VENDOR;
 		},
 
@@ -101,7 +101,7 @@ sap.ui.define([
 		 * @returns {boolean} Indicates whether a new variant needs an ABAP package
 		 * @public
 		 */
-		doesCurrentLayerRequirePackage: function () {
+		doesCurrentLayerRequirePackage: function() {
 			var sCurrentLayer = this.getCurrentLayer();
 			return (sCurrentLayer === Layer.VENDOR) || (sCurrentLayer === Layer.PARTNER) || (sCurrentLayer === Layer.CUSTOMER_BASE);
 		},
@@ -113,7 +113,7 @@ sap.ui.define([
 		 * @param {sap.ui.core.service.Service} oURLParsingService Unified Shell URL Parsing Service
 		 * @return {string} maxLayer
 		 */
-		getMaxLayer: function (oURLParsingService) {
+		getMaxLayer: function(oURLParsingService) {
 			var sParseMaxLayer = LayerUtils.getMaxLayerTechnicalParameter(hasher.getHash(), oURLParsingService);
 			return sParseMaxLayer || getUrlParameter(this.FL_MAX_LAYER_PARAM) || LayerUtils._sTopLayer;
 		},
@@ -148,7 +148,7 @@ sap.ui.define([
 		 * @returns {boolean} <code>true</code> if the first input layer is higher than the second input layer, otherwise <code>false</code>
 		 * @public
 		 */
-		isOverLayer: function (sObjectsLayer, sComparedLayer) {
+		isOverLayer: function(sObjectsLayer, sComparedLayer) {
 			return this.getLayerIndex(sObjectsLayer) > this.getLayerIndex(sComparedLayer);
 		},
 
@@ -189,7 +189,7 @@ sap.ui.define([
 		 * @returns {boolean} <code>true</code> if the top layer is also the max layer, otherwise <code>false</code>
 		 * @public
 		 */
-		isSapUiLayerParameterProvided: function () {
+		isSapUiLayerParameterProvided: function() {
 			return !!getUrlParameter("sap-ui-layer");
 		},
 
@@ -199,7 +199,7 @@ sap.ui.define([
 		 * @returns {string} the current layer
 		 * @public
 		 */
-		getCurrentLayer: function () {
+		getCurrentLayer: function() {
 			var sLayer = getUrlParameter("sap-ui-layer") || "";
 			return sLayer.toUpperCase() || Layer.CUSTOMER;
 		},