fix(timeRange): shiftArray should not mutate its argument (#2420)

plouc · Nov 14, 2023 · e4a343f · e4a343f · vercel · Nov 14, 2023
1 parent 1620db2
commit e4a343f
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/packages/calendar/src/compute/timeRange.ts b/packages/calendar/src/compute/timeRange.ts
@@ -184,12 +184,8 @@ const getTimeInterval = (firstWeekday: Weekday) => {
 function shiftArray<T>(arr: T[], x: number): T[] {
     if (!arr.length || !x) return arr
 
-    for (let i = 0; i < x; i++) {
-        const shifted = arr.shift() as T
-        arr.push(shifted)
-    }
-
-    return arr
+    x = x % arr.length
+    return arr.slice(x, arr.length).concat(arr.slice(0, x))
 }
 
 function computeGrid({