Improve trigger 'scope' attribute support (RT#119997)
[dbsrgits/SQL-Translator.git] / lib / SQL / Translator / Schema.pm
index d2f9ba4..efa5bda 100644 (file)
@@ -1,25 +1,5 @@
 package SQL::Translator::Schema;
 
-# ----------------------------------------------------------------------
-# $Id: Schema.pm,v 1.17 2004-10-15 02:23:30 allenday Exp $
-# ----------------------------------------------------------------------
-# Copyright (C) 2002-4 SQLFairy Authors
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; version 2.
-#
-# This program is distributed in the hope that it will be useful, but
-# WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-# General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
-# 02111-1307  USA
-# -------------------------------------------------------------------
-
 =pod
 
 =head1 NAME
@@ -29,9 +9,13 @@ SQL::Translator::Schema - SQL::Translator schema object
 =head1 SYNOPSIS
 
   use SQL::Translator::Schema;
-  my $schema = SQL::Translator::Schema->new;
-  my $table  = $schema->add_table( name => 'foo' );
-  my $view   = $schema->add_view( name => 'bar', sql => '...' );
+  my $schema   =  SQL::Translator::Schema->new(
+      name     => 'Foo',
+      database => 'MySQL',
+  );
+  my $table    = $schema->add_table( name => 'foo' );
+  my $view     = $schema->add_view( name => 'bar', sql => '...' );
+
 
 =head1 DESCSIPTION
 
@@ -42,44 +26,63 @@ returns the database structure.
 
 =cut
 
-use strict;
-use Class::Base;
+use Moo;
 use SQL::Translator::Schema::Constants;
 use SQL::Translator::Schema::Procedure;
 use SQL::Translator::Schema::Table;
 use SQL::Translator::Schema::Trigger;
 use SQL::Translator::Schema::View;
-use SQL::Translator::Schema::Graph;
+use Sub::Quote qw(quote_sub);
+
 use SQL::Translator::Utils 'parse_list_arg';
+use Carp;
 
-use base 'Class::Base';
-use vars qw[ $VERSION $TABLE_ORDER $VIEW_ORDER $TRIGGER_ORDER $PROC_ORDER ];
+extends 'SQL::Translator::Schema::Object';
 
-$VERSION = sprintf "%d.%02d", q$Revision: 1.17 $ =~ /(\d+)\.(\d+)/;
+our $VERSION = '1.59';
 
-# ----------------------------------------------------------------------
-sub init {
 
-=pod
+has _order => (is => 'ro', default => quote_sub(q{ +{ map { $_ => 0 } qw/
+    table
+    view
+    trigger
+    proc
+  /} }),
+);
 
-=head2 new
+sub as_graph_pm {
 
-Object constructor.
+=pod
 
-  my $schema   =  SQL::Translator::Schema->new(
-      name     => 'Foo',
-      database => 'MySQL',
-  );
+=head2 as_graph_pm
+
+Returns a Graph::Directed object with the table names for nodes.
 
 =cut
 
-    my ( $self, $config ) = @_;
-    $self->params( $config, qw[ name database parser_args producer_args ] )
-      || return undef;
-    return $self;
+    require Graph::Directed;
+
+    my $self = shift;
+    my $g    = Graph::Directed->new;
+
+    for my $table ( $self->get_tables ) {
+        my $tname  = $table->name;
+        $g->add_vertex( $tname );
+
+        for my $field ( $table->get_fields ) {
+            if ( $field->is_foreign_key ) {
+                my $fktable = $field->foreign_key_reference->reference_table;
+
+                $g->add_edge( $fktable, $tname );
+            }
+        }
+    }
+
+    return $g;
 }
 
-# ----------------------------------------------------------------------
+has _tables => ( is => 'ro', init_arg => undef, default => quote_sub(q{ +{} }) );
+
 sub add_table {
 
 =pod
@@ -88,7 +91,7 @@ sub add_table {
 
 Add a table object.  Returns the new SQL::Translator::Schema::Table object.
 The "name" parameter is required.  If you try to create a table with the
-same name as an existing table, you will get an error and the table will 
+same name as an existing table, you will get an error and the table will
 not be created.
 
   my $t1 = $schema->add_table( name => 'foo' ) or die $schema->error;
@@ -103,41 +106,86 @@ not be created.
 
     if ( UNIVERSAL::isa( $_[0], $table_class ) ) {
         $table = shift;
-        $table->schema( $self );
+        $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 = $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;
 
-    if ( defined $self->{'tables'}{ $table_name } ) {
-        return $self->error(qq[Can't create table: "$table_name" exists]);
+    if ( defined $self->_tables->{$table_name} ) {
+        return $self->error(qq[Can't use table name "$table_name": table exists]);
+    }
+    else {
+        $self->_tables->{$table_name} = $table;
+    }
+
+    return $table;
+}
+
+sub drop_table {
+
+=pod
+
+=head2 drop_table
+
+Remove a table from the schema. Returns the table object if the table was found
+and removed, an error otherwise. The single parameter can be either a table
+name or an C<SQL::Translator::Schema::Table> object. The "cascade" parameter
+can be set to 1 to also drop all triggers on the table, default is 0.
+
+  $schema->drop_table('mytable');
+  $schema->drop_table('mytable', cascade => 1);
+
+=cut
+
+    my $self        = shift;
+    my $table_class = 'SQL::Translator::Schema::Table';
+    my $table_name;
+
+    if ( UNIVERSAL::isa( $_[0], $table_class ) ) {
+        $table_name = shift->name;
     }
     else {
-        $self->{'tables'}{ $table_name } = $table;
+        $table_name = shift;
     }
+    my %args    = @_;
+    my $cascade = $args{'cascade'};
+
+    if ( !exists $self->_tables->{$table_name} ) {
+        return $self->error(qq[Can't drop table: "$table_name" doesn't exist]);
+    }
+
+    my $table = delete $self->_tables->{$table_name};
 
+    if ($cascade) {
+
+        # Drop all triggers on this table
+        $self->drop_trigger()
+          for ( grep { $_->on_table eq $table_name } values %{ $self->_triggers } );
+    }
     return $table;
 }
 
-# ----------------------------------------------------------------------
+has _procedures => ( is => 'ro', init_arg => undef, default => quote_sub(q{ +{} }) );
+
 sub add_procedure {
 
 =pod
 
 =head2 add_procedure
 
-Add a procedure object.  Returns the new
-SQL::Translator::Schema::Procedure object.  The "name" parameter is
-required.  If you try to create a procedure with the same name as an
-existing procedure, you will get an error and the procedure will not
-be created.
+Add a procedure object.  Returns the new SQL::Translator::Schema::Procedure
+object.  The "name" parameter is required.  If you try to create a procedure
+with the same name as an existing procedure, you will get an error and the
+procedure will not be created.
 
   my $p1 = $schema->add_procedure( name => 'foo' );
   my $p2 = SQL::Translator::Schema::Procedure->new( name => 'bar' );
@@ -151,33 +199,69 @@ be created.
 
     if ( UNIVERSAL::isa( $_[0], $procedure_class ) ) {
         $procedure = shift;
-        $procedure->schema( $self );
+        $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 = $procedure_class->new( \%args )
+          or return $self->error( $procedure_class->error );
     }
 
-    $procedure->order( ++$PROC_ORDER );
-    my $procedure_name = $procedure->name or return 
-        $self->error('No procedure name');
+    $procedure->order( ++$self->_order->{proc} );
+    my $procedure_name = $procedure->name
+      or return $self->error('No procedure name');
 
-    if ( defined $self->{'procedures'}{ $procedure_name } ) { 
+    if ( defined $self->_procedures->{$procedure_name} ) {
         return $self->error(
-            qq[Can't create procedure: "$procedure_name" exists]
-        );
+            qq[Can't create procedure: "$procedure_name" exists] );
     }
     else {
-        $self->{'procedures'}{ $procedure_name } = $procedure;
+        $self->_procedures->{$procedure_name} = $procedure;
     }
 
     return $procedure;
 }
 
-# ----------------------------------------------------------------------
+sub drop_procedure {
+
+=pod
+
+=head2 drop_procedure
+
+Remove a procedure from the schema. Returns the procedure object if the
+procedure was found and removed, an error otherwise. The single parameter
+can be either a procedure name or an C<SQL::Translator::Schema::Procedure>
+object.
+
+  $schema->drop_procedure('myprocedure');
+
+=cut
+
+    my $self       = shift;
+    my $proc_class = 'SQL::Translator::Schema::Procedure';
+    my $proc_name;
+
+    if ( UNIVERSAL::isa( $_[0], $proc_class ) ) {
+        $proc_name = shift->name;
+    }
+    else {
+        $proc_name = shift;
+    }
+
+    if ( !exists $self->_procedures->{$proc_name} ) {
+        return $self->error(
+            qq[Can't drop procedure: "$proc_name" doesn't exist]);
+    }
+
+    my $proc = delete $self->_procedures->{$proc_name};
+
+    return $proc;
+}
+
+has _triggers => ( is => 'ro', init_arg => undef, default => quote_sub(q{ +{} }) );
+
 sub add_trigger {
 
 =pod
@@ -186,7 +270,7 @@ sub add_trigger {
 
 Add a trigger object.  Returns the new SQL::Translator::Schema::Trigger object.
 The "name" parameter is required.  If you try to create a trigger with the
-same name as an existing trigger, you will get an error and the trigger will 
+same name as an existing trigger, you will get an error and the trigger will
 not be created.
 
   my $t1 = $schema->add_trigger( name => 'foo' );
@@ -201,30 +285,66 @@ not be created.
 
     if ( UNIVERSAL::isa( $_[0], $trigger_class ) ) {
         $trigger = shift;
-        $trigger->schema( $self );
+        $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 = $trigger_class->new( \%args )
+          or return $self->error( $trigger_class->error );
     }
 
-    $trigger->order( ++$TRIGGER_ORDER );
-    my $trigger_name = $trigger->name or return $self->error('No trigger name');
+    $trigger->order( ++$self->_order->{trigger} );
 
-    if ( defined $self->{'triggers'}{ $trigger_name } ) { 
+    my $trigger_name = $trigger->name or return $self->error('No trigger name');
+    if ( defined $self->_triggers->{$trigger_name} ) {
         return $self->error(qq[Can't create trigger: "$trigger_name" exists]);
     }
     else {
-        $self->{'triggers'}{ $trigger_name } = $trigger;
+        $self->_triggers->{$trigger_name} = $trigger;
+    }
+
+    return $trigger;
+}
+
+sub drop_trigger {
+
+=pod
+
+=head2 drop_trigger
+
+Remove a trigger from the schema. Returns the trigger object if the trigger was
+found and removed, an error otherwise. The single parameter can be either a
+trigger name or an C<SQL::Translator::Schema::Trigger> object.
+
+  $schema->drop_trigger('mytrigger');
+
+=cut
+
+    my $self          = shift;
+    my $trigger_class = 'SQL::Translator::Schema::Trigger';
+    my $trigger_name;
+
+    if ( UNIVERSAL::isa( $_[0], $trigger_class ) ) {
+        $trigger_name = shift->name;
+    }
+    else {
+        $trigger_name = shift;
+    }
+
+    if ( !exists $self->_triggers->{$trigger_name} ) {
+        return $self->error(
+            qq[Can't drop trigger: "$trigger_name" doesn't exist]);
     }
 
+    my $trigger = delete $self->_triggers->{$trigger_name};
+
     return $trigger;
 }
 
-# ----------------------------------------------------------------------
+has _views => ( is => 'ro', init_arg => undef, default => quote_sub(q{ +{} }) );
+
 sub add_view {
 
 =pod
@@ -233,7 +353,7 @@ sub add_view {
 
 Add a view object.  Returns the new SQL::Translator::Schema::View object.
 The "name" parameter is required.  If you try to create a view with the
-same name as an existing view, you will get an error and the view will 
+same name as an existing view, you will get an error and the view will
 not be created.
 
   my $v1 = $schema->add_view( name => 'foo' );
@@ -242,39 +362,68 @@ not be created.
 
 =cut
 
-    my $self        = shift;
+    my $self       = shift;
     my $view_class = 'SQL::Translator::Schema::View';
     my $view;
 
     if ( UNIVERSAL::isa( $_[0], $view_class ) ) {
         $view = shift;
-        $view->schema( $self );
+        $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 } ) { 
+    if ( defined $self->_views->{$view_name} ) {
         return $self->error(qq[Can't create view: "$view_name" exists]);
     }
     else {
-        $self->{'views'}{ $view_name } = $view;
+        $self->_views->{$view_name} = $view;
     }
 
     return $view;
 }
 
-# ----------------------------------------------------------------------
-sub database {
+sub drop_view {
 
 =pod
 
+=head2 drop_view
+
+Remove a view from the schema. Returns the view object if the view was found
+and removed, an error otherwise. The single parameter can be either a view
+name or an C<SQL::Translator::Schema::View> object.
+
+  $schema->drop_view('myview');
+
+=cut
+
+    my $self       = shift;
+    my $view_class = 'SQL::Translator::Schema::View';
+    my $view_name;
+
+    if ( UNIVERSAL::isa( $_[0], $view_class ) ) {
+        $view_name = shift->name;
+    }
+    else {
+        $view_name = shift;
+    }
+
+    if ( !exists $self->_views->{$view_name} ) {
+        return $self->error(qq[Can't drop view: "$view_name" doesn't exist]);
+    }
+
+    my $view = delete $self->_views->{$view_name};
+
+    return $view;
+}
+
 =head2 database
 
 Get or set the schema's database.  (optional)
@@ -283,12 +432,8 @@ Get or set the schema's database.  (optional)
 
 =cut
 
-    my $self = shift;
-    $self->{'database'} = shift if @_;
-    return $self->{'database'} || '';
-}
+has database => ( is => 'rw', default => quote_sub(q{ '' }) );
 
-# ----------------------------------------------------------------------
 sub is_valid {
 
 =pod
@@ -312,7 +457,6 @@ Returns true if all the tables and views are valid.
     return 1;
 }
 
-# ----------------------------------------------------------------------
 sub get_procedure {
 
 =pod
@@ -325,14 +469,13 @@ Returns a procedure by the name provided.
 
 =cut
 
-    my $self       = shift;
+    my $self = shift;
     my $procedure_name = shift or return $self->error('No procedure name');
-    return $self->error( qq[Table "$procedure_name" does not exist] ) unless
-        exists $self->{'procedures'}{ $procedure_name };
-    return $self->{'procedures'}{ $procedure_name };
+    return $self->error(qq[Table "$procedure_name" does not exist])
+      unless exists $self->_procedures->{$procedure_name};
+    return $self->_procedures->{$procedure_name};
 }
 
-# ----------------------------------------------------------------------
 sub get_procedures {
 
 =pod
@@ -345,23 +488,21 @@ Returns all the procedures as an array or array reference.
 
 =cut
 
-    my $self   = shift;
-    my @procedures = 
-        map  { $_->[1] } 
-        sort { $a->[0] <=> $b->[0] } 
-        map  { [ $_->order, $_ ] }
-        values %{ $self->{'procedures'} };
+    my $self       = shift;
+    my @procedures =
+      map  { $_->[1] }
+      sort { $a->[0] <=> $b->[0] }
+      map  { [ $_->order, $_ ] } values %{ $self->_procedures };
 
-    if ( @procedures ) {
+    if (@procedures) {
         return wantarray ? @procedures : \@procedures;
     }
     else {
         $self->error('No procedures');
-        return wantarray ? () : undef;
+        return;
     }
 }
 
-# ----------------------------------------------------------------------
 sub get_table {
 
 =pod
@@ -374,14 +515,21 @@ Returns a table by the name provided.
 
 =cut
 
-    my $self       = shift;
+    my $self = shift;
     my $table_name = shift or return $self->error('No table name');
-    return $self->error( qq[Table "$table_name" does not exist] ) unless
-        exists $self->{'tables'}{ $table_name };
-    return $self->{'tables'}{ $table_name };
+    my $case_insensitive = shift;
+    if ( $case_insensitive ) {
+      $table_name = uc($table_name);
+      foreach my $table ( keys %{$self->_tables} ) {
+         return $self->_tables->{$table} if $table_name eq uc($table);
+      }
+      return $self->error(qq[Table "$table_name" does not exist]);
+    }
+    return $self->error(qq[Table "$table_name" does not exist])
+      unless exists $self->_tables->{$table_name};
+    return $self->_tables->{$table_name};
 }
 
-# ----------------------------------------------------------------------
 sub get_tables {
 
 =pod
@@ -395,22 +543,20 @@ Returns all the tables as an array or array reference.
 =cut
 
     my $self   = shift;
-    my @tables = 
-        map  { $_->[1] } 
-        sort { $a->[0] <=> $b->[0] } 
-        map  { [ $_->order, $_ ] }
-        values %{ $self->{'tables'} };
+    my @tables =
+      map  { $_->[1] }
+      sort { $a->[0] <=> $b->[0] }
+      map  { [ $_->order, $_ ] } values %{ $self->_tables };
 
-    if ( @tables ) {
+    if (@tables) {
         return wantarray ? @tables : \@tables;
     }
     else {
         $self->error('No tables');
-        return wantarray ? () : undef;
+        return;
     }
 }
 
-# ----------------------------------------------------------------------
 sub get_trigger {
 
 =pod
@@ -423,14 +569,13 @@ Returns a trigger by the name provided.
 
 =cut
 
-    my $self       = shift;
+    my $self = shift;
     my $trigger_name = shift or return $self->error('No trigger name');
-    return $self->error( qq[Table "$trigger_name" does not exist] ) unless
-        exists $self->{'triggers'}{ $trigger_name };
-    return $self->{'triggers'}{ $trigger_name };
+    return $self->error(qq[Trigger "$trigger_name" does not exist])
+      unless exists $self->_triggers->{$trigger_name};
+    return $self->_triggers->{$trigger_name};
 }
 
-# ----------------------------------------------------------------------
 sub get_triggers {
 
 =pod
@@ -443,23 +588,21 @@ Returns all the triggers as an array or array reference.
 
 =cut
 
-    my $self   = shift;
-    my @triggers = 
-        map  { $_->[1] } 
-        sort { $a->[0] <=> $b->[0] } 
-        map  { [ $_->order, $_ ] }
-        values %{ $self->{'triggers'} };
+    my $self     = shift;
+    my @triggers =
+      map  { $_->[1] }
+      sort { $a->[0] <=> $b->[0] }
+      map  { [ $_->order, $_ ] } values %{ $self->_triggers };
 
-    if ( @triggers ) {
+    if (@triggers) {
         return wantarray ? @triggers : \@triggers;
     }
     else {
         $self->error('No triggers');
-        return wantarray ? () : undef;
+        return;
     }
 }
 
-# ----------------------------------------------------------------------
 sub get_view {
 
 =pod
@@ -472,14 +615,13 @@ Returns a view by the name provided.
 
 =cut
 
-    my $self      = shift;
+    my $self = shift;
     my $view_name = shift or return $self->error('No view name');
-    return $self->error('View "$view_name" does not exist') unless
-        exists $self->{'views'}{ $view_name };
-    return $self->{'views'}{ $view_name };
+    return $self->error('View "$view_name" does not exist')
+      unless exists $self->_views->{$view_name};
+    return $self->_views->{$view_name};
 }
 
-# ----------------------------------------------------------------------
 sub get_views {
 
 =pod
@@ -493,43 +635,41 @@ Returns all the views as an array or array reference.
 =cut
 
     my $self  = shift;
-    my @views = 
-        map  { $_->[1] } 
-        sort { $a->[0] <=> $b->[0] } 
-        map  { [ $_->order, $_ ] }
-        values %{ $self->{'views'} };
+    my @views =
+      map  { $_->[1] }
+      sort { $a->[0] <=> $b->[0] }
+      map  { [ $_->order, $_ ] } values %{ $self->_views };
 
-    if ( @views ) {
+    if (@views) {
         return wantarray ? @views : \@views;
     }
     else {
         $self->error('No views');
-        return wantarray ? () : undef;
+        return;
     }
 }
 
-# ----------------------------------------------------------------------
 sub make_natural_joins {
 
 =pod
 
 =head2 make_natural_joins
 
-Creates foriegn key relationships among like-named fields in different
+Creates foreign key relationships among like-named fields in different
 tables.  Accepts the following arguments:
 
 =over 4
 
-=item * join_pk_only 
+=item * join_pk_only
 
-A True or False argument which determins whether or not to perform 
+A True or False argument which determines whether or not to perform
 the joins from primary keys to fields of the same name in other tables
 
 =item * skip_fields
 
 A list of fields to skip in the joins
 
-=back 4
+=back
 
   $schema->make_natural_joins(
       join_pk_only => 1,
@@ -541,31 +681,30 @@ A list of fields to skip in the joins
     my $self         = shift;
     my %args         = @_;
     my $join_pk_only = $args{'join_pk_only'} || 0;
-    my %skip_fields  = map { s/^\s+|\s+$//g; $_, 1 } @{ 
-        parse_list_arg( $args{'skip_fields'} ) 
-    };
+    my %skip_fields  =
+      map { s/^\s+|\s+$//g; $_, 1 } @{ parse_list_arg( $args{'skip_fields'} ) };
 
     my ( %common_keys, %pk );
     for my $table ( $self->get_tables ) {
         for my $field ( $table->get_fields ) {
             my $field_name = $field->name or next;
-            next if $skip_fields{ $field_name };
-            $pk{ $field_name } = 1 if $field->is_primary_key;
-            push @{ $common_keys{ $field_name } }, $table->name;
+            next if $skip_fields{$field_name};
+            $pk{$field_name} = 1 if $field->is_primary_key;
+            push @{ $common_keys{$field_name} }, $table->name;
         }
-    } 
-   
+    }
+
     for my $field ( keys %common_keys ) {
-        next if $join_pk_only and !defined $pk{ $field };
+        next if $join_pk_only and !defined $pk{$field};
 
-        my @table_names = @{ $common_keys{ $field } };
+        my @table_names = @{ $common_keys{$field} };
         next unless scalar @table_names > 1;
 
         for my $i ( 0 .. $#table_names ) {
-            my $table1 = $self->get_table( $table_names[ $i ] ) or next;
+            my $table1 = $self->get_table( $table_names[$i] ) or next;
 
             for my $j ( 1 .. $#table_names ) {
-                my $table2 = $self->get_table( $table_names[ $j ] ) or next;
+                my $table2 = $self->get_table( $table_names[$j] ) or next;
                 next if $table1->name eq $table2->name;
 
                 $table1->add_constraint(
@@ -574,18 +713,13 @@ A list of fields to skip in the joins
                     reference_table  => $table2->name,
                     reference_fields => $field,
                 );
-            }               
+            }
         }
-    } 
+    }
 
     return 1;
 }
 
-# ----------------------------------------------------------------------
-sub name {
-
-=pod
-
 =head2 name
 
 Get or set the schema's name.  (optional)
@@ -594,44 +728,25 @@ Get or set the schema's name.  (optional)
 
 =cut
 
-    my $self = shift;
-    $self->{'name'} = shift if @_;
-    return $self->{'name'} || '';
-}
+has name => ( is => 'rw', default => quote_sub(q{ '' }) );
 
-=head2 parser_args
+=pod
 
-=cut
+=head2 translator
 
-sub parser_args {
-    my $self = shift;
-    return $self->{'parser_args'};
-}
-
-=head2 producer_args
+Get the SQL::Translator instance that instantiated the parser.
 
 =cut
 
-sub producer_args {
-    my $self = shift;
-    return $self->{'producer_args'};
-}
-
-# ----------------------------------------------------------------------
-sub DESTROY {
-    my $self = shift;
-    undef $_ for values %{ $self->{'tables'} };
-    undef $_ for values %{ $self->{'views'}  };
-}
+has translator => ( is => 'rw', weak_ref => 1 );
 
 1;
 
-# ----------------------------------------------------------------------
-
 =pod
 
 =head1 AUTHOR
 
-Ken Y. Clark E<lt>kclark@cpan.orgE<gt>
+Ken Youens-Clark E<lt>kclark@cpan.orgE<gt>.
 
 =cut
+