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

Themes: reintroduce check for defaultBackgroundID: make sure it _does_ exist as global layer

Browse source
This commit is contained in:
Pieter Vander Vennet 2025-04-17 02:33:49 +02:00
parent fffc959c0d
commit 6a1c5fec97
2 changed files with 5 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/Models/ThemeConfig/Conversion/ValidateTheme.ts
View file

@ -149,16 +149,14 @@ export class ValidateTheme extends DesugaringStep<ThemeConfigJson> {
}
if (json.defaultBackgroundId) {
/*
TODO re-enable this check
const backgroundId = json.defaultBackgroundId
const isCategory =
backgroundId === "photo" || backgroundId === "map" || backgroundId === "osmbasedmap"
if (!isCategory && !ValidateTheme._availableLayers.has(backgroundId)) {
const options = Array.from(ValidateTheme._availableLayers)
const nearby = Utils.sortedByLevenshteinDistance(backgroundId, options, (t) => t)
const knownIds = Array.from(AvailableRasterLayers.allAvailableGlobalLayers).map(l => l.properties.id)
const available = new Set(knownIds)
if (!isCategory && !available.has(backgroundId)) {
const nearby = Utils.sortedByLevenshteinDistance(backgroundId, knownIds, (t) => t)
context
.enter("defaultBackgroundId")
.err(
@ -166,7 +164,7 @@ export class ValidateTheme extends DesugaringStep<ThemeConfigJson> {
.slice(0, 5)
.join(", ")}`,
)
}*/
}
}
for (let i = 0; i < theme.layers.length; i++) {