Osmwithspace/MapComplete
1
0
Fork 0
forked from MapComplete/MapComplete
Code Pull requests Activity

Fix: add better error message if "requiredLayers" didn't set an ID

Browse source
This commit is contained in:
Pieter Vander Vennet 2025-05-06 13:52:29 +02:00
parent 3c7441d93b
commit 01d924374e
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/Models/ThemeConfig/DependencyCalculator.ts
View file

@ -15,8 +15,12 @@ export default class DependencyCalculator {
} }
const deps: { id: string; minZoom?: number; neededBy: string }[] = [] const deps: { id: string; minZoom?: number; neededBy: string }[] = []
if (tr.requiredLayers) { for (const req of tr.requiredLayers ?? []) {
deps.push(...tr.requiredLayers.map((req) => ({ ...req, neededBy: tr.id }))) const dependency = { ...req, neededBy: tr.id }
if (!dependency.id) {
throw `The tagRendering ${tr.id} specifies a \`requiredBy\`, but this \`requiredBy\` does _not_ have an \`id\`-field`
}
deps.push(dependency)
} }
// All translated snippets // All translated snippets