[Bast-commits] r3980 - in branches/DBIx-Class-Schema-Loader/current: . lib/DBIx/Class/Schema/Loader/DBI t

[ilmari at dev.catalyst.perl.org]

Author: ilmari
Date: 2008-01-27 09:01:42 +0000 (Sun, 27 Jan 2008)
New Revision: 3980

Modified:
   branches/DBIx-Class-Schema-Loader/current/Changes
   branches/DBIx-Class-Schema-Loader/current/lib/DBIx/Class/Schema/Loader/DBI/mysql.pm
   branches/DBIx-Class-Schema-Loader/current/t/11mysql_common.t
Log:
Add support for extra => { unsigned => 1 } for MySQL.

Modified: branches/DBIx-Class-Schema-Loader/current/Changes
===================================================================
--- branches/DBIx-Class-Schema-Loader/current/Changes	2008-01-27 08:58:38 UTC (rev 3979)
+++ branches/DBIx-Class-Schema-Loader/current/Changes	2008-01-27 09:01:42 UTC (rev 3980)
@@ -2,6 +2,7 @@
 
 0.04999_01 Not yet released
         - Add support for vendor-specific extra column attributes.
+        - Add support for extra => { unsigned => 1 } for MySQL.
         - Set join_type => 'LEFT OUTER' for nullable foreign keys
           (patch from Bernhard Weißhuhn)
         - Set is_auto_increment for auto-increment columns (RT #31473)

Modified: branches/DBIx-Class-Schema-Loader/current/lib/DBIx/Class/Schema/Loader/DBI/mysql.pm
===================================================================
--- branches/DBIx-Class-Schema-Loader/current/lib/DBIx/Class/Schema/Loader/DBI/mysql.pm	2008-01-27 08:58:38 UTC (rev 3979)
+++ branches/DBIx-Class-Schema-Loader/current/lib/DBIx/Class/Schema/Loader/DBI/mysql.pm	2008-01-27 09:01:42 UTC (rev 3980)
@@ -127,6 +127,17 @@
     return $info->{mysql_is_auto_increment};
 }
 
+sub _column_extra_attr {
+    my ($self, $info) = @_;
+    
+    my $extra_attr;
+    if ($info->{mysql_type_name} =~ /\bunsigned\b/i) {
+        $extra_attr->{unsigned} = 1;
+    }
+
+    return $extra_attr;
+}
+
 =head1 SEE ALSO
 
 L<DBIx::Class::Schema::Loader>, L<DBIx::Class::Schema::Loader::Base>,

Modified: branches/DBIx-Class-Schema-Loader/current/t/11mysql_common.t
===================================================================
--- branches/DBIx-Class-Schema-Loader/current/t/11mysql_common.t	2008-01-27 08:58:38 UTC (rev 3979)
+++ branches/DBIx-Class-Schema-Loader/current/t/11mysql_common.t	2008-01-27 09:01:42 UTC (rev 3980)
@@ -1,6 +1,7 @@
 use strict;
 use lib qw(t/lib);
 use dbixcsl_common_tests;
+use Test::More;
 
 my $dsn         = $ENV{DBICTEST_MYSQL_DSN} || '';
 my $user        = $ENV{DBICTEST_MYSQL_USER} || '';
@@ -19,6 +20,25 @@
     skip_rels        => $test_innodb ? 0 : $skip_rels_msg,
     no_inline_rels   => 1,
     no_implicit_rels => 1,
+    extra            => {
+        create => [
+            qq{
+                CREATE TABLE mysql_loader_test1 (
+                    id INTEGER UNSIGNED NOT NULL PRIMARY KEY
+                )
+            },
+        ],
+        drop   => [ qw/ mysql_loader_test1 / ],
+        count  => 1,
+        run    => sub {
+            my ($schema, $monikers, $classes) = @_;
+        
+            my $rs = $schema->resultset($monikers->{mysql_loader_test1});
+            my $column_info = $rs->result_source->column_info('id');
+        
+            is($column_info->{extra}->{unsigned}, 1, 'Unsigned MySQL columns');
+        },
+    }
 );
 
 if( !$dsn || !$user ) {