[GH-ISSUE #60] Commit 0d9d9da problem #50

Closed
opened 2026-02-28 14:39:15 +03:00 by kerem · 13 comments
Owner

Originally created by @lesar8 on GitHub (Oct 26, 2024).
Original GitHub issue: https://github.com/NTifyApp/NTify/issues/60

I just tested the latest build 0d9d9da, which implemented the "lazy loading for playlists". Unfortunately this build had problem, when scrolling, the playlist try to fetch new songs but it's actually duplicate the first 100 songs (idk the correct number) from the list. The more I scroll, the more duplicate songs it get

P/s: I would rather create an optional setting for this, because it takes very long time to scroll to the whole list if a playlist has a lot of songs. Seriously, it tooks me 15 minutes to fully fetch my Playlist.. I like the previous behavior better when all the song get fetched at the same times.

Originally created by @lesar8 on GitHub (Oct 26, 2024). Original GitHub issue: https://github.com/NTifyApp/NTify/issues/60 I just tested the latest build 0d9d9da, which implemented the "lazy loading for playlists". Unfortunately this build had problem, when scrolling, the playlist try to fetch new songs but it's actually duplicate the first 100 songs (idk the correct number) from the list. The more I scroll, the more duplicate songs it get P/s: I would rather create an optional setting for this, because it takes very long time to scroll to the whole list if a playlist has a lot of songs. Seriously, it tooks me 15 minutes to fully fetch my Playlist.. I like the previous behavior better when all the song get fetched at the same times.
kerem closed this issue 2026-02-28 14:39:15 +03:00
Author
Owner

@werwolf2303 commented on GitHub (Oct 27, 2024):

Ok I will add this functionality later. I'm playing Mafia 1. Sry

<!-- gh-comment-id:2440025675 --> @werwolf2303 commented on GitHub (Oct 27, 2024): Ok I will add this functionality later. I'm playing Mafia 1. Sry
Author
Owner

@werwolf2303 commented on GitHub (Oct 28, 2024):

69c2c8e

<!-- gh-comment-id:2442060088 --> @werwolf2303 commented on GitHub (Oct 28, 2024): [69c2c8e](https://github.com/SpotifyXP/SpotifyXP/commit/69c2c8e3a0f39c43b8e896a2b5fdb74621791dd7)
Author
Owner

@lesar8 commented on GitHub (Oct 28, 2024):

The duplicate bug was fixed, thank you. But enable the "load add tracks" option does not work when access Playlist from the search results. I still need to scrolling down to fetch songs.

Also, this "lazy loading" feature gives a lot of error messages when scrolling down:
[java.lang.RuntimeException: com.spotifyxp.deps.se.michaelthelin.spotify.exceptions.detailed.TooManyRequestsException: API rate limit exceeded]

<!-- gh-comment-id:2442564057 --> @lesar8 commented on GitHub (Oct 28, 2024): The duplicate bug was fixed, thank you. But enable the "load add tracks" option does not work when access Playlist from the search results. I still need to scrolling down to fetch songs. Also, this "lazy loading" feature gives a lot of error messages when scrolling down: `[java.lang.RuntimeException: com.spotifyxp.deps.se.michaelthelin.spotify.exceptions.detailed.TooManyRequestsException: API rate limit exceeded]`
Author
Owner

@werwolf2303 commented on GitHub (Oct 30, 2024):

I fixed the issue with the lazy loading but I can't do anything about the TooManyRequestsException thing because Spotify rate limits SpotifyXP, so just wait a few seconds and it should work again

<!-- gh-comment-id:2447950410 --> @werwolf2303 commented on GitHub (Oct 30, 2024): I fixed the issue with the lazy loading but I can't do anything about the TooManyRequestsException thing because Spotify rate limits SpotifyXP, so just wait a few seconds and it should work again
Author
Owner

@lesar8 commented on GitHub (Oct 30, 2024):

Ok, i tried the latest build and found a serious bug here, change any of the settings from UI does not save to config.json.

<!-- gh-comment-id:2448466917 --> @lesar8 commented on GitHub (Oct 30, 2024): Ok, i tried the latest build and found a serious bug here, change any of the settings from UI does not save to config.json.
Author
Owner

@werwolf2303 commented on GitHub (Oct 31, 2024):

I changed the config so that SpotifyXP needs a restart before the settings apply

<!-- gh-comment-id:2449759408 --> @werwolf2303 commented on GitHub (Oct 31, 2024): I changed the config so that SpotifyXP needs a restart before the settings apply
Author
Owner

@lesar8 commented on GitHub (Nov 1, 2024):

I changed the config so that SpotifyXP needs a restart before the settings apply

Problem is the settings did not save after I exit the app and restart, tried multiple times but no luck. Does it work for you?

<!-- gh-comment-id:2451041516 --> @lesar8 commented on GitHub (Nov 1, 2024): > I changed the config so that SpotifyXP needs a restart before the settings apply Problem is the settings did not save after I exit the app and restart, tried multiple times but no luck. Does it work for you?
Author
Owner

@werwolf2303 commented on GitHub (Nov 12, 2024):

Sorry for the long delay. I will fix it later today

<!-- gh-comment-id:2470087799 --> @werwolf2303 commented on GitHub (Nov 12, 2024): Sorry for the long delay. I will fix it later today
Author
Owner

@werwolf2303 commented on GitHub (Nov 12, 2024):

Should be fixed in the newest nightly build

<!-- gh-comment-id:2471533509 --> @werwolf2303 commented on GitHub (Nov 12, 2024): Should be fixed in the newest nightly build
Author
Owner

@lesar8 commented on GitHub (Nov 12, 2024):

Thanks, it's fixed for me.

I think i found another bug with the "lazy loading" feature. Here is steps to reproduce this bug:

  1. Uncheck "Load all tracks" option, then Restart.
  2. Select "Search Playlists" filter and search for the keyword "rateyourmusic", then access "Top 1,000 Highest Rated Songs of All Time" (any large playlists will do the job too). Scroll down a couple of times for the songs to be fetch.
  3. Select "Search Albums" filter and search for whatever you want, then access any search results. Just scroll down and you will see those song from the "Top 1,000" playlist got merged with songs from the Album !!

bug

<!-- gh-comment-id:2471788483 --> @lesar8 commented on GitHub (Nov 12, 2024): Thanks, it's fixed for me. I think i found another bug with the "lazy loading" feature. Here is steps to reproduce this bug: 1. Uncheck "Load all tracks" option, then Restart. 2. Select "Search Playlists" filter and search for the keyword "rateyourmusic", then access "Top 1,000 Highest Rated Songs of All Time" (any large playlists will do the job too). Scroll down a couple of times for the songs to be fetch. 3. Select "Search Albums" filter and search for whatever you want, then access any search results. Just scroll down and you will see those song from the "Top 1,000" playlist got merged with songs from the Album !! ![bug](https://github.com/user-attachments/assets/9ea018c7-9fd7-4542-bf27-442e0175d205)
Author
Owner

@werwolf2303 commented on GitHub (Nov 13, 2024):

I will look into that. Thanks for reporting

<!-- gh-comment-id:2473630189 --> @werwolf2303 commented on GitHub (Nov 13, 2024): I will look into that. Thanks for reporting
Author
Owner

@lesar8 commented on GitHub (Nov 13, 2024):

Thank you, it is fixed now.

There is another bug that that I run into. Here is steps to reproduce this bug:

  1. Uncheck "Load all tracks" option, then Restart.
  2. Select "Search Playlists" filter and search for the keyword "rateyourmusic", then access "Top 1,000 Highest Rated Songs of All Time" (again..). Scroll down 9 or 10 times and the bug got triggered, songs stop fetching and returns a lot of errors [java.lang.RuntimeException: java.lang.NullPointerException]. The more you scroll, the more errors it creates. This always happens to me whenever i test it.

bug

<!-- gh-comment-id:2474932501 --> @lesar8 commented on GitHub (Nov 13, 2024): Thank you, it is fixed now. There is another bug that that I run into. Here is steps to reproduce this bug: 1. Uncheck "Load all tracks" option, then Restart. 2. Select "Search Playlists" filter and search for the keyword "rateyourmusic", then access "Top 1,000 Highest Rated Songs of All Time" (again..). Scroll down 9 or 10 times and the bug got triggered, songs stop fetching and returns a lot of errors **[java.lang.RuntimeException: java.lang.NullPointerException]**. The more you scroll, the more errors it creates. This always happens to me whenever i test it. ![bug](https://github.com/user-attachments/assets/844abc6d-3354-43a4-a10d-19a6305d2513)
Author
Owner

@werwolf2303 commented on GitHub (Nov 13, 2024):

This also happened on my side. I think it's the Spotify servers blocking the requests due to the excessive API calls. I have to think about caching or cool downs

<!-- gh-comment-id:2475034044 --> @werwolf2303 commented on GitHub (Nov 13, 2024): This also happened on my side. I think it's the Spotify servers blocking the requests due to the excessive API calls. I have to think about caching or cool downs
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/NTify#50
No description provided.