Skip to content

Commit 6ed7b67

Browse files
authored
Merge pull request #186 from EyeSeeTea/fix/togglemultiple-de-disabled-vacuous-truth
fix(toggleMultiple): avoid vacuous-truth disable when no orgUnit toggles
2 parents a2cf1d9 + 46c7325 commit 6ed7b67

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "d2-autogen-forms",
33
"description": "D2 Autogenerated Forms",
4-
"version": "1.5.1",
4+
"version": "1.5.3",
55
"license": "GPL-3.0",
66
"author": "EyeSeeTea team",
77
"homepage": ".",

src/domain/common/entities/ToggleMultiple.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,8 @@ export function isToggleMultipleDeDisabled(
122122
(toggle): toggle is OrgUnitToggle => toggle.type === "orgUnit" && toggle.dataElement.code === dataElement.code
123123
);
124124

125+
if (orgUnitToggles.length === 0) return false;
126+
125127
const isToggleDisabledForOrgUnit = (toggle: OrgUnitToggle) => {
126128
const { orgUnitCodes, dataElement: toggleDe } = toggle;
127129
if (!toggleDe.disabled) return false;

0 commit comments

Comments
 (0)