[mythtv-commits] Ticket #7887: Additional MusicExitAction option for MythMusic

[MythTV]

#7887: Additional MusicExitAction option for MythMusic
-----------------------------------+----------------------------------------
 Reporter:  mythtv@…               |       Owner:  ijr    
     Type:  patch                  |      Status:  new    
 Priority:  minor                  |   Milestone:  unknown
Component:  Plugin - MythMusic     |     Version:  0.22   
 Severity:  medium                 |     Mlocked:  0      
-----------------------------------+----------------------------------------

Comment(by Jonathan Martens <jonathan@…>):

 Your action is an undesired (and to many unexpected change IMHO. After
 pressing Escape once the user is prompted with the choices you stated
 above.
 If the user decides to really exit they should confirm either 1 or 2, if
 the user decides to cancel it can select option 3. The use can (on a
 keyboard) also press Escape to effectuate option 3. Escape should be used
 for backing out, not confirming like you desire IMHO. I doubt that the
 cancel function (ESC) should be used to exit from this prompt other than
 the current behavior return to MythMusic as the user cancels the exit.

 You are trying to override the cancel to do both, cause option 1: "stop
 playing and exit" as well as your new option 4: "stop playing and exit",
 this is a race condition which will not likely be added to the source I
 think.

 You can disable prompting IIRC in the database, which might solve your
 issues in home automation.

 Are you sure your home automation can not send a plain ENTER in order to
 confirm the first (default option) to stop playing and leave MythMusic
 after the first Escape which causes the prompt?

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/7887#comment:1>
MythTV <http://www.mythtv.org/>
MythTV