Merge pull request #82 from center-for-threat-informed-defense/AF-126…

…_location_validation AF-126 Location Validation
center-for-threat-informed-defense · Jul 20, 2023 · 42bca91 · 42bca91
2 parents 1230825 + a895803
commit 42bca91
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/src/attack_flow_builder/src/assets/builder.config.validator.ts b/src/attack_flow_builder/src/assets/builder.config.validator.ts
@@ -169,6 +169,22 @@ class AttackFlowValidator extends DiagramValidator {
                     this.addError(id, "Invalid email address.")
                 }
                 break;
+            case "location": // Additional validation for location object
+                const region = node.props.value.get("region");
+                const country = node.props.value.get("country");
+                const latitude = node.props.value.get("latitude");
+                const longitude = node.props.value.get("longitude");
+
+                // Verify one of the required properties is set
+                if(!region?.isDefined() && !country?.isDefined() && !(latitude?.isDefined() || longitude?.isDefined())) {
+                    this.addError(id, "Location requires one of the following properties: Region, Country, Latitude+Longitude.");
+                }
+
+                // Latitude + Longitude check
+                if(latitude?.isDefined() !== longitude?.isDefined()) {
+                    this.addError(id, "Latitude and Longitude must be supplied together.");
+                }
+                break;
         }
     }