X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FRelationship%2FHasOne.pm;h=aa94a08b80317a0b51eef7bc232d98e41d0b9b5d;hb=25594f038dc8bb8d8c3f110ccfbebe30774043be;hp=d114a398497d98810ba76fd9c0786ca036bec2a6;hpb=103647d504eeadac7d179057e9f4d5eda0cd7c1b;p=dbsrgits%2FDBIx-Class-Historic.git diff --git a/lib/DBIx/Class/Relationship/HasOne.pm b/lib/DBIx/Class/Relationship/HasOne.pm index d114a39..aa94a08 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,19 +9,18 @@ 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"; + $class->ensure_class_loaded($f_class); unless (ref $cond) { my ($pri, $too_many) = $class->primary_columns; - $class->throw( "might_have/has_one can only infer join for a single primary key; ${class} has more" ) + $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_key,$guess); if (defined $cond && length $cond) { $f_key = $cond; $guess = "caller specified foreign key '$f_key'"; @@ -29,11 +29,11 @@ sub _has_one { $guess = "using given relationship '$rel' for foreign key"; } else { ($f_key, $too_many) = $f_class->primary_columns; - $class->throw( "might_have/has_one can only infer join for a single primary key; ${f_class} has more" ) + $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)") + $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}" }; }