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

Implement file copy from "videodb://" source and refine target filenames #1585

Closed
wants to merge 3 commits into from

Conversation

koying
Copy link
Contributor

@koying koying commented Oct 9, 2012

This is still quite experimental and would like to receive comments about it.

The main point is to be able to copy, in the file browser, from a videodb:// source to a "hard" destination.
The use case being to be able to export files from the database for mobile use, being android, rpi or other, even non-xbmc.

Main points of attention:

  1. To align with the music database, I've added the extension to the "videodb://" url's.
    This is not stricto sensu mandatory, but I think it is, both for consistency with the music db and with the url concept, a good idea.
    Obviously, it is quite dangerous and is likely to break any piece of code assuming a videodb url without extension. Any hint on where this is likely to break is welcome.
  2. The concept of exporting a video file for "mobile" xbmc usage would ideally also require to export the .nfo/.tbn and implementation of the "mythical" nfo only scrapping.

Thanks for giving feedback

@koying
Copy link
Contributor Author

koying commented Oct 9, 2012

P.S. Please note that for the moment XBMC is inconsistent as it is showing the "Copy" option in the context menu of a "videodb://" item while it is not implemented.

@ghost
Copy link

ghost commented Oct 9, 2012

can you do up a fix for ^^ ?

@ghost
Copy link

ghost commented Oct 9, 2012

  1. for starters make sure the videodatabasedirectory classes can handle it. then check the contextmenu and action hanlding related functions in the video and music (due to mvids there's some code there as well) windows

@koying
Copy link
Contributor Author

koying commented Oct 9, 2012

Yep, will do

@ghost ghost assigned jmarshallnz Oct 17, 2013
@koying koying closed this May 24, 2015
@MartijnKaijser MartijnKaijser modified the milestones: WIP / RFC (code improvements needed), Abandoned, obsolete or superseeded Jun 12, 2015
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

3 participants