? sites/default/files
? sites/default/settings.php
Index: includes/database/pgsql/schema.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/database/pgsql/schema.inc,v
retrieving revision 1.17
diff -u -p -r1.17 schema.inc
--- includes/database/pgsql/schema.inc	4 Aug 2009 06:32:17 -0000	1.17
+++ includes/database/pgsql/schema.inc	7 Aug 2009 10:59:06 -0000
@@ -143,9 +143,6 @@ class DatabaseSchema_pgsql extends Datab
     if ($spec['type'] == 'serial') {
       unset($spec['not null']);
     }
-    if (!empty($spec['unsigned'])) {
-      $sql .= " CHECK ($name >= 0)";
-    }
 
     if (in_array($spec['type'], array('varchar', 'char', 'text')) && isset($spec['length'])) {
       $sql .= '(' . $spec['length'] . ')';
@@ -154,6 +151,10 @@ class DatabaseSchema_pgsql extends Datab
       $sql .= '(' . $spec['precision'] . ', ' . $spec['scale'] . ')';
     }
 
+    if (!empty($spec['unsigned'])) {
+      $sql .= " CHECK ($name >= 0)";
+    }
+
     if (isset($spec['not null']) && $spec['not null']) {
       $sql .= ' NOT NULL';
     }
Index: modules/simpletest/tests/schema.test
===================================================================
RCS file: /cvs/drupal/drupal/modules/simpletest/tests/schema.test,v
retrieving revision 1.8
diff -u -p -r1.8 schema.test
--- modules/simpletest/tests/schema.test	13 Jul 2009 21:51:41 -0000	1.8
+++ modules/simpletest/tests/schema.test	7 Aug 2009 10:59:06 -0000
@@ -129,4 +129,58 @@ class SchemaTestCase extends DrupalWebTe
       $this->assertEqual($comment, $description, t('The comment matches the schema description.'));
     }
   }
+  
+  /**
+   * Tests creating unsigned columns and data integrity thereof.
+   */
+  function testUnsignedColumns() {
+    // First create the table with just a serial column.
+    $table_name = 'unsigned_table';
+    $table_spec = array(
+      'fields' => array('serial_column' => array('type' => 'serial', 'unsigned' => TRUE, 'not null' => TRUE)),
+      'primary key' => array('serial_column'),
+    );
+    $ret = array();
+    db_create_table($ret, $table_name, $table_spec);
+    
+    // Now set up columns for the other types.
+    $types = array('int', 'float', 'numeric');
+    foreach ($types as $type) {
+      $column_spec = array('type' => $type, 'unsigned'=> TRUE);
+      if ($type == 'numeric') {
+        $column_spec += array('precision' => 10, 'scale' => 0);
+      }
+      $column_name = $type . '_column';
+      $table_spec['fields'][$column_name] = $column_spec;
+      db_add_field($ret, $table_name, $column_name, $column_spec);
+    }
+    
+    // Finally, check each column and try to insert invalid values into them.
+    foreach($table_spec['fields'] as $column_name => $column_spec) {
+      $this->assertTrue(db_column_exists($table_name, $column_name), t('Unsigned @type column was created.', array('@type' => $column_spec['type'])));
+      $this->assertFalse($this->tryUnsignedInsert($table_name, $column_name), t('Unsigned @type column rejected a negative value.', array('@type' => $column_spec['type'])));
+    }
+  }
+  
+  /**
+   * Tries to insert a negative value into columns defined as unsigned.
+   *
+   * @param $table_name 
+   *   The table to insert
+   * @param $column_name 
+   *   The column to insert
+   * @return 
+   *   TRUE if the insert succeeded, FALSE otherwise
+   */
+  function tryUnsignedInsert($table_name, $column_name) {
+    try {
+      db_insert($table_name)
+         ->fields(array($column_name => -1))
+         ->execute();
+      return TRUE;
+    }
+    catch (Exception $e) {
+      return FALSE;
+    }
+  }
 }