[Zope] Two mysql / timestamp related fixes

[Tino Wildenhain]

Gaute Amundsen schrieb:
> Hi.
> 
> Just had som troubles when I added timestamp columns to a table.
> Commnets?
> 
> 
> my setup:
> Mysql: 4.1.20-log
>  Zope Version: (Zope 2.7.5-1.fc3, python 2.3.4, linux2)
> 
> 
> 1 )
> First I got a ValueError 'query' when using a very simple zsql method.
> (consisting of only "<dtml-var sql >")

Thats a very bad idea. Better make zsql methods
for every query you want to issue instead of
rebuilding your own (obviously not working)
templating for SQL. Quoting is done automatically
when you use <dtml-sqlvar ...>

> (I am not totally sure this is related to the timestamp  column)
> 
> I appled the fix to customDefaultReport.dtml  mentioned in 
> http://mail.zope.org/pipermail/zope-db/2005-June/004219.html
> 
> <         <a 
> href="&dtml-URL;?query_start=&dtml-previous-sequence-start-number;&query=<dtml-var 
> query url_quote>">
> ---
>>         <a 
> href="&dtml-URL;&dtml-sequence-query;query_start=&dtml-previous-sequence-start-number;">
> 25d24
> <          <a 
> href="&dtml-URL;?query_start=&dtml-next-sequence-start-number;&query=<dtml-var 
> query url_quote>">
> 26a26
>>          <a 
> href="&dtml-URL;&dtml-sequence-query;query_start=&dtml-next-sequence-start-number;">
> 
> This does not seem to be exactly the same fix as described in: 
> http://www.zope.org/Collectors/Zope/1812
> 
> Should I have this problem in Zope 2.7.5-1.fc3?
> 
> 
> 2)
> 
> ValueError: invalid literal for int(): 0-  
> When trying to do a "select *" 
> 
> this is definitely connected to the timestamp  column, because this change 
> fixed it:
> 
> diff /usr/lib/zope/lib/python/Products/ZMySQLDA/db.py /usr/lib/zope/lib/python/Products/ZMySQLDA/db.py~
> 173d172
> <     conv[FIELD_TYPE.TIMESTAMP] = DateTime_or_None
> 
> I have found no mention of this anywhere?
> Have anyone else had this problem
> Comments?
> 
> Regards
> 
> Gaute Amundsen
> 
> _______________________________________________
> Zope maillist  -  Zope at zope.org
> http://mail.zope.org/mailman/listinfo/zope
> **   No cross posts or HTML encoding!  **
> (Related lists - 
>  http://mail.zope.org/mailman/listinfo/zope-announce
>  http://mail.zope.org/mailman/listinfo/zope-dev )