Opened 14 years ago
Closed 14 years ago
#7648 closed defect (fixed)
New MythTV.py breaks isRecording
Reported by: | 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)
Change History (7)
Changed 14 years ago by
Attachment: | MythTV.py.diffu added |
---|
Changed 14 years ago by
Attachment: | 7648.trunk.diff added |
---|
Changed 14 years ago by
Attachment: | 7648.fixes.diff added |
---|
comment:1 Changed 14 years ago by
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 14 years ago by
I can confirm the patch works on SVN trunk, at least as far as allowing the mythcal script to run.
aaron
comment:3 Changed 14 years ago by
Owner: | changed from Isaac Richards to Raymond Wagner |
---|---|
Status: | new → accepted |
Restore isRecording and getCurrentRecording APIs