Added "show_warnings" and "add_drop_table" options to sql_translator.pl and
[dbsrgits/SQL-Translator.git] / lib / SQL / Translator.pm
index 00c2dea..872f53b 100644 (file)
@@ -1,9 +1,9 @@
 package SQL::Translator;
 
 # ----------------------------------------------------------------------
-# $Id: Translator.pm,v 1.4 2002-03-21 18:50:53 dlc Exp $
+# $Id: Translator.pm,v 1.14 2002-11-26 03:59:57 kycl4rk Exp $
 # ----------------------------------------------------------------------
-# Copyright (C) 2002 Ken Y. Clark <kycl4rk@users.sourceforge.net>,
+# Copyright (C) 2002 Ken Y. Clark <kclark@cpan.org>,
 #                    darren chamberlain <darren@cpan.org>
 #
 # This program is free software; you can redistribute it and/or
@@ -28,37 +28,49 @@ SQL::Translator - convert schema from one database to another
 =head1 SYNOPSIS
 
   use SQL::Translator;
-  my $translator = SQL::Translator->new;
 
-  my $output = $translator->translate(
-                      from     => "MySQL",
-                      to       => "Oracle",
-                      filename => $file,
-               ) or die $translator->error;
+  my $translator     = SQL::Translator->new(
+      xlate          => $xlate || {},    # Overrides for field translation
+      debug          => $debug,          # Print debug info
+      trace          => $trace,          # Print Parse::RecDescent trace
+      no_comments    => $no_comments,    # Don't include comments in output
+      show_warnings  => $show_warnings,  # Print name mutations, conflicts
+      add_drop_table => $add_drop_table, # Add "drop table" statements
+  );
+
+  my $output     = $translator->translate(
+      from       => "MySQL",
+      to         => "Oracle",
+      filename   => $file,
+  ) or die $translator->error;
+
   print $output;
 
 =head1 DESCRIPTION
 
 This module attempts to simplify the task of converting one database
-create syntax to another through the use of Parsers and Producers.
-The idea is that any Parser can be used with any Producer in the
-conversion process.  So, if you wanted PostgreSQL-to-Oracle, you would
-use the PostgreSQL parser and the Oracle producer.
-
-Currently, the existing parsers use Parse::RecDescent, but this not
-a requirement, or even a recommendation.  New parser modules don't
-necessarily have to use Parse::RecDescent, as long as the module
-implements the appropriate API.  With this separation of code, it is
-hoped that developers will find it easy to add more database dialects
-by using what's written, writing only what they need, and then
-contributing their parsers or producers back to the project.
+create syntax to another through the use of Parsers (which understand
+the sourced format) and Producers (which understand the destination
+format).  The idea is that any Parser can be used with any Producer in
+the conversion process.  So, if you wanted PostgreSQL-to-Oracle, you
+would use the PostgreSQL parser and the Oracle producer.
 
 =cut
 
 use strict;
-use vars qw($VERSION $DEFAULT_SUB $DEBUG);
-$VERSION = sprintf "%d.%02d", q$Revision: 1.4 $ =~ /(\d+)\.(\d+)/;
-$DEBUG = 1 unless defined $DEBUG;
+use vars qw( $VERSION $REVISION $DEFAULT_SUB $DEBUG $ERROR );
+use base 'Class::Base';
+
+$VERSION  = '0.01';
+$REVISION = sprintf "%d.%02d", q$Revision: 1.14 $ =~ /(\d+)\.(\d+)/;
+$DEBUG    = 0 unless defined $DEBUG;
+$ERROR    = "";
+
+use Carp qw(carp);
+
+use File::Spec::Functions qw(catfile);
+use File::Basename qw(dirname);
+use IO::Dir;
 
 # ----------------------------------------------------------------------
 # The default behavior is to "pass through" values (note that the
@@ -67,10 +79,6 @@ $DEBUG = 1 unless defined $DEBUG;
 # ----------------------------------------------------------------------
 $DEFAULT_SUB = sub { $_[1] } unless defined $DEFAULT_SUB;
 
-*isa = \&UNIVERSAL::isa;
-
-use Carp qw(carp);
-
 =head1 CONSTRUCTOR
 
 The constructor is called B<new>, and accepts a optional hash of options.
@@ -80,9 +88,17 @@ Valid options are:
 
 =item parser (aka from)
 
