X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FRelationship%2FHasOne.pm;h=4efbec012449cad72500055b94cd0e9ffc14baef;hb=0d67fe74ce59e5c6b6594c0a9235b9e311bc4d04;hp=b9ab33d901fae682c582fa2ba9cef0908683a6ef;hpb=dcf8330b075eee8c71d30e0fa31289d162482fe4;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Relationship/HasOne.pm b/lib/DBIx/Class/Relationship/HasOne.pm index b9ab33d..4efbec0 100644 --- a/lib/DBIx/Class/Relationship/HasOne.pm +++ b/lib/DBIx/Class/Relationship/HasOne.pm @@ -1,4 +1,5 @@ -package DBIx::Class::Relationship::HasOne; +package # hide from PAUSE + DBIx::Class::Relationship::HasOne; use strict; use warnings; @@ -8,33 +9,36 @@ sub might_have { } sub has_one { - shift->_has_one(undef => @_); + shift->_has_one(undef() => @_); } sub _has_one { my ($class, $join_type, $rel, $f_class, $cond, $attrs) = @_; eval "require $f_class"; + if ($@) { + $class->throw_exception($@) unless $@ =~ /Can't locate/; + } + unless (ref $cond) { - my ($pri, $too_many) = keys %{ $class->_primaries }; - $class->throw( "might_have/has_one can only infer join for a single primary key; ${class} has more" ) + my ($pri, $too_many) = $class->primary_columns; + $class->throw_exception( "might_have/has_one can only infer join for a single primary key; ${class} has more" ) if $too_many; - my $f_key; - my $f_class_loaded = eval { $f_class->_columns }; - my $guess; + my $f_class_loaded = eval { $f_class->columns }; + my ($f_key,$guess); if (defined $cond && length $cond) { $f_key = $cond; $guess = "caller specified foreign key '$f_key'"; - } elsif ($f_class_loaded && $f_class->_columns->{$rel}) { + } elsif ($f_class_loaded && $f_class->has_column($rel)) { $f_key = $rel; $guess = "using given relationship '$rel' for foreign key"; } else { - ($f_key, $too_many) = keys %{ $f_class->_primaries }; - $class->throw( "might_have/has_one can only infer join for a single primary key; ${f_class} has more" ) + ($f_key, $too_many) = $f_class->primary_columns; + $class->throw_exception( "might_have/has_one can only infer join for a single primary key; ${f_class} has more" ) if $too_many; $guess = "using primary key of foreign class for foreign key"; } - $class->throw("No such column ${f_key} on foreign class ${f_class} ($guess)") - if $f_class_loaded && !exists $f_class->_columns->{$f_key}; + $class->throw_exception("No such column ${f_key} on foreign class ${f_class} ($guess)") + if $f_class_loaded && !$f_class->has_column($f_key); $cond = { "foreign.${f_key}" => "self.${pri}" }; } $class->add_relationship($rel, $f_class,