Merge pull request #1872 from kamalqureshi/form_not_populating

raheeliftikhar5 · web-flow · commit 67643f69b770 · 2025-07-15T23:37:58.000-07:00
Child element on Form not populating
diff --git a/client/packages/lowcoder/src/comps/comps/formComp/formComp.tsx b/client/packages/lowcoder/src/comps/comps/formComp/formComp.tsx
@@ -208,13 +208,17 @@ const FormBaseComp = (function () {
     );
   })
     .setPropertyViewFn((children) => {
+      const editorContext = useContext(EditorContext);
+      const isLogicMode = editorContext.editorModeStatus === "logic" || editorContext.editorModeStatus === "both";
+      const isLayoutMode = editorContext.editorModeStatus === "layout" || editorContext.editorModeStatus === "both";
+      
       return (
         <>
           <Section name={sectionNames.basic}>
             {children.resetAfterSubmit.propertyView({ label: trans("formComp.resetAfterSubmit") })}
           </Section>
 
-          {(useContext(EditorContext).editorModeStatus === "logic" || useContext(EditorContext).editorModeStatus === "both") && (
+          {isLogicMode && (
             <><Section name={sectionNames.interaction}>
                 {children.onEvent.getPropertyView()}
                 {disabledPropertyView(children)}
@@ -225,22 +229,22 @@ const FormBaseComp = (function () {
             </>
           )}
 
-          {(useContext(EditorContext).editorModeStatus === "layout" || useContext(EditorContext).editorModeStatus === "both") && (
+          {isLayoutMode && (
             <>
               <Section name={sectionNames.layout}>
                 {children.container.getPropertyView()}
               </Section>
             </>
           )}
 
-          {(useContext(EditorContext).editorModeStatus === "logic" || useContext(EditorContext).editorModeStatus === "both") && (
+          {isLogicMode && (
             <Section name={sectionNames.advanced}>
               {children.initialData.propertyView({ label: trans("formComp.initialData") })}
               {children.invalidFormMessage.propertyView({ label: trans("formComp.invalidFormMessage") })}
             </Section>
           )}
 
-          {(useContext(EditorContext).editorModeStatus === "layout" || useContext(EditorContext).editorModeStatus === "both") && (
+          {isLayoutMode && (
             <>
               <Section name={sectionNames.style}>
                 {children.container.stylePropertyView()}
@@ -383,9 +387,7 @@ let FormTmpComp = class extends FormBaseComp implements IForm {
       case CompActionTypes.UPDATE_NODES_V2: {
         const ret = super.reduce(action);
         // When the initial value changes, update the form
-        if (ret.children.initialData !== this.children.initialData) {
-          // FIXME: kill setTimeout ?
-          setTimeout(() => {
+          requestAnimationFrame(() => {
             this.dispatch(
               customAction<SetDataAction>(
                 {
@@ -396,7 +398,6 @@ let FormTmpComp = class extends FormBaseComp implements IForm {
               )
             );
           });
-        }
         return ret;
       }
       case CompActionTypes.CUSTOM:
diff --git a/client/packages/lowcoder/src/redux/sagas/orgSagas.ts b/client/packages/lowcoder/src/redux/sagas/orgSagas.ts
@@ -367,13 +367,15 @@ export function* fetchWorkspacesSaga(action: ReduxAction<{page: number, pageSize
       const apiData = response.data.data;
       
       // Transform orgId/orgName to match Org interface
-      const transformedItems = apiData.data.map(item => ({
-        id: item.orgView.orgId,
-        name: item.orgView.orgName,
-        createdAt: item.orgView.createdAt,
-        updatedAt: item.orgView.updatedAt,
-        isCurrentOrg: item.isCurrentOrg,
-      }));
+      const transformedItems = apiData.data
+        .filter(item => item.orgView && item.orgView.orgId) 
+        .map(item => ({
+          id: item.orgView.orgId,
+          name: item.orgView.orgName,
+          createdAt: item.orgView.createdAt,
+          updatedAt: item.orgView.updatedAt,
+          isCurrentOrg: item.isCurrentOrg,
+        }));
         
       yield put({
         type: ReduxActionTypes.FETCH_WORKSPACES_SUCCESS,
