[mythtv-commits] Ticket #9913: ShellEscape should put double quotes around strings with a single quote

[MythTV]

#9913: ShellEscape should put double quotes around strings with a single quote
-----------------------------------------------+---------------------------
 Reporter:  Doug Haber <doug@…>                |           Type:  Patch -
   Status:  new                                |  Bug Fix
Milestone:  unknown                            |       Priority:  minor
  Version:  Trunk Head                         |      Component:  MythTV -
 Keywords:                                     |  General
                                               |       Severity:  medium
                                               |  Ticket locked:  0
-----------------------------------------------+---------------------------
 Hi,[[BR]]
 [[BR]]
 I was running mythmetadatalookup and it failed on a recording that had a
 subtitle of: [[BR]]
 Tackin'[[BR]]

 The reason was that it passed the following to the ttvdb.py script:

 {{{
 ttvdb.py -l en -N 90571 Tackin'
 }}}

 The command line was confused by the single quote.  Attached is a simple
 patch that puts double quotes around strings with a single quote in them
 so that they look like:


 {{{
 ttvdb.py -l en -N 90571 "Tackin'"
 }}}

 I wasn't sure if always putting double quotes around strings would break
 anything else so I kept the patch as unintrusive as possible.

 Thanks! [[BR]]
 [[BR]]

 Doug

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/9913>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center