code review fix(2)

src/ui/components/Widgets/Chart/Chart.tsx

@@ -65,6 +65,7 @@ export const Chart = (props: ChartNoWidgetProps) => {
         ignoreUsedParams,
         onInnerParamsChanged,
         disableChartLoader,
+        actionParamsEnabled,
     } = props;
 
     const innerParamsRef = React.useRef<DataProps['params'] | null>(null);
@@ -170,7 +171,7 @@ export const Chart = (props: ChartNoWidgetProps) => {
         ignoreUsedParams,
         clearedOuterParams,
         onInnerParamsChanged,
-        enableActionParams: false,
+        enableActionParams: actionParamsEnabled,
     });
 
     /**

src/ui/components/Widgets/Chart/types.ts

@@ -103,6 +103,7 @@ type ChartKitBaseWrapperProps = ChartsProps & {
     skipReload?: boolean;
 
     renderPluginLoader?: () => React.ReactNode;
+    actionParamsEnabled?: boolean;
 };
 
 export type ChartWidgetProviderPropsWithRefProps = Omit<

src/ui/constants/common.ts

@@ -255,6 +255,7 @@ export const URL_OPTIONS = {
     EMBEDDED: '_embedded',
     NO_CONTROLS: '_no_controls',
     LANGUAGE: '_lang',
+    ACTION_PARAMS_ENABLED: '_ap_enabled',
 };
 
 export const DLS_SUBJECT = {

src/ui/libs/DatalensChartkit/ChartKit/helpers/apply-hc-handlers.ts

@@ -1,11 +1,12 @@
 import type {Highcharts} from '@gravity-ui/chartkit/highcharts';
+import {pickActionParamsFromParams} from '@gravity-ui/dashkit';
 import {wrap} from 'highcharts';
 import get from 'lodash/get';
 import has from 'lodash/has';
 import merge from 'lodash/merge';
 import set from 'lodash/set';
-import {GraphWidgetEventScope, URL_ACTION_PARAMS_PREFIX} from 'shared';
 
+import type {GraphWidgetEventScope} from '../../../../../shared';
 import type {GraphWidget} from '../../types';
 import type {ChartKitAdapterProps} from '../types';
 
@@ -30,19 +31,6 @@ export const fixPieTotals = (args: {data: GraphWidget}) => {
     });
 };
 
-function getActionParams(params: Record<string, string | string[]> = {}) {
-    const actionParams: Record<string, string | string[]> = {};
-    Object.entries(params).forEach(([key, value]) => {
-        if (key.startsWith(URL_ACTION_PARAMS_PREFIX)) {
-            actionParams[key.slice(URL_ACTION_PARAMS_PREFIX.length)] = Array.isArray(value)
-                ? value
-                : [value];
-        }
-    });
-
-    return actionParams;
-}
-
 export const applySetActionParamsEvents = (args: {
     action: ShapedAction;
     data: GraphWidget;
@@ -72,7 +60,7 @@ export const applySetActionParamsEvents = (args: {
         set(data, pathToScatterMarkerStates, {});
     }
 
-    const actionParams = getActionParams(get(data, 'unresolvedParams', {}));
+    const actionParams = pickActionParamsFromParams(get(data, 'unresolvedParams', {}));
 
     wrap(
         get(data, pathToChartEvents),

src/ui/units/preview/components/Preview/Preview.tsx

@@ -65,7 +65,7 @@ const Preview: React.FC<PreviewProps> = (props) => {
         isEmbedded,
     } = props;
 
-    const {noControls} = Utils.getOptionsFromSearch(search);
+    const {noControls, actionParamsEnabled} = Utils.getOptionsFromSearch(search);
 
     const possibleEntryId = React.useMemo(() => extractEntryId(idOrSource), [idOrSource]);
 
@@ -197,6 +197,7 @@ const Preview: React.FC<PreviewProps> = (props) => {
                     onChartLoad={onChartLoad}
                     onChartRender={onChartRender}
                     noControls={noControls}
+                    actionParamsEnabled={actionParamsEnabled}
                     forwardedRef={chartKitRef as unknown as React.RefObject<ChartKitType>}
                     splitTooltip={hasSplitTooltip}
                     menuType="preview"

src/ui/utils/utils.ts

@@ -237,6 +237,7 @@ export default class Utils {
             noControls:
                 searchParams.get(URL_OPTIONS.NO_CONTROLS) === '1' ||
                 searchParams.get(URL_OPTIONS.NO_CONTROLS) === 'true', // deprecated
+            actionParamsEnabled: searchParams.get(URL_OPTIONS.ACTION_PARAMS_ENABLED) === '1',
         };
     }