Jean Felder authored Feb 25, 2018 and Jean Felder committed Mar 04, 2018

Player keeps an internal reference to the currently visible view, wich
can be different from the currently played model. The main idea is to
isolate player model. when set_playlist method from player is called, a
simple copy is done. Therefore, when view changes, player model is not
updated. This new model needs to stay synchronized by calling add_song
or remove_song appropriately.
This is not a fix. Just a workaround before player.py rewrite.
Add an enum for player model fields.
Remove 'song-position-changed' signal from playlists as it is not
necessary anymore.

Closes: #136