"spotify_player get key playback" sometimes returns outdated information

[DOD-101]

"spotify_player get key playback" sometimes returns outdated information

[aome510]

Making sure to delete the Arch package and then I ran it form ~/.cargo/bin.

cargo build doesn't produce the binary in ~/.cargo/bin though, it should be inside the repo's target/debug

[aome510]

I think the issue may have to do with the fact that the application doesn't call the spotify api each time

This is intended to speed up the command's latency. The player should be able to infer its playback state most of the time. However, if you control the player's playback using external tool, e.g from other Spotify client, the state may not be up-to-date. I'm well aware of this drawback and will be working on a patch

[DOD-101]

Would it be possible to - perhaps simply in the interim - implement a flag / setting that tells it to always call the API?

[aome510]

yes, but it feels more like a hack than a proper fix

[aome510]

@DOD-101 should be fixed with #418. Can you try the latest main?

[DOD-101]

Sure thing.

[DOD-101]

Alrighty, so after a little bit of testing, I have found the following:

1. If the song changes by itself, it now works as intended.
2. If however one changes the song manually in any way it will take 1.5 to 2 min to update.

So it's kinda fixed, but at the same time it probably shouldn't take that long to "recognize" the change.

[aome510]

How did you test the change? WDYM by "one changes the song manually in any way"?

[DOD-101]

How did you test the change?

I cloned the git repo and then cargo build it with only the daemon feature flag added.
Making sure to delete the Arch package and then I ran it form ~/.cargo/bin.
Then I followed the steps outlined in the original post.

WDYM by "one changes the song manually in any way"?

I meant that if you either skip to the next song (via the shortcut) or by simply selecting some other song in the tui.
Should have been more clear here: This only applies to changing the song in another client. If one changes the song via the "spotify_player playback next" command it will update almost instantly, as if it just finished playing.

[DOD-101]

I'm sorry.
I accidentally ran cargo install instead of cargo build (not familiar with rust). My bad.

It works like a charm now. Thanks. Keep up the great work.

[aome510]

Great! Thanks for confirming