Bug 412334

Summary: Now Playing view no longer shows or updates metadata or lyrics until you manually re-play the current song
Product: [Applications] Elisa Reporter: Nate Graham <nate>
Component: generalAssignee: Matthieu Gallien <matthieu_gallien>
Status: RESOLVED FIXED    
Severity: major    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 19.12.0
Attachments: Screen recording

Description Nate Graham 2019-09-25 19:23:36 UTC
Building Elisa from git master. The described regression appears to have been introduced by https://cgit.kde.org/elisa.git/commit/?id=a1ca107d948fff02d4b7e58cb12f1b03843f1e36


STEPS TO REPRODUCE
1. Add some songs to the global playlist and begin playback
2. Go to the Now Playing view


OBSERVED RESULT
The artwork and title appear, but none of the metadata or lyrics.

If you manually restart playback for the current song (e.g. by double-clicking the song in the global playlist) then the missing information will appear.

However, when the current song completes, the metadata and lyrics display in the Now Playing view do not automatically update themselves to show
Comment 1 Matthieu Gallien 2019-09-27 20:23:34 UTC
I will take care of it
Comment 2 Matthieu Gallien 2019-09-27 21:13:13 UTC
Proposed fix https://phabricator.kde.org/D24274
Comment 3 Matthieu Gallien 2019-09-28 07:57:20 UTC
Git commit d8d6d037724269993a7ed9ed3e2b37df2e04fd32 by Matthieu Gallien.
Committed on 28/09/2019 at 07:55.
Pushed by mgallien into branch 'master'.

fix now playing view no longer updating to follow playing track

Summary:
properly initialize the metadata model in now playing view

Test Plan: Now playing view works OK and no warnings are printed in console

Reviewers: #elisa, ngraham, astippich, jguidon

Reviewed By: #elisa, ngraham

Differential Revision: https://phabricator.kde.org/D24274

M  +1    -0    src/elisaapplication.cpp
M  +31   -0    src/manageheaderbar.cpp
M  +27   -0    src/manageheaderbar.h
M  +1    -0    src/qml/ContentView.qml
M  +4    -3    src/qml/ContextView.qml

https://commits.kde.org/elisa/d8d6d037724269993a7ed9ed3e2b37df2e04fd32
Comment 4 Nate Graham 2019-11-21 14:25:55 UTC
Created attachment 124046 [details]
Screen recording

I'm afraid a variant of the issue appears to have returned in 19.12 and git master. I'm attaching a screen recording.
Comment 5 Matthieu Gallien 2019-11-22 06:37:04 UTC
Possible fix for that https://invent.kde.org/kde/elisa/merge_requests/37
Comment 6 Matthieu Gallien 2019-11-22 14:00:13 UTC
Git commit 292b18e1f6789e945f5751f30230df051bb4963f by Matthieu Gallien.
Committed on 22/11/2019 at 06:33.
Pushed by mgallien into branch 'release/19.12'.

bring back a working ContextView

M  +4    -9    src/manageheaderbar.cpp
M  +5    -5    src/manageheaderbar.h
M  +1    -1    src/qml/ContentView.qml
M  +5    -5    src/qml/ContextView.qml

https://invent.kde.org/kde/elisa/commit/292b18e1f6789e945f5751f30230df051bb4963f