With this patch Term Fields fully supports Postgres DB. With out it, the field is added to the term_fields table, but the alter table on term_fields_term fails and nothing really works after that.

+++ term_fields.admin.inc Wed Nov 18 09:55:46 2009
@@ -134,7 +134,7 @@
case 'textfield':
case 'textarea':
case 'select':
- $data_type = 'LONGTEXT';
+ $data_type = 'TEXT';
break;
case 'numeric':
case 'file':

CommentFileSizeAuthor
#2 term_fields_pg_data_type_bug1.patch3.37 KBhass

Comments

hass’s picture

hass commented

This is for sure a bug as it's an unsupported data type (http://drupal.org/node/159605), but needs an update function to correct longtext if used in past.

hass’s picture

hass commented
Status: Active » Needs review
StatusFileSize
new term_fields_pg_data_type_bug1.patch3.37 KB

Ok, there is no update hook required as size => big is longtext on mysql and we are therefore fine. Changed the code to use API functions and the core schema specs. Untested... I'm still waiting for getting the review patch committed and than this one may needs a re-role.

hass’s picture

hass commented
Title: LONGTEXT -> TEXT to allow postgres support » Invalid database schema definitions break postgres support
hass’s picture

hass commented
Status: Needs review » Fixed

Has been committed.

Status: Fixed » Closed (fixed)

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