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: | 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)
Change History (10)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
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
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
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
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
Attachment: | diff-dump.txt added |
---|
patch fix incorrect use of offset in recordedmarkup
comment:6 Changed 12 years ago by
This was fixed in my fork in https://github.com/paul-h/mythtv/commit/fcfe61b1bb
comment:7 Changed 12 years ago by
Owner: | set to Paul Harrison <mythtv@…> |
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:9 Changed 12 years ago by
Milestone: | unknown → 0.26.1 |
---|---|
Owner: | changed from Paul Harrison <mythtv@…> to paulh |
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.