Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#10871 closed Bug Report - General (fixed)

MythArchive is complaining about an invalid field

Reported by: Chris kapp <chris.kapp@…> Owned by: paulh
Priority: minor Milestone: 0.26.1
Component: Plugin - MythArchive Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

After attempting to use MythArchive?, the backend log in Mythweb is showing the two following entries:

Driver error was [2/1054]: QMYSQL3: Unable to prepare statement Database error was: Unknown column 'offset' in 'field list'

Error preparing query: SELECT chanid, starttime, mark, offset, type FROM recordedmarkup WHERE chanid = :CHANID and starttime = :STARTTIME;

My current version of Mythtv is fixes/025(v0.25.1-49-g65913e7)

Attachments (1)

diff-dump.txt (2.0 KB) - added by blm-ubunet@… 12 years ago.
patch fix incorrect use of offset in recordedmarkup

Download all attachments as: .zip

Change History (10)

comment:1 Changed 13 years ago by J.Pilk@…

I have just created a short dvd iso file using a cutlist. It worked perfectly. I suggest that you experiment with a short recording first. I think that :CHANID and :STARTTIME above should be numeric strings. The logs didn't include the command generated, because there was no error. If you can't make it work seek help on the users list, with info about the kind of file you are trying to encode.

I'm still on 0.24.3 but I don't think that's relevant here.

comment:2 Changed 13 years ago by Chris kapp <chris.kapp@…>

Here is some more information. This is only happening when I use the option "Create Archive" and doesn't appear to cause issues with the archive dvd. My recordedmarkup table shows the following:

Field Type Null Key Default Extra
chanid int(10) unsigned NO PRI 0
starttime datetime NO PRI 0000-00-00 00:00:00
mark mediumint(8) unsigned NO PRI 0
type tinyint(4) NO PRI 0
data int(11) unsigned YES NULL

I also had a conversation in mythtv-users irc earlier today. (http://irc.mythtv.org/ircLog/channel/1/2012-06-30) where stuartm confirmed the field 'offset' was dropped a long time ago.

comment:3 Changed 13 years ago by J.Pilk@…

OK. 'offset' is in recordedseek, in bytes. Cutlist processing in mytharchive uses frame counts anyway. I've never seen a progress log when using mytharchive in 'Create Archive' mode in 0.25. I thought I had opened a ticket about that, but maybe it was just this:

http://www.gossamer-threads.com/lists/mythtv/users/509728#509728

I see that the final section of that post duplicates this ticket :-)

comment:4 Changed 12 years ago by J.Pilk@…

It's trying to copy the recordedmarkup table but has its structure wrong. That explains why the import process doesn't find that table.

It looks as if changes are needed in mytharchivehelper/main.cpp at lines 777, 788, 789 and 1324, 1325, 1329.

I'm not set up to build from source yet. Looking at code for 0.25-fixes.

comment:5 Changed 12 years ago by blm-ubunet@…

Attach patch for 0.25+fixes git. Tested by: Archived a recording with cut-list etc. The generated .xml file now contains correct recordedmarkup & recordedseek data. Previously the recordedmarkup was missing. Deleted original recording & then re-imported from archive. "New" recording has complete metadata, cut-list & seektable.

Changed 12 years ago by blm-ubunet@…

Attachment: diff-dump.txt added

patch fix incorrect use of offset in recordedmarkup

comment:6 Changed 12 years ago by paulh <mythtv@…>

comment:7 Changed 12 years ago by Paul Harrison <mythtv@…>

Owner: set to Paul Harrison <mythtv@…>
Resolution: fixed
Status: newclosed

In 003f3023f2483f88f84df73b41fe2f976bb119d0/mythtv:

MythArchive?: fix the import/export of recordings to the native archive format

  • This updates the native import/export to work with the current DB table fields. Fixes #10871
  • Add storage group support so the FE running MythArchive? no longer needs local access to the recordings. Fixes #8657.

comment:8 Changed 12 years ago by Paul Harrison <mythtv@…>

In 854cb0c3b663d6aa37d9ea4484964bd1c899ec81/mythtv:

MythArchive?: fix the import/export of recordings to the native archive format

  • This updates the native import/export to work with the current DB table fields. Fixes #10871
  • Add storage group support so the FE running MythArchive? no longer needs local access to the recordings. Fixes #8657.

(cherry picked from commit 003f3023f2483f88f84df73b41fe2f976bb119d0)

comment:9 Changed 12 years ago by paulh

Milestone: unknown0.26.1
Owner: changed from Paul Harrison <mythtv@…> to paulh
Note: See TracTickets for help on using tickets.