package DBIx::DBSchema::Column;
use strict;
-use vars qw(@ISA $VERSION);
-#use Carp;
-#use Exporter;
+use vars qw($VERSION);
+use Carp;
use DBIx::DBSchema::_util qw(_load_driver _dbh);
-#@ISA = qw(Exporter);
-@ISA = qw();
-
-$VERSION = '0.07';
+$VERSION = '0.11';
=head1 NAME
if ( ref($_[0]) ) {
$self = shift;
} else {
+ #carp "Old-style $class creation without named parameters is deprecated!";
+ #croak "FATAL: old-style $class creation no longer supported;".
+ # " use named parameters";
+
$self = { map { $_ => shift } qw(name type null length default local) };
}
my $driver = $dbh ? _load_driver($dbh) : '';
+ ##
+ # type mapping
+ ##
+
my %typemap;
%typemap = eval "\%DBIx::DBSchema::DBD::${driver}::typemap" if $driver;
my $type = defined( $typemap{uc($self->type)} )
? $typemap{uc($self->type)}
: $self->type;
- my $null = $self->null;
+ ##
+ # set default for the callback...
+ ##
my $default;
+ my $orig_default = $self->default;
if ( defined($self->default) && !ref($self->default) && $self->default ne ''
&& ref($dbh)
# false laziness: nicked from FS::Record::_quote
} else {
$default = ref($self->default) ? ${$self->default} : $self->default;
}
+ $self->default($default);
- #this should be a callback into the driver
- if ( $driver eq 'mysql' ) { #yucky mysql hack
- $null ||= "NOT NULL";
- $self->local('AUTO_INCREMENT') if uc($self->type) eq 'SERIAL';
- } elsif ( $driver =~ /^(?:Pg|SQLite)$/ ) { #yucky Pg/SQLite hack
- $null ||= "NOT NULL";
- $null =~ s/^NULL$//;
- }
+ ##
+ # callback into the database-specific driver
+ ##
+
+ my $dbd = "DBIx::DBSchema::DBD::$driver";
+ my $hashref = $dbd->column_callback( $dbh, $self->table_name, $self );
+
+ $self->default($orig_default);
+
+ $type = $hashref->{'effective_type'}
+ if $hashref->{'effective_type'};
+
+ my $null = $self->null;
+
+ #we seem to do this for mysql/Pg/SQLite, i think this should be the default
+ #add something to $hashref if drivers need to overrdide?
+ $null ||= "NOT NULL";
+
+ $null =~ s/^NULL$// unless $hashref->{'explicit_null'};
+
+ $default = $hashref->{'effective_default'}
+ if $hashref->{'effective_default'};
+
+ my $local = $self->local;
+ $local = $hashref->{'effective_local'}
+ if $hashref->{'effective_local'};
+
+ ##
+ # return column line
+ ##
join(' ',
$self->name,
? 'DEFAULT '. $default
: ''
),
- ( ( $driver eq 'mysql' && defined($self->local) )
- ? $self->local
- : ''
- ),
+ ( defined($local) ? $local : ''),
);
}
my $driver = $dbh ? _load_driver($dbh) : '';
- my @after_add = ();
-
- my $real_type = '';
- if ( $driver eq 'Pg' && $self->type eq 'serial' ) {
- $real_type = 'serial';
- $self->type('int');
-
- push @after_add, sub {
- my($table, $column) = @_;
-
- #needs more work for old Pg
-
- my $nextval;
- if ( $dbh->{'pg_server_version'} > 70300 ) {
- $nextval = "nextval('public.${table}_${column}_seq'::text)";
- } else {
- $nextval = "nextval('${table}_${column}_seq'::text)";
- }
-
- (
- "ALTER TABLE $table ALTER COLUMN $column SET DEFAULT $nextval",
- "CREATE SEQUENCE ${table}_${column}_seq",
- "UPDATE $table SET $column = $nextval WHERE $column IS NULL",
- #"ALTER TABLE $table ALTER $column SET NOT NULL",
- );
+ my @sql = ();
+ my $table = $self->table_name;
- };
+ my $dbd = "DBIx::DBSchema::DBD::$driver";
+ my $hashref = $dbd->add_column_callback( $dbh, $table, $self );
+ my $real_type = '';
+ if ( $hashref->{'effective_type'} ) {
+ $real_type = $self->type;
+ $self->type($hashref->{'effective_type'});
}
my $real_null = undef;
- if ( $driver eq 'Pg' && ! $self->null ) {
+ if ( exists($hashref->{'effective_null'}) ) {
$real_null = $self->null;
- $self->null('NULL');
-
- if ( $dbh->{'pg_server_version'} > 70300 ) {
-
- push @after_add, sub {
- my($table, $column) = @_;
- "ALTER TABLE $table ALTER $column SET NOT NULL";
- };
-
- } else {
-
- push @after_add, sub {
- my($table, $column) = @_;
- "UPDATE pg_attribute SET attnotnull = TRUE ".
- " WHERE attname = '$column' ".
- " AND attrelid = ( SELECT oid FROM pg_class WHERE relname = '$table' )";
- };
-
- }
-
+ $self->null($hashref->{'effective_null'});
}
- my @r = ();
- my $table = $self->table_name;
- my $column = $self->name;
+ push @sql, "ALTER TABLE $table ADD COLUMN ". $self->line($dbh);
- push @r, "ALTER TABLE $table ADD COLUMN ". $self->line($dbh);
+ push @sql, @{ $hashref->{'sql_after'} } if $hashref->{'sql_after'};
- push @r, &{$_}($table, $column) foreach @after_add;
-
- push @r, "ALTER TABLE $table ADD PRIMARY KEY ( ".
+ push @sql, "ALTER TABLE $table ADD PRIMARY KEY ( ".
$self->table_obj->primary_key. " )"
if $self->name eq $self->table_obj->primary_key;
$self->type($real_type) if $real_type;
$self->null($real_null) if defined $real_null;
- @r;
+ @sql;
}
my $name = $self->name;
-# my $driver = $dbh ? _load_driver($dbh) : '';
+ my $driver = $dbh ? _load_driver($dbh) : '';
+
+ my @sql = ();
- my @r = ();
+ my $dbd = "DBIx::DBSchema::DBD::$driver";
+ my $hashref = $dbd->alter_column_callback( $dbh, $table, $self, $new );
# change the name...
# change the type...
- # change nullability from NOT NULL to NULL
- if ( ! $self->null && $new->null ) {
- push @r, "ALTER TABLE $table ALTER COLUMN $name DROP NOT NULL";
- }
+ if ( $hashref->{'sql_alter_null' } ) {
- # change nullability from NULL to NOT NULL...
- # this one could be more complicated, need to set a DEFAULT value and update
- # the table first...
- if ( $self->null && ! $new->null ) {
- push @r, "ALTER TABLE $table ALTER COLUMN $name SET NOT NULL";
- }
+ push @sql, $hashref->{'sql_alter_null'};
+ } else {
+
+ # change nullability from NOT NULL to NULL
+ if ( ! $self->null && $new->null ) {
+
+ push @sql, "ALTER TABLE $table ALTER COLUMN $name DROP NOT NULL";
+
+ }
+
+ # change nullability from NULL to NOT NULL...
+ # this one could be more complicated, need to set a DEFAULT value and update
+ # the table first...
+ if ( $self->null && ! $new->null ) {
+
+ push @sql, "ALTER TABLE $table ALTER COLUMN $name SET NOT NULL";
+
+ }
+
+ }
+
# change other stuff...
- @r;
+ @sql;
+
+}
+=item sql_drop_column [ DBH ]
+
+Returns a list of SQL statements to drop this column from an existing table.
+
+The optional database handle or DBI data source/username/password is not yet
+used.
+=cut
+
+sub sql_drop_column {
+ my( $self, $dbh ) = ( shift, _dbh(@_) );
+
+ my $table = $self->table_name;
+ my $name = $self->name;
+
+ ("ALTER TABLE $table DROP COLUMN $name"); # XXX what about indexes???
}
=back
=head1 COPYRIGHT
Copyright (c) 2000-2006 Ivan Kohler
+Copyright (c) 2007 Freeside Internet Services, Inc.
All rights reserved.
This program is free software; you can redistribute it and/or modify it under
the same terms as Perl itself.
=head1 BUGS
+The new() method should warn that
+"Old-style $class creation without named parameters is deprecated!"
+
Better documentation is needed for sql_add_column
line() and sql_add_column() hav database-specific foo that should be abstracted