X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FRequirementSpecItem.pm;h=b74ebefd07e411711032679fec60a5b49a326e7b;hb=4fbfbceb32294fde4a5a551fb2cdb6afafb66c57;hp=c6026ce08d7e677120b1887019e4e12f6d46aad3;hpb=d41efcfe571f88b63aa53492e3e3460d948232d6;p=kivitendo-erp.git diff --git a/SL/Controller/RequirementSpecItem.pm b/SL/Controller/RequirementSpecItem.pm index c6026ce08..b74ebefd0 100644 --- a/SL/Controller/RequirementSpecItem.pm +++ b/SL/Controller/RequirementSpecItem.pm @@ -39,7 +39,7 @@ sub action_ajax_list { if (!$::form->{clicked_id}) { # Clicked on "sections" in the tree. Do nothing. - return $self->render; + return $self->render($self->js); } my $clicked_item = SL::DB::RequirementSpecItem->new(id => $::form->{clicked_id})->load; @@ -87,21 +87,21 @@ sub action_dragged_and_dropped { my $old_type = $self->item->item_type; my $new_type = !$dropped_item ? 'section' : $position =~ m/before|after/ ? $dropped_item->item_type : $dropped_item->child_type; - $self->item->db->do_transaction(sub { + $self->item->db->with_transaction(sub { $self->item->remove_from_list; $self->item->parent_id($position =~ m/before|after/ ? $dropped_item->parent_id : $dropped_item->id) if $dropped_item; $self->item->item_type($new_type); $self->item->add_to_list(position => $position, reference => $::form->{dropped_id} || undef); }); - $self->item(SL::DB::RequirementSpecItem->new(id => $self->item->id)->load); - my $new_section = $self->item->section; - my $new_visible_section = SL::DB::RequirementSpecItem->new(id => $self->visible_item->id)->load->section; - return $self->invalidate_version->render if !$old_visible_section || ($new_type eq 'section'); # From here on $old_visible_section is definitely set. + $self->item(SL::DB::RequirementSpecItem->new(id => $self->item->id)->load); + my $new_section = $self->item->section; + my $new_visible_section = SL::DB::RequirementSpecItem->new(id => $self->visible_item->id)->load->section; + my $old_parent = SL::DB::RequirementSpecItem->new(id => $old_parent_id)->load; my $old_section = $old_parent->section;