Aubin Paul wrote: > On Fri, Dec 19, 2003 at 07:15:28PM +0100, Dirk Meyer wrote: >> That's something I'm thinking of changing. Some items have 'file', >> some 'filename'. DVD items have filename == 1 for the first >> track. Some also have 'url'. This is ugly. >> Remove the filename attribute and replace it with url (file:// for >> files and dvd://, vcd://, http:// ... for other stuff). This is a >> better way to play files. To take care of the fileops, mover and >> shopping card plugin, we could add an attribute 'files'. This is a >> list of _files_ which needed to be moved/copied/deleted for this >> item. > > Both mplayer and xine use the "mrl" format for media files, so it > might be a good idea to follow their rules since it's fairly > straightforward and intuitive.
OK, I will remove the 'mode' and 'filename' attribute and create the 'url' attribute. This will break something because there is at least one part of the code I will forget to change :-) But not this year. > Why not make it a list for all files; and single unit Items would just > have one entry... I think it would be easier to just do a "for" loop > for ANY item then checking if it's a string and then doing one thing > and if it's a list doing another. > > It would make it easier for people contributing code because they'd > know... "filename" is always a list of files. I was thinking of adding move/delete/copy functions to the item itself, based in self.files. If someone (like the gphoto plugin) needs different functions to do this, the functions can be overwritten. Maybe also bool functions to check if moving is possible (e.g. if self.media: copy == true, move, delete == false. Dischi -- multitasking = screwing up several things at once ------------------------------------------------------- This SF.net email is sponsored by: IBM Linux Tutorials. Become an expert in LINUX or just sharpen your skills. Sign up for IBM's Free Linux Tutorials. Learn everything from the bash shell to sys admin. Click now! http://ads.osdn.com/?ad_id=1278&alloc_id=3371&op=click _______________________________________________ Freevo-cvslog mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/freevo-cvslog
