X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FResultSourceProxy.pm;h=1e1f307d35210a82358aa7af99a255218c6c2ef0;hb=05a5ca4b941f0741e91d0972334ab96b96e13f4d;hp=feb0a593c154674275e9a936d34480e01700c3f4;hpb=157ce0cfdee831491de41596b44e19cd7b45dfeb;p=dbsrgits%2FDBIx-Class-Historic.git

diff --git a/lib/DBIx/Class/ResultSourceProxy.pm b/lib/DBIx/Class/ResultSourceProxy.pm
index feb0a59..1e1f307 100644
--- a/lib/DBIx/Class/ResultSourceProxy.pm
+++ b/lib/DBIx/Class/ResultSourceProxy.pm
@@ -4,34 +4,26 @@ package # hide from PAUSE
 use strict;
 use warnings;
 
-use base qw/DBIx::Class/;
-use Scalar::Util qw/blessed/;
-use Carp::Clan qw/^DBIx::Class/;
+use base 'DBIx::Class';
 
-sub iterator_class  { shift->result_source_instance->resultset_class(@_) }
-sub resultset_class { shift->result_source_instance->resultset_class(@_) }
-sub result_class { shift->result_source_instance->result_class(@_) }
-sub source_info { shift->result_source_instance->source_info(@_) }
+use Scalar::Util 'blessed';
+use DBIx::Class::_Util 'quote_sub';
+use namespace::clean;
 
-sub set_inherited_ro_instance {
-    my $self = shift;
-
-    croak "Cannot set @{[shift]} on an instance" if blessed $self;
-
-    return $self->set_inherited(@_);
-}
+__PACKAGE__->mk_group_accessors('inherited_ro_instance' => 'source_name');
 
-sub get_inherited_ro_instance {
-    return shift->get_inherited(@_);
-}
+sub get_inherited_ro_instance {  shift->get_inherited(@_) }
 
-__PACKAGE__->mk_group_accessors('inherited_ro_instance' => 'source_name');
+sub set_inherited_ro_instance {
+  my $self = shift;
 
+  $self->throw_exception ("Cannot set @{[shift]} on an instance")
+    if blessed $self;
 
-sub resultset_attributes {
-  shift->result_source_instance->resultset_attributes(@_);
+  $self->set_inherited(@_);
 }
 
+
 sub add_columns {
   my ($class, @cols) = @_;
   my $source = $class->result_source_instance;
@@ -44,76 +36,56 @@ sub add_columns {
   }
 }
 
-sub add_column {
-  shift->add_columns(@_);
-}
+sub add_column { shift->add_columns(@_) }
 
-sub has_column {
-  shift->result_source_instance->has_column(@_);
-}
-
-sub column_info {
-  shift->result_source_instance->column_info(@_);
-}
-
-sub column_info_from_storage {
-  shift->result_source_instance->column_info_from_storage(@_);
-}
 
-sub columns {
-  shift->result_source_instance->columns(@_);
-}
-
-sub remove_columns {
-  shift->result_source_instance->remove_columns(@_);
+sub add_relationship {
+  my ($class, $rel, @rest) = @_;
+  my $source = $class->result_source_instance;
+  $source->add_relationship($rel => @rest);
+  $class->register_relationship($rel => $source->relationship_info($rel));
 }
 
-*remove_column = \&remove_columns;
 
-sub set_primary_key {
-  shift->result_source_instance->set_primary_key(@_);
-}
+# legacy resultset_class accessor, seems to be used by cdbi only
+sub iterator_class { shift->result_source_instance->resultset_class(@_) }
 
-sub primary_columns {
-  shift->result_source_instance->primary_columns(@_);
-}
+for my $method_to_proxy (qw/
+  source_info
+  result_class
+  resultset_class
+  resultset_attributes
 
-sub _pri_cols {
-  shift->result_source_instance->_pri_cols(@_);
-}
+  columns
+  has_column
 
-sub add_unique_constraint {
-  shift->result_source_instance->add_unique_constraint(@_);
-}
+  remove_column
+  remove_columns
 
-sub unique_constraints {
-  shift->result_source_instance->unique_constraints(@_);
-}
+  column_info
+  columns_info
+  column_info_from_storage
 
-sub unique_constraint_names {
-  shift->result_source_instance->unique_constraint_names(@_);
-}
+  set_primary_key
+  primary_columns
+  sequence
 
-sub unique_constraint_columns {
-  shift->result_source_instance->unique_constraint_columns(@_);
-}
+  add_unique_constraint
+  add_unique_constraints
 
-sub add_relationship {
-  my ($class, $rel, @rest) = @_;
-  my $source = $class->result_source_instance;
-  $source->add_relationship($rel => @rest);
-  $class->register_relationship($rel => $source->relationship_info($rel));
-}
+  unique_constraints
+  unique_constraint_names
+  unique_constraint_columns
 
-sub relationships {
-  shift->result_source_instance->relationships(@_);
-}
+  relationships
+  relationship_info
+  has_relationship
+/) {
+  quote_sub __PACKAGE__."::$method_to_proxy", sprintf( <<'EOC', $method_to_proxy );
+    DBIx::Class::_ENV_::ASSERT_NO_INTERNAL_INDIRECT_CALLS and DBIx::Class::_Util::fail_on_internal_call;
+    shift->result_source_instance->%s (@_);
+EOC
 
-sub relationship_info {
-  shift->result_source_instance->relationship_info(@_);
 }
 
-sub has_relationship {
-  shift->result_source_instance->has_relationship(@_);
-}
 1;