Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#8115 closed defect (fixed)

Fails to create DVD: IndexError

Reported by: Antonio Marcos López Alonso <amlopezalonso@…> 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)

mythburn.diff (771 bytes) - added by lynchmv@… 14 years ago.
Mytharchive diff for successful archiving of videos to DVD
mythburn2.diff (8.6 KB) - added by Raymond Wagner 14 years ago.

Download all attachments as: .zip

Change History (11)

Changed 14 years ago by lynchmv@…

Attachment: mythburn.diff added

Mytharchive diff for successful archiving of videos to DVD

comment:1 Changed 14 years ago by lynchmv@…

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.

Changed 14 years ago by Raymond Wagner

Attachment: mythburn2.diff added

comment:2 Changed 14 years ago by Raymond Wagner

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.

comment:3 in reply to:  2 ; Changed 14 years ago by anonymous

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 in reply to:  3 Changed 14 years ago by Raymond Wagner

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:5 Changed 14 years ago by amlopezalonso@…

Still on even in 0.23-fixes. Should I file a new bug report?

comment:6 Changed 14 years ago by robertm

Ticket locked: set

No. This ticket isn't closed.

comment:7 Changed 14 years ago by paulh

Resolution: fixed
Status: newclosed

(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 Changed 14 years ago by paulh

(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 Changed 14 years ago by paulh

Milestone: unknown0.23-fixes
Ticket locked: unset
Version: 0.22-fixes0.23-fixes
Note: See TracTickets for help on using tickets.