Opened 10 years ago

Closed 10 years ago

#7648 closed defect (fixed)

New MythTV.py breaks isRecording

Reported by: finlay@… Owned by: Raymond Wagner
Priority: major Milestone: 0.22
Component: MythTV - General Version: 0.22
Severity: high Keywords:
Cc: Ticket locked: no

Description

The 0.22+ version of the pytohn bindings in MythTV.py the isRecording and getCurrentRecording APIs were changed and do not accept the recorder value as an int. The earlier bindings were able to accept a recorder value that was an int. This looks like it's due to the change to use the BACKEND_SEP value instead of hardcoding it. Since getRecorderList returns a list of recorders as ints it would seem logical to pass an int value to isRecording.

This bug breaks many programs that use these APIs.

I will attach a patch to fix the problem.

Attachments (3)

MythTV.py.diffu (806 bytes) - added by finlay@… 10 years ago.
Restore isRecording and getCurrentRecording APIs
7648.trunk.diff (846 bytes) - added by Raymond Wagner <raymond@…> 10 years ago.
7648.fixes.diff (846 bytes) - added by Raymond Wagner <raymond@…> 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by finlay@…

Attachment: MythTV.py.diffu added

Restore isRecording and getCurrentRecording APIs

Changed 10 years ago by Raymond Wagner <raymond@…>

Attachment: 7648.trunk.diff added

Changed 10 years ago by Raymond Wagner <raymond@…>

Attachment: 7648.fixes.diff added

comment:1 Changed 10 years ago by Raymond Wagner <raymond@…>

Patches for -fixes and trunk. I altered the way I dealt with 'BACKEND_SEP' to make it cleaner for long sequences, but in a couple instances like this, I didn't make sure to keep type consistent.

comment:2 Changed 10 years ago by memoryguy@…

I can confirm the patch works on SVN trunk, at least as far as allowing the mythcal script to run.

aaron

comment:3 Changed 10 years ago by Raymond Wagner

Owner: changed from Isaac Richards to Raymond Wagner
Status: newaccepted

comment:4 Changed 10 years ago by Raymond Wagner

Resolution: fixed
Status: acceptedclosed

Fixed in [23117].

Note: See TracTickets for help on using tickets.