+=item parser_args
+
 =item producer (aka to)
 
-=item filename
+=item producer_args
+
+=item filename (aka file)
+
+=item data
+
+=item debug
 
 =back
 
@@ -92,9 +108,8 @@ advantage is gained by passing options to the constructor.
 
 =cut
 
-# {{{ new
 # ----------------------------------------------------------------------
-# new([ARGS])
+# init([ARGS])
 #   The constructor.
 #
 #   new takes an optional hash of arguments.  These arguments may
@@ -105,37 +120,114 @@ advantage is gained by passing options to the constructor.
 #   given directly to the parser or producer methods, respectively.
 #   See the appropriate method description below for details about
 #   what each expects/accepts.
-#
-#   TODO
-#     * Support passing an input (filename or string) as with
-#       translate
 # ----------------------------------------------------------------------
-sub new {
-    my $class = shift;
-    my $args  = isa($_[0], 'HASH') ? shift : { @_ };
-    my $self  = bless { } => $class;
+sub init {
+    my ( $self, $config ) = @_;
 
-    # ------------------------------------------------------------------
+    #
     # Set the parser and producer.
     #
     # If a 'parser' or 'from' parameter is passed in, use that as the
     # parser; if a 'producer' or 'to' parameter is passed in, use that
     # as the producer; both default to $DEFAULT_SUB.
-    # ------------------------------------------------------------------
-    $self->parser(  $args->{'parser'}   || $args->{'from'} || $DEFAULT_SUB);
-    $self->producer($args->{'producer'} || $args->{'to'}   || $DEFAULT_SUB);
+    #
+    $self->parser  ($config->{'parser'}   || $config->{'from'} || $DEFAULT_SUB);
+    $self->producer($config->{'producer'} || $config->{'to'}   || $DEFAULT_SUB);
+
+    #
+    # Set the parser_args and producer_args
+    #
+    for my $pargs ( qw[ parser_args producer_args ] ) {
+        $self->$pargs( $config->{$pargs} ) if defined $config->{ $pargs };
+    }
+
+    #
+    # Set the data source, if 'filename' or 'file' is provided.
+    #
+    $config->{'filename'} ||= $config->{'file'} || "";
+    $self->filename( $config->{'filename'} ) if $config->{'filename'};
+
+    #
+    # Finally, if there is a 'data' parameter, use that in 
+    # preference to filename and file
+    #
+    if ( my $data = $config->{'data'} ) {
+        $self->data( $data );
+    }
+
+    #
+    # Set various other options.
+    #
+    $self->{'debug'} = defined $config->{'debug'} ? $config->{'debug'} : $DEBUG;
+
 
-    # ------------------------------------------------------------------
-    # Clear the error
-    # ------------------------------------------------------------------
-    $self->error_out("");
+    $self->add_drop_table( $config->{'add_drop_table'} );
+    
+    $self->custom_translate( $config->{'xlate'} );
+
+    $self->no_comments( $config->{'no_comments'} );
+
+    $self->show_warnings( $config->{'show_warnings'} );
+
+    $self->trace( $config->{'trace'} );
 
     return $self;
 }
-# }}}
 
 =head1 METHODS
 
+# ----------------------------------------------------------------------
+=head2 B<add_drop_table>
+
+Toggles whether or not to add "DROP TABLE" statements just before the 
+create definitions.
+
+=cut
+
+sub add_drop_table {
+    my $self = shift;
+    if ( defined (my $arg = shift) ) {
+        $self->{'add_drop_table'} = $arg ? 1 : 0;
+    }
+    return $self->{'add_drop_table'} || 0;
+}
+
+
+# ----------------------------------------------------------------------
+=head2 B<custom_translate>
+
+Allows the user to override default translation of fields.  For example,
+if a MySQL "text" field would normally be converted to a "long" for Oracle,
+the user could specify to change it to a "CLOB."  Accepts a hashref where
+keys are the "from" value and values are the "to," returns the current
+value of the field.
+
+=cut
+
+sub custom_translate {
+    my $self = shift;
+    $self->{'custom_translate'} = shift if @_;
+    return $self->{'custom_translate'} || {};
+}
+
+# ----------------------------------------------------------------------
+=head2 B<no_comments>
+
+Toggles whether to print comments in the output.  Accepts a true or false
+value, returns the current value.
+
+=cut
+
+sub no_comments {
+    my $self = shift;
+    my $arg  = shift;
+    if ( defined $arg ) {
+        $self->{'no_comments'} = $arg ? 1 : 0;
+    }
+    return $self->{'no_comments'} || 0;
+}
+
+# ----------------------------------------------------------------------
 =head2 B<producer>
 
 The B<producer> method is an accessor/mutator, used to retrieve or
@@ -173,7 +265,7 @@ instead of the default "produce".
   # This will invoke My::Groovy::Producer::transmogrify($tr, $data),
   # assuming that My::Groovy::Producer::transmogrify is not a module
   # on disk.
-  # $tr->producer("My::Groovy::Producer::transmogrify);
+  $tr->producer("My::Groovy::Producer::transmogrify");
 
   # This will invoke the referenced subroutine directly, as
   # $subref->($tr, $data);
@@ -184,29 +276,48 @@ containing the classname to which the above B<produce> function
 belongs.  In the case of anonymous subroutines, this method returns
 the string "CODE".
 
+Finally, there is a method named B<producer_args>, which is both an
+accessor and a mutator.  Arbitrary data may be stored in name => value
+pairs for the producer subroutine to access:
+
+  sub My::Random::producer {
+      my ($tr, $data) = @_;
+      my $pr_args = $tr->producer_args();
+
+      # $pr_args is a hashref.
+
+Extra data passed to the B<producer> method is passed to
+B<producer_args>:
+
+  $tr->producer("xSV", delimiter => ',\s*');
+
+  # In SQL::Translator::Producer::xSV:
+  my $args = $tr->producer_args;
+  my $delimiter = $args->{'delimiter'}; # value is ,\s*
+
 =cut
 
-# {{{ producer and producer_type
+# producer and producer_type
 sub producer {
     my $self = shift;
 
-    # {{{ producer as a mutator
+    # producer as a mutator
     if (@_) {
         my $producer = shift;
 
-        # {{{ Passed a module name (string containing "::")
+        # Passed a module name (string containing "::")
         if ($producer =~ /::/) {
             my $func_name;
 
-            # {{{ Module name was passed directly
+            # Module name was passed directly
             # We try to load the name; if it doesn't load, there's
             # a possibility that it has a function name attached to
             # it.
             if (load($producer)) {
                 $func_name = "produce";
-            } # }}}
+            } 
 
-            # {{{ Module::function was passed
+            # Module::function was passed
             else {
                 # Passed Module::Name::function; try to recover
                 my @func_parts = split /::/, $producer;
@@ -216,41 +327,82 @@ sub producer {
                 # If this doesn't work, then we have a legitimate
                 # problem.
                 load($producer) or die "Can't load $producer: $@";
-            } # }}}
+            }
 
-            # {{{ get code reference and assign
+            # get code reference and assign
             $self->{'producer'} = \&{ "$producer\::$func_name" };
             $self->{'producer_type'} = $producer;
-            $self->debug("Got 'producer': $producer\::$func_name");
-            # }}}
-        } # }}}
+            $self->debug("Got producer: $producer\::$func_name\n");
+        } 
 
-        # {{{ passed an anonymous subroutine reference
+        # passed an anonymous subroutine reference
         elsif (isa($producer, 'CODE')) {
             $self->{'producer'} = $producer;
             $self->{'producer_type'} = "CODE";
-            $self->debug("Got 'producer': code ref");
-        } # }}}
+            $self->debug("Got producer: code ref\n");
+        } 
 
-        # {{{ passed a string containing no "::"; relative package name
+        # passed a string containing no "::"; relative package name
         else {
             my $Pp = sprintf "SQL::Translator::Producer::$producer";
             load($Pp) or die "Can't load $Pp: $@";
             $self->{'producer'} = \&{ "$Pp\::produce" };
             $self->{'producer_type'} = $Pp;
-            $self->debug("Got producer: $Pp");
-        } # }}}
+            $self->debug("Got producer: $Pp\n");
+        }
 
         # At this point, $self->{'producer'} contains a subroutine
         # reference that is ready to run
