Unified music addon frontend

Sorry guys I will wait for Spotify release for ATV, thank you

akezeke Wrote:

• bool isReady();
• bool getTracks(CFileItemList& items,CStdString path, CStdString artistName, int albumId);
• bool getAlbums(CFileItemList& items,CStdString path, CStdString artistName);
• bool getArtists(CFileItemList& items,CStdString path);
• bool getPlaylists(CFileItemList& items);
• bool getTopLists(CFileItemList& items);
• bool getCustomEntries(CFileItemList& items);
• bool search(CStdString query,CFileItemList& items);
• ICodec* getCodec();
  

/David

1) why on earth are you copying those strings again?
2) we usually chase off camels with a large stick

spiff Wrote:1) why on earth are you copying those strings again?
2) we usually chase off camels with a large stick

1) Do you mean the path strings? It is so I can return the relevant tracks, albums etc. without the path I cant know for which album the tracks are supposed to come from for example.

2) Thanks, dont really understand that one

1) yes - passing the strings is fine, copying is not. pass by const reference..
2) camelCasing - do GetCodec() not getCodec()

spiff Wrote:1) yes - passing the strings is fine, copying is not. pass by const reference..
2) camelCasing - do GetCodec() not getCodec()

1) Yeah you are right, the code is somewhat hasty written and I guess there are a lot of faulty logic like that, I will take care of it.

2) Didn't realize that you went against C++ common practice on that one, I read through http://forum.xbmc.org/showthread.php?tid=5238&page=2 but didn't see anything about functions, are all functions supposed to be named like that or is it some special cases?

yeh, that list is a bit outdated.

yes, use of NaturalCasing is encouraged - elupus gets a twitchy eye otherwise
member variables is often of the form m_camelCase though, those seems to pass

Suggest that you consider using this as a unified framework and API for both music and videos in the future, so for now just keep movies and tv shows in mind a little bit, as it would be good the implementation could be extended for that too later.

http://www.youtube.com/watch?v=UH8YmxsHNWQ

topfs2 Wrote:What we want in general is to allow addons to populate the library, i.e. allow multiple library backends. This for video and for music.

You guys might want to check out the new IceLibrary addon which enables the user to add stream links for movies from Icefilms.info into the XBMC video library.
http://forum.xbmc.org/showthread.php?tid=109572

That is, I think that you should look at the overall concept of this addon, more than the method used by it at the code level, as the result it achieves is probably the experince that the end users want, no matter what the source is.

Batch Wrote:The plugin was inspired by
Icefilms.info addon, created by anarchintosh
XBMC.Library supplemental tool, created by bradvido88

As you can read it that thread it uses a patch hack that allows for an addon to add strm files to the movies library.
http://trac.xbmc.org/ticket/8730

So the idea behind this addon, as well as with the XBMC.MyLibrary supplemental tool that was the inspiration for it, is that they let you add individual content from any source, such as UPnP or audio and video plugins for online internet services to XBMC's native video library for a totally integrated experince.

http://forum.xbmc.org/showthread.php?tid=98210
http://code.google.com/p/xbmc-mylibrary/

bradvido88 Wrote:So, this program acts as an integration layer between your video source and xbmc's video database like so:
[Video source (plugin/upnp/etc)] <<--->>[XBMC.MyLibrary] <<--->>[XBMC Video Library]

The XBMC.MyLibrary tool also enables the user to download streaming content, so to save local cached copies of the videos, similar to Spotify's offline mode for when you do not have high enough bandwith to stream the content directly of the internet.

anybody had a look at the concept MediaPortal2 has for it's plugable content providers/backends? Just for an inspiration...

I know this is an old(ish) thread, but since akezeke is pointing at it from the readme at https://github.com/akezeke/spotyxbmc2 I presume that it is still valid.

Is there any news on this topic? I think that the discussion is sound and interesting and defining a solid plugin API is definitely a good start.

I too am very curious. It would be nice if someone posted an update in this thread.