X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FRelationship%2FBase.pm;h=45909856f66e869935f306b95d1a151d1eff6b2f;hb=55de06f116aadb02326d4613a4db7117941e5350;hp=27479955522da5769a23937314e0119ec1273e2f;hpb=8918977e98800ece6e7eefcf292d7512138ad48d;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Relationship/Base.pm b/lib/DBIx/Class/Relationship/Base.pm index 2747995..4590985 100644 --- a/lib/DBIx/Class/Relationship/Base.pm +++ b/lib/DBIx/Class/Relationship/Base.pm @@ -3,7 +3,7 @@ package DBIx::Class::Relationship::Base; use strict; use warnings; -use base qw/Class::Data::Inheritable/; +use base qw/DBIx::Class/; __PACKAGE__->mk_classdata('_relationships', { } ); @@ -43,14 +43,16 @@ sub add_relationship { die "Can't create relationship without join condition" unless $cond; $attrs ||= {}; eval "require $f_class;"; + if ($@) { + $class->throw($@) unless $@ =~ /Can't locate/; + } my %rels = %{ $class->_relationships }; $rels{$rel} = { class => $f_class, cond => $cond, attrs => $attrs }; $class->_relationships(\%rels); - #warn %{$f_class->_columns}; - return unless eval { %{$f_class->_columns}; }; # Foreign class not loaded + return unless eval { $f_class->can('columns'); }; # Foreign class not loaded eval { $class->_resolve_join($rel, 'me') }; if ($@) { # If the resolve failed, back out and re-throw the error @@ -117,14 +119,14 @@ sub _cond_key { if (my $alias = $attrs->{_aliases}{$type}) { my $class = $attrs->{_classes}{$alias}; $self->throw("Unknown column $field on $class as $alias") - unless exists $class->_columns->{$field}; + unless $class->has_column($field); return join('.', $alias, $field); } else { $self->throw( "Unable to resolve type ${type}: only have aliases for ". join(', ', keys %{$attrs->{_aliases} || {}}) ); } } - return $self->NEXT::ACTUAL::_cond_key($attrs, $key); + return $self->next::method($attrs, $key); } sub _cond_value { @@ -134,7 +136,7 @@ sub _cond_value { unless ($value =~ s/^self\.//) { $self->throw( "Unable to convert relationship to WHERE clause: invalid value ${value}" ); } - unless ($self->_columns->{$value}) { + unless ($self->has_column($value)) { $self->throw( "Unable to convert relationship to WHERE clause: no such accessor ${value}" ); } return $self->get_column($value); @@ -144,7 +146,7 @@ sub _cond_value { if (my $alias = $attrs->{_aliases}{$type}) { my $class = $attrs->{_classes}{$alias}; $self->throw("Unknown column $field on $class as $alias") - unless exists $class->_columns->{$field}; + unless $class->has_column($field); return join('.', $alias, $field); } else { $self->throw( "Unable to resolve type ${type}: only have aliases for ". @@ -152,7 +154,7 @@ sub _cond_value { } } - return $self->NEXT::ACTUAL::_cond_value($attrs, $key, $value) + return $self->next::method($attrs, $key, $value) } =item search_related