-    } # }}}
+
+        # Anything left?  If so, it's producer_args
+        $self->producer_args(@_) if (@_);
+    }
 
     return $self->{'producer'};
 };
 
+# ----------------------------------------------------------------------
+# producer_type
+#
+# producer_type is an accessor that allows producer subs to get
+# information about their origin.  This is poptentially important;
+# since all producer subs are called as subroutine refernces, there is
+# no way for a producer to find out which package the sub lives in
+# originally, for example.
+# ----------------------------------------------------------------------
 sub producer_type { $_[0]->{'producer_type'} }
-# }}}
 
+# ----------------------------------------------------------------------
+# producer_args
+#
+# Arbitrary name => value pairs of paramters can be passed to a
+# producer using this method.
+#
+# XXX All calls to producer_args with a value clobbers old values!
+#     Should probably check if $_[0] is undef, and delete stored
+#     args if it is:
+#
+#     if (@_) {
+#         unless (defined $_[0]) {
+#             %{ $self->{'producer_args'} } = ();
+#         }
+#         my $args = isa($_[0], 'HASH') ? shift : { @_ };
+#         %{ $self->{'producer_args'} } = (
+#                                           %{ $self->{'producer_args'} },
+#                                           %{ $args }
+#                                         );
+#     }
+# ----------------------------------------------------------------------
+sub producer_args {
+    my $self = shift;
+    if (@_) {
+        my $args = isa($_[0], 'HASH') ? shift : { @_ };
+        $self->{'producer_args'} = $args;
+    }
+    $self->{'producer_args'};
+}
+
+# ----------------------------------------------------------------------
 =head2 B<parser>
 
 The B<parser> method defines or retrieves a subroutine that will be
