[mythtv-commits] Ticket #10046: Channel insert/updates now strip off leading/trailing whitespace in callsign and name

[MythTV]

#10046: Channel insert/updates now strip off leading/trailing whitespace in
callsign and name
-----------------------------------------------+---------------------------
 Reporter:  Jerry Rubinow <jerrymr@…>          |          Owner:
     Type:  Patch - Bug Fix                    |         Status:  new
 Priority:  minor                              |      Milestone:  unknown
Component:  MythTV - General                   |        Version:  Trunk
 Severity:  medium                             |  Head
 Keywords:  database channel callsign          |     Resolution:
                                               |  Ticket locked:  0
-----------------------------------------------+---------------------------

Comment (by Jerry Rubinow <jerrymr@…>):

 Here's is an updated and tested patch which trims callsign, channel
 number, and channel name in libmythtv/channelsettings.cpp,
 libmythtv/channelutil.cpp, libmythtv/tv_rec.cpp, and in mythweb in
 modules/tv/set_channels.php.  I've verified the changes work correctly in
 the mythtv-setup channel editor, when using 'e' in the program guide, and
 in mythweb.

 It seems like there's an extra place in the code that modifies the channel
 table which doesn't have a corresponding channel editor I was able to
 find.  Also, I found two more places in the code, in mythfilldatabase and
 in libmythtv/datadirect.cpp, which update the channel table.  I wasn't
 sure if these places should trim as well.  They are currently unmodified.

 These two patches (one for master, one for mythweb) supplant the earlier
 attachments.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10046#comment:2>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center