After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 708103 - Playing default playlist is broken
Playing default playlist is broken
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other All
: Normal normal
: 3.10
Assigned To: Arnel Borja
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2013-09-15 05:30 UTC by Arnel Borja
Modified: 2013-09-16 14:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
player: No next/previous track if there is no playlist (1.18 KB, patch)
2013-09-15 05:30 UTC, Arnel Borja
committed Details | Review
mpris: Use filter instead of the view model (981 bytes, patch)
2013-09-16 12:34 UTC, Arnel Borja
committed Details | Review

Description Arnel Borja 2013-09-15 05:30:46 UTC
I broke this during the search merge, so here's the fix.
Comment 1 Arnel Borja 2013-09-15 05:30:48 UTC
Created attachment 254960 [details] [review]
player: No next/previous track if there is no playlist

Fixes Play on MPRIS when there is no playlist.
Comment 2 Vadim Rutkovsky 2013-09-16 07:59:41 UTC
Playing default playlist still crashes for me:

1) If I'm not on Songs view - crash:
Traceback (most recent call last):
  • File "/opt/gnome/lib/python3.3/site-packages/gnomemusic/mpris.py", line 153 in _play_first_song
    model.disconnect(self.first_song_handler)
  • File "/opt/gnome/lib64/python3.3/site-packages/gi/overrides/GObject.py", line 438 in wrapper
    return func(_get_instance_for_signal(obj), *args, **kwargs)
TypeError: argument instance: Expected GObject.Object, but got PyCapsule

2) If I'm on Songs view, then g-m keeps on trying to play first song (which is undiscoverable) - and doesn't move on
Comment 3 Arnel Borja 2013-09-16 12:27:20 UTC
(In reply to comment #2)
> Playing default playlist still crashes for me:
> 
> 2) If I'm on Songs view, then g-m keeps on trying to play first song (which is
> undiscoverable) - and doesn't move on
> 

I think this is a different bug though, this would happen too when all files get deleted on disk - Music will be in an infinite loop if in repeat all, repeat song or shuffle mode.
Comment 4 Arnel Borja 2013-09-16 12:34:14 UTC
Created attachment 255021 [details] [review]
mpris: Use filter instead of the view model
Comment 5 Vadim Rutkovsky 2013-09-16 13:50:15 UTC
Review of attachment 255021 [details] [review]:

+1, but it still doesn't fix problem 2) - g-m is stuck on the same song instead of skipping it
Comment 6 Arnel Borja 2013-09-16 14:10:37 UTC
Comment on attachment 254960 [details] [review]
player: No next/previous track if there is no playlist

Committed as 37e7e12.
Comment 7 Arnel Borja 2013-09-16 14:10:58 UTC
Comment on attachment 255021 [details] [review]
mpris: Use filter instead of the view model

Committed as da1bc54.
Comment 8 Arnel Borja 2013-09-16 14:11:26 UTC
(In reply to comment #5)
> Review of attachment 255021 [details] [review]:
> 
> +1, but it still doesn't fix problem 2) - g-m is stuck on the same song instead
> of skipping it

I'll file another bug report for this.