Audit and minimize use of last major indirect method: search()
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / Admin.pm
index e85b784..300c485 100644 (file)
@@ -1,59 +1,24 @@
-#
-#===============================================================================
-#
-#         FILE:  Admin.pm
-#
-#  DESCRIPTION:  Administrative functions for DBIx::Class Schemata
-#
-#        FILES:  ---
-#         BUGS:  ---
-#        NOTES:  ---
-#       AUTHOR:  Gordon Irving (), <Gordon.irving@sophos.com>
-#      VERSION:  1.0
-#      CREATED:  28/11/09 12:27:15 GMT
-#     REVISION:  ---
-#===============================================================================
-
 package DBIx::Class::Admin;
 
+use warnings;
+use strict;
+
+# check deps
+BEGIN {
+  require DBIx::Class::Optional::Dependencies;
+  if (my $missing = DBIx::Class::Optional::Dependencies->req_missing_for ('admin') ) {
+    die "The following extra modules are required for DBIx::Class::Admin: $missing\n";
+  }
+}
+
+use JSON::Any qw(DWIW PP JSON CPANEL XS);
 use Moose;
-use MooseX::Types 
-       -declare => [qw( DBICConnectInfo )];
-use MooseX::Types::Moose qw/Int HashRef ArrayRef Str Any/;
+use MooseX::Types::Moose qw/Int Str Any Bool/;
+use DBIx::Class::Admin::Types qw/DBICConnectInfo DBICHashRef/;
 use MooseX::Types::JSON qw(JSON);
 use MooseX::Types::Path::Class qw(Dir File);
-use Try::Tiny;
-
-use parent 'Class::C3::Componentised';
-
-use JSON::Any;
-
-use namespace::autoclean;
-
-my @_deps = qw(Moose MooseX::Types MooseX::Types::JSON MooseX::Types::Path::Class Try::Tiny parent JSON::Any Class::C3::Componentised namespace::autoclean);
-
-coerce ArrayRef,
-       from JSON,
-       via { _json_to_data ($_) };
-
-coerce HashRef,
-       from JSON,
-       via { _json_to_data($_) };
-
-subtype DBICConnectInfo,
-       as ArrayRef;
-
-coerce DBICConnectInfo,
-       from JSON,
-        via { return _json_to_data($_) } ;
-
-coerce DBICConnectInfo,
-       from Str,
-               via { return _json_to_data($_) };
-
-coerce DBICConnectInfo,
-       from HashRef,
-        via { [ $_->{dsn}, $_->{user}, $_->{password} ]  };
+use MooseX::Types::LoadableClass qw(LoadableClass);
+use namespace::clean;
 
 =head1 NAME
 
@@ -61,224 +26,292 @@ DBIx::Class::Admin - Administration object for schemas
 
 =head1 SYNOPSIS
 
-       use DBIx::Class::Admin;
+  $ dbicadmin --help
 
-       # ddl manipulation
-       my $admin = DBIx::Class::Admin->new(
-               schema_class=> 'MY::Schema',
-               sql_dir=> $sql_dir,
-               connect_info => { dsn => $dsn, user => $user, password => $pass },
-       );
+  $ dbicadmin --schema=MyApp::Schema \
+    --connect='["dbi:SQLite:my.db", "", ""]' \
+    --deploy
 
-       # create SQLite sql
-       $admin->create('SQLite');
+  $ dbicadmin --schema=MyApp::Schema --class=Employee \
+    --connect='["dbi:SQLite:my.db", "", ""]' \
+    --op=update --set='{ "name": "New_Employee" }'
 
-       # create SQL diff for an upgrade
-       $admin->create('SQLite', {} , "1.0");
+  use DBIx::Class::Admin;
 
-       # upgrade a database
-       $admin->upgrade();
+  # ddl manipulation
+  my $admin = DBIx::Class::Admin->new(
+    schema_class=> 'MY::Schema',
+    sql_dir=> $sql_dir,
+    connect_info => { dsn => $dsn, user => $user, password => $pass },
+  );
 
-       # install a version for an unversioned schema
-       $admin->install("3.0");
+  # create SQLite sql
+  $admin->create('SQLite');
 
-=head1 Attributes
+  # create SQL diff for an upgrade
+  $admin->create('SQLite', {} , "1.0");
 
-=head2 lib
+  # upgrade a database
+  $admin->upgrade();
 
