X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FTable.pm;h=ddce803e8c6d261b2cf104380415cc893d126548;hb=49995cc8a37e1628fd93f52c58f3d9e2e34341d2;hp=fa8ecc002c840d8fb86d7a1a79cfa9bba3bf5b1f;hpb=58a4bd18e28485398f71429b129ac9e2e1073e92;p=dbsrgits%2FDBIx-Class.git

diff --git a/lib/DBIx/Class/Table.pm b/lib/DBIx/Class/Table.pm
index fa8ecc0..ddce803 100644
--- a/lib/DBIx/Class/Table.pm
+++ b/lib/DBIx/Class/Table.pm
@@ -5,110 +5,74 @@ use warnings;
 
 use DBIx::Class::ResultSet;
 
-use base qw/DBIx::Class/;
-
-__PACKAGE__->mk_classdata('_columns' => {});
-
-__PACKAGE__->mk_classdata('_table_name');
-
-__PACKAGE__->mk_classdata('table_alias'); # FIXME: Doesn't actually do anything yet!
+use Carp qw/croak/;
 
-__PACKAGE__->mk_classdata('_resultset_class' => 'DBIx::Class::ResultSet');
+use base qw/DBIx::Class/;
+__PACKAGE__->load_components(qw/AccessorGroup/);
 
-sub iterator_class { shift->_resultset_class(@_) }
+__PACKAGE__->mk_group_accessors('simple' =>
+  qw/_columns name resultset_class result_class storage/);
 
 =head1 NAME 
 
-DBIx::Class::Table - Basic table methods
+DBIx::Class::Table - Table object
 
 =head1 SYNOPSIS
 
 =head1 DESCRIPTION
 
-This class is responsible for defining and doing basic operations on 
-L<DBIx::Class> objects.
+This class is responsible for defining and doing table-level operations on 
+L<DBIx::Class> classes.
 
 =head1 METHODS
 
-=over 4
-
 =cut
 
-sub _register_columns {
-  my ($class, @cols) = @_;
-  my $names = { %{$class->_columns} };
-  while (my $col = shift @cols) {
-    $names->{$col} = (ref $cols[0] ? shift : {});
-  }
-  $class->_columns($names); 
+sub new {
+  my ($class, $attrs) = @_;
+  $class = ref $class if ref $class;
+  my $new = bless({ %{$attrs || {}} }, $class);
+  $new->{resultset_class} ||= 'DBIx::Class::ResultSet';
+  $new->{_columns} ||= {};
+  $new->{name} ||= "!!NAME NOT SET!!";
+  return $new;
 }
 
-sub _mk_column_accessors {
-  my ($class, @cols) = @_;
-  $class->mk_group_accessors('column' => @cols);
-}
-
-=item add_columns
-
-  __PACKAGE__->add_columns(qw/col1 col2 col3/);
-
-Adds columns to the current package, and creates accessors for them
-
-=cut
-
 sub add_columns {
-  my ($class, @cols) = @_;
-  $class->_register_columns(@cols);
-  $class->_mk_column_accessors(@cols);
-}
-
-sub resultset_instance {
-  my $class = shift;
-  $class->next::method($class->construct_resultset);
-}
-
-sub construct_resultset {
-  my $class = shift;
-  my $rs_class = $class->_resultset_class;
-  eval "use $rs_class;";
-  return $rs_class->new($class);
+  my ($self, @cols) = @_;
+  while (my $col = shift @cols) {
+    $self->add_column($col => (ref $cols[0] ? shift : {}));
+  }
 }
 
-sub _select_columns {
-  return keys %{$_[0]->_columns};
+sub add_column {
+  my ($self, $col, $info) = @_;
+  $self->_columns->{$col} = $info || {};
 }
 
-=item table
+=head2 add_columns
 
-  __PACKAGE__->table('tbl_name');
+  $table->add_columns(qw/col1 col2 col3/);
 
-=cut
-
-sub table {
-  shift->_table_name(@_);
-}
-
-=item find_or_create
+  $table->add_columns('col1' => \%col1_info, 'col2' => \%col2_info, ...);
 
-  $class->find_or_create({ key => $val, ... });
-
-Searches for a record matching the search condition; if it doesn't find one,
-creates one and returns that instead
+Adds columns to the table object. If supplied key => hashref pairs uses
+the hashref as the column_info for that column.
 
 =cut
 
-sub find_or_create {
-  my $class    = shift;
-  my $hash     = ref $_[0] eq "HASH" ? shift: {@_};
-  my $exists = $class->find($hash);
-  return defined($exists) ? $exists : $class->create($hash);
+sub resultset {
+  my $self = shift;
+  my $rs_class = $self->resultset_class;
+  eval "use $rs_class;";
+  return $rs_class->new($self);
 }
 
-=item has_column                                                                
+=head2 has_column                                                                
                                                                                 
   if ($obj->has_column($col)) { ... }                                           
                                                                                 
-Returns 1 if the object has a column of this name, 0 otherwise                  
+Returns 1 if the table has a column of this name, 0 otherwise.                  
                                                                                 
 =cut                                                                            
 
@@ -117,35 +81,33 @@ sub has_column {
   return exists $self->_columns->{$column};
 }
 
-=item column_info                                                               
+=head2 column_info                                                               
                                                                                 
   my $info = $obj->column_info($col);                                           
                                                                                 
-Returns the column metadata hashref for the column                              
+Returns the column metadata hashref for a column.
                                                                                 
 =cut                                                                            
 
 sub column_info {
   my ($self, $column) = @_;
-  die "No such column $column" unless exists $self->_columns->{$column};
+  croak "No such column $column" unless exists $self->_columns->{$column};
   return $self->_columns->{$column};
 }
 
-=item columns                                                                   
+=head2 columns                                                                   
                                                                                 
   my @column_names = $obj->columns;                                             
                                                                                 
 =cut                                                                            
 
 sub columns {
-  die "columns() is a read-only accessor, did you mean add_columns()?" if (@_ > 1);
+  croak "columns() is a read-only accessor, did you mean add_columns()?" if (@_ > 1);
   return keys %{shift->_columns};
 }
 
 1;
 
-=back
-
 =head1 AUTHORS
 
 Matt S. Trout <mst@shadowcatsystems.co.uk>