[mythtv-commits] Ticket #8901: Add undo/redo functionality to the cutlist editor

MythTV mythtv at cvs.mythtv.org
Sat Sep 18 14:23:50 UTC 2010


#8901: Add undo/redo functionality to the cutlist editor
------------------------------------------------+---------------------------
 Reporter:  Jim Stichnoth <stichnot@…>          |           Owner:  mdean     
     Type:  patch                               |          Status:  accepted  
 Priority:  minor                               |       Milestone:  unknown   
Component:  MythTV - General                    |         Version:  Trunk Head
 Severity:  medium                              |      Resolution:            
 Keywords:                                      |   Ticket locked:  0         
------------------------------------------------+---------------------------

Comment (by Jim Stichnoth <stichnot@…>):

 The v3 patch adds auto-save functionality like #7916.  When an edit is
 made (as defined by an operation pushed onto the undo stack, hence the
 inclusion with this ticket), the working cutlist is stored into the
 recordedmarkup table using a set of temporary mark types.

 When starting the editor, an auto-save cutlist is automatically loaded if
 available.  It might be desirable to display an OSD notification, or allow
 the auto-save cutlist to be only explicitly loaded by extending the
 TV::ShowOSDAlreadyEditing() menu.

 The DeleteMap was extended to cache a pointer to the current PlayerContext
 object, which is required for saving a cutlist.  As such, the
 PlayerContext* argument in most of the DeleteMap methods is redundant and
 could be removed.

 DeleteMap::LoadMap() and DeleteMap::Clear() were modified from the
 previous patch so that they don't by default push an undo entry or auto-
 save, since they are sometimes called as part of higher level actions.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/8901#comment:3>
MythTV <http://www.mythtv.org/>
MythTV Media Center


More information about the mythtv-commits mailing list