[mythtv-commits] Ticket #9537: MediaMonitor JumpTo plugin broken in commit ed4d961aed836fade236d006b2797f0c044970de

[MythTV]

#9537: MediaMonitor JumpTo plugin broken in commit
ed4d961aed836fade236d006b2797f0c044970de
-------------------------------------------------+-------------------------
 Reporter:  Lawrence Rust <lvr@…>                |           Type:  Bug
   Status:  new                                  |  Report
Milestone:  unknown                              |       Priority:  minor
  Version:  Trunk Head                           |      Component:  MythTV
 Keywords:                                       |  - General
                                                 |       Severity:  medium
                                                 |  Ticket locked:  0
-------------------------------------------------+-------------------------
 In commit ed4d961aed836fade236d006b2797f0c044970de the call to
 GetMythMainWindow()->JumpTo("Main Menu") was moved from
 MediaMonitor::JumpToMediaHandler into the plugins MythMusic, MythVideo &
 MythGallery.

 However, JumpTo() doesn't complete synchronously, it calls
 QCoreApplication::postEvent to do the work.  When the plugin starts its
 event loop it finds the jump event and immediately exits without playing
 the inserted media.

 It is necessary to execute the following code after calling JumpTo():
 {{{
     GetMythMainWindow()->JumpTo("Main Menu");
     QTime t; t.start();
     while (GetMythMainWindow()->IsExitingToMain() && t.elapsed() < 2000)
         qApp->processEvents(); // Ensure jump is executed before calling
 handler

 }}}

 Because of this it may be preferable to keep the call to JumpTo() in
 MediaMonitor::JumpToMediaHandler and append the above code rather than
 replicate it in all plugins.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/9537>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center