refactor: use number parseint

Author: kim

cypress/support/server.ts

@@ -593,8 +593,8 @@ export const mockGetMemberStorageFiles = (
     ({ url, reply }) => {
       const params = new URL(url).searchParams;
 
-      const page = window.parseInt(params.get('page') ?? '1');
-      const pageSize = window.parseInt(params.get('pageSize') ?? '10', 10);
+      const page = Number.parseInt(params.get('page') ?? '1');
+      const pageSize = Number.parseInt(params.get('pageSize') ?? '10', 10);
 
       const result = files.slice((page - 1) * pageSize, page * pageSize);
 
@@ -792,8 +792,8 @@ export const mockGetAccessibleItems = (items: ItemForTest[]): void => {
     ({ url, reply }) => {
       const params = new URL(url).searchParams;
 
-      const page = parseInt(params.get('page') ?? '1', 10);
-      const pageSize = parseInt(params.get('pageSize') ?? '10', 10);
+      const page = Number.parseInt(params.get('page') ?? '1', 10);
+      const pageSize = Number.parseInt(params.get('pageSize') ?? '10', 10);
 
       // warning: we don't check memberships
       const root = items.filter(isRootItem);
@@ -824,8 +824,8 @@ export const mockGetOwnRecycledItemData = (
 
       const params = new URL(url).searchParams;
 
-      const page = parseInt(params.get('page') ?? '1', 10);
-      const pageSize = parseInt(params.get('pageSize') ?? '10', 10);
+      const page = Number.parseInt(params.get('page') ?? '1', 10);
+      const pageSize = Number.parseInt(params.get('pageSize') ?? '10', 10);
 
       const result = recycledItemData.slice(
         (page - 1) * pageSize,

src/modules/analytics/charts/ActionsByTimeOfDayChart.tsx

@@ -65,7 +65,7 @@ const ActionsByTimeOfDayChart = ({
             data={Object.values(
               data as { [key: string]: { hour: string } },
             ).toSorted((a, b) =>
-              parseInt(a.hour) > parseInt(b.hour) ? 1 : -1,
+              Number.parseInt(a.hour) > Number.parseInt(b.hour) ? 1 : -1,
             )}
           >
             <CartesianGrid strokeDasharray="2" />

src/ui/Avatar/stringToColor.ts

@@ -7,9 +7,9 @@ import { hslToRgb } from '@mui/material';
  */
 export const getColorFromId = (id: string): string => {
   const rawHue = id.slice(0, 2);
-  const hue = (parseInt(rawHue, 16) / 256) * 360;
+  const hue = (Number.parseInt(rawHue, 16) / 256) * 360;
   const rawSaturation = id[2];
-  const saturation = 65 + (16 - parseInt(rawSaturation, 16));
+  const saturation = 65 + (16 - Number.parseInt(rawSaturation, 16));
   const newRgb = hslToRgb(`hsl(${hue}, ${saturation}, ${60})`);
   return newRgb;
 };

src/ui/items/H5PItem.tsx

@@ -86,7 +86,7 @@ const H5PItem = ({
           return;
         }
         // height should be int
-        const newHeight = parseInt(event.data.height);
+        const newHeight = Number.parseInt(event.data.height);
         iframeRef.current.height = newHeight.toString();
       };
 

vite.config.ts

@@ -32,7 +32,7 @@ const config = ({ mode }: { mode: string }): UserConfig => {
   }
 
   // compute the port to use
-  const PORT = parseInt(VITE_PORT ?? '3114', 10);
+  const PORT = Number.parseInt(VITE_PORT ?? '3114', 10);
   // compute whether we should open the browser
   // this defines if we should automatically open the browser
   const shouldOpen = BROWSER && BROWSER !== 'none';