"operator does not exist: bigint = timestamp with time zone" with Postgres

[dave@…]

I'm not sure what triggers this, or what effect it has, but I'm getting a bunch of these in my trac.log:

2013-03-27 12:21:03,479 Trac[dbhelper] ERROR: There was a problem executing sql:DELETE FROM ticket_change  
             WHERE ticket=%s and author=%s and time=%s and field=%s 
     with parameters:(58138, u'dave', datetime.datetime(2013, 3, 27, 17, 21, 3, 277000, tzinfo=<FixedOffset "UTC" 0:00:00>), 'hours')
Exception:operator does not exist: bigint = timestamp with time zone
LINE 2: ...     WHERE ticket=58138 and author='dave' and time='2013-03-...
                                                             ^
HINT:  No operator matches the given name and argument type(s). You might need to add explicit type casts.
Traceback (most recent call last):
  File "build\bdist.win32\egg\timingandestimationplugin\dbhelper.py", line 59, in execute_in_trans
    @env.with_transaction()
  File "C:\Python27\lib\site-packages\trac-1.0.1-py2.7-win32.egg\trac\db\api.py", line 97, in transaction_wrapper
    fn(ldb)
  File "build\bdist.win32\egg\timingandestimationplugin\dbhelper.py", line 65, in fn
    cur.execute(sql, params)
  File "C:\Python27\lib\site-packages\trac-1.0.1-py2.7-win32.egg\trac\db\util.py", line 65, in execute
    return self.cursor.execute(sql_escape_percent(sql), args)
ProgrammingError: operator does not exist: bigint = timestamp with time zone
LINE 2: ...     WHERE ticket=58138 and author='dave' and time='2013-03-...
                                                             ^
HINT:  No operator matches the given name and argument type(s). You might need to add explicit type casts.

TimingAndEstimationPlugin 0.12 with permissions branch, Trac 1.0.1, PostgreSQL database.

Perhaps this is the fix?

timingandestimationplugin/ticket_daemon.py

@@ -69 +69 @@
 
 def delete_ticket_change( comp, ticket_id, author, change_time, field):
     """ removes a ticket change from the database """
+    if type(change_time) == datetime.datetime:
+        change_time = to_timestamp(change_time)
     sql = """DELETE FROM ticket_change  
              WHERE ticket=%s and author=%s and time=%s and field=%s""" 
     dbhelper.execute_non_query(comp.env, sql, ticket_id, author, change_time, field)

[Ryan J Ollos]

I don't know anything about this change, but I would just add that a more common pattern I see in the Trac codebase is:

if isinstance(change_time, datetime.datetime)

... so I would assume that is the more Pythonic way to do a comparison.

[Russ Tyndall]

(In [12828]) use ensure correct timestamp type in delete_ticket_change re #10978

[Russ Tyndall]

Thanks for the bug report and proposed fix! I am using the non-permissions branch primarily, so I definitely appreciate the help finding / fixing these bugs.

Cheers, Russ

[Russ Tyndall]

(In [12829]) use isinstance instead of checking types eql re #10978