[Ned Scott]

Anyone ever notice that we don't have a dedicated actionID for going to the next video in a playlist? All the options we have now won't skip the video itself if there are chapters. I understand sharing bigstepforward and next chapter, but next video and next chapter should be separate.

[jmarshall]

If there are chapters is the key here. We do actually have NextItem/PrevItem functions, but perhaps they're overridden to perform chapter duty? IMO chapter duty is better to override the large skip forward/back.

[Ned Scott]

From Action IDs (wiki) we have:

Code:

SkipNext                 Skip to the next item in a playlist or scene in a video.
SkipPrevious             Skip to the previous item in a playlist or scene in a video.
BigStepForward           Step forward 10min in video.
BigStepBack              Step back 10min in video.
nextscene                ? (ACTION_NEXT_SCENE)
previousscene            ? (ACTION_PREV_SCENE)

The last two don't seem to do anything during playback. I'm not sure what they do.

Playing videos on a playlist without chapters:

Code:

SkipNext                 Next video in playlist
SkipPrevious             Previous video in playlist
BigStepForward           Step forward 10min in video.
BigStepBack              Step back 10min in video.

Playing videos on a playlist with chapters:

Code:

SkipNext                 Next chapter
SkipPrevious             Previous chapter
BigStepForward           Next chapter
BigStepBack              Previous chapter

I even tried PlayerControl(Next) and PlayerControl(Previous), but they simply do the same thing as SkipNext and SkipPrevious.

So if you're in a playlist and you're on a video with chapters, you then have no way to go to the next video in the playlist.

[jmarshall]

So we already override BigStep* -> Next/Prev Chapter.

The trick now is to make sure we don't override SkipNext/Previous.

[jmarshall]

Test away:

https://github.com/xbmc/xbmc/pull/2809

[Ned Scott]

awesome, will do.