Skip to content

Commit f414e81

Browse files
refactor: improve getDisableReason parameter handling in isFieldDisabled function
1 parent 01dc87b commit f414e81

1 file changed

Lines changed: 21 additions & 6 deletions

File tree

src/visualBuilder/utils/isFieldDisabled.ts

Lines changed: 21 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -177,11 +177,24 @@ export const isFieldDisabled = (
177177
}
178178

179179
const isDisabled = Object.values(flags).some(Boolean);
180-
const reason = getDisableReason(flags, {
181-
stageName: entryWorkflowStageDetails?.stage?.name,
182-
entryWorkflowStageDetails,
183-
entryPermissions,
184-
});
180+
181+
const getDisableReasonParams: {
182+
stageName?: string;
183+
entryWorkflowStageDetails?: WorkflowStageDetails;
184+
entryPermissions?: EntryPermissions;
185+
} = {};
186+
if (entryWorkflowStageDetails?.stage?.name !== undefined) {
187+
getDisableReasonParams.stageName = entryWorkflowStageDetails.stage.name;
188+
}
189+
if (entryWorkflowStageDetails !== undefined) {
190+
getDisableReasonParams.entryWorkflowStageDetails =
191+
entryWorkflowStageDetails;
192+
}
193+
if (entryPermissions !== undefined) {
194+
getDisableReasonParams.entryPermissions = entryPermissions;
195+
}
196+
197+
const reason = getDisableReason(flags, getDisableReasonParams);
185198

186199
let workflowRequestUi: "request" | "pending" | undefined;
187200
if (
@@ -198,5 +211,7 @@ export const isFieldDisabled = (
198211
}
199212
}
200213

201-
return { isDisabled, reason, workflowRequestUi };
214+
return workflowRequestUi !== undefined
215+
? { isDisabled, reason, workflowRequestUi }
216+
: { isDisabled, reason };
202217
};

0 commit comments

Comments
 (0)