2018-01-03, 19:04
(2017-12-31, 09:26)ksooo Wrote: That's not correct.Of course I already tried using GetRecordingStreamProperties() to implement it like in v17, but this function works different than the old v17 functionality. Originally, the (now removed) strStreamURL argument got the filename (e.g. "recording.ts") and the ReadEditDecisionLists() function found the EDL. If I pass the same file name via GetRecordingStreamProperties(), the ReadEditDecisionLists() function receives a "pvr://recording/something.pvr" argument and it does not check the new stream URL for EDL info. So, right now the old v17 way is not possible anymore.
If you implement GetRecordingStreamProperties you can pass the recording url to Kodi and let Kodi do the playback like in v17.
Why can't you implement GetRecordingEdl? This has nothing to do with any stream vs. URL API changes, imo.
As said earlier, I never implemented EDL support in the addon since I'm not using it myself. I can implement GetRecordingEdl() but that means that I have to read the files, and transform them to a times list in the PVR addon.
This is a bit weird since Kodi already knows how to read these files. I would prefer to learn the EDL file to Kodi instead. Maybe we could add a property to the GetRecordingStreamProperties() list for this purpose...