Opened 13 years ago
Closed 11 years ago
#10280 closed Patch - Bug Fix (Works for me)
Quick manual channel selection in tv browse mode doesn't work if there is no 'browse all channels' option activated
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | MythTV - General | Version: | Master Head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
The problem occurs if you are in LiveTV channel browse mode, have "Setup"->"Video"->"Playback OSD"->"Browse all cahnnels" off, "Setup"->"Video"->"Playback OSD"->"Always use browse mode in Live TV" on, have more than one card per source and trying to enter channel number manually. After you finish entering the channel number it just disappeares from the screen without putting entered channel to browse OSD.
There are actually two problems resides in two places:
- tv_play.cpp, function TV::CommitQueuedInput?, line around 12374, ctx->playingInfo->GetSourceID() always returns '0' because playingInfo.sourceid is not set (I was'nt able to find where it should be set, so my solution is to get sourceid by a different way)
- tvbrowsehelper.cpp, function TVBrowseHelper::TVBrowseHelper, around line 52 - during querying all channels list for cardid and sourceid comparision 'GROUP BY "channum, callsign"' drops all result rows with same channel numbers leaving just the first one (in my case one source has 4 cardid's, so each channel in this source has to have 4 cardids), so I just added "cardid" as additional GROUP BY parameter.
So I'm attaching a small patch that solves above mentioned problems and allows to use 'quick channel selection in browse mode'. Best regards, Dennis
Attachments (2)
Change History (5)
Changed 13 years ago by
Attachment: | browse.patch added |
---|
Changed 13 years ago by
Attachment: | browse.2.patch added |
---|
comment:1 Changed 11 years ago by
I can't reproduce this in 0.27, could you please confirm that this bug still exists?
comment:2 Changed 11 years ago by
Status: | new → infoneeded_new |
---|
comment:3 Changed 11 years ago by
Resolution: | → Works for me |
---|---|
Status: | infoneeded_new → closed |
Was reported working in 0.27, and no confirmation/disagreement from original reporter in 7 months.
corrected patch