Because the type name is the shortened version and schema module receives the full type name from the DBMS an error is reported by schema module.

"timestamp" is short for "timestamp without time zone".

Patch follows.

CommentFileSizeAuthor
#9 365996_200901291748+1100.patch617 byteswebchick
#1 365996_200901291748+1100.patch617 bytessammys
#1 365996_200901291750+1100-D6.patch606 bytessammys

Comments

sammys’s picture

sammys commented
Status: Active » Needs review
StatusFileSize
new 365996_200901291750+1100-D6.patch606 bytes
new 365996_200901291748+1100.patch617 bytes

Patches are attached. They are fairly trivial so pretty sure it won't take someone long to review it.

You'll find a reference to confirm the types are, in fact, named as i've claimed.

http://www.postgresql.org/docs/8.3/static/datatype-datetime.html#DATATYP...

Status: Needs review » Needs work

The last submitted patch failed testing.

sammys’s picture

sammys commented
Status: Needs work » Needs review

Patch does actually work.

damien tournoud’s picture

damien tournoud commented
Component: database system » postgresql database
Priority: Minor » Normal
Status: Needs review » Reviewed & tested by the community

I'm Damien Tournoud and I endorse this patch ;)

Status: Reviewed & tested by the community » Needs work

The last submitted patch failed testing.

sammys’s picture

sammys commented
Status: Needs work » Needs review

I'd like Gabor to look at it and mark it RTBC.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

That patch is still good to go.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Issue tags: +Quick fix

.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
StatusFileSize
new 365996_200901291748+1100.patch617 bytes

Re-uploading. I can't commit something that testing bot doesn't like, and it looks like it was choking on this earlier.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Version: 7.x-dev » 6.x-dev

Committed to HEAD. Thanks!

Looks like #1 still applies cleanly to D6.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Committed to Drupal 6 too, thanks.

Status: Fixed » Closed (fixed)
Issue tags: -PostgreSQL, -Quick fix

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

Issue tags: +PostgreSQL, +Quick fix

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.