[CLDN-1565] Fixing bugs (#189)

* [CLDN-1565] Fixing bugs * removing uneeded import
CybercentreCanada · Apr 19, 2023 · 06ee1bb · 06ee1bb
1 parent 71c917c
commit 06ee1bb
Showing 1 changed file with 17 additions and 7 deletions.
diff --git a/superset-frontend/src/cccs-viz/plugins/plugin-chart-cccs-grid/src/plugin/controlPanel.tsx b/superset-frontend/src/cccs-viz/plugins/plugin-chart-cccs-grid/src/plugin/controlPanel.tsx
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-import React from 'react'
+import React from 'react';
 
 import {
   t,
@@ -48,7 +48,6 @@ import { StyledColumnOption } from 'src/explore/components/optionRenderers';
 
 import DrillActionConfig from '../components/controls/JumpActionConfigControll';
 
-
 export const PAGE_SIZE_OPTIONS = formatSelectOptions<number>([
   [0, t('page_size.all')],
   10,
@@ -104,6 +103,17 @@ const validateAggControlValues = (
     : [];
 };
 
+const validateAggColumnValues = (
+  controls: ControlStateMapping,
+  values: any[],
+  state: ControlPanelState,
+) => {
+  const result = validateAggControlValues(controls, values);
+  if (result.length === 0 && isAggMode({ controls })) {
+    return [];
+  }
+  return result;
+};
 
 // function isIP(v: unknown) {
 //   if (typeof v === 'string' && v.trim().length > 0) {
@@ -229,7 +239,7 @@ const config: ControlPanelConfig = {
 
                 return newState;
               },
-              rerender: ['metrics', 'percent_metrics', ],
+              rerender: ['metrics', 'percent_metrics'],
             },
           },
         ],
@@ -332,7 +342,7 @@ const config: ControlPanelConfig = {
               rerender: ['principalColumns'],
               visibility: isRawMode,
               canCopy: true,
-            } 
+            },
           },
         ],
         [
@@ -422,9 +432,9 @@ const config: ControlPanelConfig = {
                     controlState: ControlState,
                   ) => {
                     const { controls } = state;
-                    const originalMapStateToProps = isRawMode({ controls }) ?
-                      sharedControls?.columns?.mapStateToProps :
-                      sharedControls?.groupby?.mapStateToProps;
+                    const originalMapStateToProps = isRawMode({ controls })
+                      ? sharedControls?.columns?.mapStateToProps
+                      : sharedControls?.groupby?.mapStateToProps;
                     const newState =
                       originalMapStateToProps?.(state, controlState) ?? {};
                     const choices = isRawMode({ controls })