[mythtv-commits] Ticket #1831: mythfilldatabase broken for xmltvid containing spaces

[MythTV]

#1831: mythfilldatabase broken for xmltvid containing spaces
------------------------------------+---------------------------------------
 Reporter:  urnest at onthenet.com.au  |       Owner:  ijr
     Type:  defect                  |      Status:  new
 Priority:  major                   |   Milestone:     
Component:  mythtv                  |     Version:     
 Severity:  medium                  |  
------------------------------------+---------------------------------------
 mythfilldatabase, with xml_tv_grab only uses part of the channel id, up to
 the first space. e.g. xmltv data entry with
 {{{
 <programme channel="TEN Digital" start="20060520130000 +0000"
 stop="20060520150000 +0000">
 }}}
 ... gives a messages:
 {{{
  Unknown xmltv channel identifier: TEN
 Skipping channel.
 }}}
 ... with 'TEN' rather than 'TEN Digital'. It also puts in a new channel,
 which is probably another bug?:
 {{{
 mysql> select name, chanid, xmltvid from channel where name = 'TEN
 DIGITAL';
     :
 | TEN Digital |   1007 | TEN Digital |
 | TEN Digital |      4 | TEN Digital |
     :
 }}}
 ... I already had 1007 and I now have 4 too.

 Here is the patch I use for my guide-data:
 {{{
 xju at xjutv ~/2/mythtv/programs/mythfilldatabase $ svn diff filldata.cpp
 Index: filldata.cpp
 ===================================================================
 --- filldata.cpp        (revision 9947)
 +++ filldata.cpp        (working copy)
 @@ -1318,10 +1318,10 @@
      pginfo->endts = text;

      text = element.attribute("channel", "");
 -    QStringList split = QStringList::split(" ", text);
 +    pginfo->channel = text;
 +
 +    QStringList split;

 -    pginfo->channel = split[0];
 -
      text = element.attribute("clumpidx", "");
      if (!text.isEmpty())
      {
 }}}

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/1831>
MythTV <http://www.mythtv.org/>
MythTV