[mythtv-commits] Ticket #2754: Colon in subtitle confuses mythfrontend, MythWeb
MythTV
mythtv at cvs.mythtv.org
Sat Dec 2 03:55:07 UTC 2006
#2754: Colon in subtitle confuses mythfrontend, MythWeb
----------------------------+-----------------------------------------------
Reporter: ylee at pobox.com | Owner: ijr
Type: defect | Status: new
Priority: minor | Milestone: unknown
Component: mythtv | Version: 0.20
Severity: medium | Resolution:
----------------------------+-----------------------------------------------
Comment (by nigel):
Yeechang, I can't easily test this, so could you try this patch:
[code]
Index: mythrename.pl
===================================================================
--- mythrename.pl (revision 12168)
+++ mythrename.pl (working copy)
@@ -228,9 +228,11 @@
# Format the name
my $name =
$show->format_name($format,$separator,$replacement,$dest,$underscores);
# Get a shell-safe version of the filename (yes, I know it's not
needed in this case, but I'm anal about such things)
- my $safe_file = $show->{'local_path'};
- $safe_file =~ s/'/'\\''/sg;
- $safe_file = "'$safe_file'";
+ my $safe_name = quotemeta($name);
+ if ( $name ne $safe_name ) {
+ print "Changing unsafe filename '$name' to '$safe_name'\n";
+ $name = $safe_name;
+ }
# Figure out the suffix
my $suffix = ($show->{'finfo'}->{'is_mpeg'}) ? '.mpg' : '.nuv';
# Link destination
[/code]
--
Ticket URL: <http://svn.mythtv.org/trac/ticket/2754#comment:1>
MythTV <http://www.mythtv.org/>
MythTV
More information about the mythtv-commits
mailing list