[mythtv] Trailing spaces in callsigns

Michael T. Dean mtdean at thirdcontact.com
Fri Sep 16 04:31:15 UTC 2011


On 09/15/2011 09:58 PM, Jerry Rubinow wrote:
> On Thu, Sep 15, 2011 at 1:44 PM, Michael T. Dean<mtdean at thirdcontact.com>wrote:
>
>> On 09/15/2011 12:15 PM, Jerry Rubinow wrote:
>>> On Thu, Sep 15, 2011 at 12:06 PM, Stuart Morgan wrote:
>>>> On Thursday 15 Sep 2011 07:44:51 Jerry Rubinow wrote:
>>>>> Would a patch be accepted which trimmed off trailing whitespace in the channel callsigns before doing the comparison for merging channels in the program guide?
>>>> Would it not be easier in the long term to trim the whitespace at the time the callsign is inserted into the database instead of all the places that it's read back from the database?
>>>>
>>>> A one-off DB update could address the existing instances of trailing/leading whitespace.
>> Ah, yeah, I misread it thinking he was asking about fixing the callsigns
>> in the editors so they're put in the database correctly.  But, yeah, I
>> agree it should be fixed upon insert--meaning we only need a fix for the
>> 4 channel editors (mythtv-setup, frontend, MythWeb, and HTTP setup).
> I started looking at mythtv-setup, channeleditor.cpp to be specific.  I see
> where it reads the DB and populates the UI with the channel info, but I
> don't see how edited info gets back into the DB.  Background: lots of C++
> experience, not so much with QT, some DB work, enough to puzzle my way
> through understanding a query.
>

Looks like ChannelUtil::CreateChannel() and 
ChannelUtil::UpdateChannel(), which are probably used by the frontend 
editor, too.

Also, it seems that the xmltv config(?) uses ChannelData::handleChannels().

Mike


More information about the mythtv-dev mailing list