Modify

Ticket #10871 (closed Bug Report - General: fixed)

Opened 11 months ago

Last modified 2 months ago

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

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

Change History

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

patch fix incorrect use of offset in recordedmarkup

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

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

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

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

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

Add a 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.