From khaelin at gmail.com  Sat Jul 31 15:06:19 2010
From: khaelin at gmail.com (Nicolas Martyanoff)
Date: Sat, 31 Jul 2010 17:06:19 +0200
Subject: [postmodern-devel] Error with the :PLIST result style
Message-ID: <20100731170619.35c1a18e@valhala.home>


Hi,

I tried using the :PLIST result style with the following query:

(postmodern:query (:select '* :from 'accounts) :plist)

Infortunately, it doesn't work because this form expands to:

(T
 (CL-POSTGRES:EXEC-QUERY POSTMODERN:*DATABASE*
                         (S-SQL:SQL (:SELECT '* :FROM 'ACCOUNTS))
                         'POSTMODERN::SYMBOL-PLIST-ROW-READER))

Where T isn't a function.

Using other result styles works as intended.


The source of the problem seems to be that in *RESULT-STYLES*
(postmodern/query.lisp), :PLIST and :PLISTS are respectively associated
with T and NIL, while all other result styles use SINGLE-ROW or
ALL-ROWS.

Am I doing something wrong or is this really a bug ?

Regards,

-- 
Nicolas Martyanoff
   http://codemore.org
   khaelin at gmail.com



From marijnh at gmail.com  Sat Jul 31 17:27:49 2010
From: marijnh at gmail.com (Marijn Haverbeke)
Date: Sat, 31 Jul 2010 19:27:49 +0200
Subject: [postmodern-devel] Error with the :PLIST result style
In-Reply-To: <20100731170619.35c1a18e@valhala.home>
References: <20100731170619.35c1a18e@valhala.home>
Message-ID: <AANLkTikqFZ8rxmek9csWtOwXM+Wuh3ENtdpKo0D9q_uX@mail.gmail.com>

Hi Nicolas,

This has been fixed a while ago in git. It had been caused by a bad
merge. Checkout the repository to get the fixed version. (I'll
probably do a new release soon, although there isn't much new stuff to
release.)

Best,
Marijn



From khaelin at gmail.com  Sat Jul 31 19:33:34 2010
From: khaelin at gmail.com (Nicolas Martyanoff)
Date: Sat, 31 Jul 2010 21:33:34 +0200
Subject: [postmodern-devel] Error with the :PLIST result style
In-Reply-To: <AANLkTikqFZ8rxmek9csWtOwXM+Wuh3ENtdpKo0D9q_uX@mail.gmail.com>
References: <20100731170619.35c1a18e@valhala.home>
	<AANLkTikqFZ8rxmek9csWtOwXM+Wuh3ENtdpKo0D9q_uX@mail.gmail.com>
Message-ID: <20100731213334.0eaa3043@valhala.home>

On Sat, 31 Jul 2010 19:27:49 +0200
Marijn Haverbeke <marijnh at gmail.com> wrote:

> This has been fixed a while ago in git. It had been caused by a bad
> merge. Checkout the repository to get the fixed version. (I'll
> probably do a new release soon, although there isn't much new stuff to
> release.)

I just tested with the last git version, works for me.

Thank you for your help :)

Regards,

-- 
Nicolas Martyanoff
   http://codemore.org
   khaelin at gmail.com