abs3nt
/
gospt
1
2
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases 20 Wiki Activity

fix loadmoreitems being called a billion times
ci/woodpecker/push/woodpecker Pipeline was successful Details
ci/woodpecker/tag/woodpecker Pipeline was successful Details

This commit is contained in:
abs3nt 2023-04-14 19:45:59 -07:00
parent ec60177128
commit c124ec4eb9
Signed by: abs3nt
GPG Key ID: FDC6662313FA9386
2 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/tui/loader.go
View File

@ -8,10 +8,13 @@ import (
)
func (m *mainModel) LoadMoreItems() {
loading = true
defer func() {
loading = false
}()
switch m.mode {
case "artist":
albums, err := m.commands.ArtistAlbums(m.ctx, m.artist.ID, (page + 1))
page++
if err != nil {
return
}
@ -31,7 +34,6 @@ func (m *mainModel) LoadMoreItems() {
return
case "artists":
artists, err := m.commands.UserArtists(m.ctx, (page + 1))
page++
if err != nil {
return
}
@ -51,7 +53,6 @@ func (m *mainModel) LoadMoreItems() {
return
case "album":
tracks, err := m.commands.AlbumTracks(m.ctx, m.album.ID, (page + 1))
page++
if err != nil {
return
}
@ -72,7 +73,6 @@ func (m *mainModel) LoadMoreItems() {
return
case "albums":
albums, err := m.commands.UserAlbums(m.ctx, (page + 1))
page++
if err != nil {
return
}
@ -92,7 +92,6 @@ func (m *mainModel) LoadMoreItems() {
return
case "main":
playlists, err := m.commands.Playlists(m.ctx, (page + 1))
page++
if err != nil {
return
}
@ -111,7 +110,6 @@ func (m *mainModel) LoadMoreItems() {
return
case "playlist":
tracks, err := m.commands.PlaylistTracks(m.ctx, m.playlist.ID, (page + 1))
page++
if err != nil {
return
}
@ -132,7 +130,6 @@ func (m *mainModel) LoadMoreItems() {
return
case "tracks":
tracks, err := m.commands.TrackList(m.ctx, (page + 1))
page++
if err != nil {
return
}

3
src/tui/main.go
View File

@ -25,6 +25,7 @@ var (
playbackContext string
main_updates chan *mainModel
page = 1
loading = false
)
type Mode string
@ -512,7 +513,7 @@ func (m *mainModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
default:
}
// Call for more items if needed
if m.list.Paginator.Page == m.list.Paginator.TotalPages-1 && m.list.Cursor() == 0 {
if m.list.Paginator.Page == m.list.Paginator.TotalPages-1 && m.list.Cursor() == 0 && !loading {
// if last request was still full request more
if len(m.list.Items())%50 == 0 {
go m.LoadMoreItems()