Added tests for the core APIs, refactored some

[dbsrgits/DBIx-Class-Historic.git] / lib / DBIx / Class / Table.pm

diff --git a/lib/DBIx/Class/Table.pm b/lib/DBIx/Class/Table.pm
index 78a97bd..51d7ceb 100644 (file)
--- a/lib/DBIx/Class/Table.pm
+++ b/lib/DBIx/Class/Table.pm
@@ -9,6 +9,8 @@ __PACKAGE__->mk_classdata('_columns' => {});
 
 __PACKAGE__->mk_classdata('_table_name');
 
+__PACKAGE__->mk_classdata('table_alias'); # FIXME XXX
+
 sub new {
   my ($class, $attrs) = @_;
   $class = ref $class if ref $class;
@@ -28,11 +30,16 @@ sub insert {
   my $sth = $self->_get_sth('insert', [ keys %{$self->{_column_data}} ],
                               $self->_table_name, undef);
   $sth->execute(values %{$self->{_column_data}});
+  $sth->finish;
   $self->{_in_database} = 1;
   $self->{_dirty_columns} = {};
   return $self;
 }
 
+sub in_database {
+  return $_[0]->{_in_database};
+}
+
 sub create {
   my ($class, $attrs) = @_;
   die "create needs a hashref" unless ref $attrs eq 'HASH';
@@ -48,6 +55,7 @@ sub update {
                               $self->_table_name, $self->_ident_cond);
   my $rows = $sth->execute( (map { $self->{_column_data}{$_} } @to_update),
                   $self->_ident_values );
+  $sth->finish;
   if ($rows == 0) {
     die "Can't update $self: row not found";
   } elsif ($rows > 1) {
@@ -60,6 +68,7 @@ sub update {
 sub delete {
   my $self = shift;
   if (ref $self) {
+    die "Not in database" unless $self->{_in_database};
     #warn $self->_ident_cond.' '.join(', ', $self->_ident_values);
     my $sth = $self->_get_sth('delete', undef,
                                 $self->_table_name, $self->_ident_cond);
@@ -123,7 +132,8 @@ sub add_columns {
 sub retrieve_from_sql {
   my ($class, $cond, @vals) = @_;
   $cond =~ s/^\s*WHERE//i;
-  my @cols = $class->_select_columns;
+  my $attrs = (ref $vals[$#vals] eq 'HASH' ? pop(@vals) : {});
+  my @cols = $class->_select_columns($attrs);
   my $sth = $class->_get_sth( 'select', \@cols, $class->_table_name, $cond);
   #warn "$cond @vals";
   return $class->sth_to_objects($sth, \@vals, \@cols);
@@ -140,6 +150,7 @@ sub sth_to_objects {
     $new->{_in_database} = 1;
     push(@found, $new);
   }
+  $sth->finish;
   return @found;
 }
 
@@ -176,6 +187,7 @@ sub copy {
 
 sub _cond_resolve {
   my ($self, $query, $attrs) = @_;
+  return '1 = 1' unless keys %$query;
   my $op = $attrs->{'cmp'} || '=';
   my $cond = join(' AND ',
                map { (defined $query->{$_}