fix: normalize character status filtering logic to handle unknown and null values
This commit is contained in:
@@ -167,9 +167,12 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Status filter
|
// Status filter
|
||||||
if (characterFilters.status.length > 0 && !characterFilters.status.includes(char.status)) {
|
if (characterFilters.status.length > 0) {
|
||||||
|
const normalizedStatus = normalizeStatus(char.status);
|
||||||
|
if (!characterFilters.status.includes(normalizedStatus)) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// Height filter
|
// Height filter
|
||||||
if (characterFilters.hasHeight && (char.height === null || char.height === undefined)) {
|
if (characterFilters.hasHeight && (char.height === null || char.height === undefined)) {
|
||||||
@@ -357,6 +360,35 @@
|
|||||||
generateNewCharacter();
|
generateNewCharacter();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function normalizeStatus(status: unknown): string {
|
||||||
|
if (status == null) {
|
||||||
|
return 'Unknown';
|
||||||
|
}
|
||||||
|
|
||||||
|
if (typeof status !== 'string') {
|
||||||
|
return String(status);
|
||||||
|
}
|
||||||
|
|
||||||
|
const value = status.trim();
|
||||||
|
if (value === '') {
|
||||||
|
return 'Unknown';
|
||||||
|
}
|
||||||
|
|
||||||
|
switch (value.toLowerCase()) {
|
||||||
|
case 'alive':
|
||||||
|
return 'Alive';
|
||||||
|
case 'dead':
|
||||||
|
case 'deceased':
|
||||||
|
return 'Dead';
|
||||||
|
case 'unknown':
|
||||||
|
case 'inconnu':
|
||||||
|
case '-':
|
||||||
|
return 'Unknown';
|
||||||
|
default:
|
||||||
|
return value;
|
||||||
|
}
|
||||||
|
}
|
||||||
</script>
|
</script>
|
||||||
|
|
||||||
<svelte:head>
|
<svelte:head>
|
||||||
|
|||||||
Reference in New Issue
Block a user