Skip to content

Commit 80ba976

Browse files
committed
feat: add additional validation to all lat/long data entry in CreateEditSites.tsx
1 parent 071aea1 commit 80ba976

File tree

1 file changed

+34
-2
lines changed

1 file changed

+34
-2
lines changed

src/admin/CreateEditSite.tsx

Lines changed: 34 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,11 +164,21 @@ export default function CreateEditSite({ mode }: CreateEditSiteProps) {
164164
alert('Name is required');
165165
return false;
166166
}
167-
if (longitude === '' || isNaN(Number(longitude))) {
167+
if (
168+
longitude === '' ||
169+
isNaN(Number(longitude)) ||
170+
Number(longitude) < -180 ||
171+
Number(longitude) > 180
172+
) {
168173
alert('Valid Longitude is required');
169174
return false;
170175
}
171-
if (latitude === '' || isNaN(Number(latitude))) {
176+
if (
177+
latitude === '' ||
178+
isNaN(Number(latitude)) ||
179+
Number(latitude) < -90 ||
180+
Number(latitude) > 90
181+
) {
172182
alert('Valid Latitude is required');
173183
return false;
174184
}
@@ -184,6 +194,28 @@ export default function CreateEditSite({ mode }: CreateEditSiteProps) {
184194
alert('At least one Cell ID is required');
185195
return false;
186196
}
197+
if (boundaryEnabled) {
198+
for (const point of boundaryPoints) {
199+
if (
200+
point.lat === '' ||
201+
isNaN(Number(point.lat)) ||
202+
Number(point.lat) < -90 ||
203+
Number(point.lat) > 90
204+
) {
205+
alert('Valid Latitude for Boundary Point is required');
206+
return false;
207+
}
208+
if (
209+
point.lng === '' ||
210+
isNaN(Number(point.lng)) ||
211+
Number(point.lng) < -180 ||
212+
Number(point.lng) > 180
213+
) {
214+
alert('Valid Longitude for Boundary Point is required');
215+
return false;
216+
}
217+
}
218+
}
187219
return true;
188220
};
189221

0 commit comments

Comments
 (0)