[z3-checkins] Re: r6442 - z3/sqlos/trunk

[Joseph Kocherhans]

dreamcatcher at codespeak.net wrote:
> Author: dreamcatcher
> Date: Tue Sep 14 02:10:20 2004
> New Revision: 6442
> 
> Modified:
>    z3/sqlos/trunk/__init__.py
> Log:
> Those were swapped.
> 
> Modified: z3/sqlos/trunk/__init__.py
> ==============================================================================
> --- z3/sqlos/trunk/__init__.py	(original)
> +++ z3/sqlos/trunk/__init__.py	Tue Sep 14 02:10:20 2004
> @@ -46,12 +46,12 @@
>  
>  
>  def DateTimeConverter(value, db=None):
> -    return repr(value.isoformat())
> +    return repr(value.date().isoformat())
>  
>  registerConverter(datetime, DateTimeConverter)
>  
>  def DateConverter(value, db=None):
> -    return repr(value.date().isoformat())
> +    return repr(value.isoformat())
>  
>  registerConverter(date, DateConverter)

This is the way things were originally and I asked you to switch them 
(which you did) in r6064. To my understanding the code is broken this 
way. The DateTimeConverter will strip off any time values. Is that how 
it's meant to work?

Joseph