From: Peter Rabbitson Date: Fri, 13 Jun 2014 14:37:31 +0000 (+0200) Subject: More fallout of the _resolve_cond refactor 03f6d1f7 X-Git-Tag: v0.082800~172 X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=dbsrgits%2FDBIx-Class.git;a=commitdiff_plain;h=5592d633292c0fecf45b554ab6d4b235b140dc15 More fallout of the _resolve_cond refactor 03f6d1f7 I had in the back of my mind "one can't just create random objects" but figured it may work anyway - WROOOOOOOONG. Instead pass through the hash as before, will need to rethink that part before settling on an API for resolve_relationship_condition. FML --- diff --git a/lib/DBIx/Class/ResultSource.pm b/lib/DBIx/Class/ResultSource.pm index fb4c27d..ce88cbb 100644 --- a/lib/DBIx/Class/ResultSource.pm +++ b/lib/DBIx/Class/ResultSource.pm @@ -1704,11 +1704,10 @@ sub _resolve_condition { else { $res_args[$_] ||= {}; + # hate everywhere - have to pass in as a plain hash + # pretending to be an object at least for now $self->throw_exception("Unsupported object-like structure encountered: $res_args[$_]") unless ref $res_args[$_] eq 'HASH'; - - # hate everywhere - $res_args[$_] = $self->relationship_info($rel_name)->{source}->result_class->new($res_args[$_]); } } @@ -1881,7 +1880,16 @@ sub _resolve_relationship_condition { ; for my $i (0..$#$obj_cols) { - if (defined $args->{self_resultobj} and ! $obj->has_column_loaded($obj_cols->[$i])) { + + # FIXME - temp shim + if (! blessed $obj) { + $cond->{"$plain_alias.$plain_cols->[$i]"} = $obj->{$obj_cols->[$i]}; + } + elsif ( + defined $args->{self_resultobj} + and + ! $obj->has_column_loaded($obj_cols->[$i]) + ) { $self->throw_exception(sprintf "Unable to resolve relationship '%s' from object '%s': column '%s' not "