-add a library search path
-=cut
-has lib => (
-       is              => 'ro',
-       isa             => Dir,
-       coerce  => 1,
-       trigger => \&_set_inc,
-);
+  # install a version for an unversioned schema
+  $admin->install("3.0");
 
-sub _set_inc {
-       my ($self, $lib) = @_;
-       push @INC, $lib->stringify;
-}
+=head1 REQUIREMENTS
+
+The Admin interface has additional requirements not currently part of
+L<DBIx::Class>. See L<DBIx::Class::Optional::Dependencies> for more details.
+
+=head1 ATTRIBUTES
 
 =head2 schema_class
 
 the class of the schema to load
+
 =cut
+
 has 'schema_class' => (
-       is              => 'ro',
-       isa             => 'Str',
-       coerce  => 1,
+  is  => 'ro',
+  isa => LoadableClass,
 );
 
+
 =head2 schema
 
 A pre-connected schema object can be provided for manipulation
+
 =cut
+
 has 'schema' => (
-       is                      => 'ro',
-       isa                     => 'DBIx::Class::Schema',
-       lazy_build      => 1,
+  is          => 'ro',
+  isa         => 'DBIx::Class::Schema',
+  lazy_build  => 1,
 );
 
 sub _build_schema {
-       my ($self)  = @_;
-       $self->ensure_class_loaded($self->schema_class);
+  my ($self)  = @_;
 
-       $self->connect_info->[3]->{ignore_version} =1;
-       return $self->schema_class->connect(@{$self->connect_info()} ); # ,  $self->connect_info->[3], { ignore_version => 1} );
+  $self->connect_info->[3]{ignore_version} = 1;
+  return $self->schema_class->clone->connection(@{$self->connect_info});
 }
 
 =head2 resultset
 
 a resultset from the schema to operate on
+
 =cut
+
 has 'resultset' => (
-       is                      => 'rw',
-       isa                     => Str,
+  is  => 'rw',
+  isa => Str,
 );
 
+
 =head2 where
 
 a hash ref or json string to be used for identifying data to manipulate
+
 =cut
 
 has 'where' => (
-       is                      => 'rw',
-       isa                     => HashRef,
-       coerce          => 1,
+  is      => 'rw',
+  isa     => DBICHashRef,
+  coerce  => 1,
 );
 
+
 =head2 set
+
 a hash ref or json string to be used for inserting or updating data
+
 =cut
 
 has 'set' => (
-       is                      => 'rw',
-       isa                     => HashRef,
-       coerce          => 1,
+  is      => 'rw',
+  isa     => DBICHashRef,
+  coerce  => 1,
 );
 
+
 =head2 attrs
-a hash ref or json string to be used for passing additonal info to the ->search call
+
+a hash ref or json string to be used for passing additional info to the ->search call
+
 =cut
+
 has 'attrs' => (
-       is                      => 'rw',
-       isa                     => HashRef,
-       coerce          => 1,
+  is      => 'rw',
+  isa     => DBICHashRef,
+  coerce  => 1,
 );
+
+
 =head2 connect_info
 
 connect_info the arguments to provide to the connect call of the schema_class
-=cut
 
+=cut
 
 has 'connect_info' => (
-       is                      => 'ro',
-       isa                     => DBICConnectInfo,
-       lazy_build      => 1,
-       coerce          => 1,
+  is          => 'ro',
+  isa         => DBICConnectInfo,
+  lazy_build  => 1,
+  coerce      => 1,
 );
 
 sub _build_connect_info {
-       my ($self) = @_;
-       return $self->_find_stanza($self->config, $self->config_stanza);
+  my ($self) = @_;
+  return $self->_find_stanza($self->config, $self->config_stanza);
 }
 
+
 =head2 config_file
 
 config_file provide a config_file to read connect_info from, if this is provided
 config_stanze should also be provided to locate where the connect_info is in the config
-The config file should be in a format readable by Config::General
+The config file should be in a format readable by Config::Any.
+
 =cut
+
 has config_file => (
-       is                      => 'ro',
-       isa                     => File,
-       coerce          => 1,
+  is      => 'ro',
+  isa     => File,
+  coerce  => 1,
 );
 
+
 =head2 config_stanza
 
-config_stanza for use with config_file should be a '::' deliminated 'path' to the connection information
+config_stanza for use with config_file should be a '::' delimited 'path' to the connection information
 designed for use with catalyst config files
+
 =cut
+
 has 'config_stanza' => (
-       is                      => 'ro',
-       isa                     => 'Str',
+  is  => 'ro',
+  isa => Str,
 );
 
