fix: useEffect order

lovegaoshi · Dec 11, 2023 · 886c7a4 · 886c7a4
1 parent c929fcb
commit 886c7a4
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/src/components/playlist/usePlaylistRN.ts b/src/components/playlist/usePlaylistRN.ts
@@ -124,6 +124,19 @@ export default (playlist: NoxMedia.Playlist) => {
     }
   };
 
+  /**
+   * playlistShouldReRender is a global state that indicates playlist should be
+   * refreshed. right now its only called when the playlist is updated in updatePlaylist.
+   * this should in turn clear all searching, checking and filtering.
+   */
+  useEffect(() => {
+    resetSelected();
+    setChecking(false);
+    // setSearching(false);
+    setRows(playlist.songList);
+    setCachedSongs(Array.from(noxCache.noxMediaCache.cache.keys()));
+  }, [playlist, playlistShouldReRender]);
+
   useEffect(() => {
     if (
       playerSetting.autoRSSUpdate &&
@@ -145,19 +158,6 @@ export default (playlist: NoxMedia.Playlist) => {
     }
   }, [playlist]);
 
-  /**
-   * playlistShouldReRender is a global state that indicates playlist should be
-   * refreshed. right now its only called when the playlist is updated in updatePlaylist.
-   * this should in turn clear all searching, checking and filtering.
-   */
-  useEffect(() => {
-    resetSelected();
-    setChecking(false);
-    setSearching(false);
-    setRows(playlist.songList);
-    setCachedSongs(Array.from(noxCache.noxMediaCache.cache.keys()));
-  }, [playlist, playlistShouldReRender]);
-
   useEffect(() => handleSearch(debouncedSearchText), [debouncedSearchText]);
 
   useEffect(() => {