fix: use Set with .has() instead of array .includes() for acceptsSortSite

mnashawati · mnashawati · commit 3a3753a18c8e · 2026-03-31T10:54:12.000+01:00
Addresses SonarCloud warning to use Set for existence checks.
diff --git a/src/helpers/acceptsSortSite.ts b/src/helpers/acceptsSortSite.ts
@@ -4,6 +4,6 @@ export default function (headers: Record<string, any>): boolean {
         return false;
     }
 
-    const acceptToArray = accept.split(',').map(a=>a.trim());
-    return acceptToArray.includes('application/x-ms-application') && acceptToArray.includes('application/x-ms-xbap');
+    const acceptToArray = new Set(accept.split(',').map(a=>a.trim()));
+    return acceptToArray.has('application/x-ms-application') && acceptToArray.has('application/x-ms-xbap');
 }

Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,6 @@ export default function (headers: Record<string, any>): boolean {`
`4`	`4`	`return false;`
`5`	`5`	`}`
`6`	`6`
`7`		`- const acceptToArray = accept.split(',').map(a=>a.trim());`
`8`		`- return acceptToArray.includes('application/x-ms-application') && acceptToArray.includes('application/x-ms-xbap');`
	`7`	`+ const acceptToArray = new Set(accept.split(',').map(a=>a.trim()));`
	`8`	`+ return acceptToArray.has('application/x-ms-application') && acceptToArray.has('application/x-ms-xbap');`
`9`	`9`	`}`