@finos/legend-query-builder 4.15.11 → 4.15.13

package/src/stores/QueryBuilderState.ts

@@ -69,10 +69,10 @@ import {
   buildLambdaVariableExpressions,
   buildRawLambdaFromLambdaFunction,
   PrimitiveType,
+  PackageableElementExplicitReference,
   RuntimePointer,
   QueryExplicitExecutionContext,
   attachFromQuery,
-  PackageableElementExplicitReference,
 } from '@finos/legend-graph';
 import { buildLambdaFunction } from './QueryBuilderValueSpecificationBuilder.js';
 import type {
@@ -108,7 +108,6 @@ import { QUERY_BUILDER_EVENT } from '../__lib__/QueryBuilderEvent.js';
 import { QUERY_BUILDER_SETTING_KEY } from '../__lib__/QueryBuilderSetting.js';
 import { QueryBuilderChangeHistoryState } from './QueryBuilderChangeHistoryState.js';
 import { type QueryBuilderWorkflowState } from './query-workflow/QueryBuilderWorkFlowState.js';
-import type { QueryBuilder_LegendApplicationPlugin_Extension } from './QueryBuilder_LegendApplicationPlugin_Extension.js';
 
 // eslint-disable-next-line @typescript-eslint/no-empty-object-type
 export interface QueryableSourceInfo {}
@@ -351,26 +350,6 @@ export abstract class QueryBuilderState implements CommandRegistrar {
     return queryExeContext;
   }
 
-  async propagateExecutionContextChange(
-    isGraphBuildingNotRequired?: boolean,
-  ): Promise<void> {
-    const propagateFuncHelpers = this.applicationStore.pluginManager
-      .getApplicationPlugins()
-      .flatMap(
-        (plugin) =>
-          (
-            plugin as QueryBuilder_LegendApplicationPlugin_Extension
-          ).getExtraQueryBuilderPropagateExecutionContextChangeHelper?.() ?? [],
-      );
-    for (const helper of propagateFuncHelpers) {
-      const propagateFuncHelper = helper(this, isGraphBuildingNotRequired);
-      if (propagateFuncHelper) {
-        await propagateFuncHelper();
-        return;
-      }
-    }
-  }
-
   /**
    * Gets information about the current queryBuilderState.
    * This information can be used as a part of analytics

package/src/stores/QueryBuilder_LegendApplicationPlugin_Extension.ts

@@ -37,7 +37,7 @@ export type CuratedTemplateQuerySpecification = {
   loadCuratedTemplateQuery(
     templateQuery: CuratedTemplateQuery,
     queryBuilderState: QueryBuilderState,
-  ): Promise<void>;
+  ): void;
 };
 
 export type LoadQueryFilterOption = {
@@ -95,11 +95,6 @@ export type QueryBuilderMenuActionConfiguration = {
   ) => React.ReactNode;
 };
 
-export type QueryBuilderPropagateExecutionContextChangeHelper = (
-  queryBuilderState: QueryBuilderState,
-  isGraphBuildingNotRequired?: boolean,
-) => (() => Promise<void>) | undefined;
-
 export interface QueryBuilder_LegendApplicationPlugin_Extension
   extends LegendApplicationPlugin {
   /**
@@ -163,9 +158,4 @@ export interface QueryBuilder_LegendApplicationPlugin_Extension
    * Get the list of export menu action configurations
    */
   getExtraQueryBuilderExportMenuActionConfigurations?(): QueryBuilderMenuActionConfiguration[];
-
-  /**
-   * Get the list of Query Builder Propagate Execution Context Change Helper
-   */
-  getExtraQueryBuilderPropagateExecutionContextChangeHelper?(): QueryBuilderPropagateExecutionContextChangeHelper[];
 }

package/src/stores/workflows/ServiceQueryBuilderState.ts

@@ -156,25 +156,23 @@ export class ServiceQueryBuilderState extends QueryBuilderState {
    * - If no class is chosen, try to choose a compatible class
    * - If the chosen class is compatible with the new chosen mapping, do nothing, otherwise, try to choose a compatible class
    */
-  override async propagateExecutionContextChange(
-    isGraphBuildingNotRequired?: boolean,
-  ): Promise<void> {
-    if (this.selectedExecutionContext) {
-      const mapping = this.selectedExecutionContext.mapping;
-      this.changeMapping(mapping, { keepQueryContent: true });
-      this.changeRuntime(this.selectedExecutionContext.runtimeValue);
-
-      const compatibleClasses = getMappingCompatibleClasses(
-        mapping,
-        this.graphManagerState.usableClasses,
-      );
-      // if there is no chosen class or the chosen one is not compatible
-      // with the mapping then pick a compatible class if possible
-      if (!this.class || !compatibleClasses.includes(this.class)) {
-        const possibleNewClass = getNullableFirstEntry(compatibleClasses);
-        if (possibleNewClass) {
-          this.changeClass(possibleNewClass);
-        }
+  propagateExecutionContextChange(
+    executionContext: ServiceExecutionContext,
+  ): void {
+    const mapping = executionContext.mapping;
+    this.changeMapping(mapping, { keepQueryContent: true });
+    this.changeRuntime(executionContext.runtimeValue);
+
+    const compatibleClasses = getMappingCompatibleClasses(
+      mapping,
+      this.graphManagerState.usableClasses,
+    );
+    // if there is no chosen class or the chosen one is not compatible
+    // with the mapping then pick a compatible class if possible
+    if (!this.class || !compatibleClasses.includes(this.class)) {
+      const possibleNewClass = getNullableFirstEntry(compatibleClasses);
+      if (possibleNewClass) {
+        this.changeClass(possibleNewClass);
       }
     }
   }