if ( blessed($updates) && $updates->isa('DBIx::Class::Row') ) {
return $updates;
}
+ # warn Dumper( $updates ); use Data::Dumper;
# direct column accessors
my %columns;
my %post_updates;
my %other_methods;
my %columns_by_accessor = _get_columns_by_accessor( $self );
-
for my $name ( keys %$updates ) {
my $source = $self->result_source;
+ if( $name eq 'id'
+# && scalar @{$source->primary_columns} == 1
+ && !$source->has_column( 'id' )
+ ){
+ my @ids = ( $updates->{id} );
+ if( ref $updates->{id} ){
+ @ids = @{ $updates->{id} };
+ }
+ my $i = 0;
+ for my $key ( $source->primary_columns ){
+ $columns{ $key } = $ids[ $i++ ];
+ }
+ next;
+ }
if ( $columns_by_accessor{$name}
&& !( $source->has_relationship($name) && ref( $updates->{$name} ) )
)
$columns{$name} = $updates->{$name};
next;
}
- if( !( $source->has_relationship($name) && ref( $updates->{$name} ) ) ){
+ if( !( $source->has_relationship($name) ) ){
$other_methods{$name} = $updates->{$name};
+ next;
}
- next if !$source->has_relationship($name);
my $info = $source->relationship_info($name);
if (
_master_relation_cond(
$post_updates{$name} = $updates->{$name};
}
}
- # warn 'columns: ' . Dumper( \%columns ); use Data::Dumper;
+ # warn 'other: ' . Dumper( \%other_methods ); use Data::Dumper;
my @missing =
grep { !exists $columns{$_} && !exists $fixed_fields{$_} } $self->result_source->primary_columns;
$object = $self->find( \%columns, { key => 'primary' } );
}
$object ||= $self->new( {} );
-
# first update columns and other accessors - so that later related records can be found
for my $name ( keys %columns ) {
- $object->$name( $updates->{$name} );
+ $object->$name( $columns{$name} );
}
for my $name ( keys %other_methods) {
$object->$name( $updates->{$name} ) if $object->can( $name );
my ($pk) = _get_pk_for_related( $self, $name);
my @rows;
my $result_source = $object->$name->result_source;
- for my $elem ( @{ $updates->{$name} } ) {
+ my @updates;
+ if( ! defined $value ){
+ next;
+ }
+ elsif( ref $value ){
+ @updates = @{ $value };
+ }
+ else{
+ @updates = ( $value );
+ }
+ for my $elem ( @updates ) {
if ( ref $elem ) {
push @rows, $result_source->resultset->find($elem);
}
}
else {
my $sub_updates = $updates->{$name};
- $sub_updates = { %$sub_updates, %$resolved } if $resolved && ref( $sub_updates ) eq 'HASH';
my $sub_object;
- if( $info->{attrs}{accessor} eq 'single' && defined $object->$name ){
- $sub_object = recursive_update(
- resultset => $related_result,
- updates => $sub_updates,
- object => $object->$name
- );
+ if( ref $sub_updates ){
+ $sub_updates = { %$sub_updates, %$resolved } if $resolved && ref( $sub_updates ) eq 'HASH';
+ # for might_have relationship
+ if( $info->{attrs}{accessor} eq 'single' && defined $object->$name ){
+ $sub_object = recursive_update(
+ resultset => $related_result,
+ updates => $sub_updates,
+ object => $object->$name
+ );
+ }
+ else{
+ $sub_object =
+ recursive_update( resultset => $related_result, updates => $sub_updates );
+ }
+ }
+ elsif( ! ref $sub_updates ){
+ $sub_object = $related_result->find( $sub_updates );
}
- else{
- $sub_object =
- recursive_update( resultset => $related_result, updates => $sub_updates );
- }
$object->set_from_related( $name, $sub_object );
}
}