this is popping up all over the place. Postgres (at least as far as 8.3) does not have unsigned types. In http://drupal.org/node/109060, it appears that the Drupal core puts in some new domains for unsigned integers, but not for floating-point types. So we need to add this domain:

CREATE DOMAIN real_unsigned
AS real
CONSTRAINT real_unsigned_check CHECK ((VALUE >= 0));
ALTER DOMAIN real_unsigned OWNER TO drupal;

CommentFileSizeAuthor
#4 20090203_unsigned.txt1.73 KBVinceW
#2 float_change.patch7.15 KBcha0s

Comments

cha0s’s picture

cha0s commented
Assigned: Unassigned » cha0s
Status: Active » Postponed (maintainer needs more info)

Could you provide steps to reproduce?

EDIT: nevermind, I just got my server fully set up for pgsql, merely enabling the modules threw a bunch of those errors. =) Thanks...

cha0s’s picture

cha0s commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new float_change.patch7.15 KB

We talked and decided it was better to implement the proper validation where necessary and change our floats to signed. Here's a patch reflecting that...

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed

Excellent. Committed.

VinceW’s picture

VinceW commented
Status: Fixed » Closed (won't fix)
StatusFileSize
new 20090203_unsigned.txt1.73 KB

LS,

I'm having this problem a few weeks now. After reading this thread i even tried a fresh install with:

Postgres 8.3.1 (client_encoding = 'UTF8')
Drupal 6.9
Ubercart 6.x-2.0-beta3 (uc_order.install,v 1.9.2.7 2009/01/19 20:31:26)

Did not work (for me). Guess i may be doing something wrong.

Ubercart CA did install correctly, but when installing uc_orders i got a bunch of pg_ warnings.

Hope someone can point me a direction to solve this.

Regards,
VinceW

rszrama’s picture

rszrama commented
Status: Closed (won't fix) » Fixed

Ahh, yeah, this fix isn't in the beta3. It's in the latest development code available from our Bazaar repository for D6. On a clean beta3 you should be able to apply the patch from #2, tho.

VinceW’s picture

VinceW commented

LS,

Patch applied. Ubercart core-modules installed on Postgresql 8.3.1 without any errors.

When applying the patch, i noticed in: uc_taxes/uc_taxes.install another 'unsigned' (on line 16).

Perhaps not relevant, or forgotten. Thought i let you know.

tnx for pointing out.

VinceW

Island Usurper’s picture

Island Usurper commented

That's for the serial column, and Drupal tells Postgres how to handle that. This patch is only to remove the unsigned flag from float or real columns.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.