Skip to content

Commit ec46074

Browse files
authored
Fix validation issue on Both Sides applicability (#3155)
Signed-off-by: basseche <[email protected]>
1 parent 0f896dc commit ec46074

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

src/components/dialogs/limits/limits-utils.tsx

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,20 @@ export interface OperationalLimitsId {
1616
export const areOperationalLimitsGroupUnique = (array: OperationalLimitsId[]) => {
1717
const equipmentApplicabilityElements: string[] = array
1818
.filter((item: OperationalLimitsId) => item.applicability === APPLICABILITY.EQUIPMENT.id)
19-
.map((item) => item.name);
19+
.map((item: OperationalLimitsId) => item.name);
2020

2121
if (
22-
equipmentApplicabilityElements.map((item) => array.filter((arrayItem) => arrayItem.name === item).length > 1)
23-
.length > 0
22+
// never more than one EQUIPMENT limit set of a given name
23+
equipmentApplicabilityElements.filter(
24+
(item: string) =>
25+
array.filter((arrayItem: OperationalLimitsId): boolean => arrayItem.name === item).length > 1
26+
).length > 0
2427
) {
2528
return false;
2629
}
2730

2831
const otherApplicabilityElements: string[] = array
2932
.filter((item: OperationalLimitsId) => item.applicability !== APPLICABILITY.EQUIPMENT.id)
30-
.map((item) => item.name + item.applicability);
33+
.map((item: OperationalLimitsId) => item.name + item.applicability);
3134
return areArrayElementsUnique(otherApplicabilityElements);
3235
};

0 commit comments

Comments
 (0)