Alexander/homepage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: nutrition coverage double-counting excluded ingredients

Browse Source 
Excluded (manually disregarded) ingredients were incrementing the total
count twice — once in the loop body and again in the exclusion check —
deflating the displayed coverage percentage.
This commit is contained in:
Alexander Bocken
2026-04-03 09:00:24 +02:00
parent 88f3909634
commit d462a6ae1b
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/lib/js/nutrition.svelte.ts
View File

@@ -274,7 +274,7 @@ export function createNutritionCalculator(
total++; total++;
const m = mappingIndex.get(`${ing.sectionIndex}-${ing.ingredientIndex}`); const m = mappingIndex.get(`${ing.sectionIndex}-${ing.ingredientIndex}`);
// Manually excluded ingredients count as covered // Manually excluded ingredients count as covered
if (m?.excluded) { total++; mapped++; continue; } if (m?.excluded) { mapped++; continue; }
if (m && m.matchMethod !== 'none') mapped++; if (m && m.matchMethod !== 'none') mapped++;
} }
return total > 0 ? mapped / total : 1; return total > 0 ? mapped / total : 1;