Re: [PATCHES] to_char and i18n - Mailing list pgsql-hackers

From Karel Zak
Subject Re: [PATCHES] to_char and i18n
Date
Msg-id 1135621589.9937.8.camel@petra
Whole thread Raw
In response to Re: to_char and i18n  (Euler Taveira de Oliveira <eulerto@yahoo.com.br>)
Responses Re: [PATCHES] to_char and i18n  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-hackers
On Sun, 2005-12-25 at 17:56 -0300, Euler Taveira de Oliveira wrote:
> --- Euler Taveira de Oliveira <eulerto@yahoo.com.br> escreveu:
>
> > I have a patch like this. But this was for 7.4.x. I have to take a
> > look
> > at it.
> >
> The patch is attached. It implements day and month i18n. I fixed a few
> misspelling comments. Docs is attached too.
>
> template1=# select to_char(now(), 'Day, DD Month YYYY');
>            to_char
> ------------------------------
>  Sunday   , 25 December  2005
> (1 registro)
>
> template1=# select to_char(now(), 'TMDay, DD TMMonth YYYY');
>           to_char
> ---------------------------
>  Domingo, 25 Dezembro 2005
> (1 registro)
>
> template1=#
>
>
> Comments?

I think it looks like a good patch. There's small problem that the
current to_char() output is possible use as argument for to_timestamp()
or to_date() function. It means you should implement vice-versa
conversion from string with TMMonth/TMDay to timestamp.

   to_timestamp('Domingo, 25 Dezembro 2005', 'TMDay, DD TMMonth YYYY')

Or.. at least describe in the docs that this way is unsupported
for 'TM' prefix.

    Karel

--
Karel Zak <zakkr@zf.jcu.cz>


pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: [PATCHES] default resource limits
Next
From: Tom Lane
Date:
Subject: Re: [DOCS] Online backup vs Continuous backup