It's almost 2010 - load_components ('Core') is like ewwww

[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / PK / Auto.pm

diff --git a/lib/DBIx/Class/PK/Auto.pm b/lib/DBIx/Class/PK/Auto.pm
index 5bf069a..e2f717f 100644 (file)
--- a/lib/DBIx/Class/PK/Auto.pm
+++ b/lib/DBIx/Class/PK/Auto.pm
@@ -5,35 +5,23 @@ use base qw/DBIx::Class/;
 use strict;
 use warnings;
 
-=head1 NAME 
+=head1 NAME
 
 DBIx::Class::PK::Auto - Automatic primary key class
 
 =head1 SYNOPSIS
 
-  # In your table classes (replace PK::Auto::SQLite with your database)
-  __PACKAGE__->load_components(qw/PK::Auto::SQLite Core/);
-  __PACKAGE__->set_primary_key('id');
+use base 'DBIx::Class::Core';
+__PACKAGE__->set_primary_key('id');
 
 =head1 DESCRIPTION
 
 This class overrides the insert method to get automatically incremented primary
 keys.
 
-You don't want to be using this directly - instead load the appropriate one for
-your database, e.g. C<PK::Auto::SQLite>, in your table classes:
+PK::Auto is now part of Core.
 
-  __PACKAGE__->load_components(qw/PK::Auto::SQLite Core/);
-
-Note that C<PK::Auto::SQLite> is specified as the leftmost argument.
-
-Alternatively, you can load the components separately:
-
-  __PACKAGE__->load_components(qw/Core/);
-  __PACKAGE__->load_components(qw/PK::Auto::SQLite/);
-
-This can be used, for example, if you have different databases and need to
-determine the appropriate C<PK::Auto> class at runtime.
+See L<DBIx::Class::Manual::Component> for details of component interactions.
 
 =head1 LOGIC
 
@@ -44,26 +32,7 @@ fetching the assigned value afterwards.
 
 =head2 insert
 
-Overrides C<insert> so that it will get the value of autoincremented primary
-keys.
-
-=cut
-
-sub insert {
-  my ($self, @rest) = @_;
-  my $ret = $self->next::method(@rest);
-
-  my ($pri, $too_many) = grep { !defined $self->get_column($_) } $self->primary_columns;
-  return $ret unless defined $pri; # if all primaries are already populated, skip auto-inc
-  $self->throw_exception( "More than one possible key found for auto-inc on ".ref $self )
-    if defined $too_many;
-
-  my $id = $self->result_source->storage->last_insert_id($self->result_source,$pri);
-  $self->throw_exception( "Can't get last insert id" ) unless $id;
-  $self->store_column($pri => $id);
-
-  return $ret;
-}
+The code that was handled here is now in Row for efficiency.
 
 =head2 sequence
 
@@ -72,7 +41,12 @@ associated with looking up the sequence automatically.
 
 =cut
 
-__PACKAGE__->mk_classdata('sequence');
+sub sequence {
+    my ($self,$seq) = @_;
+    foreach my $pri ($self->primary_columns) {
+        $self->column_info($pri)->{sequence} = $seq;
+    }
+}
 
 1;