X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FSchema%2FLoader%2FRelBuilder.pm;h=6f32ff1e6766fe0263a3367ad2578fdd9cdd6104;hb=ae151d4f7f9d1bff0e912b03ff97c1aec918f867;hp=40ae096f764c07d1adf144ea4da64b2389706c53;hpb=533d98c061845dd1d49e6ff94f7acee618b23764;p=dbsrgits%2FDBIx-Class-Schema-Loader.git diff --git a/lib/DBIx/Class/Schema/Loader/RelBuilder.pm b/lib/DBIx/Class/Schema/Loader/RelBuilder.pm index 40ae096..6f32ff1 100644 --- a/lib/DBIx/Class/Schema/Loader/RelBuilder.pm +++ b/lib/DBIx/Class/Schema/Loader/RelBuilder.pm @@ -18,7 +18,7 @@ use String::ToIdentifier::EN::Unicode (); use Class::Unload (); use Class::Inspector (); -our $VERSION = '0.07029'; +our $VERSION = '0.07033'; # Glossary: # @@ -149,8 +149,8 @@ sub new { # validate the relationship_attrs arg if( defined $self->relationship_attrs ) { - ref $self->relationship_attrs eq 'HASH' - or croak "relationship_attrs must be a hashref"; + (ref $self->relationship_attrs eq 'HASH' || ref $self->relationship_attrs eq 'CODE') + or croak "relationship_attrs must be a hashref or coderef"; } return $self; @@ -257,16 +257,30 @@ sub _default_relationship_attrs { +{ # The attributes from the database override the default attributes, which in # turn are overridden by user supplied attributes. sub _relationship_attrs { - my ( $self, $reltype, $db_attrs ) = @_; + my ( $self, $reltype, $db_attrs, $params ) = @_; my $r = $self->relationship_attrs; my %composite = ( %{ $self->_default_relationship_attrs->{$reltype} || {} }, %{ $db_attrs || {} }, - %{ $r->{all} || {} }, - %{ $r->{$reltype} || {} }, + ( + ref $r eq 'HASH' ? ( + %{ $r->{all} || {} }, + %{ $r->{$reltype} || {} }, + ) + : + () + ), ); + if (ref $r eq 'CODE') { + $params->{attrs} = \%composite; + + my %ret = %{ $r->(%$params) || {} }; + + %composite = %ret if %ret; + } + return %composite ? \%composite : undef; } @@ -279,10 +293,10 @@ sub _strip_id_postfix { } sub _remote_attrs { - my ($self, $local_moniker, $local_cols, $fk_attrs) = @_; + my ($self, $local_moniker, $local_cols, $fk_attrs, $params) = @_; # get our set of attrs from _relationship_attrs, which uses the FK attrs if available - my $attrs = $self->_relationship_attrs('belongs_to', $fk_attrs) || {}; + my $attrs = $self->_relationship_attrs('belongs_to', $fk_attrs, $params) || {}; # If any referring column is nullable, make 'belongs_to' an # outer join, unless explicitly set by relationship_attrs @@ -409,12 +423,22 @@ sub generate_code { $remote_relname = $self->_resolve_relname_collision($local_moniker, $local_cols, $remote_relname); $local_relname = $self->_resolve_relname_collision($remote_moniker, $remote_cols, $local_relname); + my $rel_attrs_params = { + rel_name => $remote_relname, + local_source => $self->schema->source($local_moniker), + remote_source => $self->schema->source($remote_moniker), + local_table => $rel->{local_table}, + local_cols => $local_cols, + remote_table => $rel->{remote_table}, + remote_cols => $remote_cols, + }; + push(@{$all_code->{$local_class}}, { method => $local_method, args => [ $remote_relname, $remote_class, \%cond, - $self->_remote_attrs($local_moniker, $local_cols, $rel->{attrs}), + $self->_remote_attrs($local_moniker, $local_cols, $rel->{attrs}, $rel_attrs_params), ], extra => { local_class => $local_class, @@ -430,12 +454,22 @@ sub generate_code { delete $rev_cond{$_}; } + $rel_attrs_params = { + rel_name => $local_relname, + local_source => $self->schema->source($remote_moniker), + remote_source => $self->schema->source($local_moniker), + local_table => $rel->{remote_table}, + local_cols => $remote_cols, + remote_table => $rel->{local_table}, + remote_cols => $local_cols, + }; + push(@{$all_code->{$remote_class}}, { method => $remote_method, args => [ $local_relname, $local_class, \%rev_cond, - $self->_relationship_attrs($remote_method), + $self->_relationship_attrs($remote_method, {}, $rel_attrs_params), ], extra => { local_class => $remote_class,