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

FIX: No files are returned from JSON-RPC Files.GetDirectory for http sources #458

Merged
merged 1 commit into from Oct 6, 2011

Conversation

koying
Copy link
Contributor

@koying koying commented Oct 2, 2011

CHTTPDirectory::Exists is returning true not only for directories but also for files.

This breaks CFileOperations::GetDirectory for http as the item is rejected if it is a directory in CFileOperations::FillFileItem.
The result is that no files are returned from JSON-RPC for http sources.

This patch uses CFileCurl::Stat rather than CFileCurl::Exists.

…t also for files.

This broke CFileOperations::GetDirectory for http as the item is rejected if it is a directory in CFileOperations::FillFileItem.
@elupus
Copy link
Contributor

elupus commented Oct 2, 2011

If cfilecurl set that flag somewhat properly, patch is okey.

@koying
Copy link
Contributor Author

koying commented Oct 3, 2011

If what is read has mime-type text/html, it is assumed to be a directory.

It is explicit at FileCurl.cpp:1149

elupus added a commit that referenced this pull request Oct 6, 2011
FIX: No files are returned from JSON-RPC Files.GetDirectory for http sources
@elupus elupus merged commit 592fba8 into xbmc:master Oct 6, 2011
tru pushed a commit to plexinc/plex-home-theater-public that referenced this pull request Jan 31, 2013
koying added a commit to koying/xbmc that referenced this pull request Nov 27, 2017
FIX: mouse handling when FB size != GUI size
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.

None yet

2 participants