Modify

Opened 3 years ago

Closed 2 years ago

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

Download all attachments as: .zip

Change History (10)

comment:1 Changed 3 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 3 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 3 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 3 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 2 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 2 years ago by blm-ubunet@…

patch fix incorrect use of offset in recordedmarkup

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

comment:7 Changed 2 years 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 2 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 2 years 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 owner will remain paulh.
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.