+
 =head2 config
 
-Instead of loading from a file the configuration can be provided directly as a hash ref.  Please note 
+Instead of loading from a file the configuration can be provided directly as a hash ref.  Please note
 config_stanza will still be required.
+
 =cut
+
 has config => (
-       is                      => 'ro',
-       isa                     => HashRef,
-       lazy_build      => 1,
+  is          => 'ro',
+  isa         => DBICHashRef,
+  lazy_build  => 1,
 );
 
 sub _build_config {
-       my ($self) = @_;
-       try { require 'Config::Any'; } catch { die "Config::Any is required to parse the config file"; };
+  my ($self) = @_;
 
-       my $cfg = Config::Any->load_files ( {files => [$self->config_file], use_ext =>1, flatten_to_hash=>1});
+  my $cfg = Config::Any->load_files ( {files => [$self->config_file], use_ext =>1, flatten_to_hash=>1});
 
-       # just grab the config from the config file
-       $cfg = $cfg->{$self->config_file};
-       return $cfg;
+  # just grab the config from the config file
+  $cfg = $cfg->{$self->config_file};
+  return $cfg;
 }
 
+
 =head2 sql_dir
 
 The location where sql ddl files should be created or found for an upgrade.
+
 =cut
+
 has 'sql_dir' => (
-       is                      => 'ro',
-       isa                     => Dir,
-       coerce          => 1,
+  is      => 'ro',
+  isa     => Dir,
+  coerce  => 1,
+);
+
+
+=head2 sql_type
+
+The type of sql dialect to use for creating sql files from schema
+
+=cut
+
+has 'sql_type' => (
+  is     => 'ro',
+  isa    => Str,
 );
 
 =head2 version
 
 Used for install, the version which will be 'installed' in the schema
+
 =cut
+
 has version => (
-       is                      => 'rw',
-       isa                     => 'Str',
+  is  => 'rw',
+  isa => Str,
 );
 
+
 =head2 preversion
 
-Previouse version of the schema to create an upgrade diff for, the full sql for that version of the sql must be in the sql_dir
+Previous version of the schema to create an upgrade diff for, the full sql for that version of the sql must be in the sql_dir
+
 =cut
+
 has preversion => (
-       is                      => 'rw',
-       isa                     => 'Str',
+  is  => 'rw',
+  isa => Str,
 );
 
+
 =head2 force
 
 Try and force certain operations.
+
 =cut
+
 has force => (
-       is                      => 'rw',
-       isa                     => 'Bool',
+  is  => 'rw',
+  isa => Bool,
 );
 
+
 =head2 quiet
 
 Be less verbose about actions
+
 =cut
+
 has quiet => (
-       is                      => 'rw',
-       isa                     => 'Bool',
+  is  => 'rw',
+  isa => Bool,
 );
 
 has '_confirm' => (
-       is              => 'bare',
-       isa             => 'Bool',
+  is  => 'bare',
+  isa => Bool,
 );
 
+
+=head2 trace
+
+Toggle DBIx::Class debug output
+
+=cut
+
+has trace => (
+    is => 'rw',
+    isa => Bool,
+    trigger => \&_trigger_trace,
+);
+
+sub _trigger_trace {
+    my ($self, $new, $old) = @_;
+    $self->schema->storage->debug($new);
+}
+
+
 =head1 METHODS
 
 =head2 create
