Fixed new history video sort order according to sort filter

Author: anurag2787

src/renderer/store/modules/history.js

@@ -4,7 +4,7 @@ import { HISTORY_SORT_BY_VALUES } from '../../../constants'
 
 const state = {
   historyCacheSorted: [],
-  useAscendingOrder: HISTORY_SORT_BY_VALUES.DateAddedNewest,
+  sortOrder: HISTORY_SORT_BY_VALUES.DateAddedNewest,
 
   // Vuex doesn't support Maps, so we have to use an object here instead
   // TODO: switch to a Map during the Pinia migration
@@ -21,7 +21,7 @@ const getters = {
   },
 
   getHistorySortOrder(state) {
-    return state.useAscendingOrder
+    return state.sortOrder
   }
 }
 
@@ -116,7 +116,7 @@ const mutations = {
   },
 
   setHistorySortOrder(state, order) {
-    state.useAscendingOrder = order
+    state.sortOrder = order
   },
 
   setHistoryCacheById(state, historyCacheById) {
@@ -134,7 +134,19 @@ const mutations = {
       state.historyCacheSorted.splice(i, 1)
     }
 
-    state.historyCacheSorted.unshift(record)
+    // Correct position to add according to the sort order
+    const insertIndex = state.historyCacheSorted.findIndex((currentRecord) =>
+      state.sortOrder === HISTORY_SORT_BY_VALUES.DateAddedOldest
+        ? record.timeWatched < currentRecord.timeWatched
+        : record.timeWatched > currentRecord.timeWatched
+    )
+
+    if (insertIndex === -1) {
+      state.historyCacheSorted.push(record)
+    } else {
+      state.historyCacheSorted.splice(insertIndex, 0, record)
+    }
+
     vueSet(state.historyCacheById, record.videoId, record)
   },
 
@@ -177,7 +189,7 @@ const mutations = {
 
   sortHistory(state) {
     state.historyCacheSorted.sort((a, b) =>
-      state.useAscendingOrder === HISTORY_SORT_BY_VALUES.DateAddedOldest
+      state.sortOrder === HISTORY_SORT_BY_VALUES.DateAddedOldest
         ? a.timeWatched - b.timeWatched
         : b.timeWatched - a.timeWatched
     )