Merge pull request #177 from CodeForPhilly/feat/zbl-buildissue20251218

ZacharyLeahan · web-flow · commit 29e2d69113ce · 2025-12-18T19:54:09.000-05:00
Improve summaryData computation by adding null checks for family and …
diff --git a/src/components/GardenPlanner.vue b/src/components/GardenPlanner.vue
@@ -285,7 +285,9 @@ const summaryData = computed(() => {
       familyMap.set(plantFamily, { plants: new Map() });
     }
 
-    const family = familyMap.get(plantFamily)!;
+    const family = familyMap.get(plantFamily);
+    if (!family) continue;
+    
     if (!family.plants.has(placed.plantId)) {
       family.plants.set(placed.plantId, {
         plantId: placed.plantId,
@@ -296,7 +298,9 @@ const summaryData = computed(() => {
       });
     }
 
-    const plantEntry = family.plants.get(placed.plantId)!;
+    const plantEntry = family.plants.get(placed.plantId);
+    if (!plantEntry) continue;
+    
     plantEntry.count++;
     plantEntry.coordinates.push(centerCoord);
   }

Original file line number	Diff line number	Diff line change
`@@ -285,7 +285,9 @@ const summaryData = computed(() => {`
`285`	`285`	`familyMap.set(plantFamily, { plants: new Map() });`
`286`	`286`	`}`
`287`	`287`
`288`		`- const family = familyMap.get(plantFamily)!;`
	`288`	`+ const family = familyMap.get(plantFamily);`
	`289`	`+ if (!family) continue;`
	`290`	`+`
`289`	`291`	`if (!family.plants.has(placed.plantId)) {`
`290`	`292`	`family.plants.set(placed.plantId, {`
`291`	`293`	`plantId: placed.plantId,`
`@@ -296,7 +298,9 @@ const summaryData = computed(() => {`
`296`	`298`	`});`
`297`	`299`	`}`
`298`	`300`
`299`		`- const plantEntry = family.plants.get(placed.plantId)!;`
	`301`	`+ const plantEntry = family.plants.get(placed.plantId);`
	`302`	`+ if (!plantEntry) continue;`
	`303`	`+`
`300`	`304`	`plantEntry.count++;`
`301`	`305`	`plantEntry.coordinates.push(centerCoord);`
`302`	`306`	`}`