Skip to content

Commit 55da829

Browse files
Refactored Category Code
Refactored code to avoid Build tool thinking there might be an off chance that a filter would be an Undefined Object. Both removed indexing and filtered out any undefined or null values before the loop begins.
1 parent c8a9b4d commit 55da829

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/xmltv.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,12 @@ export function buildProgramsXml(data: GridApiResponse): string {
7878
xml += ` <desc>${escapeXml(event.program.shortDesc)}</desc>\n`;
7979
}
8080

81-
if (event.filter && event.filter.length > 0) {
82-
for (let i = 0; i < event.filter.length; i++) {
83-
const category = event.filter(i).match(/^(filter)-(.*?)$/);
81+
const filters = event.filter?.filter(item => item);
82+
if (filters?.length) {
83+
for (const filter of filters) {
84+
const category = filter.match(/^(filter)-(.*?)$/);
8485
if (category) {
85-
xml += ` <category lang="en">${category[2]}</category>`;
86+
xml += ' <category lang="en">${category[2]}</category>`;
8687
}
8788
}
8889
}

0 commit comments

Comments
 (0)