[mythtv] Trailing spaces in callsigns

[Michael T. Dean]

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).

> It's always possible for a user to manually insert incorrectly terminated
> strings into the DB though, as frowned upon as it is to manually edit the
> channel table.  What about doing a normalization on the callsign entries in
> the channel table every time mythbackend starts?

But we don't support direct DB edits, so if a user breaks his own 
configuration by editing the database without the benefit of all the 
lines of MythTV code that define and enforce data constraints (which, 
BTW, are not defined in the database), then they can edit the DB 
directly (or using one of our supported channel editors) to fix it.  :)

Mike