[mythtv] Trailing spaces in callsigns
Michael T. Dean
mtdean at thirdcontact.com
Thu Sep 15 17:44:47 UTC 2011
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
>> 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
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. :)
More information about the mythtv-dev