Sancaros/phantasmal-world
1
0
Fork 0
mirror of https://github.com/DaanVandenBosch/phantasmal-world.git synced 2025-04-05 07:18:29 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed bug in QuestEditorUiPersister config sanitization code. When removing old components results in empty containers, these containers are now also removed.

Browse Source
This commit is contained in:
Daan Vanden Bosch 2020-07-14 22:08:36 +02:00
parent 767397d26d
commit d0d6b7ff1a
1 changed files with 24 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
src/quest_editor/persistence/QuestEditorUiPersister.ts
View File

@ -65,33 +65,22 @@ export class QuestEditorUiPersister extends Persister {
return undefined; return undefined;
} }
switch (config.type) { if (config.type === "component") {
case "component": // Remove corrupted components.
{ if (!("componentName" in config)) {
// Remove corrupted components. return undefined;
if (!("componentName" in config)) { }
return undefined;
}
const component = components.get(config.componentName); const component = components.get(config.componentName);
// Remove deprecated components. // Remove deprecated components.
if (!component) { if (!component) {
return undefined; return undefined;
} }
found.add(config.componentName); found.add(config.componentName);
config.id = component.id; config.id = component.id;
config.title = component.title; config.title = component.title;
}
break;
case "stack":
// Remove empty stacks.
if (config.content == undefined || config.content.length === 0) {
return undefined;
}
break;
} }
// Sanitize child items. // Sanitize child items.
@ -101,6 +90,17 @@ export class QuestEditorUiPersister extends Persister {
.filter(item => item) as ItemConfigType[]; .filter(item => item) as ItemConfigType[];
} }
// Remove empty containers.
switch (config.type) {
case "row":
case "column":
case "stack":
if (config.content == undefined || config.content.length === 0) {
return undefined;
}
break;
}
// Remove corrupted activeItemIndex properties. // Remove corrupted activeItemIndex properties.
const cfg = config as any; const cfg = config as any;