fixed issue where adding event handlers inside list/grid adds multiple event handlers

raheeliftikhar5 · raheeliftikhar5 · commit fa1447000e29 · 2024-09-27T14:44:49.000+05:00
diff --git a/client/packages/lowcoder/src/components/ThemeSettingsSelector.tsx b/client/packages/lowcoder/src/components/ThemeSettingsSelector.tsx
@@ -428,7 +428,6 @@ export default function ThemeSettingsSelector(props: ColorConfigProps) {
             size="small"
             checked={showComponentLoaders}
             onChange={(value) => {
-              debugger;
               setComponentLoaders(value)
               configChange({ themeSettingKey, showComponentLoadingIndicators: value});
             }}
diff --git a/client/packages/lowcoder/src/comps/comps/tableComp/tableUtils.tsx b/client/packages/lowcoder/src/comps/comps/tableComp/tableUtils.tsx
@@ -465,8 +465,8 @@ export function genSelectionParams(
   filterData: RecordType[],
   selection: string
 ): Record<string, unknown> | undefined {
-  const idx = filterData.findIndex((row) => row[OB_ROW_ORI_INDEX] === selection);
-  if (idx < 0) {
+  const idx = filterData?.findIndex((row) => row[OB_ROW_ORI_INDEX] === selection);
+  if (!Boolean(filterData) || idx < 0) {
     return undefined;
   }
   const currentRow = filterData[idx];
diff --git a/client/packages/lowcoder/src/comps/generators/withSelectedMultiContext.tsx b/client/packages/lowcoder/src/comps/generators/withSelectedMultiContext.tsx
@@ -87,10 +87,6 @@ export function withSelectedMultiContext<TCtor extends MultiCompConstructor>(
         || isCustomAction<LazyCompReadyAction>(action, "LazyCompReady")
         || isCustomAction<ModuleReadyAction>(action, "moduleReady")
       ) && action.path[1] === SELECTED_KEY) {
-        if (action.path[0] === MAP_KEY && action.path[1] === SELECTED_KEY) {
-          action.path[1] = this.selection;
-          comp = super.reduce(action);
-        }
         // broadcast
         const newAction = {
           ...action,
