Opened 12 years ago

Closed 11 years ago

Last modified 10 years ago

#12260 closed Patch - Bug Fix (fixed)

Patch: Making recordedid the primary key for recorded breaks the python bindings

Reported by: doug@… Owned by: Raymond Wagner
Priority: minor Milestone: 0.28
Component: Bindings - Python Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

There are a number of scripts that depend on the Recorded class taking the chanid/starttime instead of the recordedid as the primary key. This patch updates the Recorded class to continue using the chanid/starttime pair. A future patch can add recordedid as a second option so as not to break backwards compatibility.

Attachments (2)

dataheap.patch (685 bytes ) - added by doug@… 12 years ago.
0001-Fix-python-bindings-for-Recorded-Seek-Mark.patch (1.1 KB ) - added by angela.schmid@… 11 years ago.

Download all attachments as: .zip

Change History (10)

by doug@…, 12 years ago

Attachment: dataheap.patch added

comment:1 by Raymond Wagner, 12 years ago

Milestone: unknown0.28
Status: newaccepted
Version: UnspecifiedMaster Head

by angela.schmid@…, 11 years ago

comment:2 by angela.schmid@…, 11 years ago

Too bad this has not been fixed for 0.28. I had problems with Lossless Cut.

The attached fix still uses the recordedid as the primary key for recorded. Instead it set the attributes to be used for recordedseek and recordedmarkup.

comment:3 by angelaschmid <angela.schmid@…>, 11 years ago

Resolution: fixed
Status: acceptedclosed

In fed010634249265550383d45e1d95ebde8194639/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:4 by Roger Siddons <rsiddons@…>, 10 years ago

In ce1935b0ebc0918391f05893be82cd275cbd3969/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:5 by Roger Siddons <rsiddons@…>, 10 years ago

In 2ead1a6c9ebc7c39d868c6baeb5f209cd937c087/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:6 by john.sweeney@…, 10 years ago

I am still getting an error related to this problem:

File "/usr/lib/python2.7/dist-packages/MythTV/dataheap.py", line 332, in init

DBDataWrite.init(self, data, db)

@------------- @ data = [1034, datetime(2016, 7, 28, 11, 30, tzinfo=<MythTV.utility.dt.offsettzinfo object at 0x7fdea8fecc90>)] @ self = <Unknown due to: 'NoneType' object has no attribute 'isoformat'> @ db = <MythDB 'sql://mythconverg@bondo:3306/' at 0x7fdea8b60610>

I am using silence.py, and after an OS upgrade this problem has appeared.

comment:7 by mark@…, 10 years ago

Sorry folks but has this problem been fixed,.. and if so what is the recommended solution as the wiki claims silence.py works with 0.28.... I have read stuff about appending "/" to folder names,..already present and updating a python bindings library,. and mine is already up to date... So I am a little stumped as to how to fix silence. many tx

in reply to:  7 comment:8 by mark@…, 10 years ago

Replying to mark@…:

Sorry folks but has this problem been fixed,.. and if so what is the recommended solution as the wiki claims silence.py works with 0.28.... I have read stuff about appending "/" to folder names,..already present and updating a python bindings library,. and mine is already up to date... So I am a little stumped as to how to fix silence. many tx

update,.. Yes issue is fixed,.. add PPA fixes 0.28 and update/upgrade... ( I had Kernel issues which messed all updates including my fixes0.28,.. and confused me )

Note: See TracTickets for help on using tickets.