[mythtv-commits] Ticket #603: Removable media support in MythGallery

MythTV mythtv at cvs.mythtv.org
Wed Mar 15 17:24:07 UTC 2006

#603: Removable media support in MythGallery
 Reporter:  Aaron McCarthy <mccarthy.aaron at gmail.com>  |        Owner:  danielk 
     Type:  enhancement                                |       Status:  assigned
 Priority:  minor                                      |    Milestone:  0.20    
Component:  mythgallery                                |      Version:  head    
 Severity:  medium                                     |   Resolution:          
Comment (by danielk):

 Aaron can you look at this again?

 I didn't get as far as testing importing code, but I looked at the
 tag/copy/move/delete UI.

 There are some UI improvements which would make this much easier to use:
  * Clear marks after operation
  * Allow wholesale clearing all marks
  * If there are no marks and operation is delete, mark then delete
 selected item
  * Exit submenu on ESCAPE
  * Exit "Show Devices" on exit from submenu
  * Bind DELETE key to 'Delete Marked'
  * Show progress bar while performing operation. (The first thing I tried
 to copy took several minutes...)

 Plus there some code problems that need to be addressed:
  * Use Qt or C API for renaming/unlinking/copying files or directories.
 (Using the shell means needing to worry about shell escaping... As is, the
 core is unsafe.)
  * GetMedias() is unsafe if a device is removed/unmounted while you are
 looking at its returned list...
  * Avoid using #define's and #ifdef's when possible. For example you
 disable some code when _WIN32 is defined in IconView where it doesn't look
 like you need to.

Ticket URL: <http://svn.mythtv.org/trac/ticket/603>
MythTV <http://www.mythtv.org/>

More information about the mythtv-commits mailing list