@sapui5/sap.suite.ui.generic.template 1.142.5 → 1.142.7

package/src/sap/suite/ui/generic/template/genericUtilities/controlStateWrapperFactory/SmartFilterBarWrapper.js

@@ -1,6 +1,5 @@
 sap.ui.define([
-	"sap/ui/model/FilterOperator"
-], function (FilterOperator) {
+], function () {
 	"use strict";
 
 
@@ -23,29 +22,24 @@ sap.ui.define([
 	 * @returns {object}
 	 * @alias sap.suite.ui.generic.template.genericUtilities.controlStateWrapperFactory.SmartFilterBarWrapper
 	 */
-	function SmartFilterBarWrapper(vTarget, oFactory, mParams) {
+	function SmartFilterBarWrapper(oControl, oFactory, mParams) {
 		var bIsApplying = false;
 		var aBasicFilters = [];
-		var aMissingNavFilters = [];
-		var oSmartFilterBar, oControlAssignedResolve, oPreliminaryState;
-		var oControlAssignedPromise = new Promise(function (resolve) {
-			oControlAssignedResolve = resolve;
-		});
+		var oSmartFilterBar = oControl;
 
-		function fnInitialize() {
-			if (typeof vTarget !== "string") {
-				fnSetControl(vTarget);
-			}
+		var oVariantManagementInitializedPromise;
 
+		function fnInitialize() {
 			// Filters visible initially (after initialization). Stored here to be
 			// able to compare with currently visible ones to identify added and
 			// removed ones	without need to instantiate all possible filter items
 			// (which would harm performance). Can only be retrieved in initialized event.
-			oControlAssignedPromise.then(function () {
-				oSmartFilterBar.getInitializedPromise().then(function () {
-					aBasicFilters = oSmartFilterBar.getAllFilterItems(true);
-				});
+			oSmartFilterBar.getInitializedPromise().then(function () {
+				aBasicFilters = oSmartFilterBar.getAllFilterItems(true);
 			});
+
+			// Initialize variant management promise
+			oVariantManagementInitializedPromise = oSmartFilterBar.getInitializedPromise();
 		}
 
 		// Attach to variantFetch and variantLoad events to be able to store and restore also custom filters and state of other controls (in case of page VM) with variant.
@@ -69,7 +63,7 @@ sap.ui.define([
 			// in SFB instead of SVM where it would make more sense.
 			// Here the event is afterVariantLoad, as the variant content known to SVM/SFB has first to be applied to the SFB, which will also pass the custom data accordingly. Only
 			// after this has happened, we can get the custom data from SFB.
-			oSmartFilterBar.attachAfterVariantLoad(function(oEvent){
+			oSmartFilterBar.attachAfterVariantLoad(function(){
 				var oCustomData = oSmartFilterBar.getCustomFilterData();
 				// variant stored with 1.103 or later: all customFilter data stored in property customFilters
 				// legacy variant stored with 1.102 or earlier: customFilters (from storing point of view) separated according to their origin
@@ -81,91 +75,11 @@ sap.ui.define([
 
 				mParams.oCustomFiltersWrapper.setState(oCustomFiltersState);
 				oSVMWrapperCallbacks.setManagedControlStates(oCustomData[dataPropertyNameGeneric]);
-				/* SFB header state of a variant (standard or custom) gets determined by the Apply Automatically checkbox's value of the corresponding
-				   variant i.e. if the checkbox is checked, then the header should be collapsed and vice versa. */
-				oSVMWrapperCallbacks.setHeaderState(!oEvent.getParameter("executeOnSelect"));
-				aMissingNavFilters = fnGetNavigationProperties();
 			});
 
 		}
 
-		function fnGetNavigationProperties() {
-			//Fetch the navigation properties
-			const oMetaModel = oSmartFilterBar.getModel().getMetaModel(),
-				sEntitySet = oSmartFilterBar.getEntitySet(),
-				oDataEntitySet = oMetaModel.getODataEntitySet(sEntitySet),
-				oDataEntityType = oMetaModel.getODataEntityType(oDataEntitySet.entityType),
-				aNavigationProperties = oDataEntityType['navigationProperty'];
-			//Fetch the added filters in the current variant
-			const sCurrentVariantID = oSmartFilterBar.getVariantManagement().getCurrentVariantId(),
-				oCurrentVariant = oSmartFilterBar.getVariantManagement().getAllVariants().find(function(variant) {
-					return variant.getId() === sCurrentVariantID;
-				});
-			if (!oCurrentVariant
-				|| !oCurrentVariant.getContent()
-				|| !oCurrentVariant.getContent().searchListReportVariant
-				|| !oCurrentVariant.getContent().searchListReportVariant.filterBarVariant
-				|| !aNavigationProperties
-				|| !aNavigationProperties.length
-			) {
-				return [];
-			}
-			const oManifestNavigationProperties = oSmartFilterBar.getNavigationProperties() ?
-				oSmartFilterBar.getNavigationProperties().split(",").reduce(function(accumulator, currentValue){
-					accumulator[currentValue] = true;
-					return accumulator;
-				}, {}) : {},
-				oNavigationProperties = aNavigationProperties.reduce(function(accumulator, currentValue){
-					accumulator[currentValue.name] = true;
-					return accumulator;
-				}, {}),
-				oSmartFilterBarVariant = JSON.parse(oCurrentVariant.getContent().searchListReportVariant.filterBarVariant),
-				aMissing = [];
-			delete oSmartFilterBarVariant["_CUSTOM"];
-
-			// Compare the filter source and navigation properties
-			// Take into account if parameter is already specified in page setting in manifest.json - settings->filterSettings->navigationProperties
-			// If parameter exist in navigationProperties -> SFB will handle specific parameter and code ignore it
-			for (const sFilterKey in oSmartFilterBarVariant) {
-				const sParamName = sFilterKey.split(".")[0]; // take first part from navigation parameter. Example: to_Currency.Currency_Code -> to_Currency
-				if (!oNavigationProperties[sParamName] || oManifestNavigationProperties[sParamName]
-				) {
-					// Parameter is
-					// 1) not navigation property -> we don't process it
-					// 2) is defined in manifest.json - settings->filterSettings->navigationProperties -> value will be handled by SFB
-					continue;
-				}
-				if (oSmartFilterBarVariant[sFilterKey].items && oSmartFilterBarVariant[sFilterKey].items.length) {
-					aMissing.push(oSmartFilterBarVariant[sFilterKey].items.map(function(entry) {
-						return {
-							exclude: false,
-							field: sFilterKey,
-							operation: FilterOperator.EQ,
-							value1: entry.key
-						};
-					}));
-					continue;
-				}
-				if (oSmartFilterBarVariant[sFilterKey].ranges && oSmartFilterBarVariant[sFilterKey].ranges.length) {
-					aMissing.push(oSmartFilterBarVariant[sFilterKey].ranges.map(function(entry) {
-						return {
-							exclude: entry.exclude,
-							field: entry.keyField,
-							operation: entry.operation,
-							value1: entry.value1,
-							value2: entry.value2
-						};
-					}));
-				}
-			}
-			return aMissing;
-		}
-
 		function fnGetState() {
-			if (!oSmartFilterBar) {
-				return oPreliminaryState;
-			}
-
 			var oUiState = oSmartFilterBar.getUiState();
 			// UiState is not Serializable, but a managed object, containing information only partly relevant
 			// relevant information are
@@ -217,43 +131,40 @@ sap.ui.define([
 			}
 
 			bIsApplying = true;
-			oPreliminaryState = oState;
 
-			oControlAssignedPromise.then(function() {
-				// SFB expects a UIState object - not serializable, but a managed object, actually containing also information not belonging to SFBs state
-				// => get current UIState object from SFB, replace only relevant information, and set it again
-				var oUiState = oSmartFilterBar.getUiState(),
-					oSmartVariant = oSmartFilterBar.getSmartVariant(),
-					bIsCurrentVariantModifiedBeforeSetState = oSmartVariant.currentVariantGetModified();
-
-				oUiState.getSelectionVariant().SelectOptions = oPreliminaryState && oPreliminaryState.selectOptions;
-				oUiState.getSelectionVariant().Parameters = oPreliminaryState && oPreliminaryState.parameters;
-				oUiState.setSemanticDates(oPreliminaryState && oPreliminaryState.semanticDates);
-				// setState is meant to set the state absolutely, no merge needed => replace and strictMode can be set to true.
-				// (replace=false can be used to set additional selectOptions but keeping the existing ones. strictMode=false is used to map filters to parameters. Both might make sense in navigation
-				// scenarios (navigation parameter to be merged with defaults, property used in source app as filter but in target as parameter), but not when just restoring to a state (provided from the
-				// same control))
-				oSmartFilterBar.setUiState(oUiState, {replace: true, strictMode: true});
-
-				mParams.oCustomFiltersWrapper.setState(oPreliminaryState && oPreliminaryState.customFilters);
-
-				// set visibility
-				// TODO:
-				// - How to deal with old states (not containing all information about visibility) -> legacy state handler?
-				// - restoring from old state when annotation has changed (new selection fields)?
-				oSmartFilterBar.getAllFilterItems().forEach(function(oFilterItem){
-					if (oPreliminaryState && oPreliminaryState.addedFilterItems && oPreliminaryState.addedFilterItems.includes(oFilterItem.getName())){
-						oFilterItem.setVisibleInFilterBar();
-					}
-					if (oPreliminaryState && oPreliminaryState.removedFilterItems && oPreliminaryState.removedFilterItems.includes(oFilterItem.getName())){
-						oFilterItem.setVisibleInFilterBar(false);
-					}
-				});
-
-				// Retaining the old value of "modified" flag in the smart variant
-				oSmartVariant.currentVariantSetModified(bIsCurrentVariantModifiedBeforeSetState);
+			// SFB expects a UIState object - not serializable, but a managed object, actually containing also information not belonging to SFBs state
+			// => get current UIState object from SFB, replace only relevant information, and set it again
+			var oUiState = oSmartFilterBar.getUiState(),
+				oSmartVariant = oSmartFilterBar.getSmartVariant(),
+				bIsCurrentVariantModifiedBeforeSetState = oSmartVariant.currentVariantGetModified();
+
+			oUiState.getSelectionVariant().SelectOptions = oState && oState.selectOptions;
+			oUiState.getSelectionVariant().Parameters = oState && oState.parameters;
+			oUiState.setSemanticDates(oState && oState.semanticDates);
+			// setState is meant to set the state absolutely, no merge needed => replace and strictMode can be set to true.
+			// (replace=false can be used to set additional selectOptions but keeping the existing ones. strictMode=false is used to map filters to parameters. Both might make sense in navigation
+			// scenarios (navigation parameter to be merged with defaults, property used in source app as filter but in target as parameter), but not when just restoring to a state (provided from the
+			// same control))
+			oSmartFilterBar.setUiState(oUiState, {replace: true, strictMode: true});
+
+			mParams.oCustomFiltersWrapper.setState(oState && oState.customFilters);
+
+			// set visibility
+			// TODO:
+			// - How to deal with old states (not containing all information about visibility) -> legacy state handler?
+			// - restoring from old state when annotation has changed (new selection fields)?
+			oSmartFilterBar.getAllFilterItems().forEach(function(oFilterItem){
+				if (oState && oState.addedFilterItems && oState.addedFilterItems.includes(oFilterItem.getName())){
+					oFilterItem.setVisibleInFilterBar();
+				}
+				if (oState && oState.removedFilterItems && oState.removedFilterItems.includes(oFilterItem.getName())){
+					oFilterItem.setVisibleInFilterBar(false);
+				}
 			});
 
+			// Retaining the old value of "modified" flag in the smart variant
+			oSmartVariant.currentVariantSetModified(bIsCurrentVariantModifiedBeforeSetState);
+
 			// Apparently, SFB does not always correctly adapt the adapt filters count automatically. The following method has been provided espacially to trigger the same.
 			// TODO: Verify, whether this is really needed (or was rather an artifact of old structure) - if yes, this API should be made public, if no, we should remove the call.
 			oSmartFilterBar.refreshFiltersCount();
@@ -268,25 +179,11 @@ sap.ui.define([
 				}
 			}
 
-			oControlAssignedPromise.then(function () {
-				oSmartFilterBar.attachFilterChange(function () {
-					// Don't forward filter change event while dialog is open - changes should only be registered when dialog is closed
-					if (!oSmartFilterBar.isDialogOpen()) {
-						handleStateChanged();
-					}
-				});
-				// unclear, whether this is needed, or filterChange event is anyway raised again (after dialog is closed)
-				oSmartFilterBar.attachFiltersDialogClosed(handleStateChanged);
-
-				// do we need to provide and handle change events from custom filters?
-				// contra:
-				// - SFB raises filteChange event also for custom filters
-				// - maybe also needed to be suppressed while dialog is open
-				// pro:
-				// - cleaner from architectural perspective
-				// - SFB cannot deal correctly with unknown custom controls
-				// - existing method in extensionAPI (onCustomAppStateChange)
-				mParams.oCustomFiltersWrapper.attachStateChanged(handleStateChanged);
+			oSmartFilterBar.attachFilterChange(function () {
+				// Don't forward filter change event while dialog is open - changes should only be registered when dialog is closed
+				if (!oSmartFilterBar.isDialogOpen()) {
+					handleStateChanged();
+				}
 			});
 			// unclear, whether this is needed, or filterChange event is anyway raised again (after dialog is closed)
 			oSmartFilterBar.attachFiltersDialogClosed(handleStateChanged);
@@ -314,15 +211,6 @@ sap.ui.define([
 			});
 		}
 
-		function fnSetControl(oControl) {
-			oSmartFilterBar = oControl;
-			oControlAssignedResolve(oSmartFilterBar);
-		}
-
-		function fnGetMissingNavProperties() {
-			return aMissingNavFilters;
-		}
-
 		// check waiting for initialization (like in SmartTableWrapper/SmartChartWrapper)
 		// currently, whole appState restoring is waiting for initialized event from sfb - but probably it would be enough to wait for it here
 		fnInitialize();
@@ -331,15 +219,14 @@ sap.ui.define([
 			// generic properties (provided by all state wrappers)
 			getState: fnGetState,
 			setState: fnSetState,
-			setControl: fnSetControl,
 			attachStateChanged: fnAttachStateChanged,
 			// specific properties (needed to workaround direct connection between SFB and SVM)
 			setSVMWrapperCallbacks: setSVMWrapperCallbacks,
 			bVMConnection: oSmartFilterBar.getSmartVariant(),
 			suppressSelection: oSmartFilterBar.setSuppressSelection.bind(oSmartFilterBar), // if multiple reasons for suppressing overlap, a counter (to avoid to early resume) could be implemented here
-			getMissingNavProperties: fnGetMissingNavProperties
+			oVariantManagementInitializedPromise: oVariantManagementInitializedPromise
 		};
 	}
 
 	return SmartFilterBarWrapper;
-});
+});

package/src/sap/suite/ui/generic/template/genericUtilities/controlStateWrapperFactory/SmartTableChartCommon.js

@@ -4,41 +4,64 @@ sap.ui.define([
 
     /**
      * Constructor for SmartTableChartCommon
-     * ...
-     * @param {object} mParams
-     * @param mParams.oCustomFiltersWrapper - wrapper for custom filters (from SFB point of view)
-     *   (currently containing: generic (currently only editState), app extension, and adaptation extension - but this is knowledge of LR, i.e. of iAppStateHandler, not of SFB)
+     * @param {sap.ui.core.Control} oControl - The SmartTable or SmartChart control
+     * @param {object} oController - The controller instance
+     * @param {object} oFactory - The ControlStateWrapperFactory instance
+     * @param {string} sInitializationEvent - The initialization event name (e.g., "initialise")
+     * @param {object} [mExtensions] - Optional extension functions for control-specific state handling
+     * @param {function} [mExtensions.fnExtendGetUiState] - Function to extend UiState during getState
+     *   Called with (oStateUiState, oControlUiState) where modifications to oStateUiState add to the state
+     * @param {function} [mExtensions.fnExtendSetUiState] - Function to extend UiState during setState
+     *   Called with (oStateUiState, oControlUiState) where oStateUiState contains the state to apply
      * @returns {object}
      */
 
     // deals with state of SFB itself, without SVM (see SmartVariantManagementWrapper) and go-button
     // (does not contain a state from SFB point of view - however, we remember whether it was pressed once
     // to restore data - this information is not considered being part of SFB)
-    function SmartTableChartCommon(vTarget, oController, oFactory, sInitializationEvent) {
-        var oSmartControl, oVariantManagementControl, bVariantManagementActive;
-        var oControlAssignedResolve, oPreliminaryState, bSmartControlInitialized;
+    function SmartTableChartCommon(oControl, oController, oFactory, sInitializationEvent, mExtensions) {
+        var oSmartControl = oControl;
+        var oVariantManagementControl, bVariantManagementActive;
         var oSmartControlStateWrapper;
         var oVariantManagementStateWrapper;
+        var oVariantManagementInitializedResolve;
+        var fnResolveReady;
 
-        var oControlAssignedPromise = new Promise(function (resolve) {
-            oControlAssignedResolve = resolve;
+        var oVariantManagementInitializedPromise = new Promise(function(resolve){
+            oVariantManagementInitializedResolve = resolve;
         });
 
-        if (typeof vTarget !== "string") {
-            fnSetControl(vTarget);
+        var oReadyPromise = new Promise(function(resolve) {
+            fnResolveReady = resolve;
+        });
+
+        // Initialize control state wrapper and check if control is already initialized
+        oSmartControlStateWrapper = getControlWrapper(oSmartControl);
+        if (!oSmartControl.isInitialised()) {
+            // In case Smart Chart/Table is not yet initialized
+            // listen to the initialize event & then initialize the wrapper
+            oSmartControl.attachEvent(sInitializationEvent, fnControlInitialized);
+        } else {
+            fnControlInitialized();
         }
 
         // creates a simple wrapper for a SmartTable or SmartChart (a control using UIState) (ignoring initialization and vm)
         function getControlWrapper(oControl) {
             var bIsApplying = false;		// avoid forwarding change event when caused by us applying a state
             function fnGetState() {
-                var oUiState = oControl.getUiState(); // unfortunately not serializable, but a managed object
+                var oControlUiState = oControl.getUiState(); // unfortunately not serializable, but a managed object
+                var oStateUiState = {
+                    oPresentationVariant: oControlUiState.getPresentationVariant(),
+                    oSelectionVariant: oControlUiState.getSelectionVariant()
+                };
+
+                // Allow extension to add control-specific properties (e.g., tableSettings for SmartTable)
+                if (mExtensions && mExtensions.fnExtendGetUiState) {
+                    mExtensions.fnExtendGetUiState(oStateUiState, oControlUiState);
+                }
+
                 return {
-                    oUiState: {
-                        oPresentationVariant: oUiState.getPresentationVariant(),
-                        oSelectionVariant: oUiState.getSelectionVariant(),
-                        oTableSettings: oUiState.getTableSettings()
-                    }
+                    oUiState: oStateUiState
                 };
             }
 
@@ -53,15 +76,20 @@ sap.ui.define([
                     bIsApplying = true;
                     // don't create UiState (managed object) from scratch, but fetch it from control and only apply known properties from state - thus, if any other properties would be added, we don't
                     // interfere with them
-                    var oUiState = oControl.getUiState();
+                    var oControlUiState = oControl.getUiState();
                     if (
-                        oUiState.getProperty("variantName") === "" || 
-                        oControl.getVariantManagement().getModified() && oControl.getVariantManagement().getStandardVariantKey() === oUiState.getProperty("variantName")
+                        oControlUiState.getProperty("variantName") === "" ||
+                        oControl.getVariantManagement().getModified() && oControl.getVariantManagement().getStandardVariantKey() === oControlUiState.getProperty("variantName")
                     ){
-                        oUiState.setPresentationVariant(oState.oUiState.oPresentationVariant);
-                        oUiState.setSelectionVariant(oState.oUiState.oSelectionVariant);
-                        oUiState.setTableSettings(oState.oUiState.oTableSettings);
-                        oControl.setUiState(oUiState);
+                            oControlUiState.setPresentationVariant(oState.oUiState.oPresentationVariant);
+                            oControlUiState.setSelectionVariant(oState.oUiState.oSelectionVariant);
+
+                            // Allow extension to set control-specific properties (e.g., tableSettings for SmartTable)
+                            if (mExtensions && mExtensions.fnExtendSetUiState) {
+                                mExtensions.fnExtendSetUiState(oState.oUiState, oControlUiState);
+                            }
+
+                            oControl.setUiState(oControlUiState);
                     }
                     bIsApplying = false;
                 } else {
@@ -88,76 +116,58 @@ sap.ui.define([
                 setState: fnSetState,
                 getLocalId: fnGetLocalId,
                 bVMConnection: oControl && oControl.getSmartVariant && !!oControl.getSmartVariant(),
-                attachStateChanged: fnAttachStateChanged
+                attachStateChanged: fnAttachStateChanged,
+                onAfterVariantInitialise: oVariantManagementInitializedResolve
             };
         }
 
         function fnGetState() {
-            if (bSmartControlInitialized) {
-                if (oVariantManagementStateWrapper) {
-                    // oVariantManagementStateWrapper would be undefined only in cases SmartControl is not yet initialized
-                    // or VariantManagement is not enabled.
-                    return oVariantManagementStateWrapper.getState();
-                }
-
-                if (oSmartControlStateWrapper && !bVariantManagementActive) {
-                    // VariantManagement is not turned on & SmartControlStateWrapper instance
-                    return oSmartControlStateWrapper.getState();
-                }
+            if (oVariantManagementStateWrapper) {
+                // Variant management is active
+                return oVariantManagementStateWrapper.getState();
             }
 
-            // Control is not available and StateWrapper is not initialized
-            return oPreliminaryState;
+            // No variant management - return control state directly
+            return oSmartControlStateWrapper.getState();
         }
 
         function fnSetState(oState) {
-            oPreliminaryState = oState;
-            oControlAssignedPromise.then(function () {
-                // map legacy states - before separating VM state from state of managed control, inner state information
-                // (presentation variant and selection variant) were put as oUiState on the same level as VM state
-                // information (variant id and dirty indicator), now they are contained in map managedControlStates
-                if (oState){
-                    if (oState.oUiState && !oState.managedControlStates) {
-                        oState.managedControlStates = Object.create(null);
-                        oState.managedControlStates[oSmartControlStateWrapper.getLocalId()] = {oUiState: oState.oUiState};
-                    }
-                    if (oState.bVariantModified !== undefined) {
-                        oState.modified = oState.bVariantModified;
-                    }
-                    if (oState.sVariantId !== undefined) {
-                      oState.variantId = oState.sVariantId;
-                    }
-                }
-
+            if (!oState) {
+                // If no state provided, set standard variant on control
                 if (bVariantManagementActive) {
-                    // In case VariantManagement is configured
-                    oVariantManagementStateWrapper.setState(oPreliminaryState);
-                    return;
+                    oVariantManagementStateWrapper.setState(oState);
+                } else {
+                    oSmartControlStateWrapper.setState(oState);
                 }
-
-                // SmartTable/Chart shall be used
-                oSmartControlStateWrapper.setState(oPreliminaryState);
-            });
-        }
-
-        function fnSetControl(oControl) {
-            oSmartControl = oControl;
-            oSmartControlStateWrapper = getControlWrapper(oSmartControl);
-            if (!oSmartControl.isInitialised()) {
-                // In case Smart Chart/Table is not yet initialized
-                // listen to the initialize event & then resolve the control
-                oSmartControl.attachEvent(sInitializationEvent, fnControlInitialized);
                 return;
             }
 
-            fnControlInitialized();
+            // Map legacy states - before separating VM state from state of managed control, inner state information
+            // (presentation variant and selection variant) were put as oUiState on the same level as VM state
+            // information (variant id and dirty indicator), now they are contained in map managedControlStates
+            if (oState.oUiState && !oState.managedControlStates) {
+                oState.managedControlStates = Object.create(null);
+                oState.managedControlStates[oSmartControlStateWrapper.getLocalId()] = {oUiState: oState.oUiState};
+            }
+            if (oState.bVariantModified !== undefined) {
+                oState.modified = oState.bVariantModified;
+            }
+            if (oState.sVariantId !== undefined) {
+                oState.variantId = oState.sVariantId;
+            }
+
+            if (bVariantManagementActive) {
+                // In case VariantManagement is configured
+                oVariantManagementStateWrapper.setState(oState);
+            } else {
+                // SmartTable/Chart shall be used
+                oSmartControlStateWrapper.setState(oState);
+            }
         }
 
         function fnControlInitialized() {
-            bSmartControlInitialized = true;
-
             // deal with VM
-            // 3 possibilties with respect to VM
+            // 3 possibilities with respect to VM
             // a) no VM (relevant for this control) at all
             // b) control managed as part of page wide variant management
             // c) control creates own VM
@@ -181,39 +191,35 @@ sap.ui.define([
                     oVariantManagementStateWrapper = oFactory.getControlStateWrapper(oVariantManagementControl, {
                         managedControlWrappers: [ oSmartControlStateWrapper ]
                     });
-                    oControlAssignedResolve();
+                    fnResolveReady();
                 } else {
-                    // Resolve the ControlAssignedPromise only once the VariantManagement is
-                    // also initialized
+                    // Wait for VariantManagement to be initialized
                     oSmartControl.attachAfterVariantInitialise(fnControlInitialized);
                 }
-
-                return;
+            } else {
+                // No variant management or page-level variant management - ready immediately
+                fnResolveReady();
             }
-            // There is no VariantManagement configured for this SmartChart/SmartTable
-            oControlAssignedResolve(oSmartControl);
         }
 
         function fnAttachStateChanged(fnHandler) {
-            oControlAssignedPromise.then(function () {
-                if (bVariantManagementActive) {
-                    oVariantManagementStateWrapper.attachStateChanged(fnHandler);
-                    return;
-                }
-
+            if (bVariantManagementActive) {
+                oVariantManagementStateWrapper.attachStateChanged(fnHandler);
+            } else {
                 oSmartControlStateWrapper.attachStateChanged(fnHandler);
-
-            });
+            }
         }
 
         return {
             getState: fnGetState,
             setState: fnSetState,
-            setControl: fnSetControl,
             attachStateChanged: fnAttachStateChanged,
-            bVMConnection: oSmartControl && oSmartControl.getSmartVariant && !!oSmartControl.getSmartVariant()
+            bVMConnection: oSmartControl && oSmartControl.getSmartVariant && !!oSmartControl.getSmartVariant(),
+            oVariantManagementInitializedPromise: oVariantManagementInitializedPromise,
+            onAfterVariantInitialise: oVariantManagementInitializedResolve,
+            oReadyPromise: oReadyPromise
         };
     }
 
     return SmartTableChartCommon;
-});
+});

package/src/sap/suite/ui/generic/template/genericUtilities/controlStateWrapperFactory/SmartTableWrapper.js

@@ -2,10 +2,29 @@ sap.ui.define([
 	"sap/suite/ui/generic/template/genericUtilities/controlStateWrapperFactory/SmartTableChartCommon"
 ], function(SmartTableChartCommon) {
 	"use strict";
-	
+
+	/**
+	 * Constructor for SmartTableWrapper
+	 * @param {sap.ui.comp.smarttable.SmartTable} oSmartTable - The SmartTable control
+	 * @param {object} oController - The controller instance
+	 * @param {object} oFactory - The ControlStateWrapperFactory instance
+	 * @returns {object} Wrapper object for SmartTable
+	 */
 	function SmartTableWrapper(oSmartTable, oController, oFactory){
-		return new SmartTableChartCommon(oSmartTable, oController, oFactory, "initialise");
+		// SmartTable extends SmartTableChartCommon with tableSettings support
+		return new SmartTableChartCommon(oSmartTable, oController, oFactory, "initialise", {
+			// Extension to add tableSettings to the state
+			fnExtendGetUiState: function(oStateUiState, oControlUiState) {
+				oStateUiState.oTableSettings = oControlUiState.getTableSettings();
+			},
+			// Extension to apply tableSettings from the state
+			fnExtendSetUiState: function(oStateUiState, oControlUiState) {
+				if (oStateUiState.oTableSettings) {
+					oControlUiState.setTableSettings(oStateUiState.oTableSettings);
+				}
+			}
+		});
 	}
 
 	return SmartTableWrapper;
-});
+});