@@ -258,7 +410,7 @@ called to perform the parsing.  The basic idea is the same as that of
 B<producer> (see above), except the default subroutine name is
 "parse", and will be invoked as $module_name::parse($tr, $data).
 Also, the parser subroutine will be passed a string containing the
-entirety of the data to be parsed (or possibly a reference to a string?).
+entirety of the data to be parsed.
 
   # Invokes SQL::Translator::Parser::MySQL::parse()
   $tr->parser("MySQL");
@@ -273,29 +425,31 @@ entirety of the data to be parsed (or possibly a reference to a string?).
     return $dumper->Dump;
   });
 
+There is also B<parser_type> and B<parser_args>, which perform
+analogously to B<producer_type> and B<producer_args>
+
 =cut
 
-# {{{ parser and parser_type
 sub parser {
     my $self = shift;
 
-    # {{{ parser as a mutator
+    # parser as a mutator
     if (@_) {
         my $parser = shift;
 
-        # {{{ Passed a module name (string containing "::")
+        # Passed a module name (string containing "::")
         if ($parser =~ /::/) {
             my $func_name;
 
-            # {{{ Module name was passed directly
+            # Module name was passed directly
             # We try to load the name; if it doesn't load, there's
             # a possibility that it has a function name attached to
             # it.
             if (load($parser)) {
                 $func_name = "parse";
-            } # }}}
+            }
 
-            # {{{ Module::function was passed
+            # Module::function was passed
             else {
                 # Passed Module::Name::function; try to recover
                 my @func_parts = split /::/, $parser;
@@ -305,42 +459,75 @@ sub parser {
                 # If this doesn't work, then we have a legitimate
                 # problem.
                 load($parser) or die "Can't load $parser: $@";
-            } # }}}
+            } 
 
-            # {{{ get code reference and assign
+            # get code reference and assign
             $self->{'parser'} = \&{ "$parser\::$func_name" };
             $self->{'parser_type'} = $parser;
-            $self->debug("Got parser: $parser\::$func_name");
-            # }}}
-        } # }}}
+            $self->debug("Got parser: $parser\::$func_name\n");
+        }
 
-        # {{{ passed an anonymous subroutine reference
-        elsif (isa($parser, 'CODE')) {
-            $self->{'parser'} = $parser;
+        # passed an anonymous subroutine reference
+        elsif ( isa( $parser, 'CODE' ) ) {
+            $self->{'parser'}      = $parser;
             $self->{'parser_type'} = "CODE";
-            $self->debug("Got 'parser': code ref");
-        } # }}}
+            $self->debug("Got parser: code ref\n");
+        } 
 
