@sapui5/sap.fe.templates 1.108.8 → 1.108.10

package/src/sap/fe/templates/ObjectPage/ObjectPageController.controller.ts

@@ -29,8 +29,10 @@ import RootViewBaseController from "sap/fe/core/rootView/RootViewBaseController"
 import ChartRuntime from "sap/fe/macros/chart/ChartRuntime";
 import CommonHelper from "sap/fe/macros/CommonHelper";
 import DelegateUtil from "sap/fe/macros/DelegateUtil";
+import type TableAPI from "sap/fe/macros/table/TableAPI";
 import TableUtils from "sap/fe/macros/table/Utils";
 import SelectionVariant from "sap/fe/navigation/SelectionVariant";
+import type SubSectionBlock from "sap/fe/templates/ObjectPage/controls/SubSectionBlock";
 import type { default as ObjectPageExtensionAPI } from "sap/fe/templates/ObjectPage/ExtensionAPI";
 import { default as ExtensionAPI } from "sap/fe/templates/ObjectPage/ExtensionAPI";
 import EditFlowOverrides from "sap/fe/templates/RootContainer/overrides/EditFlow";
@@ -43,6 +45,7 @@ import Core from "sap/ui/core/Core";
 import Message from "sap/ui/core/message/Message";
 import OverrideExecution from "sap/ui/core/mvc/OverrideExecution";
 import type View from "sap/ui/core/mvc/View";
+import type Table from "sap/ui/mdc/Table";
 import type Binding from "sap/ui/model/Binding";
 import type JSONModel from "sap/ui/model/json/JSONModel";
 import type Context from "sap/ui/model/odata/v4/Context";
@@ -205,6 +208,60 @@ class ObjectPageController extends PageController {
 		return oTable && oTable.getRowBinding();
 	}
 
+	/**
+	 * Find the last visible subsection and add the sapUxAPObjectPageSubSectionFitContainer CSS class if it contains only a gridTable.
+	 *
+	 * @param subSections The sub sections to look for
+	 * @private
+	 */
+	private checkSectionsForGridTable(subSections: ObjectPageSubSection[]) {
+		const changeClassForTables = (event: Event, lastVisibleSubSection: ObjectPageSubSection) => {
+			const blocks = [...lastVisibleSubSection.getBlocks(), ...lastVisibleSubSection.getMoreBlocks()];
+			if (
+				blocks.length === 1 &&
+				this.searchTableInBlock(blocks[0] as SubSectionBlock)
+					?.getType()
+					?.isA("sap.ui.mdc.table.GridTableType")
+			) {
+				//In case there is only a single table in a subSection we fit that to the whole page so that the scrollbar comes only on table and not on page
+				lastVisibleSubSection.addStyleClass("sapUxAPObjectPageSubSectionFitContainer");
+				lastVisibleSubSection.detachEvent("modelContextChange", changeClassForTables, this);
+			}
+		};
+		for (let subSectionIndex = subSections.length - 1; subSectionIndex >= 0; subSectionIndex--) {
+			if (subSections[subSectionIndex].getVisible()) {
+				const lastVisibleSubSection = subSections[subSectionIndex];
+				// We need to attach this event in order to manage the Object Page Lazy Loading mechanism
+				lastVisibleSubSection.attachEvent("modelContextChange", lastVisibleSubSection, changeClassForTables, this);
+				break;
+			}
+		}
+	}
+
+	/**
+	 * Find a table in blocks of section.
+	 *
+	 * @param block One sub section block
+	 * @returns Table if exists
+	 */
+	private searchTableInBlock(block: SubSectionBlock) {
+		const control = block.content;
+		let tableAPI: TableAPI | undefined;
+		if (block.isA("sap.fe.templates.ObjectPage.controls.SubSectionBlock")) {
+			// The table may currently be shown in a full screen dialog, we can then get the reference to the TableAPI
+			// control from the custom data of the place holder panel
+			if (control.isA("sap.m.Panel") && control.data("FullScreenTablePlaceHolder")) {
+				tableAPI = control.data("tableAPIreference");
+			} else if (control.isA("sap.fe.macros.table.TableAPI")) {
+				tableAPI = control as TableAPI;
+			}
+			if (tableAPI) {
+				return tableAPI.content as Table;
+			}
+		}
+		return undefined;
+	}
+
 	onBeforeRendering() {
 		PageController.prototype.onBeforeRendering.apply(this);
 		// In the retrieveTextFromValueList scenario we need to ensure in case of reload/refresh that the meta model in the methode retrieveTextFromValueList of the FieldRuntime is available
@@ -221,6 +278,17 @@ class ObjectPageController extends PageController {
 			.catch(function (oError: any) {
 				Log.error("Error while retrieving the resource bundle", oError);
 			});
+		let subSections: ObjectPageSubSection[];
+		if (this._getObjectPageLayoutControl().getUseIconTabBar()) {
+			const sections = this._getObjectPageLayoutControl().getSections();
+			for (const section of sections) {
+				subSections = section.getSubSections();
+				this.checkSectionsForGridTable(subSections);
+			}
+		} else {
+			subSections = this._getAllSubSections();
+			this.checkSectionsForGridTable(subSections);
+		}
 	}
 
 	_onBeforeBinding(oContext: any, mParameters: any) {
@@ -349,7 +417,7 @@ class ObjectPageController extends PageController {
 				// Due to the left alignment of the Draft switch and the collaborative draft avatar controls
 				// there is a ToolbarSpacer in the actions aggregation which we need to exclude here!
 				// Due to the ACC report, we also need not to check for the InvisibleText elements
-				if (oAction.isA("sap.fe.macros.ShareAPI")) {
+				if (oAction.isA("sap.fe.macros.share.ShareAPI")) {
 					// since ShareAPI does not have a disable property
 					// hence there is no need to check if it is disbaled or not
 					return oAction.getVisible();
@@ -960,7 +1028,6 @@ class ObjectPageController extends PageController {
 	}
 
 	_findControlInSubSection(aParentElement: any, aSubsection: any, aControls: any, bIsChart?: boolean) {
-		const aSubSectionTables = [];
 		for (let element = 0; element < aParentElement.length; element++) {
 			let oElement = aParentElement[element].getContent instanceof Function && aParentElement[element].getContent();
 			if (bIsChart) {
@@ -987,10 +1054,6 @@ class ObjectPageController extends PageController {
 			}
 			if (oElement && oElement.isA && oElement.isA("sap.ui.mdc.Table")) {
 				aControls.push(oElement);
-				aSubSectionTables.push({
-					"table": oElement,
-					"gridTable": oElement.getType().isA("sap.ui.mdc.table.GridTableType")
-				});
 			}
 
 			if (oElement && oElement.isA && oElement.isA("sap.fe.macros.chart.ChartAPI")) {
@@ -1003,15 +1066,6 @@ class ObjectPageController extends PageController {
 				aControls.push(oElement);
 			}
 		}
-		if (
-			aSubSectionTables.length === 1 &&
-			aParentElement.length === 1 &&
-			aSubSectionTables[0].gridTable &&
-			!aSubsection.hasStyleClass("sapUxAPObjectPageSubSectionFitContainer")
-		) {
-			//In case there is only a single table in a section we fit that to the whole page so that the scrollbar comes only on table and not on page
-			aSubsection.addStyleClass("sapUxAPObjectPageSubSectionFitContainer");
-		}
 	}
 
 	_getAllSubSections() {