Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 22 months ago

Closed 13 months ago

Last modified 13 months 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@… 17 months ago.
patch fix incorrect use of offset in recordedmarkup

Download all attachments as: .zip

Change History (10)

comment:1 Changed 22 months 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 22 months 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 22 months 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 22 months 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 17 months 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 17 months ago by blm-ubunet@…

patch fix incorrect use of offset in recordedmarkup

comment:6 Changed 16 months ago by paulh <mythtv@…>

comment:7 Changed 13 months ago by Paul Harrison <mythtv@…>

  • Owner set to Paul Harrison <mythtv@…>
  • Resolution set to fixed
  • Status changed from new to closed

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 13 months 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 13 months ago by paulh

  • Milestone changed from unknown to 0.26.1
  • Owner changed from Paul Harrison <mythtv@…> to paulh

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.