Closed (fixed)
Project:
Page Title
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Nov 2009 at 08:21 UTC
Updated:
13 Jul 2011 at 16:02 UTC
Jump to comment: Most recent file
Comments
Comment #1
valthebaldPeople, please!
The world is not stuck to mysql :)
Please find attached patch against 6.x-2.3
Comment #2
Andy Dorman commentedFWIW, I just applied the patch to a new install using 6.x-2.3 and it works great with PgSQL 8.4
Comment #3
nicholasthompsonThanks guys - I've never used PostgreSQL so have no experience (or system) for testing this.
I have applied a similar fix to that in your patch (I actually just switched the single and double quotes around... Works on MySQL and looks like it achieves the same thing as your patch).
Marking as TBP...
Comment #4
beasley commentedJust tried the dev version on PostgreSQL 8.1.19 and it works fine.
Comment #5
nicholasthompsonThanks Beasley :)
Comment #6
beasley commentedNo problem. Thanks for the really useful module!
Comment #7
john franklin commentedThis patch has been sitting around for a full eight months. Can we please get this into a release soon?
Comment #8
davidwhthomas commentedThis module uses invalid SQL syntax which breaks on postgres.
Basically, quote marks are around the wrong way
should be
In all cases.
In then works fine on mysql and postgres.
Without this, postgres users see constant error messages on screen.
Otherwise, a useful module, thanks.
Comment #9
onewomanbiz commentedWhere / how have you used this patch?
I tried the patch through the PGSQL interface and came up with a pgsql error in the code.
I tried looking at database.pgsql.inc line 139, and dreamweaver'd the whole file looking for erroneous " to change them manually to ', everything is already apostrophed.
I get error node column doesn't exist, time_zone column not found...all basically the same error in line 139 database.pgsql.inc.
Any suggestions?
Comment #10
nicholasthompsonThis is fixed in both D6 and D7 - as far as I know.
Comment #11
valthebaldconfirm - D6 version works with postgres 8.4
Comment #12
nicholasthompsonthank you valthebald :)