NetworkControl interface plays the wrong recording

[stichnot@…]

The frontend telnet interface plays the wrong recording. In the logs below, I entered the command "play program 1021 2009-10-21T20:58:00". Regardless of the program requested, it always plays the first recording in the currently selected recording group.

2009-10-23 08:57:36.756 NetworkControl: Trying to PLAY program '1021' @ '2009-10-21T20:58:00'
2009-10-23 08:57:36.827 TV: StartTV() -- begin
2009-10-23 08:57:36.827 TV: ctor
2009-10-23 08:57:36.881 TV: DrawUnusedRects() -- begin
2009-10-23 08:57:36.881 TV: DrawUnusedRects() -- end
2009-10-23 08:57:36.882 TV: tv->Playback() -- begin
2009-10-23 08:57:36.890 TV: tv->Playback() -- end
2009-10-23 08:57:36.891 TV: StartTV -- process events begin
2009-10-23 08:57:36.934 TV: HandleStateChange(0) -- begin
2009-10-23 08:57:36.935 TV: Attempting to change from None to Watching WatchingPreRecorded
2009-10-23 08:57:36.938 RingBuf(myth://192.168.0.205:6543/1111_20091019195800.mpg): OpenFile(myth://192.168.0.205:6543/1111_20091019195800.mpg, 12)
2009-10-23 08:57:37.075 RingBuf(myth://192.168.0.205:6543/1111_20091019195800.mpg): CalcReadAheadThresh(0 KB)
                         -> threshhold(64 KB) min read(0 KB) blk size(32 KB)
2009-10-23 08:57:37.125 TV: StartPlayer(0, Watching WatchingPreRecorded, main) -- begin

$ mythfrontend --version
Please include all output in bug reports.
MythTV Version   : 22575M
MythTV Branch    : trunk
Network Protocol : 50
Library API      : 0.22.20091022-1
QT Version       : 4.5.2
Options compiled in:
 linux release using_oss using_alsa using_backend using_dvb using_frontend using_hdhomerun using_hdpvr using_iptv using_ivtv using_joystick_menu using_lirc using_mheg using_opengl_video using_opengl_vsync using_qtwebkit using_v4l using_x11 using_xrandr using_xv using_bindings_perl using_bindings_python using_opengl using_vdpau using_ffmpeg_threads using_live using_mheg

[foobum <foobum@…>]

Attached patch fixes it by iterating over recording list in NetworkControl (rather than MythUIButtonList::SetValueByData()) and comparing chanid and recstartts. Corrects MythUIButtonList::GetItemNext().

[chrisribe@…]

Uses play() instead of playSelected(). I'm not really sure of the implications, but it works. Also added an extra delay to NetworkControl::processPlay() to keep UI from getting stuck on Watch Recordings screen.

[cpinkham]

This should have been fixed by [23376] and [23381].