Opened 16 years ago
Closed 15 years ago
Last modified 15 years ago
#8115 closed defect (fixed)
Fails to create DVD: IndexError
Reported by: | Owned by: | paulh | |
---|---|---|---|
Priority: | major | Milestone: | 0.23-fixes |
Component: | Plugin - MythArchive | Version: | 0.23-fixes |
Severity: | medium | Keywords: | DVD creation, videos |
Cc: | Ticket locked: | no |
Description
Creating a DVD made of at least one video fails with an IndexError message (but no problem if the DVD is filled with recordings only). It seems not to be related to the video type (avi or mpg) as some of them get succesfully inserted into the ISO file while others not.
Another possibly related behavior is that the list of selected videos/recordings to be in the DVD shows strange filesize values ranging from 0 or a few MB to several TB.
Changed file permissions to 777 to no avail.
See mythburn.log error below:
Traceback (most recent call last):
File "/usr/share/mythtv/mytharchive/scripts/mythburn.py", line 5434, in main
processJob(job)
File "/usr/share/mythtv/mytharchive/scripts/mythburn.py", line 5218, in processJob
title = expandItemText(infoDOM,"%title",1,0,0,0,0)
File "/usr/share/mythtv/mytharchive/scripts/mythburn.py", line 935, in expandItemText
if getText( infoDOM.getElementsByTagName("coverfile")[0]) =="":
IndexError: list index out of range
Regards Antonio
Attachments (2)
Change History (11)
by , 16 years ago
Attachment: | mythburn.diff added |
---|
comment:1 by , 16 years ago
After toying around with this very issue, I *think* I have it nailed down to the sqlstatement for the elif file.attributestype.value=="video" section. The sqlstatement uses the full file path for the query, which in turn, returns 0 rows, so the nodes do not get created in info.xml properly. I changed the sqlstatement to use file.attributesfilename.value.replace("'", "
'") instead of just filename and have been able to successfully burn DVDs of videos. Patch attached.
by , 16 years ago
Attachment: | mythburn2.diff added |
---|
follow-up: 3 comment:2 by , 16 years ago
This is a somewhat improved patch, which doesn't break burning of local MythVideo content. It also cleans up the SQL calls to better handle escaping of strings.
The bigger issue here is that the script has never been set up to handle MythVideo content in storage groups. Both patches only work as a fluke, because lynchmv happens to have 'VideoStartupDir' pointed to the same path as his storage group.
follow-up: 4 comment:3 by , 16 years ago
Replying to wagnerrp:
This is a somewhat improved patch, which doesn't break burning of local MythVideo content. It also cleans up the SQL calls to better handle escaping of strings.
The bigger issue here is that the script has never been set up to handle MythVideo content in storage groups. Both patches only work as a fluke, because lynchmv happens to have 'VideoStartupDir' pointed to the same path as his storage group.
How do you apply this patch?
comment:4 by , 16 years ago
Replying to anonymous:
How do you apply this patch?
Trac is only to be used for tracking down and fixing bugs. If you wish to discuss or otherwise seek assistance, please use the mailing lists.
comment:7 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
(In [24878]) Fix adding MythVideo files to a DVD created with MythArchive when the file is stored in a storage group. Also includes some DB query clean ups. Patch from wagnerrp. Fixes #8115
comment:8 by , 15 years ago
(In [24879]) Backport [24878] from trunk.
Fix adding MythVideo files to a DVD created with MythArchive when the file is stored in a storage group. Also includes some DB query clean ups. Patch from wagnerrp. Fixes #8115
comment:9 by , 15 years ago
Milestone: | unknown → 0.23-fixes |
---|---|
Ticket locked: | unset |
Version: | 0.22-fixes → 0.23-fixes |
Mytharchive diff for successful archiving of videos to DVD