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 679938 - [gapless] Repeat always on
[gapless] Repeat always on
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: Playback
2.4.1
Other Linux
: Normal normal
: ---
Assigned To: Andrés G. Aragoneses (IRC: knocte)
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-07-14 17:41 UTC by Chow Loong Jin
Modified: 2014-02-04 19:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (5.41 KB, patch)
2014-01-29 00:05 UTC, Andrés G. Aragoneses (IRC: knocte)
reviewed Details | Review

Description Chow Loong Jin 2012-07-14 17:41:28 UTC
Originally reported at:
  https://bugs.launchpad.net/bugs/990831

I use ubuntu 12.04 amd64 and banshee 2.4.0-2ubuntu1. I have repeat option off (the default setting, I haven't changed it), but when the last song of an album is finished then banshee automatic plays the first one.
Comment 1 Andrés G. Aragoneses (IRC: knocte) 2014-01-25 00:44:27 UTC
Hi Chow, it's me again :)

Did you report this because you also reproduce it, or just to report it upstream? If it's the former, then can you tell me if it's OGG specific?

Thanks!
Comment 2 Chow Loong Jin 2014-01-25 06:41:43 UTC
Just forwarding upstream, I think. Looks like the bug has been marked a duplicate of https://bugs.launchpad.net/ubuntu/+source/banshee/+bug/709860, but I see your comment from 8 hours ago there. :)
Comment 3 Andrés G. Aragoneses (IRC: knocte) 2014-01-25 12:47:42 UTC
Are you able to reproduce it anyway?
Comment 4 Andrés G. Aragoneses (IRC: knocte) 2014-01-26 16:43:34 UTC
Based on https://bugs.launchpad.net/ubuntu/+source/banshee/+bug/709860/comments/9 I'm adding the [gapless] token in the summary.
Comment 5 Ismael Olea 2014-01-28 20:23:45 UTC
Configuration:

 - banshee git 2.6 stable at 20140125[1]
 - gapless set up
 - no repeat set up

Test:
 - playing one mp3: it's played just one time
 - playing one ogg: seems to play infinite times
 - playing one wma: seems to play infinite times
 - playing one wav: it's played just one time

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=6453707

Feel free to ask me another tests with same or newer builds.
Comment 6 Andrés G. Aragoneses (IRC: knocte) 2014-01-29 00:05:29 UTC
Created attachment 267460 [details] [review]
Proposed patch

Thanks for testing Ismael! Can you test this patch?
Comment 7 Ismael Olea 2014-01-29 20:10:06 UTC
Review of attachment 267460 [details] [review]:

Configuration:

 - banshee git 2.6 stable at 20140125 + patch[1]
 - gapless set up
 - no repeat set up

Tests:
 - playing one wav: it's played just one time
 - playing one ogg: it's played just one time
 - playing one mp3: it's played just one time
 - playing one wma: it's played just one time

Conclusion: 
 - in these scenarios the patch seems to work always
 - in this test it hasn't scrobbled to lastfm :-m

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=6469789
Comment 8 Ismael Olea 2014-01-29 20:10:29 UTC
Review of attachment 267460 [details] [review]:

Configuration:

 - banshee git 2.6 stable at 20140125 + patch[1]
 - gapless set up
 - no repeat set up

Tests:
 - playing one wav: it's played just one time
 - playing one ogg: it's played just one time
 - playing one mp3: it's played just one time
 - playing one wma: it's played just one time

Conclusion: 
 - in these scenarios the patch seems to work always
 - in this test it hasn't scrobbled to lastfm :-m

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=6469789
Comment 9 Andrés G. Aragoneses (IRC: knocte) 2014-02-01 16:55:17 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.

(Fixed in master and the stable-2.6 branch, which means the fix will be in 2.9.1 and 2.6.2.)

(In reply to comment #8)
> Conclusion: 
>  - in this test it hasn't scrobbled to lastfm :-m

Please do more tests than one, and if you confirm that lastfm scrobbling is broken after this fix and not before this fix, then create a new bug please (I added a unit test for this bug, so if lastfm gets broken because of this, we will need to add more unit tests for lastfm).
Thanks
Comment 10 Ismael Olea 2014-02-04 19:33:52 UTC
Ok.

Seems my tests where done with extremely short files so, seems to me, they didn't reach the threshold of the audioscrobbler.

So, yes, it's scrobbles.

The bad news: in the specific case of a single song to play it scrobbles twice :-)
Comment 11 Andrés G. Aragoneses (IRC: knocte) 2014-02-04 19:49:21 UTC
(In reply to comment #10)
> The bad news: in the specific case of a single song to play it scrobbles twice
> :-)

I think there's already a bug report open for that case. Please look for it and once you find it, cc Phil Trimble to it, because he usually investigates and fixes LastFM bugs very efficiently.