Re: PostgreSQL driver for Joomla review

From: Thom Brown <thombrown(at)gmail(dot)com>
To: Alban Hertroys <dalroi(at)solfertje(dot)student(dot)utwente(dot)nl>
Cc: pgsql-php(at)postgresql(dot)org, PGSQL Mailing List <pgsql-general(at)postgresql(dot)org>
Subject: Re: PostgreSQL driver for Joomla review
Date: 2009-10-20 19:00:19
Message-ID: bddc86150910201200s6535464o42bc480ed2eb56b8@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-php

2009/10/20 Alban Hertroys <dalroi(at)solfertje(dot)student(dot)utwente(dot)nl>:
> You miss a keyword in your query in renameTable; it should be " RENAME TO ",
> not " TO ".

Thanks for spotting that. I've made my amendments for next submit.

> Wouldn't it be convenient to have an EXPLAIN ANALYSE version of explain()?
> Maybe with a boolean parameter?

The problem is, I don't see where they're using this function, so if
they're using it to EXPLAIN an INSERT, UPDATE or DELETE statement,
ANALYZE would execute it... unless I roll it back straight after.

> In insertObject() you have a query "SELECT $keyName AS 'id' FROM $table'" -
> That line contains two syntax errors: 'id' (You probably meant "id") and
> $table' (spurious trailing quote).

Again, you're right and well spotted. :) Fixed both of those.

Cheers for helping out Alban! :D

Thom

In response to

Browse pgsql-general by date

  From Date Subject
Next Message Scott Marlowe 2009-10-20 19:23:07 Re: cast numeric with scale and precision to numeric plain
Previous Message Rich Shepard 2009-10-20 18:59:54 Re: multi-company design/structure ?

Browse pgsql-php by date

  From Date Subject
Next Message Csaba Nagy 2009-10-21 08:52:42 Re: PostgreSQL driver for Joomla review
Previous Message Alban Hertroys 2009-10-20 17:29:07 Re: PostgreSQL driver for Joomla review