Skip to content

Commit

Permalink
Fix Excel mapping
Browse files Browse the repository at this point in the history
  • Loading branch information
tombrunet committed Sep 25, 2023
1 parent 816cd31 commit d44fb09
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions common/module/src/report/ACReporterXLSX.ts
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,7 @@ export class ACReporterXLSX implements IReporter {
}
for (const ruleId in policyInfo) {
policyInfo[ruleId].tkLevels = dropDupes(policyInfo[ruleId].tkLevels);
policyInfo[ruleId].cps = dropDupes(policyInfo[ruleId].cps);
policyInfo[ruleId].tkLevels.sort();
policyInfo[ruleId].cps.sort();
}

// const buffer: any = await workbook.xlsx.writeBuffer();
Expand Down Expand Up @@ -706,14 +704,16 @@ export class ACReporterXLSX implements IReporter {
})
let wcagLevels = dropDupes(cps.map(cp => cp.wcagLevel));
wcagLevels.sort();
let cpStrs = dropDupes(cps.map(cp => `${cp.num} ${cp.name}`));
cpStrs.sort();
let row = [
storedScan.pageTitle,
storedScan.engineReport.summary.URL,
storedScan.label,
this.stringHash(item.ruleId + item.path.dom),
`${valueMap[item.value[0]][item.value[1]]}${item.ignored ? ` (Archived)` : ``}`,
polInfo.tkLevels.join(", "),
cps.join(", "),
cpStrs.join("; "),
wcagLevels.join(", "),
item.ruleId,
item.message.substring(0, 32767), //max ength for MS Excel 32767 characters
Expand Down

0 comments on commit d44fb09

Please sign in to comment.