Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#12458 closed Bug Report - General (fixed)

Python bindings no longer add missing 'videocategory'.'category' during method update() for Video object

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

Description

The traceback points to a bad cursor.execute statement at line 870 of dataheap.py in method 'def _cat_toid(self):' I adjusted the code as follows to correct the errant behavior.

cursor.execute("INSERT INTO videocategory (category) VALUES (%s)", [self.category])

Traceback (most recent call last):

File "<stdin>", line 1, in <module> File "/mnt/data/mythtv/Myth-Rec-to-Vid.py", line 288, in update_vid

self.vid.update()

File "/usr/lib/python2.7/site-packages/MythTV/database.py", line 429, in update

self._push()

File "/usr/lib/python2.7/site-packages/MythTV/dataheap.py", line 885, in _push

self._cat_toid()

File "/usr/lib/python2.7/site-packages/MythTV/dataheap.py", line 870, in _cat_toid

self.category)

File "/usr/lib/python2.7/site-packages/MythTV/_conn_mysqldb.py", line 71, in execute

raise MythDBError(MythDBError.DB_RAW, e.args)

MythTV.exceptions.MythDBError: MySQL error: not all arguments converted during string formatting

Change History (3)

comment:1 Changed 4 years ago by Ben Klopfenstein <benklop@…>

I've submitted a PR with your fix as described here (https://github.com/MythTV/mythtv/pull/99) - i hope that will grease the wheels on getting this integrated. If there's anything I should do, let me know.

comment:2 Changed 4 years ago by Karl Dietz <dekarl@…>

Resolution: fixed
Status: newclosed

In 7a6cddb04ccabc6737671907ff9cf29db26f5347/mythtv:

Merge remote-tracking branch 'benklop/fix-dataheap-binding' into HEAD

Fixes #12458

comment:3 Changed 4 years ago by Karl Egly

Milestone: unknown0.28
Version: UnspecifiedMaster Head
Note: See TracTickets for help on using tickets.