-        # {{{ passed a string containing no "::"; relative package name
+        # passed a string containing no "::"; relative package name
         else {
-            my $Pp = sprintf "SQL::Translator::Parser::$parser";
-            load($Pp) or die "Can't load $Pp: $@";
-            $self->{'parser'} = \&{ "$Pp\::parse" };
+            my $Pp = "SQL::Translator::Parser::$parser";
+            load( $Pp ) or die "Can't load $Pp: $@";
+            $self->{'parser'}      = \&{ "$Pp\::parse" };
             $self->{'parser_type'} = $Pp;
-            $self->debug("Got parser: $Pp");
-        } # }}}
+            $self->debug("Got parser: $Pp\n");
+        } 
 
+        #
         # At this point, $self->{'parser'} contains a subroutine
         # reference that is ready to run
-    } # }}}
-
+        #
+        $self->parser_args( @_ ) if (@_);
+    }
 
     return $self->{'parser'};
 }
 
+# ----------------------------------------------------------------------
 sub parser_type { $_[0]->{'parser_type'} }
-# }}}
 
+# ----------------------------------------------------------------------
+# XXX See notes on producer_args, above
+sub parser_args {
+    my $self = shift;
+    if (@_) {
+        my $args = isa($_[0], 'HASH') ? shift : { @_ };
+        $self->{'parser_args'} = $args;
+    }
+    $self->{'parser_args'};
+} 
+
+# ----------------------------------------------------------------------
+=head2 B<show_warnings>
+
+Toggles whether to print warnings of name conflicts, identifier
+mutations, etc.  Probably only generated by producers to let the user
+know when something won't translate very smoothly (e.g., MySQL "enum"
+fields into Oracle).  Accepts a true or false value, returns the
+current value.
+
+=cut
+
+sub show_warnings {
+    my $self = shift;
+    my $arg  = shift;
+    if ( defined $arg ) {
+        $self->{'show_warnings'} = $arg ? 1 : 0;
+    }
+    return $self->{'show_warnings'} || 0;
+}
+
+# ----------------------------------------------------------------------
 =head2 B<translate>
 
 The B<translate> method calls the subroutines referenced by the
@@ -387,32 +574,153 @@ You get the idea.
 
 =back
 
+# ----------------------------------------------------------------------
+=head2 B<filename>, B<data>
+
+Using the B<filename> method, the filename of the data to be parsed
+can be set. This method can be used in conjunction with the B<data>
+method, below.  If both the B<filename> and B<data> methods are
+invoked as mutators, the data set in the B<data> method is used.
+
+    $tr->filename("/my/data/files/create.sql");
+
+or:
+
+    my $create_script = do {
+        local $/;
+        open CREATE, "/my/data/files/create.sql" or die $!;
+        <CREATE>;
+    };
+    $tr->data(\$create_script);
+
+B<filename> takes a string, which is interpreted as a filename.
+B<data> takes a reference to a string, which is used as the data to be
+parsed.  If a filename is set, then that file is opened and read when
+the B<translate> method is called, as long as the data instance
+variable is not set.
+
 =cut
 
