@finos/legend-query-builder 4.14.41 → 4.14.43

package/src/components/fetch-structure/QueryBuilderTDSPanel.tsx

@@ -1138,134 +1138,27 @@ export const QueryBuilderTDSPanel = observer(
           >
             <div className="query-builder__projection__result-modifier-prompt__header">
               <button
-                className="query-builder__projection__result-modifier-prompt__header__label editable-value"
+                className="query-builder__projection__result-modifier-prompt__header__label"
                 onClick={openResultSetModifierEditor}
-                title="Configure Query Options..."
+                title="Configure result set modifiers..."
               >
                 <CogIcon className="query-builder__projection__result-modifier-prompt__header__label__icon" />
                 <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                  {tdsState.isQueryOptionsSet
-                    ? 'Query Options'
-                    : 'Set Query Options'}
+                  Query Options
                 </div>
               </button>
-              <div className="query-builder__projection__result-modifier-prompt__divider">
-                {tdsState.isQueryOptionsSet && ' - '}
-              </div>
-              {tdsState.queryBuilderState.milestoningState.businessDate && (
-                <div className="query-builder__projection__result-modifier-prompt__group">
-                  <div className="query-builder__projection__result-modifier-prompt__group__label">
-                    Business Date
-                  </div>
-                  <button
-                    className="query-builder__projection__result-modifier-prompt__header__label editable-value"
-                    onClick={openResultSetModifierEditor}
-                  >
-                    <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                      {getNameOfValueSpecification(
-                        tdsState.queryBuilderState.milestoningState.getMilestoningParameterValue(
-                          tdsState.queryBuilderState.milestoningState
-                            .businessDate,
-                        ) ??
-                          tdsState.queryBuilderState.milestoningState
-                            .businessDate,
-                        tdsState.queryBuilderState,
-                      )}
-                    </div>
-                  </button>
-                </div>
-              )}
-              {tdsState.queryBuilderState.milestoningState.processingDate && (
-                <div className="query-builder__projection__result-modifier-prompt__group">
-                  <div className="query-builder__projection__result-modifier-prompt__group__label">
-                    Processing Date
-                  </div>
-                  <button
-                    className="query-builder__projection__result-modifier-prompt__header__label editable-value"
-                    onClick={openResultSetModifierEditor}
-                  >
-                    <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                      {getNameOfValueSpecification(
-                        tdsState.queryBuilderState.milestoningState.getMilestoningParameterValue(
-                          tdsState.queryBuilderState.milestoningState
-                            .processingDate,
-                        ) ??
-                          tdsState.queryBuilderState.milestoningState
-                            .processingDate,
-                        tdsState.queryBuilderState,
-                      )}
-                    </div>
-                  </button>
-                </div>
-              )}
-              {tdsState.queryBuilderState.milestoningState
-                .isAllVersionsEnabled &&
-                !tdsState.queryBuilderState.milestoningState
-                  .isAllVersionsInRangeEnabled && (
-                  <div className="query-builder__projection__result-modifier-prompt__group">
-                    <div className="query-builder__projection__result-modifier-prompt__group__label">
-                      All Versions
-                    </div>
-                    <button
-                      className="query-builder__projection__result-modifier-prompt__header__label editable-value"
-                      onClick={openResultSetModifierEditor}
-                    >
-                      <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                        Yes
-                      </div>
-                    </button>
-                  </div>
-                )}
-              {tdsState.queryBuilderState.milestoningState
-                .isAllVersionsInRangeEnabled &&
-                tdsState.queryBuilderState.milestoningState.startDate &&
-                tdsState.queryBuilderState.milestoningState.endDate && (
-                  <div className="query-builder__projection__result-modifier-prompt__group">
-                    <div className="query-builder__projection__result-modifier-prompt__group__label">
-                      All Versions
-                    </div>
-                    <button
-                      className="query-builder__projection__result-modifier-prompt__header__label editable-value"
-                      onClick={openResultSetModifierEditor}
-                    >
-                      <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                        (
-                        {getNameOfValueSpecification(
-                          tdsState.queryBuilderState.milestoningState.getMilestoningParameterValue(
-                            tdsState.queryBuilderState.milestoningState
-                              .startDate,
-                          ) ??
-                            tdsState.queryBuilderState.milestoningState
-                              .startDate,
-                          tdsState.queryBuilderState,
-                        )}{' '}
-                        -{' '}
-                        {getNameOfValueSpecification(
-                          tdsState.queryBuilderState.milestoningState.getMilestoningParameterValue(
-                            tdsState.queryBuilderState.milestoningState.endDate,
-                          ) ??
-                            tdsState.queryBuilderState.milestoningState.endDate,
-                          tdsState.queryBuilderState,
-                        )}
-                        )
-                      </div>
-                    </button>
-                  </div>
-                )}
             </div>
             {tdsState.resultSetModifierState.limit && (
               <div className="query-builder__projection__result-modifier-prompt__group">
                 <div className="query-builder__projection__result-modifier-prompt__group__label">
                   Max Rows
                 </div>
-                <button
-                  className="query-builder__projection__result-modifier-prompt__header__label editable-value"
+                <div
+                  className="query-builder__projection__result-modifier-prompt__group__content"
                   onClick={openResultSetModifierEditor}
                 >
-                  <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                    {tdsState.resultSetModifierState.limit}
-                  </div>
-                </button>
+                  {tdsState.resultSetModifierState.limit}
+                </div>
               </div>
             )}
             {tdsState.resultSetModifierState.distinct && (
@@ -1273,14 +1166,12 @@ export const QueryBuilderTDSPanel = observer(
                 <div className="query-builder__projection__result-modifier-prompt__group__label">
                   Eliminate Duplicate Rows
                 </div>
-                <button
-                  className="query-builder__projection__result-modifier-prompt__header__label editable-value"
+                <div
+                  className="query-builder__projection__result-modifier-prompt__group__content"
                   onClick={openResultSetModifierEditor}
                 >
-                  <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                    Yes
-                  </div>
-                </button>
+                  Yes
+                </div>
               </div>
             )}
             {tdsState.resultSetModifierState.sortColumns.length > 0 && (
@@ -1290,15 +1181,13 @@ export const QueryBuilderTDSPanel = observer(
                 </div>
                 {tdsState.resultSetModifierState.sortColumns.map(
                   (columnState) => (
-                    <button
+                    <div
+                      className="query-builder__projection__result-modifier-prompt__group__content"
                       key={columnState.columnState.uuid}
-                      className="query-builder__projection__result-modifier-prompt__header__label editable-value"
                       onClick={openResultSetModifierEditor}
                     >
-                      <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                        {`${columnState.columnState.columnName} ${columnState.sortType}`}
-                      </div>
-                    </button>
+                      {`${columnState.columnState.columnName} ${columnState.sortType}`}
+                    </div>
                   ),
                 )}
               </div>
@@ -1308,14 +1197,12 @@ export const QueryBuilderTDSPanel = observer(
                 <div className="query-builder__projection__result-modifier-prompt__group__label">
                   Slice
                 </div>
-                <button
-                  className="query-builder__projection__result-modifier-prompt__header__label editable-value"
+                <div
+                  className="query-builder__projection__result-modifier-prompt__group__content"
                   onClick={openResultSetModifierEditor}
                 >
-                  <div className="query-builder__projection__result-modifier-prompt__header__label__title">
-                    {`${tdsState.resultSetModifierState.slice[0]},${tdsState.resultSetModifierState.slice[1]}`}
-                  </div>
-                </button>
+                  {`${tdsState.resultSetModifierState.slice[0]},${tdsState.resultSetModifierState.slice[1]}`}
+                </div>
               </div>
             )}
             {tdsState.queryBuilderState.watermarkState.value && (

package/src/components/result/QueryBuilderResultPanel.tsx

@@ -204,15 +204,7 @@ export const QueryBuilderResultPanel = observer(
     const runQuery = (): void => {
       resultState.setSelectedCells([]);
       resultState.pressedRunQuery.inProgress();
-      if (
-        queryParametersState.parameterStates.length &&
-        queryParametersState.parameterStates.find(
-          (param) =>
-            !queryBuilderState.milestoningState.isMilestoningParameter(
-              param.parameter,
-            ),
-        )
-      ) {
+      if (queryParametersState.parameterStates.length) {
         queryParametersState.parameterValuesEditorState.open(
           (): Promise<void> =>
             flowResult(resultState.runQuery()).catch(

package/src/components/shared/BasicValueSpecificationEditor.tsx

@@ -727,16 +727,26 @@ const PrimitiveCollectionInstanceValueEditor = observer(
     const convertInputValueToValueSpec = (): ValueSpecification | null => {
       const trimmedInputValue = inputValue.trim();
 
-      if (isValueAlreadySelected(trimmedInputValue)) {
-        return null;
-      }
-
       if (trimmedInputValue.length) {
-        return convertTextToPrimitiveInstanceValue(
+        const newValueSpec = convertTextToPrimitiveInstanceValue(
           expectedType,
           trimmedInputValue,
           observerContext,
         );
+
+        if (
+          newValueSpec === null ||
+          getValueSpecificationStringValue(newValueSpec) === undefined ||
+          isValueAlreadySelected(
+            guaranteeNonNullable(
+              getValueSpecificationStringValue(newValueSpec),
+            ),
+          )
+        ) {
+          return null;
+        }
+
+        return newValueSpec;
       }
       return null;
     };
@@ -830,19 +840,20 @@ const PrimitiveCollectionInstanceValueEditor = observer(
       if (!parsedData) {
         return;
       }
-      const newValues = uniq(parsedData)
-        .map((value) => {
-          const newValueSpec = convertTextToPrimitiveInstanceValue(
-            expectedType,
-            value,
-            observerContext,
-          );
-          return newValueSpec
-            ? getValueSpecificationStringValue(newValueSpec)
-            : null;
-        })
-        .filter(isNonNullable)
-        .filter((value) => !isValueAlreadySelected(value));
+      const newValues = uniq(
+        uniq(parsedData)
+          .map((value) => {
+            const newValueSpec = convertTextToPrimitiveInstanceValue(
+              expectedType,
+              value,
+              observerContext,
+            );
+            return newValueSpec
+              ? getValueSpecificationStringValue(newValueSpec)
+              : null;
+          })
+          .filter(isNonNullable),
+      ).filter((value) => !isValueAlreadySelected(value));
       setSelectedOptions([
         ...selectedOptions,
         ...newValues.map((value) => ({ label: value, value })),

package/src/components/shared/QueryBuilderVariableSelector.tsx

@@ -149,18 +149,6 @@ export const VariableViewer = observer(
     const variableTypeName =
       variable.genericType?.value.rawType.name ??
       (isConstant ? CALCULATED : undefined);
-    const isMilestoningParameter =
-      queryBuilderState.milestoningState.isMilestoningParameter(variable);
-    const milestoningParameterValue =
-      queryBuilderState.milestoningState.getMilestoningParameterValue(variable);
-    const milestoningParameterValueString = isMilestoningParameter
-      ? milestoningParameterValue
-        ? getNameOfValueSpecification(
-            milestoningParameterValue,
-            queryBuilderState,
-          )
-        : undefined
-      : undefined;
     const deleteDisabled = isReadOnly || isVariableUsed;
     const deleteTitle = isVariableUsed ? 'Used in query' : 'Remove';
     const editVariable = (): void => {
@@ -239,16 +227,6 @@ export const VariableViewer = observer(
                   <div className="query-builder__variables__variable__type__label">
                     {variableTypeName ?? 'unknown'}
                   </div>
-                  {isMilestoningParameter && (
-                    <>
-                      <div className="query-builder__variables__variable__type__label query-builder__variables__variable__type__label--milestoning">
-                        milestoning
-                      </div>
-                      <div className="query-builder__constants__value">
-                        {milestoningParameterValueString}
-                      </div>
-                    </>
-                  )}
                 </div>
               )}
             </div>

package/src/stores/QueryBuilderPropertyEditorState.ts

@@ -161,8 +161,7 @@ export const generateValueSpecificationForParameter = (
   return createNullishValue(graph);
 };
 
-//here
-export const fillDerivedPropertyParameterValues = (
+const fillDerivedPropertyParameterValues = (
   derivedPropertyExpressionState: QueryBuilderDerivedPropertyExpressionState,
 ): void => {
   const parameterValues: ValueSpecification[] =

package/src/stores/QueryBuilderState.ts

@@ -110,8 +110,6 @@ import type { QueryBuilderConfig } from '../graph-manager/QueryBuilderConfig.js'
 import { QUERY_BUILDER_EVENT } from '../__lib__/QueryBuilderEvent.js';
 import { QueryBuilderChangeHistoryState } from './QueryBuilderChangeHistoryState.js';
 import { type QueryBuilderWorkflowState } from './query-workflow/QueryBuilderWorkFlowState.js';
-import { initializeMilestoningStateFromDerivedProperty } from './milestoning/QueryBuilderMilestoningHelper.js';
-import { QueryBuilderSimpleProjectionColumnState } from './fetch-structure/tds/projection/QueryBuilderProjectionColumnState.js';
 
 export interface QuerySDLC {}
 
@@ -699,18 +697,6 @@ export abstract class QueryBuilderState implements CommandRegistrar {
         );
       }
       if (this.parametersState.parameterStates.length > 0) {
-        const fetchStructureStateImp = this.fetchStructureState.implementation;
-        if (fetchStructureStateImp instanceof QueryBuilderTDSState) {
-          fetchStructureStateImp.projectionColumns.forEach((colState) => {
-            if (colState instanceof QueryBuilderSimpleProjectionColumnState) {
-              colState.propertyExpressionState.derivedPropertyExpressionStates.forEach(
-                (state) => {
-                  initializeMilestoningStateFromDerivedProperty(state);
-                },
-              );
-            }
-          });
-        }
         this.setShowParametersPanel(true);
       }
       this.fetchStructureState.initializeWithQuery();

package/src/stores/fetch-structure/tds/QueryBuilderTDSState.ts

@@ -146,7 +146,6 @@ export class QueryBuilderTDSState
       TEMPORARY__showPostFetchStructurePanel: computed,
       derivations: computed,
       hasParserError: computed,
-      isQueryOptionsSet: computed,
       addColumn: action,
       moveColumn: action,
       removeAllColumns: action,
@@ -357,16 +356,6 @@ export class QueryBuilderTDSState
     return fetchStructureValidationIssues;
   }
 
-  get isQueryOptionsSet(): boolean {
-    return (
-      this.resultSetModifierState.limit !== undefined ||
-      this.queryBuilderState.milestoningState.isMilestonedQuery ||
-      this.resultSetModifierState.slice !== undefined ||
-      this.resultSetModifierState.sortColumns.length > 0 ||
-      this.resultSetModifierState.distinct
-    );
-  }
-
   get tdsColumns(): QueryBuilderTDSColumnState[] {
     const aggregationStateCols = this.aggregationState.columns.map(
       (c) => c.projectionColumnState,

package/src/stores/milestoning/QueryBuilderMilestoningHelper.ts

@@ -290,30 +290,6 @@ export const generateMilestonedPropertyParameterValue = (
     );
 };
 
-export const initializeMilestoningStateFromDerivedProperty = (
-  derivedPropertyExpressionState: QueryBuilderDerivedPropertyExpressionState,
-): void => {
-  const milestoningState =
-    derivedPropertyExpressionState.queryBuilderState.milestoningState;
-  const temporalTarget =
-    derivedPropertyExpressionState.propertyExpression.func.value.genericType
-      .value.rawType instanceof Class &&
-    derivedPropertyExpressionState.propertyExpression.func.value._OWNER
-      ._generatedMilestonedProperties.length !== 0
-      ? getMilestoneTemporalStereotype(
-          derivedPropertyExpressionState.propertyExpression.func.value
-            .genericType.value.rawType,
-          derivedPropertyExpressionState.queryBuilderState.graphManagerState
-            .graph,
-        )
-      : undefined;
-  if (temporalTarget) {
-    milestoningState
-      .getMilestoningImplementation(temporalTarget)
-      .initializeMilestoningParameters();
-  }
-};
-
 export const validateMilestoningPropertyExpressionChain = (
   sourceStereotype: MILESTONING_STEREOTYPE | undefined,
   targetStereotype: MILESTONING_STEREOTYPE,

package/src/stores/milestoning/QueryBuilderMilestoningState.ts

@@ -75,7 +75,7 @@ export class QueryBuilderMilestoningState implements Hashable {
       startDate: observable,
       endDate: observable,
       showMilestoningEditor: observable,
-
+      isMilestonedQuery: computed,
       setProcessingDate: action,
       setBusinessDate: action,
       setStartDate: action,
@@ -87,10 +87,6 @@ export class QueryBuilderMilestoningState implements Hashable {
       initializeAllVersionsInRangeParameters: action,
       clearAllVersionsInRangeParameters: action,
       clearGetAllParameters: action,
-
-      isAllVersionsEnabled: computed,
-      isAllVersionsInRangeEnabled: computed,
-      isMilestonedQuery: computed,
       hashCode: computed,
     });
 
@@ -384,9 +380,9 @@ export class QueryBuilderMilestoningState implements Hashable {
         this.queryBuilderState.parametersState.parameterStates.find(
           (p) => p.parameter === this.startDate,
         );
-      if (paramState) {
-        this.queryBuilderState.parametersState.removeParameter(paramState);
-      }
+      this.queryBuilderState.parametersState.removeParameter(
+        guaranteeNonNullable(paramState),
+      );
     }
     if (
       this.endDate instanceof VariableExpression &&
@@ -398,9 +394,9 @@ export class QueryBuilderMilestoningState implements Hashable {
         this.queryBuilderState.parametersState.parameterStates.find(
           (p) => p.parameter === this.endDate,
         );
-      if (paramState) {
-        this.queryBuilderState.parametersState.removeParameter(paramState);
-      }
+      this.queryBuilderState.parametersState.removeParameter(
+        guaranteeNonNullable(paramState),
+      );
     }
     this.setStartDate(undefined);
     this.setEndDate(undefined);
@@ -451,33 +447,6 @@ export class QueryBuilderMilestoningState implements Hashable {
     return milestoningParameter;
   }
 
-  updateMilestoningParameterValue(
-    parameter: VariableExpression,
-    value: ValueSpecification | undefined,
-  ): void {
-    const variableState =
-      this.queryBuilderState.parametersState.parameterStates.find(
-        (param) => param.parameter.name === parameter.name,
-      );
-    if (variableState) {
-      variableState.setValue(value);
-    }
-  }
-
-  getMilestoningParameterValue(
-    milestoningParameter: ValueSpecification,
-  ): ValueSpecification | undefined {
-    let value: ValueSpecification | undefined = milestoningParameter;
-    if (milestoningParameter instanceof VariableExpression) {
-      const paramState =
-        this.queryBuilderState.parametersState.parameterStates.find(
-          (param) => param.parameter.name === milestoningParameter.name,
-        );
-      value = paramState?.value;
-    }
-    return value;
-  }
-
   isVariableUsed(variable: VariableExpression): boolean {
     const usedInBusiness = this.businessDate
       ? isValueExpressionReferencedInValue(variable, this.businessDate)
@@ -488,33 +457,6 @@ export class QueryBuilderMilestoningState implements Hashable {
     return usedInBusiness || usedInProcessingDate;
   }
 
-  isMilestoningParameter(parameter: VariableExpression): boolean {
-    let isMilestoningParameter = false;
-    if (this.businessDate instanceof VariableExpression) {
-      isMilestoningParameter =
-        parameter.name ===
-        guaranteeType(this.businessDate, VariableExpression).name;
-    }
-    if (this.processingDate instanceof VariableExpression) {
-      isMilestoningParameter =
-        isMilestoningParameter ||
-        parameter.name ===
-          guaranteeType(this.processingDate, VariableExpression).name;
-    }
-    if (this.startDate instanceof VariableExpression) {
-      isMilestoningParameter =
-        isMilestoningParameter ||
-        parameter.name ===
-          guaranteeType(this.startDate, VariableExpression).name;
-    }
-    if (this.endDate instanceof VariableExpression) {
-      isMilestoningParameter =
-        isMilestoningParameter ||
-        parameter.name === guaranteeType(this.endDate, VariableExpression).name;
-    }
-    return isMilestoningParameter;
-  }
-
   get hashCode(): string {
     return hashArray([
       QUERY_BUILDER_STATE_HASH_STRUCTURE.MILESTONING_STATE,