Bugzilla – Bug 16551
Implement support for reportPlayDuration()
Last modified: 2010-09-23 05:36:50 UTC
When a song has played halfway through, the thrift method profileService.reportPlayDuration(playerKey, numSecondsPlayed, sessionId) should be called. We need this to make sure the number of playbacks are reported correctly to the record labels.
numSecondsPlayed should be the amount of seconds played halfway through the track? Roughly duration/2?
Michael: Yes, that's correct.
...and no need to report what track actually is playing? Haven't seen any such argument in the docs.
We get the playing track from the playerKey.
== Auto-comment from SVN commit #8854 to the network repo by mherger == == http://svn.slimdevices.com/network?view=revision&revision=8854 == Bug: 16551 Description: report play duration once we are halfway through a track
== Auto-comment from SVN commit #8855 to the network repo by mherger == == http://svn.slimdevices.com/network?view=revision&revision=8855 == Bug: 16551 Description: add reportPlayDuration handler on mysb.com side
Henrik - can you verify you're seeing my tracks being reported? (41794075018)
Nils/Henrik - do you see usage being reported? I'd like to close this bug. Thanks.
Hello. We do not see any tracks being reported halfway. Could you verify that you are doing this callback? Best regards Rune (In reply to comment #8) > Nils/Henrik - do you see usage being reported? I'd like to close this bug. > Thanks.
Some debug output is telling me that the call to reportPlayDuration succeeded: Calling WiMP:{ args => ["phmlST2sIC0+UqEWDq8Bhw==", 103], dontretry => 0, method => "reportPlayDuration", player => "000420xxxxxx", sid => "Rbn4a9jUiVeicFTBZVHUHg==", user => "41794075018", } at /Users/mh/Documents/workspace/SN/script/../lib/Net/WiMP.pm line 547. 1 at /Users/mh/Documents/workspace/SN/script/../lib/Net/WiMP.pm line 595. (the "1" is the boolean true value)
== Auto-comment from SVN commit #8884 to the network repo by mherger == == http://svn.slimdevices.com/network?view=revision&revision=8884 == Bug: 16551 Description: reportPlayDuration must handle UserLoggedOutException. Stop playback on connected players, otherwise the session would automatically be renewed on track change.
Could you please also verify that the parameters sent in are correct? reportPlayDuration(playerKey, numSecondsPlayed, sessionId) When reporting, playerkey and sessionid needs to be the same as they were when you started the song. (In reply to comment #10) > Some debug output is telling me that the call to reportPlayDuration succeeded: > > Calling WiMP:{ > args => ["phmlST2sIC0+UqEWDq8Bhw==", 103], > dontretry => 0, > method => "reportPlayDuration", > player => "000420xxxxxx", > sid => "Rbn4a9jUiVeicFTBZVHUHg==", > user => "41794075018", > } at /Users/mh/Documents/workspace/SN/script/../lib/Net/WiMP.pm line 547. > 1 at /Users/mh/Documents/workspace/SN/script/../lib/Net/WiMP.pm line 595. > > (the "1" is the boolean true value)
> When reporting, playerkey and sessionid needs to be the same as they were when > you started the song. Do you see incorrect values in the logs right now, or in the example I posted? There has been an issue which potentially could have caused the difference. But I (hopefully) fixed it earlier today.
I actually do not see any incoming reports. That is why I wondered if you maybe sent the wrong values or something.. (In reply to comment #13) > > When reporting, playerkey and sessionid needs to be the same as they were when > > you started the song. > > Do you see incorrect values in the logs right now, or in the example I posted? > There has been an issue which potentially could have caused the difference. But > I (hopefully) fixed it earlier today.
> I actually do not see any incoming reports. That is why I wondered if you maybe > sent the wrong values or something.. Shouldn't the call return a false value if it didn't log? Don't you see the call being made at all? That's quite a bit irritating. I see the calls are being made. I do get the UserLoggedOutException when I log in to WiMP from somewhere else - which means it's at least doing something. Maybe you don't see calls from mysb.com as I'm testing with my local instance. Would you see some coming from 188.62.100.139?
Are you getting UserLoggedOutException on the profileService.reportPlayDuration(playerKey, numSecondsPlayed, sessionId) method call?
Ok, please help me understand: the reportPlayDuration() call should be using the same playerKey I've been using to create the track's URL? In this case my code might indeed be wrong. Is getNextPlayer()->nextKey the same as getNextPlayerKey()?
== Auto-comment from SVN commit #8886 to the network repo by mherger == == http://svn.slimdevices.com/network?view=revision&revision=8886 == Fixed Bug: 16551 Description: we need to use the same player key when reporting the duration as we used to create the streaming URL