X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FRelationship%2FHasMany.pm;h=eecda46da1d2bb6e5d6b7bc0d107923c46c0f7fd;hb=229401a04e99e27e256fdcd24b3c34087c9c2bc1;hp=f3469977441fc200f7d55995e3c248074be7bcb0;hpb=52cd24db1219c355ff236324f4b4ae2ef93c657b;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Relationship/HasMany.pm b/lib/DBIx/Class/Relationship/HasMany.pm index f346997..eecda46 100644 --- a/lib/DBIx/Class/Relationship/HasMany.pm +++ b/lib/DBIx/Class/Relationship/HasMany.pm @@ -15,16 +15,8 @@ sub has_many { my ($class, $rel, $f_class, $cond, $attrs) = @_; unless (ref $cond) { - $class->ensure_class_loaded($f_class); - my ($pri, $too_many) = try { $class->result_source_instance->_pri_cols_or_die } - catch { - $class->throw_exception("Can't infer join condition for '$rel' on ${class}: $_"); - }; - $class->throw_exception( - "has_many can only infer join for a single primary key; ". - "${class} has more" - ) if $too_many; + my $pri = $class->result_source_instance->_single_pri_col_or_die; my ($f_key,$guess); if (defined $cond && length $cond) { @@ -33,13 +25,16 @@ sub has_many { } else { $class =~ /([^\:]+)$/; # match is safe - $class can't be '' $f_key = lc $1; # go ahead and guess; best we can do - $guess = "using our class name '$class' as foreign key"; + $guess = "using our class name '$class' as foreign key source"; } - my $f_class_loaded = try { $f_class->columns }; - $class->throw_exception( - "No such column '$f_key' on foreign class ${f_class} ($guess)" - ) if $f_class_loaded && !$f_class->has_column($f_key); +# FIXME - this check needs to be moved to schema-composition time... +# # only perform checks if the far side appears already loaded +# if (my $f_rsrc = try { $f_class->result_source_instance } ) { +# $class->throw_exception( +# "No such column '$f_key' on foreign class ${f_class} ($guess)" +# ) if !$f_rsrc->has_column($f_key); +# } $cond = { "foreign.${f_key}" => "self.${pri}" }; } @@ -51,6 +46,7 @@ sub has_many { join_type => 'LEFT', cascade_delete => $default_cascade, cascade_copy => $default_cascade, + is_depends_on => 0, %{$attrs||{}} }); }