2015-04-07, 11:35
(2015-04-07, 11:03)Montellese Wrote:Yes in the simplest case it is not needed i.e. If we deal with duplicates like movie sets do and always open a directory of all the choices it is not needed.(2015-04-07, 09:36)rsanger Wrote: I will try to remove CFileItemList, I'll probably end up putting some extra fields into CFileItem to hold the paths(but not the full CFileItems). I don't know the code very well and wouldn't want to break things with this change.I still don't see why you need all the paths in the CFileItem. You only really need them when you want to play/queue the item and then you can simply get all the paths that you need. The only disadvantage I can see with that approach is that it needs to be handled differently for video and music items.
However we miss out on some niceties:
1. With this solution we don't get a stream quality indicator (without say searching for items in the database anyway at such point we might as well cache them against the item).
2. I also personally like the idea of being able to set this to automatically pick the "Best" and have give me that without having to open up an item. Which also allows for options to "Play from here" etc.
I do have a working version of both 1. and 2. this implemented currently, so if you find the time to give this a try it would be great (either the windows build or grab it from github).
You clearly have much more experience with the Kodi code than me and if you don't think this is worth pursuing or should be implemented differently let me know.
(2015-04-07, 11:03)Montellese Wrote:(2015-04-07, 09:36)rsanger Wrote: I've noticed that the existing Context Menu->Manage->Edit Title already does this.
Yes but that will not work without adjustments as it will adjust the title of the movie but that will affect all versions because we only store that information once and link the different files to that information. So we'll need a way to store an additional version-specific title.
Hmmm... I just tried this with movies and TV-shows and it seems to be stored per item. My modifications have been based off Kodi master not your branch which had the database changes.
Thanks for all the help,
Richard