@sapui5/sap.suite.ui.generic.template 1.151.1 → 1.153.0

package/src/sap/suite/ui/generic/template/lib/ai/EasyFill/EasyFillDialogController.js

@@ -2,9 +2,8 @@ sap.ui.define([
     "sap/ui/model/json/JSONModel",
     "sap/m/MessageToast",
     "sap/suite/ui/generic/template/genericUtilities/FeLogger",
-    "sap/suite/ui/generic/template/genericUtilities/FeError",
     "sap/ui/core/ResizeHandler"
-], function(JSONModel, MessageToast, FeLogger, FeError, ResizeHandler) {
+], function(JSONModel, MessageToast, FeLogger, ResizeHandler) {
     "use strict";
 
     var sClassName = "lib.ai.EasyFill.EasyFillDialogController";
@@ -76,57 +75,74 @@ sap.ui.define([
 
         /**
          * Loads the EasyFillDialog fragment (if not already loaded) and opens it.
-         * Registers all event handlers and formatters on the fragment controller object.
+         * Uses getDialogFragmentAsync with keepAlive=true so the fragment is cached
+         * across opens — the OPL and SmartForm controls persist between invocations.
+         * A concurrent-open guard prevents the duplicate-ID error when the button is
+         * pressed before the first load completes.
          *
-         * The dialog is kept alive after first open (not destroyed on close) so that the
-         * ObjectPageLayout and SmartForm structures are preserved between invocations.
-         *
-         * @param {object} oController - MVC controller
-         * @param {object} oTemplateUtils - FE template utilities (used for getDialogFragmentAsync)
-         * @param {object} oHandlers - map of event handler and formatter functions to bind to the fragment
-         * @returns {Promise<void>}
+         * @param {object} oController   - MVC controller
+         * @param {object} oTemplateUtils - FE template utilities
+         * @param {object} oHandlers     - event handler / formatter map
          */
-        openDialog: function(oController, oTemplateUtils, oHandlers) {
+        openDialog: async function(oController, oTemplateUtils, oHandlers) {
             var sFragmentName = "sap.suite.ui.generic.template.lib.ai.EasyFill.fragments.EasyFillDialog";
+            var oView = oController.getView();
 
-            return oTemplateUtils.oCommonUtils.getDialogFragmentAsync(
-                sFragmentName,
-                oHandlers,
-                "easyFillPopup",
-                undefined,
-                true,  // keep alive — do not destroy on close
-                true   // use view as parent
-            ).then(function(oDialog) {
-                oDialog.open();
-
-                // Register a ResizeHandler on the ResponsiveSplitter
-                // The dialog is kept alive across opens so we must not register more than once.
-                if (!oDialog._easyFillResizeHandlerId) {
-                    oDialog.attachEventOnce("afterOpen", function() {
-                        // Locate the splitter by its stable fragment ID
-                        var oSplitter = oController.byId("EasyFillMainFlex") &&
-                            oController.byId("EasyFillMainFlex").findAggregatedObjects(true, function(o) {
-                                return o.isA("sap.ui.layout.ResponsiveSplitter");
-                            })[0];
-
-                        if (oSplitter) {
-                            var oDialogModel = oController.getView().getModel("easyFillDialogModel");
-                            // sapUiRSVisiblePaginator is added/removed by the splitter itself on resize
-                            oDialog._easyFillResizeHandlerId = ResizeHandler.register(oSplitter, function() {
-                                var bInPageMode = oSplitter.$().hasClass("sapUiRSVisiblePaginator");
-                                oDialogModel.setProperty("/isSplitterPageMode", bInPageMode);
-                                if (bInPageMode && !oDialogModel.getProperty("/isFullScreen")) {
-                                    // Reset fullscreen state so pane sizes are correct when dialog is widened again
-                                    oDialogModel.setProperty("/isFullScreen", true);
-                                    oController.byId("inputAreaSplitPane").getLayoutData().setSize("40%");
-                                    oController.byId("reviewAreaSplitPane").getLayoutData().setSize("60%");
-                                }
-                            });
-                        }
-                    });
+            // Guard: if a load is already in flight (concurrent button press) wait for it
+            if (oView._mEasyFillFragmentPromise) {
+                var oDialogPending;
+                try {
+                    oDialogPending = await oView._mEasyFillFragmentPromise;
+                } catch (e) {
+                    oLogger.error("openDialog: concurrent fragment load failed: " + e);
+                    return;
                 }
+                if (oDialogPending) { oDialogPending.open(); }
+                return;
+            }
+
+            // Store the in-flight promise before the first await so re-entrant calls
+            // hit the guard above instead of starting a second Fragment.load().
+            oView._mEasyFillFragmentPromise = oTemplateUtils.oCommonUtils.getDialogFragmentAsync(
+                sFragmentName, oHandlers, "easyFillPopup", undefined,
+                true,   // keepAlive — do not destroy on close
+                true    // use view as parent
+            );
+
+            var oDialog;
+            try {
+                oDialog = await oView._mEasyFillFragmentPromise;
+            } finally {
+                oView._mEasyFillFragmentPromise = null;  // always clear in-flight marker
+            }
 
-                oLogger.debug("openDialog: EasyFillDialog opened");
+            if (!oDialog) { return; }  // guard: load rejected, nothing to open
+
+            oDialog.open();
+
+            if (!oDialog._easyFillResizeHandlerId) {
+                this._registerResizeHandler(oController, oDialog);
+            }
+            oLogger.debug("openDialog: EasyFillDialog opened");
+        },
+
+        _registerResizeHandler: function(oController, oDialog) {
+            oDialog.attachEventOnce("afterOpen", function() {
+                var oSplitter = oController.byId("EasyFillMainFlex") &&
+                    oController.byId("EasyFillMainFlex").findAggregatedObjects(true, function(o) {
+                        return o.isA("sap.ui.layout.ResponsiveSplitter");
+                    })[0];
+                if (!oSplitter) { return; }
+                var oDialogModel = oController.getView().getModel("easyFillDialogModel");
+                oDialog._easyFillResizeHandlerId = ResizeHandler.register(oSplitter, function() {
+                    var bInPageMode = oSplitter.$().hasClass("sapUiRSVisiblePaginator");
+                    oDialogModel.setProperty("/isSplitterPageMode", bInPageMode);
+                    if (bInPageMode && !oDialogModel.getProperty("/isFullScreen")) {
+                        oDialogModel.setProperty("/isFullScreen", true);
+                        oController.byId("inputAreaSplitPane").getLayoutData().setSize("40%");
+                        oController.byId("reviewAreaSplitPane").getLayoutData().setSize("60%");
+                    }
+                });
             });
         },
 
@@ -158,27 +174,27 @@ sap.ui.define([
          * @param {object} oFieldCollector - EasyFillFieldCollector
          * @param {object} oEasyFillLayoutMode - EasyFillLayoutMode enum
          */
-        onSelectionChange: function(oEvent, oController, oTemplateUtils, oRb, oFieldCollector, oEasyFillLayoutMode) {
+        onSelectionChange: function(oEvent, oController, oFieldCollector, oEasyFillLayoutMode, oOPStructureMap) {
             var sSelectedKey = oEvent.getSource().getSelectedKey();
             var oDialogModel = oController.getView().getModel("easyFillDialogModel");
-            var oEntityType = oTemplateUtils.oCommonUtils.getMetaModelEntityType(
-                oController.getOwnerComponent().getEntitySet()
-            );
             oDialogModel.setProperty("/easyFillMode", sSelectedKey);
-            var oLLMResult = oDialogModel.getProperty("/llmData") || {};
-            // Use stored updatable subset when available; fall back to full LLM result
-            // only when it is absent (e.g. before the first AI submit).
-            // Important: an empty map is a valid state (all fields are non-updatable)
-            // and must not trigger fallback, otherwise non-updatable fields appear as filled.
-            var oStoredUpdatable = oDialogModel.getProperty("/updatableFields");
-            var oUpdatableFields = (oStoredUpdatable !== undefined && oStoredUpdatable !== null)
-                ? oStoredUpdatable
-                : oLLMResult;
+
+            var oStoredUpdatable    = oDialogModel.getProperty("/updatableFields")    || {};
             var oNonUpdatableFields = oDialogModel.getProperty("/nonUpdatableFields") || {};
-            var aData = (sSelectedKey === oEasyFillLayoutMode.CondensedMode)
-                ? oFieldCollector.getSectionsForCondensedView(oLLMResult, oUpdatableFields, oNonUpdatableFields)
-                : oFieldCollector.getSectionsForObjectPageView(oEntityType, oLLMResult, oRb, oNonUpdatableFields);
-            oDialogModel.setProperty("/sections", aData);
+
+            var aSections;
+            if (sSelectedKey === oEasyFillLayoutMode.CondensedMode) {
+                aSections = oFieldCollector.getSectionInfoForCondensedMode(oStoredUpdatable, oNonUpdatableFields);
+            } else if (oOPStructureMap) {
+                aSections = oFieldCollector.applyLLMResultToStructureMap(
+                    oOPStructureMap.fieldMap, oOPStructureMap.sections,
+                    oStoredUpdatable, oNonUpdatableFields
+                );
+            } else {
+                aSections = oFieldCollector.getSectionInfoForCondensedMode(oStoredUpdatable, oNonUpdatableFields);
+                oLogger.warning("onSelectionChange: oOPStructureMap not available");
+            }
+            oDialogModel.setProperty("/sections", aSections);
         },
 
         /**