fix: CLIN-2916 fix saving custom filter after adding saved custom pil…

…l to it
Ferlab-Ste-Justine · Aug 7, 2024 · 8067dcd · 8067dcd
1 parent ecf75d6
commit 8067dcd
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 5 deletions.
diff --git a/packages/ui/Release.md b/packages/ui/Release.md
@@ -1,3 +1,6 @@
+### 10.4.6-rc4 2024-08-07
+- fix: CLIN-2916 saving custom filter after adding saved custom pill to it
+
 ### 10.4.5 2024-08-05
 - feat: FLUI-138 add sentry local storage integration
 

diff --git a/packages/ui/package-lock.json b/packages/ui/package-lock.json
diff --git a/packages/ui/package.json b/packages/ui/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@ferlab/ui",
-    "version": "10.4.5",
+    "version": "10.4.6-rc4",
     "description": "Core components for scientific research data portals",
     "publishConfig": {
         "access": "public"

diff --git a/packages/ui/src/components/QueryBuilder/utils/useQueryBuilderState.tsx b/packages/ui/src/components/QueryBuilder/utils/useQueryBuilderState.tsx
@@ -1,5 +1,5 @@
 import { useEffect, useState } from 'react';
-import { isEmpty } from 'lodash';
+import { get, isEmpty } from 'lodash';
 import { v4 } from 'uuid';
 
 import { BooleanOperators, RangeOperators, TermOperators } from '../../../data/sqon/operators';
@@ -17,6 +17,7 @@ import {
     getDefaultSyntheticSqon,
     getUpdatedActiveQuery,
     getUpdatedActiveQueryByFilterGroup,
+    isBooleanOperator,
     removeFieldFromActiveQuery,
 } from '../../../data/sqon/utils';
 import { IFilter, IFilterGroup } from '../../filters/types';
@@ -215,7 +216,11 @@ export const removePillFromQueryBuilder = (pillId: string, queryBuilderId: strin
 
 export const addPillToQueryBuilder = (pill: IValueQuery, queryBuilderId: string): void => {
     const activeQuery = getActiveQuery(queryBuilderId);
-    updateQuery({ query: { ...activeQuery, content: [...activeQuery.content, pill] }, queryBuilderId });
+    let newPill = pill;
+    if (isBooleanOperator(pill) && pill.content.length === 1) {
+        newPill = { ...pill, content: get(pill.content[0], 'content', []) };
+    }
+    updateQuery({ query: { ...activeQuery, content: [...activeQuery.content, newPill] }, queryBuilderId });
 };
 
 /**