Re: patch for contrib/xml2

["Nikolay Samokhvalov" <samokhvalov(at)gmail(dot)com>]

Please wait a little bit, I'll send updated patch (xpath_array() in core)
tonight or tomorrow.

It seems so that there is no actual needs in other xpath-functions...
Just to recall:
- xpath_array() produces xml[] as a result -- so, it's possible to process
then any of returned XML fragments, with proper type casting;
- also xpath_boolean(), xpath_string() and xpath_number() were proposed,
according current functions from contrib/xml2. They are intended to return
scalar value, first found by XPath expression. I saw serious problems with
them -- e.g. it's quite ugly to return always first value of the sequence
(but it might be useful in practice, surely) and, moreover, from my POV,
xpath_string('/a/text()', '<a>very <b>simple</b> example</a>') should
produce 'very simple example' (not 'very ' and not 'very  example') -- due
to XML semantics. This differs xpath_string() from other two...

So, my proposal is to include only one function and forget about that three
until we have a better vision ...

I'll raise other (less important) questions with patch sent, shortly.

On 2/18/07, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> Peter Eisentraut wrote:
> > Bruce Momjian wrote:
> > > Should we revisit xpath_array() for 8.3, or is this all in core now?
> >
> > Not yet, but we are currently discussing how.
>
> Uh, was this handled?
>
> --
>   Bruce Momjian  <bruce(at)momjian(dot)us>          http://momjian.us
>   EnterpriseDB                               http://www.enterprisedb.com
>
>   + If your life is a hard drive, Christ can be your backup. +



--
Best regards,
Nikolay