kdf;192799 Wrote: 
> I think the best way is to proceed as part of a 7.0 conversion.  Since
> the plugin API is very different starting with 7.0, there is not much
> likelyhood that you can package a 6.5 and 7.0 compatible combined
> download.  This provides a good point for ripping out some of the more
> difficult elements.  It helps us flesh out the design of the API to see
> as many plugins as possible in operation with the API, adding what might
> be needed along the way.  Is there a particular plugin from your suite
> that would say is "simpler" than the rest?  If I start with something
> simple, I can have a go at converting it to 7.0 and trying  
> out some fixes.

I you like a "simple" plugin CustomScan or DynamicPlayList is probably
best, but I suspect that most of the web interface related problems
will only be seen with TrackStat and Custom Browse and converting these
to 7.0 at the moment will probably be a lot of work and a bad idea.

A good start is probably if I would remove the 6.2/6.3 compatibility
code from the plugins to make them 6.5 only. This way some parts of the
web interface should be possible to simplify a bit. However, I will
probably not be in the next weeks because I'm in a middle of a major
refactoring some of the code in the Custom Browse plugin that I like to
finish first.

A bit more information about the state of each plugin:

Custom Browse: 
==============
A lot of code, some parts pretty messy. I am working on a major
refactoring of parts of the code at the moment, so a conversion to 7.0
at the moment would probably be a bad idea. I think the current version
contains some 6.2/6.3 compatibility code in the html templates, this
should be removed before trying to convert anything. People have
reported that it doesn't work correctly with Nokia skin. If I remember
correctly I had some issues with the gallery/artwork handling. I want
the artwork/gallery icon that switches between artwork and standard
view in the header, I also want the sorting list box but I want to fill
it with my own sorting options. I couldn't figure out a good way to do
this with the standard SlimServer templates so it contains its own
implementation of the sort drop list and artwork icon in the heading. I
think this might be causing some of the problems.
HTML/EN/plugins/CustomBrowse/custombrowse_list.html: Main page
HTML/EN/plugins/CustomBrowse/galleryselect.html: Header with custom
sort list and artwork icon.

TrackStat
=========
A lot of code, some parts pretty messy. The major problem here was to
get the rating stars to display correctly in several skins. The problem
is that the rating starts should be placed on the right side of the
list. I had to do some skin specific code to get it to work correctly
in both default and Fishbone skin, so I suspect it probably doesn't
work good in Nokia skin since its quite different. One problem with the
rating stars is that 5 stars, play/add buttons and mixer buttons take up
a lot more space than what the standard "play/add + mixer" does. The
result is that I needed to provide also some custom css styles to get
it to display correctly.
HTML/Fishbone/plugins/TrackStat/controls: Fishbone specific code for
the rating stars and buttons
HTML/EN/plugins/TrackStat/controls: Generic code for the rest of the
skins for the rating stars and buttons
HTML/EN/plugins/TrackStat/index.html: Main page
HTML/EN/plugins/TrackStat/listitems: Items for each statistic row
displayed

DynamicPlayList
===============
A bit simplier, however there is some menu handling code that is a bit
messy that I would like to refactor if I ever get the time. The web
interface is pretty simpe compared to Custom Browse and TrackStat.
Could be a good alternative for 7.0 converstion, the only disadvantage
is that I would probably like to clean up parts of the code when I
convert it to 7.0.

CustomScan
==========
Pretty simple, no major refactoring planned. This is probably a good
place to start 7.0 conversion. The only disadvantage is that the web
interface is really simple so I suspect that you won't see all
potential problems.

CustomSkip
==========
Pretty simple, might be affected by the refactoring of Custom Browse so
a 7.0 conversion is probably not a good idea. The web interface is
pretty simple.

RandomPlayList
==============
Very simple, but it requires Dynamic Playlist to work. The web
interface is similar to the interface of the RandomMix plugin.

SQLPlayList
===========
A lot of code, will also be affected by the refactoring of Custom
Browse so due to this it is not good for 7.0 conversion at the moment.

MultiLibrary
============
Pretty simple, but it will also be affected by the refactoring of
Custom Browse so due to this it is not good for 7.0 conversion at the
moment.


-- 
erland

Erland Isaksson
'My homepage' (http://erland.homeip.net) 'My download page'
(http://erland.homeip.net/download)
(Developer of 'TrackStat, SQLPlayList, DynamicPlayList, Custom Browse,
Custom Scan,  Custom Skip, Multi Library and RandomPlayList plugins'
(http://wiki.erland.homeip.net/index.php/Category:SlimServer))
------------------------------------------------------------------------
erland's Profile: http://forums.slimdevices.com/member.php?userid=3124
View this thread: http://forums.slimdevices.com/showthread.php?t=34175

_______________________________________________
plugins mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/plugins

Reply via email to