Skip to content
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

use CFileItem::IsSamePath() to match the item in OnPlayAndQueueMedia #1952

Merged
merged 1 commit into from
Dec 18, 2012

Conversation

jmarshallnz
Copy link
Contributor

Fixes the problem introduced by 682f121

fixes #13762

… and set the original_listitem_url for bluray selection. fixes xbmc#13762
@elupus
Copy link
Contributor

elupus commented Dec 18, 2012

Looks alright to me. But the media window is getting rather messy :/. I wonder if we couldn't do away with direct playback and always make use of a playlist. Feels it would simplify things.

jmarshallnz added a commit that referenced this pull request Dec 18, 2012
use CFileItem::IsSamePath() to match the item in OnPlayAndQueueMedia
@jmarshallnz jmarshallnz merged commit 2f1fd18 into xbmc:master Dec 18, 2012
@rpatterson
Copy link

This still happens on the http://mirrors.xbmc.org/nightlies/win32/XBMCSetup-20121219-46e045e-master.exe nightly which includes this commit.

@rpatterson
Copy link

I confirmed that the last nightly this does not happen with is http://mirrors.xbmc.org/nightlies/win32/XBMCSetup-20121214-a5f1630-master.exe.

@jmarshallnz
Copy link
Contributor Author

IsFilePath fails for videodb items, as an item_start property is created while one shouldn't exist - simple fix.

It works fine for items not in the video db.

@jmarshallnz
Copy link
Contributor Author

See b686293

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants