pvr: fix channel switch for addons using other stream

[FernetMenta]

@opdenkamp
fix trac 14307 Player stops if switch channel

pOtherStream->NextStream returns always STREAM_NONE. we need to call this on m_pFile

[opdenkamp]

wasn't pOtherStream needed for mediaportal?

[FernetMenta]

Yes it is. CPVRFile is opened anyway. This returns true for SkipNext in case of live tv. Addons using stream urls open pOtherStream in addition to CPVRFile but SkipNext returns false for DVDInputStream.
This two lines have ever been wrong in this place, then it was hacked around by introducing this m_breopened flag.

[opdenkamp]

alright thanks.

@margro do you have time to test this commit with mediaportal? thanks

[afedchin]

I confirm that this fixes the issue if use pvr.demo or pvr.iptvsimple addons.

[opdenkamp]

yeah that's what this patch is supposed to fix, so that's good, but i'd like to have someone verify that we don't break mediaportal (and i think argustv too) with this

[elupus]

Should it check both?

If otherstream gives something else than none then return that. Else check
the pFile.

[FernetMenta]

Should it check both? If otherstream gives something else than none then return that. Else check the pFile.

sounds reasonable. I will update.

[margro]

I've just tested master with and without this change. The MediaPortal pvr addon was not affected by the 14307 bug.
The addition from this pull request does not break the MediaPortal pvr addon behavior either, so I'm fine with this change.
Although not yet tested, the argustv addon is not affected either (based on the same playback code).

[elupus]

Not good enough. You can only request next stream once. Now it's done
twice.

[FernetMenta]

thanks, updated now

[FernetMenta]

@opdenkamp as discussed on IRC, I'll put this to the backport list for Frodo.

[opdenkamp]

right, you were quicker than me :)