@@ -289,26 +322,34 @@ has '_confirm' => (
 
 =back
 
-L<create> will generate sql for the supplied schema_class in sql_dir.  The flavour of sql to 
-generate can be controlled by suppling a sqlt_type which should be a L<SQL::Translator> name.  
+C<create> will generate sql for the supplied schema_class in sql_dir. The
+flavour of sql to generate can be controlled by supplying a sqlt_type which
+should be a L<SQL::Translator> name.
 
 Arguments for L<SQL::Translator> can be supplied in the sqlt_args hashref.
 
 Optional preversion can be supplied to generate a diff to be used by upgrade.
+
 =cut
 
 sub create {
-       my ($self, $sqlt_type, $sqlt_args, $preversion) = @_;
+  my ($self, $sqlt_type, $sqlt_args, $preversion) = @_;
 
-       $preversion ||= $self->preversion();
+  $preversion ||= $self->preversion();
+  $sqlt_type ||= $self->sql_type();
 
-       my $schema = $self->schema();
-       # create the dir if does not exist
-       $self->sql_dir->mkpath() if ( ! -d $self->sql_dir);
+  my $schema = $self->schema();
 
-       $schema->create_ddl_dir( $sqlt_type, (defined $schema->schema_version ? $schema->schema_version : ""), $self->sql_dir->stringify, $preversion, $sqlt_args );
+  $schema->create_ddl_dir(
+    $sqlt_type,
+    (defined $schema->schema_version ? $schema->schema_version : ""),
+    $self->sql_dir,
+    $preversion,
+    $sqlt_args,
+  );
 }
 
+
 =head2 upgrade
 
 =over 4
@@ -319,20 +360,24 @@ sub create {
 
 upgrade will attempt to upgrade the connected database to the same version as the schema_class.
 B<MAKE SURE YOU BACKUP YOUR DB FIRST>
+
 =cut
 
 sub upgrade {
-       my ($self) = @_;
-       my $schema = $self->schema();
-       if (!$schema->get_db_version()) {
-               # schema is unversioned
-               die "could not determin current schema version, please either install or deploy";
-       } else {
-               my $ret = $schema->upgrade();
-               return $ret;
-       }
+  my ($self) = @_;
+  my $schema = $self->schema();
+
+  if (!$schema->get_db_version()) {
+    # schema is unversioned
+    $schema->throw_exception ("Could not determin current schema version, please either install() or deploy().\n");
+  } else {
+    $schema->upgrade_directory ($self->sql_dir) if $self->sql_dir;  # this will override whatever default the schema has
+    my $ret = $schema->upgrade();
+    return $ret;
+  }
 }
 
+
 =head2 install
 
 =over 4
@@ -341,35 +386,37 @@ sub upgrade {
 
 =back
 
-install is here to help when you want to move to L<DBIx::Class::Schema::Versioned> and have an existing 
-database.  install will take a version and add the version tracking tables and 'install' the version.  No 
-further ddl modification takes place.  Setting the force attribute to a true value will allow overriding of 
+install is here to help when you want to move to L<DBIx::Class::Schema::Versioned> and have an existing
+database.  install will take a version and add the version tracking tables and 'install' the version.  No
+further ddl modification takes place.  Setting the force attribute to a true value will allow overriding of
 already versioned databases.
+
 =cut
+
 sub install {
-       my ($self, $version) = @_;
-
-       my $schema = $self->schema();
-       $version ||= $self->version();
-       if (!$schema->get_db_version() ) {
-               # schema is unversioned
-               print "Going to install schema version\n";
-               my $ret = $schema->install($version);
-               print "retun is $ret\n";
-       }
-       elsif ($schema->get_db_version() and $self->force ) {
-               warn "forcing install may not be a good idea";
-               if($self->_confirm() ) {
-                       # FIXME private api
-                       $self->schema->_set_db_version({ version => $version});
-               }
-       }
-       else {
-               die "schema already has a version not installing, try upgrade instead";
-       }
+  my ($self, $version) = @_;
+
+  my $schema = $self->schema();
+  $version ||= $self->version();
+  if (!$schema->get_db_version() ) {
+    # schema is unversioned
+    print "Going to install schema version\n" if (!$self->quiet);
+    my $ret = $schema->install($version);
+    print "return is $ret\n" if (!$self->quiet);
+  }
+  elsif ($schema->get_db_version() and $self->force ) {
+    warn "Forcing install may not be a good idea\n";
+    if($self->_confirm() ) {
+      $self->schema->_set_db_version({ version => $version});
+    }
+  }
+  else {
+    $schema->throw_exception ("Schema already has a version. Try upgrade instead.\n");
+  }
 
 }
 
+
 =head2 deploy
 
 =over 4
@@ -378,26 +425,17 @@ sub install {
 
 =back
 
-deploy will create the schema at the connected database.  C<$args> are passed straight to 
-L<DBIx::Class::Schema/deploy>.  
+deploy will create the schema at the connected database.  C<$args> are passed straight to
+L<DBIx::Class::Schema/deploy>.
+
 =cut
+
 sub deploy {
-       my ($self, $args) = @_;
-       my $schema = $self->schema();
-       if (!$schema->get_db_version() ) {
-               # schema is unversioned
-               $schema->deploy( $args, $self->sql_dir)
-                       or die "could not deploy schema";
-       } else {
-               die "there already is a database with a version here, try upgrade instead";
-       }
+  my ($self, $args) = @_;
+  my $schema = $self->schema();
+  $schema->deploy( $args, $self->sql_dir );
 }
 
-
-# FIXME ensure option spec compatability
-#die('Do not use the where option with the insert op') if ($where);
-#die('Do not use the attrs option with the insert op') if ($attrs);
-
 =head2 insert
 
 =over 4
@@ -410,47 +448,50 @@ insert takes the name of a resultset from the schema_class and a hashref of data
 into that resultset
 
 =cut
+
 sub insert {
-       my ($self, $rs, $set) = @_;
+  my ($self, $rs, $set) = @_;
 
-       $rs ||= $self->resultset();
-       $set ||= $self->set();
-       my $resultset = $self->schema->resultset($rs);
-       my $obj = $resultset->create( $set );
-       print ''.ref($resultset).' ID: '.join(',',$obj->id())."\n" if (!$self->quiet);
+  $rs ||= $self->resultset();
+  $set ||= $self->set();
+  my $resultset = $self->schema->resultset($rs);
+  my $obj = $resultset->new_result($set)->insert;
+  print ''.ref($resultset).' ID: '.join(',',$obj->id())."\n" if (!$self->quiet);
 }
 
 
 =head2 update
 
-=over 4 
+=over 4
 
 =item Arguments: $rs, $set, $where
 
 =back
 
-update takes the name of a resultset from the schema_class, a hashref of data to update and 
-a where hash used to form the search for the rows to update. 
+update takes the name of a resultset from the schema_class, a hashref of data to update and
+a where hash used to form the search for the rows to update.
+
 =cut
+
 sub update {
-       my ($self, $rs, $set, $where) = @_;
+  my ($self, $rs, $set, $where) = @_;
 
-       $rs ||= $self->resultset();
-       $where ||= $self->where();
-       $set ||= $self->set();
-       my $resultset = $self->schema->resultset($rs);
-       $resultset = $resultset->search( ($where||{}) );
+  $rs ||= $self->resultset();
+  $where ||= $self->where();
+  $set ||= $self->set();
+  my $resultset = $self->schema->resultset($rs);
+  $resultset = $resultset->search_rs( $where )
+    if $where;
 
-       my $count = $resultset->count();
-       print "This action will modify $count ".ref($resultset)." records.\n" if (!$self->quiet);
+  my $count = $resultset->count();
+  print "This action will modify $count ".ref($resultset)." records.\n" if (!$self->quiet);
 
-       if ( $self->force || $self->_confirm() ) {
-               $resultset->update_all( $set );
-       }
+  if ( $self->force || $self->_confirm() ) {
+    $resultset->update_all( $set );
+  }
 }
 
-# FIXME
-#die('Do not use the set option with the delete op') if ($set);
+
 =head2 delete
 
 =over 4
@@ -459,26 +500,30 @@ sub update {
 
 =back
 
-delete takes the name of a resultset from the schema_class, a where hashref and a attrs to pass to ->search. 
+delete takes the name of a resultset from the schema_class, a where hashref and a attrs to pass to ->search.
 The found data is deleted and cannot be recovered.
+
 =cut
+
 sub delete {
-       my ($self, $rs, $where, $attrs) = @_;
+  my ($self, $rs, $where, $attrs) = @_;
 
-       $rs ||= $self->resultset();
-       $where ||= $self->where();
-       $attrs ||= $self->attrs();
-       my $resultset = $self->schema->resultset($rs);
-       $resultset = $resultset->search( ($where||{}), ($attrs||()) );
+  $rs ||= $self->resultset();
+  $where ||= $self->where();
+  $attrs ||= $self->attrs();
+  my $resultset = $self->schema->resultset($rs);
+  $resultset = $resultset->search_rs( ($where||{}), ($attrs||()) )
+    if $where or $attrs;
 
-       my $count = $resultset->count();
-       print "This action will delete $count ".ref($resultset)." records.\n" if (!$self->quiet);
+  my $count = $resultset->count();
+  print "This action will delete $count ".ref($resultset)." records.\n" if (!$self->quiet);
 
-       if ( $self->force || $self->_confirm() ) {
-               $resultset->delete_all();
-       }
+  if ( $self->force || $self->_confirm() ) {
+    $resultset->delete_all();
+  }
 }
 
+
 =head2 select
 
 =over 4
@@ -487,92 +532,74 @@ sub delete {
 
 =back
 
-select takes the name of a resultset from the schema_class, a where hashref and a attrs to pass to ->search. 
+select takes the name of a resultset from the schema_class, a where hashref and a attrs to pass to ->search.
 The found data is returned in a array ref where the first row will be the columns list.
 
 =cut
+
 sub select {
-       my ($self, $rs, $where, $attrs) = @_;
-
-       $rs ||= $self->resultset();
-       $where ||= $self->where();
-       $attrs ||= $self->attrs();
-       my $resultset = $self->schema->resultset($rs);
-       $resultset = $resultset->search( ($where||{}), ($attrs||()) );
-
-       my @data;
-       my @columns = $resultset->result_source->columns();
-       push @data, [@columns];# 
-
-       while (my $row = $resultset->next()) {
-               my @fields;
-               foreach my $column (@columns) {
-                       push( @fields, $row->get_column($column) );
-               }
-               push @data, [@fields];
-       }
-
-       return \@data;
+  my ($self, $rs, $where, $attrs) = @_;
+
+  $rs ||= $self->resultset();
+  $where ||= $self->where();
+  $attrs ||= $self->attrs();
+  my $resultset = $self->schema->resultset($rs);
+  $resultset = $resultset->search_rs( ($where||{}), ($attrs||()) )
+    if $where or $attrs;
+
+  my @data;
+  my @columns = $resultset->result_source->columns();
+  push @data, [@columns];#
+
+  while (my $row = $resultset->next()) {
+    my @fields;
+    foreach my $column (@columns) {
+      push( @fields, $row->get_column($column) );
+    }
+    push @data, [@fields];
+  }
+
+  return \@data;
 }
 
 sub _confirm {
-       my ($self) = @_;
-       print "Are you sure you want to do this? (type YES to confirm) \n";
-       # mainly here for testing
-       return 1 if ($self->meta->get_attribute('_confirm')->get_value($self));
-       my $response = <STDIN>;
-       return 1 if ($response=~/^YES/);
-       return;
-}
+  my ($self) = @_;
 
-sub _find_stanza {
-       my ($self, $cfg, $stanza) = @_;
-       my @path = split /::/, $stanza;
-       while (my $path = shift @path) {
-               if (exists $cfg->{$path}) {
-                       $cfg = $cfg->{$path};
-               }
-               else {
-                       die "could not find $stanza in config, $path did not seem to exist";
-               }
-       }
-       return $cfg;
-}
-
-sub _json_to_data {
-       my ($json_str) = @_;
-       my $json = JSON::Any->new(allow_barekey => 1, allow_singlequote => 1, relaxed=>1);
-       my $ret = $json->jsonToObj($json_str);
-       return $ret;
-}
+  # mainly here for testing
+  return 1 if ($self->meta->get_attribute('_confirm')->get_value($self));
 
+  print "Are you sure you want to do this? (type YES to confirm) \n";
+  my $response = <STDIN>;
 
-{  # deps check
-
-my @_missing_deps;
-foreach my $dep (@_deps) {
-       eval "require $dep";
-       if ($@) {
-               push @_missing_deps, $dep;
-       }
-}
-
-if (@_missing_deps > 0) {
-       die "The following dependecies are missing " . join ",", @_missing_deps;
+  return ($response=~/^YES/);
 }
 
-
+sub _find_stanza {
+  my ($self, $cfg, $stanza) = @_;
+  my @path = split /::/, $stanza;
+  while (my $path = shift @path) {
+    if (exists $cfg->{$path}) {
+      $cfg = $cfg->{$path};
+    }
+    else {
+      die ("Could not find $stanza in config, $path does not seem to exist.\n");
+    }
+  }
+  $cfg = $cfg->{connect_info} if exists $cfg->{connect_info};
+  return $cfg;
 }
-=head1 AUTHOR
 
-Gordon Irving <goraxe@cpan.org>
+=head1 FURTHER QUESTIONS?
 
-with code taken from dbicadmin by 
-Aran Deltac <bluefeet@cpan.org>
+Check the list of L<additional DBIC resources|DBIx::Class/GETTING HELP/SUPPORT>.
 
+=head1 COPYRIGHT AND LICENSE
 
-=head1 LICENSE
+This module is free software L<copyright|DBIx::Class/COPYRIGHT AND LICENSE>
+by the L<DBIx::Class (DBIC) authors|DBIx::Class/AUTHORS>. You can
+redistribute it and/or modify it under the same terms as the
+L<DBIx::Class library|DBIx::Class/COPYRIGHT AND LICENSE>.
 
-You may distribute this code under the same terms as Perl itself.
 =cut
+
 1;