X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FSQL%2FTranslator%2FSchema.pm;h=6268f219a643410c808123232ecc10e385171ddf;hb=1ced2a25380c28174c07743b19820ec6d1d4f74f;hp=926cf7c212940ce176a1e6d3d0903a1ada1c8d67;hpb=4ab3763d2ad756c236b757306989cafa08e7f35e;p=dbsrgits%2FSQL-Translator.git diff --git a/lib/SQL/Translator/Schema.pm b/lib/SQL/Translator/Schema.pm index 926cf7c..6268f21 100644 --- a/lib/SQL/Translator/Schema.pm +++ b/lib/SQL/Translator/Schema.pm @@ -54,12 +54,27 @@ use SQL::Translator::Schema::View; use SQL::Translator::Utils 'parse_list_arg'; use base 'SQL::Translator::Schema::Object'; -use vars qw[ $VERSION $TABLE_ORDER $VIEW_ORDER $TRIGGER_ORDER $PROC_ORDER ]; +use vars qw[ $VERSION ]; $VERSION = '1.59'; __PACKAGE__->_attributes(qw/name database translator/); +sub new { + my $class = shift; + my $self = $class->SUPER::new (@_) + or return; + + $self->{_order} = { map { $_ => 0 } qw/ + table + view + trigger + proc + /}; + + return $self; +} + # ---------------------------------------------------------------------- sub as_graph { @@ -137,13 +152,13 @@ not be created. $table->schema($self); } else { - my %args = @_; + my %args = ref $_[0] eq 'HASH' ? %{ $_[0] } : @_; $args{'schema'} = $self; $table = $table_class->new( \%args ) or return $self->error( $table_class->error ); } - $table->order( ++$TABLE_ORDER ); + $table->order( ++$self->{_order}{table} ); # We know we have a name as the Table->new above errors if none given. my $table_name = $table->name; @@ -230,14 +245,14 @@ procedure will not be created. $procedure->schema($self); } else { - my %args = @_; + my %args = ref $_[0] eq 'HASH' ? %{ $_[0] } : @_; $args{'schema'} = $self; return $self->error('No procedure name') unless $args{'name'}; $procedure = $procedure_class->new( \%args ) or return $self->error( $procedure_class->error ); } - $procedure->order( ++$PROC_ORDER ); + $procedure->order( ++$self->{_order}{proc} ); my $procedure_name = $procedure->name or return $self->error('No procedure name'); @@ -316,14 +331,14 @@ not be created. $trigger->schema($self); } else { - my %args = @_; + my %args = ref $_[0] eq 'HASH' ? %{ $_[0] } : @_; $args{'schema'} = $self; return $self->error('No trigger name') unless $args{'name'}; $trigger = $trigger_class->new( \%args ) or return $self->error( $trigger_class->error ); } - $trigger->order( ++$TRIGGER_ORDER ); + $trigger->order( ++$self->{_order}{trigger} ); my $trigger_name = $trigger->name or return $self->error('No trigger name'); if ( defined $self->{'triggers'}{$trigger_name} ) { @@ -399,13 +414,13 @@ not be created. $view->schema($self); } else { - my %args = @_; + my %args = ref $_[0] eq 'HASH' ? %{ $_[0] } : @_; $args{'schema'} = $self; return $self->error('No view name') unless $args{'name'}; $view = $view_class->new( \%args ) or return $view_class->error; } - $view->order( ++$VIEW_ORDER ); + $view->order( ++$self->{_order}{view} ); my $view_name = $view->name or return $self->error('No view name'); if ( defined $self->{'views'}{$view_name} ) { @@ -716,7 +731,7 @@ the joins from primary keys to fields of the same name in other tables A list of fields to skip in the joins -=back 4 +=back $schema->make_natural_joins( join_pk_only => 1,