New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[htsp v27] implement server based playcount and playposition #308
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor stuff.
src/Tvheadend.cpp
Outdated
int CTvheadend::GetPlayPosition ( const PVR_RECORDING &rec ) | ||
{ | ||
if (m_conn.GetProtocol() < 27) | ||
return PVR_ERROR_NOT_IMPLEMENTED; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the API documentation, this line should read return -1;
.
src/Tvheadend.cpp
Outdated
return it->second.GetPlayPosition(); | ||
} | ||
|
||
return PVR_ERROR_INVALID_PARAMETERS; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the API documentation, this line should read return -1;
.
@Glenn-1990 do you think it would be a good idea to let the user choose (addon settings) whether server-side playcount/Position shall be used? Also, version bump and changelog is missing. |
@ksooo Addressed your comments. I also added an advanced setting in case users don't like the new behaviour. |
Nice. Thank you very much for this contribution. |
It's been a while, tvheadend 4.2 is out and it includes improved playstatus support.
Kodi master does have an issue that needs to be fixed as well before this code will work as expected.
xbmc/xbmc#12226
The big advantage:
Start watching a recording in the living room and resume watching in the bedroom etc.
@ksooo