X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FRelationship%2FHasMany.pm;h=fd84b304881521afc4bfc04b2d4e4edb18d4d9ff;hb=994f4d9abe4e7e9c78c56e74b76c6bbd7e79db33;hp=eeead265c07be9ac00cb0f698bbb161fa5c9a3f9;hpb=303cf522b1c2841ecab3056cfbb4314fa05151a2;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Relationship/HasMany.pm b/lib/DBIx/Class/Relationship/HasMany.pm index eeead26..fd84b30 100644 --- a/lib/DBIx/Class/Relationship/HasMany.pm +++ b/lib/DBIx/Class/Relationship/HasMany.pm @@ -3,41 +3,49 @@ package # hide from PAUSE use strict; use warnings; +use Try::Tiny; +use namespace::clean; + +our %_pod_inherit_config = + ( + class_map => { 'DBIx::Class::Relationship::HasMany' => 'DBIx::Class::Relationship' } + ); sub has_many { my ($class, $rel, $f_class, $cond, $attrs) = @_; - $class->ensure_class_loaded($f_class); - unless (ref $cond) { - my ($pri, $too_many) = $class->primary_columns; - $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) { $f_key = $cond; $guess = "caller specified foreign key '$f_key'"; } else { - $class =~ /([^\:]+)$/; + $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 = eval { $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}" }; } + my $default_cascade = ref $cond eq 'CODE' ? 0 : 1; + $class->add_relationship($rel, $f_class, $cond, { accessor => 'multi', join_type => 'LEFT', - cascade_delete => 1, - cascade_copy => 1, + cascade_delete => $default_cascade, + cascade_copy => $default_cascade, %{$attrs||{}} }); }