changed: Extend Python/CApp GetCurrentFile to obtain Player's unstacked file

[arnova]

This extension is especially useful for our subtitles addon which currently doesn't properly handle stacked file (it currently always uses the name of the first item in the stack to create the subtitles filename).

@amet: PR for the subtitles addon will follow as soon as this has been merged

[jmarshallnz]

Why not grab the playing file URL from the stack rather than yet another member in CApplication?

[arnova]

@jmarshallnz : Good point. Updated PR accordingly.

[jmarshallnz]

I'm pretty sure that the need for this in the python API will disappear when subs moves to core. It might potentially be useful for other things, though I'm not sure if that's a good enough reason - there's other paths we can play that are meaningless for add-ons (e.g. musicdb:// paths).

[MartijnKaijser]

Meaningless? iirc addons actually use musicdb:// paths for playing. At least i do

[arnova]

Although it's usefullness may disappear in the (near?) future for now this trivial fix, fixes a real world problem (although not life threatening ;-)) for our upcoming stable release. I can imagine there are more addons that may want to know the actual file currently being played...

[jmarshallnz]

@MartijnKaijser: Yes, and you don't actually need the real path, right? i.e. the real path of the playing file isn't of use.

@arnova: I'm fine with it if subs don't hit core. If subs hit core, then no immediate need. They should hit core at next merge window, but let's hold this one in case something unforeseeable hits.

[arnova]

@jmarshallnz : Haven't seen anything concerning subtitles yet as the end of the window is closing. Merge this then?

[jmarshallnz]

#3552