-# {{{ translate
+# filename - get or set the filename
+sub filename {
+    my $self = shift;
+    if (@_) {
+        my $filename = shift;
+        if (-d $filename) {
+            my $msg = "Cannot use directory '$filename' as input source";
+            return $self->error($msg);
+        } elsif (-f _ && -r _) {
+            $self->{'filename'} = $filename;
+            $self->debug("Got filename: '$self->{'filename'}'\n");
+        } else {
+            my $msg = "Cannot use '$filename' as input source: ".
+                      "file does not exist or is not readable.";
+            return $self->error($msg);
+        }
+    }
+
+    $self->{'filename'};
+}
+
+# ----------------------------------------------------------------------
+# data - get or set the data
+# if $self->{'data'} is not set, but $self->{'filename'} is, then
+# $self->{'filename'} is opened and read, whith the results put into
+# $self->{'data'}.
+sub data {
+    my $self = shift;
+
+    # Set $self->{'data'} to $_[0], if it is provided.
+    if (@_) {
+        my $data = shift;
+        if (isa($data, "SCALAR")) {
+            $self->{'data'} =  $data;
+        }
+        elsif (! ref $data) {
+            $self->{'data'} = \$data;
+        }
+    }
+
+    # If we have a filename but no data yet, populate.
+    if (not $self->{'data'} and my $filename = $self->filename) {
+        $self->debug("Opening '$filename' to get contents.\n");
+        local *FH;
+        local $/;
+        my $data;
+
+        unless (open FH, $filename) {
+            return $self->error("Can't read file '$filename': $!");
+        }
+
+        $data = <FH>;
+        $self->{'data'} = \$data;
+
+        unless (close FH) {
+            return $self->error("Can't close file '$filename': $!");
+        }
+    }
+
+    return $self->{'data'};
+}
+
+# ----------------------------------------------------------------------
+=pod
+
+=head2 B<trace>
+
+Turns on/off the tracing option of Parse::RecDescent.
+
+=cut
+
+sub trace {
+    my $self = shift;
+    my $arg  = shift;
+    if ( defined $arg ) {
+        $self->{'trace'} = $arg ? 1 : 0;
+    }
+    return $self->{'trace'} || 0;
+}
+
+# ----------------------------------------------------------------------
 sub translate {
     my $self = shift;
-    my ($args, $parser, $producer);
+    my ($args, $parser, $parser_type, $producer, $producer_type);
+    my ($parser_output, $producer_output);
 
-    if (@_ == 1) {
+    # Parse arguments
+    if (@_ == 1) { 
+        # Passed a reference to a hash?
         if (isa($_[0], 'HASH')) {
-            # Passed a hashref
-            $self->debug("translate: Got a hashref");
+            # yep, a hashref
+            $self->debug("translate: Got a hashref\n");
             $args = $_[0];
         }
+
+        # Passed a reference to a string containing the data
         elsif (isa($_[0], 'SCALAR')) {
-            # passed a ref to a string; deref it
-            $self->debug("translate: Got a SCALAR reference (string)");
-            $args = { data => ${$_[0]} };
+            # passed a ref to a string
+            $self->debug("translate: Got a SCALAR reference (string)\n");
+            $self->data($_[0]);
         }
+
+        # Not a reference; treat it as a filename
         elsif (! ref $_[0]) {
             # Not a ref, it's a filename
-            $self->debug("translate: Got a filename");
-            $args = { filename => $_[0] };
+            $self->debug("translate: Got a filename\n");
+            $self->filename($_[0]);
         }
+
+        # Passed something else entirely.
         else {
             # We're not impressed.  Take your empty string and leave.
-            return "";
+            # return "";
+
+            # Actually, if data, parser, and producer are set, then we
+            # can continue.  Too bad, because I like my comment
+            # (above)...
+            return "" unless ($self->data     &&
+                              $self->producer &&
+                              $self->parser);
         }
     }
     else {
@@ -421,99 +729,93 @@ sub translate {
         $args = { @_ };
     }
 
-    if ((defined $args->{'filename'} || defined $args->{'file'}) &&
-         not $args->{'data'}) {
-        local *FH;
-        local $/;
+    # ----------------------------------------------------------------------
+    # Can specify the data to be transformed using "filename", "file",
+    # "data", or "datasource".
+    # ----------------------------------------------------------------------
+    if (my $filename = ($args->{'filename'} || $args->{'file'})) {
+        $self->filename($filename);
+    }
 
-        open FH, $args->{'filename'}
-            or die "Can't open $args->{'filename'} for reading: $!";
-        $args->{'data'} = <FH>;
-        close FH or die "Can't close $args->{'filename'}: $!";
+    if (my $data = ($self->{'data'} || $self->{'datasource'})) {
+        $self->data($data);
     }
 
-    #
-    # Last chance to bail out; if there's nothing in the data
-    # key of %args, back out.
-    #
-    return "" unless defined $args->{'data'};
+    # ----------------------------------------------------------------
+    # Get the data.
+    # ----------------------------------------------------------------
+    my $data = $self->data;
+    unless (length $$data) {
+        return $self->error("Empty data file!");
+    }
 
-    #
+    # ----------------------------------------------------------------
     # Local reference to the parser subroutine
-    #
+    # ----------------------------------------------------------------
     if ($parser = ($args->{'parser'} || $args->{'from'})) {
         $self->parser($parser);
-    } else {
-        $parser = $self->parser;
     }
+    $parser      = $self->parser;
+    $parser_type = $self->parser_type;
 
-    #
+    # ----------------------------------------------------------------
     # Local reference to the producer subroutine
-    #
+    # ----------------------------------------------------------------
     if ($producer = ($args->{'producer'} || $args->{'to'})) {
         $self->producer($producer);
-    } else {
-        $producer = $self->producer;
+    }
+    $producer      = $self->producer;
+    $producer_type = $self->producer_type;
+
+    # ----------------------------------------------------------------
+    # Execute the parser, then execute the producer with that output.
+    # Allowances are made for each piece to die, or fail to compile,
+    # since the referenced subroutines could be almost anything.  In
+    # the future, each of these might happen in a Safe environment,
+    # depending on how paranoid we want to be.
+    # ----------------------------------------------------------------
+    eval { $parser_output = $parser->($self, $$data) };
+    if ($@ || ! $parser_output) {
+        my $msg = sprintf "translate: Error with parser '%s': %s",
+            $parser_type, ($@) ? $@ : " no results";
+        return $self->error($msg);
     }
 
-    #
-    # Execute the parser, then execute the producer with that output
-    #
-    return $producer->($self, $parser->($self, $args->{'data'}));
-}
-# }}}
-
-=head2 B<error>
-
-The error method returns the last error.
-
-=cut
+    eval { $producer_output = $producer->($self, $parser_output) };
+    if ($@ || ! $producer_output) {
+        my $msg = sprintf "translate: Error with producer '%s': %s",
+            $producer_type, ($@) ? $@ : " no results";
+        return $self->error($msg);
+    }
 
-# {{{ error
-#-----------------------------------------------------
-sub error {
-#
-# Return the last error.
-#
-    return shift()->{'error'} || '';
+    return $producer_output;
 }
-# }}}
 
-=head2 B<error_out>
-
-Record the error and return undef.  The error can be retrieved by
-calling programs using $tr->error.
+# ----------------------------------------------------------------------
+sub list_producers {
+    require SQL::Translator::Producer;
+    my $path = catfile(dirname($INC{'SQL/Translator/Producer.pm'}), "Producer");
+    my $dh = IO::Dir->new($path);
 
-For Parser or Producer writers, primarily.  
+    my @available = map { join "::", "SQL::Translator::Producer", $_ }
+                    grep /\.pm$/, $dh->read;
 
-=cut
-
-# {{{ error_out
-sub error_out {
-    my $self = shift;
-    if ( my $error = shift ) {
-        $self->{'error'} = $error;
-    }
-    return;
+    return @available;
 }
-# }}}
 
-=head2 B<debug>
-
-If the global variable $SQL::Translator::DEBUG is set to a true value,
-then calls to $tr->debug($msg) will be carped to STDERR.  If $DEBUG is
-not set, then this method does nothing.
+# ----------------------------------------------------------------------
+sub list_parsers {
+    require SQL::Translator::Parser;
+    my $path = catfile(dirname($INC{'SQL/Translator/Parser.pm'}), "Parser");
+    my $dh = IO::Dir->new($path);
 
-=cut
+    my @available = map { join "::", "SQL::Translator::Parser", $_ }
+                    grep /\.pm$/, $dh->read;
 
-# {{{ debug
-sub debug {
-    my $self = shift;
-    carp @_ if ($DEBUG);
+    return @available;
 }
-# }}}
 
-# {{{ load
+# ----------------------------------------------------------------------
 sub load {
     my $module = do { my $m = shift; $m =~ s[::][/]g; "$m.pm" };
     return 1 if $INC{$module};
@@ -523,20 +825,24 @@ sub load {
     return if ($@);
     return 1;
 }
-# }}}
+
+# ----------------------------------------------------------------------
+sub isa { UNIVERSAL::isa($_[0], $_[1]) }
 
 1;
 
-__END__
 #-----------------------------------------------------
 # Rescue the drowning and tie your shoestrings.
 # Henry David Thoreau 
 #-----------------------------------------------------
 
-=head1 AUTHOR
+=pod
+
+=head1 AUTHORS
 
-Ken Y. Clark, E<lt>kclark@logsoft.comE<gt>,
-darren chamberlain E<lt>darren@cpan.orgE<gt>
+Ken Y. Clark, E<lt>kclark@cpan.org<gt>,
+darren chamberlain E<lt>darren@cpan.orgE<gt>,
+Chris Mungall E<lt>cjm@fruitfly.orgE<gt>
 
 =head1 COPYRIGHT