X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;ds=sidebyside;f=SL%2FController%2FRequirementSpecItem.pm;h=2cbf11447ff7c6e528ad943b08e5538ae543b05a;hb=32f83e8e1c88c04cd9a8686d8482b1473a9d73c5;hp=8ae9f573512d98ba98661d139243486f9d2813d7;hpb=48d73a85a323578cf2fafde0f6716dd164213856;p=kivitendo-erp.git

diff --git a/SL/Controller/RequirementSpecItem.pm b/SL/Controller/RequirementSpecItem.pm
index 8ae9f5735..2cbf11447 100644
--- a/SL/Controller/RequirementSpecItem.pm
+++ b/SL/Controller/RequirementSpecItem.pm
@@ -9,6 +9,8 @@ use List::MoreUtils qw(apply);
 use List::Util qw(first);
 use Time::HiRes ();
 
+use SL::Clipboard;
+use SL::Controller::Helper::RequirementSpec;
 use SL::DB::RequirementSpec;
 use SL::DB::RequirementSpecComplexity;
 use SL::DB::RequirementSpecItem;
@@ -19,11 +21,12 @@ use SL::Locale::String;
 
 use Rose::Object::MakeMethods::Generic
 (
-  scalar                  => [ qw(item visible_item visible_section) ],
-  'scalar --get_set_init' => [ qw(complexities risks) ],
+  scalar                  => [ qw(item visible_item visible_section clicked_item sections) ],
+  'scalar --get_set_init' => [ qw(complexities risks js) ],
 );
 
-__PACKAGE__->run_before('load_requirement_spec_item', only => [ qw(dragged_and_dropped ajax_update ajax_edit ajax_delete ajax_flag) ]);
+__PACKAGE__->run_before('check_auth');
+__PACKAGE__->run_before('load_requirement_spec_item', only => [ qw(dragged_and_dropped ajax_update ajax_edit ajax_delete ajax_flag ajax_copy) ]);
 __PACKAGE__->run_before('init_visible_section');
 
 #
@@ -31,25 +34,45 @@ __PACKAGE__->run_before('init_visible_section');
 #
 
 sub action_ajax_list {
-  my ($self, $js) = @_;
-
-  my $js = SL::ClientJS->new;
+  my ($self) = @_;
 
   if (!$::form->{clicked_id}) {
     # Clicked on "sections" in the tree. Do nothing.
-    return $self->render($js);
+    return $self->render($self->js);
   }
 
   my $clicked_item = SL::DB::RequirementSpecItem->new(id => $::form->{clicked_id})->load;
   $self->item($clicked_item->section);
 
   if (!$self->visible_section || ($self->visible_section->id != $self->item->id)) {
-    $self->render_list($js, $self->item, $clicked_item);
+    $self->render_list($self->item, $clicked_item);
+  } else {
+    $self->select_node($clicked_item);
+  }
+
+  $self->render($self->js);
+}
+
+sub insert_new_item_in_section_view {
+  my ($self) = @_;
+
+  $self->js->hide('#section-list-empty');
+
+  my $new_type  = $self->item->item_type;
+  my $id_prefix = $new_type eq 'sub-function-block' ? 'sub-' : '';
+  my $template  = 'requirement_spec_item/_' . (apply { s/-/_/g; $_ } $new_type);
+  my $html      = "" . $self->render($template, { output => 0 }, requirement_spec_item => $self->item);
+  my $next_item = $self->item->get_next_in_list;
+
+  if ($next_item) {
+    $self->js->insertBefore($html, '#' . $id_prefix . 'function-block-' . $next_item->id);
   } else {
-    $self->select_node($js, $clicked_item);
+    my $parent_is_section = $self->item->parent->item_type eq 'section';
+    $self->js->appendTo($html, $parent_is_section ? '#section-list' : '#sub-function-block-container-' . $self->item->parent_id);
+    $self->js->show('#sub-function-block-container-' . $self->item->parent_id) if !$parent_is_section;
   }
 
-  $self->render($js);
+  $self->replace_bottom($self->item->parent) if $new_type eq 'sub-function-block';
 }
 
 sub action_dragged_and_dropped {
@@ -68,14 +91,12 @@ sub action_dragged_and_dropped {
     $self->item->add_to_list(position => $position, reference => $::form->{dropped_id} || undef);
   });
 
-  my $js = SL::ClientJS->new;
-
   $self->item(SL::DB::RequirementSpecItem->new(id => $self->item->id)->load);
   my $new_section         = $self->item->section;
   my $new_type            = $self->item->item_type;
   my $new_visible_section = SL::DB::RequirementSpecItem->new(id => $self->visible_item->id)->load->section;
 
-  return $self->render($js) if !$old_visible_section || ($new_type eq 'section');
+  return $self->invalidate_version->render($self) if !$old_visible_section || ($new_type eq 'section');
 
   # From here on $old_visible_section is definitely set.
 
@@ -87,45 +108,31 @@ sub action_dragged_and_dropped {
 
   if ($old_visible_section->id != $new_visible_section->id) {
     # The currently visible item has been dragged to a different section.
-    return $self->render_list($js, $new_section, $self->item)
+    $self->invalidate_version;
+    return $self->render_list($new_section, $self->item)
       ->render($self);
   }
 
   if ($old_visible_section->id == $old_section->id) {
     my $id_prefix = $old_type eq 'sub-function-block' ? 'sub-' : '';
-    $js->remove('#' . $id_prefix . 'function-block-' . $self->item->id);
+    $self->js->remove('#' . $id_prefix . 'function-block-' . $self->item->id);
 
     if ($old_type eq 'sub-function-block') {
-      $self->replace_bottom($js, $old_parent) ;
-      $js->hide('#sub-function-block-container-' . $old_parent->id) if 0 == scalar(@{ $old_parent->children });
+      $self->replace_bottom($old_parent) ;
+      $self->js->hide('#sub-function-block-container-' . $old_parent->id) if 0 == scalar(@{ $old_parent->children });
 
     } elsif (0 == scalar(@{ $old_section->children })) {
-      $js->show('#section-list-empty');
+      $self->js->show('#section-list-empty');
     }
   }
 
   if ($old_visible_section->id == $new_section->id) {
-    $js->hide('#section-list-empty');
-
-    my $id_prefix = $new_type eq 'sub-function-block' ? 'sub-' : '';
-    my $template  = 'requirement_spec_item/_' . (apply { s/-/_/g; $_ } $new_type);
-    my $html      = "" . $self->render($template, { output => 0 }, requirement_spec_item => $self->item);
-    my $next_item = $self->item->get_next_in_list;
-
-    if ($next_item) {
-      $js->insertBefore($html, '#' . $id_prefix . 'function-block-' . $next_item->id);
-    } else {
-      my $parent_is_section = $self->item->parent->item_type eq 'section';
-      $js->appendTo($html, $parent_is_section ? '#section-list' : '#sub-function-block-container-' . $self->item->parent_id);
-      $js->show('#sub-function-block-container-' . $self->item->parent_id) if !$parent_is_section;
-    }
-
-    $self->replace_bottom($js, $self->item->parent) if $new_type eq 'sub-function-block';
+    $self->insert_new_item_in_section_view($self->js);
   }
 
-  # $::lxdebug->dump(0, "js", $js->to_array);
+  # $::lxdebug->dump(0, "js", $self->js->to_array);
 
-  $self->render($js);
+  $self->invalidate_version->render($self);
 }
 
 sub action_ajax_add_section {
@@ -138,7 +145,7 @@ sub action_ajax_add_section {
   my $insert_after = $::form->{id} ? SL::DB::RequirementSpecItem->new(id => $::form->{id})->load->section->id : undef;
   my $html         = $self->render('requirement_spec_item/_section_form', { output => 0 }, id_base => 'new_section', insert_after => $insert_after);
 
-  SL::ClientJS->new
+  $self->js
     ->remove('#new_section_form')
     ->hide('#column-content > *')
     ->appendTo($html, '#column-content')
@@ -161,13 +168,16 @@ sub action_ajax_add_sub_function_block {
 sub action_ajax_create {
   my ($self, %params) = @_;
 
-  my $js              = SL::ClientJS->new;
   my $prefix          = $::form->{form_prefix} || die "Missing parameter 'form_prefix'";
   my $attributes      = $::form->{$prefix}     || die "Missing parameter group '${prefix}'";
   my $insert_after    = delete $attributes->{insert_after};
 
+  if (!$attributes->{parent_id}) {
+    $attributes->{order_part_id} ||= $::instance_conf->get_requirement_spec_section_order_part_id;
+  }
+
   my @errors = $self->item(SL::DB::RequirementSpecItem->new(%{ $attributes }))->validate;
-  return $js->error(@errors)->render($self) if @errors;
+  return $self->js->error(@errors)->render($self) if @errors;
 
   $self->item->save;
   $self->item->add_to_list(position => 'after', reference => $insert_after) if $insert_after;
@@ -176,7 +186,8 @@ sub action_ajax_create {
 
   if ($type eq 'section') {
     my $node = $self->presenter->requirement_spec_item_jstree_data($self->item);
-    return $self->render_list($js, $self->item)
+    $self->invalidate_version;
+    return $self->render_list($self->item)
       ->jstree->create_node('#tree', $insert_after ? ('#fb-' . $insert_after, 'after') : ('#sections', 'last'), $node)
       ->jstree->select_node('#tree', '#fb-' . $self->item->id)
       ->render($self);
@@ -186,14 +197,15 @@ sub action_ajax_create {
   my $html     = $self->render($template, { output => 0 }, requirement_spec_item => $self->item, id_prefix => $type eq 'function-block' ? '' : 'sub-');
   my $node     = $self->presenter->requirement_spec_item_jstree_data($self->item);
 
-  $js->replaceWith('#' . $prefix . '_form', $html)
-     ->hide('#section-list-empty')
-     ->jstree->create_node('#tree', $insert_after ? ('#fb-' . $insert_after, 'after') : ('#fb-' . $self->item->parent_id, 'last'), $node)
-     ->jstree->select_node('#tree', '#fb-' . $self->item->id);
+  $self->js
+    ->replaceWith('#' . $prefix . '_form', $html)
+    ->hide('#section-list-empty')
+    ->jstree->create_node('#tree', $insert_after ? ('#fb-' . $insert_after, 'after') : ('#fb-' . $self->item->parent_id, 'last'), $node)
+    ->jstree->select_node('#tree', '#fb-' . $self->item->id);
 
-  $self->replace_bottom($js, $self->item->parent) if $type eq 'sub-function-block';
+  $self->replace_bottom($self->item->parent) if $type eq 'sub-function-block';
 
-  $js->render($self);
+  $self->invalidate_version->render($self);
 }
 
 sub action_ajax_edit {
@@ -201,27 +213,26 @@ sub action_ajax_edit {
 
   $self->item(SL::DB::RequirementSpecItem->new(id => $::form->{id})->load);
 
-  my $js = SL::ClientJS->new;
-
-  if (!$self->visible_section || ($self->visible_section->id != $self->item->section->id)) {
+  if (!$self->is_item_visible) {
     # Show section/item to edit if it is not visible.
 
-    my $html = $self->render('requirement_spec_item/_section', { output => 0 }, requirement_spec_item => $self->item);
-    $js->html('#column-content', $html);
+    my $html = $self->render('requirement_spec_item/_section', { output => 0 }, requirement_spec_item => $self->item->section);
+    $self->js->html('#column-content', $html);
   }
 
   if ($self->item->item_type =~ m/section/) {
     # Edit the section header, not an item.
     my $html = $self->render('requirement_spec_item/_section_form', { output => 0 });
 
-    $js->hide('#section-header-' . $self->item->id)
-       ->remove("#edit_section_form")
-       ->insertAfter($html, '#section-header-' . $self->item->id)
-       ->jstree->select_node('#tree', '#fb-' . $self->item->id)
-       ->focus("#edit_section_title")
-       ->val('#current_content_type', 'section')
-       ->val('#current_content_id',   $self->item->id)
-       ->render($self);
+    $self->js
+      ->hide('#section-header-' . $self->item->id)
+      ->remove("#edit_section_form")
+      ->insertAfter($html, '#section-header-' . $self->item->id)
+      ->jstree->select_node('#tree', '#fb-' . $self->item->id)
+      ->focus("#edit_section_title")
+      ->val('#current_content_type', 'section')
+      ->val('#current_content_id',   $self->item->id)
+      ->render($self);
     return;
   }
 
@@ -233,20 +244,20 @@ sub action_ajax_edit {
   my $id_base               = 'edit_function_block_' . $self->item->id;
   my $content_top_id        = '#' . $self->item->item_type . '-content-top-' . $self->item->id;
 
-  $js->hide($content_top_id)
-     ->remove("#${id_base}_form")
-     ->insertAfter($html, $content_top_id)
-     ->jstree->select_node('#tree', '#fb-' . $self->item->id)
-     ->focus("#${id_base}_description")
-     ->val('#current_content_type', $self->item->item_type)
-     ->val('#current_content_id', $self->item->id)
-     ->render($self);
+  $self->js
+    ->hide($content_top_id)
+    ->remove("#${id_base}_form")
+    ->insertAfter($html, $content_top_id)
+    ->jstree->select_node('#tree', '#fb-' . $self->item->id)
+    ->focus("#${id_base}_description")
+    ->val('#current_content_type', $self->item->item_type)
+    ->val('#current_content_id', $self->item->id)
+    ->render($self);
 }
 
 sub action_ajax_update {
   my ($self, %params) = @_;
 
-  my $js         = SL::ClientJS->new;
   my $prefix     = $::form->{form_prefix} || die "Missing parameter 'form_prefix'";
   my $attributes = $::form->{$prefix}     || {};
 
@@ -255,7 +266,7 @@ sub action_ajax_update {
   }
 
   my @errors = $self->item->assign_attributes(%{ $attributes })->validate;
-  return $js->error(@errors)->render($self) if @errors;
+  return $self->js->error(@errors)->render($self) if @errors;
 
   $self->item->save;
 
@@ -266,7 +277,7 @@ sub action_ajax_update {
 
     my $html = $self->render('requirement_spec_item/_section_header', { output => 0 }, requirement_spec_item => $self->item);
 
-    return SL::ClientJS->new
+    return $self->invalidate_version
       ->remove('#edit_section_form')
       ->html('#section-header-' . $self->item->id, $html)
       ->show('#section-header-' . $self->item->id)
@@ -282,21 +293,20 @@ sub action_ajax_update {
   my $html_top     = $self->render('requirement_spec_item/_function_block_content_top',    { output => 0 }, requirement_spec_item => $self->item, id_prefix => $id_prefix);
   $id_prefix      .= 'function-block-content-';
 
-  my $js = SL::ClientJS->new
+  $self->js
     ->remove('#' . $prefix . '_form')
     ->replaceWith('#' . $id_prefix . 'top-' . $self->item->id, $html_top)
     ->jstree->rename_node('#tree', '#fb-' . $self->item->id, $::request->presenter->requirement_spec_item_tree_node_title($self->item));
 
-  $self->replace_bottom($js, $self->item, id_prefix => $id_prefix);
-  $self->replace_bottom($js, $self->item->parent) if $type eq 'sub-function-block';
+  $self->replace_bottom($self->item, id_prefix => $id_prefix);
+  $self->replace_bottom($self->item->parent) if $type eq 'sub-function-block';
 
-  $js->render($self);
+  $self->invalidate_version->render($self);
 }
 
 sub action_ajax_delete {
   my ($self) = @_;
 
-  my $js        = SL::ClientJS->new;
   my $full_list = $self->item->get_full_list;
 
   $self->item->delete;
@@ -306,35 +316,37 @@ sub action_ajax_delete {
 
     my $new_section = first { $_->id != $self->item->id } @{ $self->item->requirement_spec->sections };
     if ($new_section) {
-      $self->render_list($js, $new_section);
+      $self->render_list($new_section);
 
     } else {
       my $html = $self->render('requirement_spec_item/_no_section', { output => 0 });
-      $js->html('#column-content', $html)
-         ->val('#current_content_type', '')
-         ->val('#current_content_id', '')
+      $self->js
+        ->html('#column-content',      $html)
+        ->val('#current_content_type', '')
+        ->val('#current_content_id',   '')
     }
 
-  } elsif ($self->visible_section && ($self->visible_section->id == $self->item->section->id)) {
+  } elsif ($self->is_item_visible) {
     # Item in currently visible section is deleted.
 
     my $type = $self->item->item_type;
-    $js->remove('#edit_function_block_' . $self->item->id . '_form')
-       ->remove('#' . $type . '-' . $self->item->id);
+    $self->js->remove('#edit_function_block_' . $self->item->id . '_form')
+             ->remove('#' . $type . '-' . $self->item->id);
 
-    $self->replace_bottom($js, $self->item->parent_id) if $type eq 'sub-function-block';
+    $self->replace_bottom($self->item->parent_id) if $type eq 'sub-function-block';
 
     if (1 == scalar @{ $full_list }) {
       if ($type eq 'function-block') {
-        $js->show('#section-list-empty');
+        $self->js->show('#section-list-empty');
       } elsif ($type eq 'sub-function-block') {
-        $js->hide('#sub-function-block-container-' . $self->item->parent_id);
+        $self->js->hide('#sub-function-block-container-' . $self->item->parent_id);
       }
     }
   }
 
-  $js->jstree->delete_node('#tree', '#fb-' . $self->item->id)
-     ->render($self);
+  $self->invalidate_version
+    ->jstree->delete_node('#tree', '#fb-' . $self->item->id)
+    ->render($self);
 }
 
 sub action_ajax_flag {
@@ -342,26 +354,131 @@ sub action_ajax_flag {
 
   $self->item->update_attributes(is_flagged => !$self->item->is_flagged);
 
-  my $is_visible = $self->visible_section && ($self->visible_section->id == $self->item->section->id);
-
-  SL::ClientJS->new
-   ->action_if($is_visible, 'toggleClass', '#' . $self->item->item_type . '-' . $self->item->id, 'flagged')
+  $self->js
+   ->action_if($self->is_item_visible, 'toggleClass', '#' . $self->item->item_type . '-' . $self->item->id, 'flagged')
    ->toggleClass('#fb-' . $self->item->id, 'flagged')
    ->render($self);
 }
 
+sub action_ajax_copy {
+  my ($self, %params) = @_;
+
+  SL::Clipboard->new->copy($self->item);
+  $self->render($self->js);
+}
+
+sub determine_paste_position {
+  my ($self) = @_;
+
+  if ($self->item->item_type eq 'section') {
+    # Sections are always pasted either directly after the
+    # clicked-upon section or at the very end.
+    return $self->clicked_item ? (undef, $self->clicked_item->section->id) : ();
+
+  } elsif ($self->item->item_type eq 'function-block') {
+    # A function block:
+    # - paste on section list: insert into last section as last element
+    # - paste on section: insert into that section as last element
+    # - paste on function block: insert after clicked-upon element
+    # - paste on sub function block: insert after parent function block of clicked-upon element
+    return !$self->clicked_item                                ? ( $self->sections->[-1]->id,              undef                          )
+         :  $self->clicked_item->item_type eq 'section'        ? ( $self->clicked_item->id,                undef                          )
+         :  $self->clicked_item->item_type eq 'function-block' ? ( $self->clicked_item->parent_id,         $self->clicked_item->id        )
+         :                                                       ( $self->clicked_item->parent->parent_id, $self->clicked_item->parent_id );
+
+  } else {                      # sub-function-block
+    # A sub function block:
+    # - paste on section list: promote to function block and insert into last section as last element
+    # - paste on section: promote to function block and insert into that section as last element
+    # - paste on function block: insert as last element in clicked-upon element
+    # - paste on sub function block: insert after clicked-upon element
+
+    # Promote sub function blocks to function blocks when pasting on a
+    # section or the section list.
+    $self->item->item_type('function-block') if !$self->clicked_item || ($self->clicked_item->item_type eq 'section');
+
+    return !$self->clicked_item                                ? ( $self->sections->[-1]->id,      undef                   )
+         :  $self->clicked_item->item_type eq 'section'        ? ( $self->clicked_item->id,        undef                   )
+         :  $self->clicked_item->item_type eq 'function-block' ? ( $self->clicked_item->id,        undef                   )
+         :                                                       ( $self->clicked_item->parent_id, $self->clicked_item->id );
+  }
+}
+
+sub assign_requirement_spec_id_rec {
+  my ($self, $item) = @_;
+
+  $item->requirement_spec_id($::form->{requirement_spec_id});
+  $self->assign_requirement_spec_id_rec($_) for @{ $item->children || [] };
+
+  return $item;
+}
+
+sub create_and_insert_node_rec {
+  my ($self, $item, $new_parent_id, $insert_after) = @_;
+
+  my $node = $self->presenter->requirement_spec_item_jstree_data($item);
+  $self->js->jstree->create_node('#tree', $insert_after ? ('#fb-' . $insert_after, 'after') : $new_parent_id ? ('#fb-' . $new_parent_id, 'last') : ('#sections', 'last'), $node);
+
+  $self->create_and_insert_node_rec($_, $item->id) for @{ $item->children || [] };
+
+  $self->js->jstree->open_node('#tree', '#fb-' . $item->id);
+}
+
+sub action_ajax_paste {
+  my ($self, %params) = @_;
+
+  my $copied = SL::Clipboard->new->get_entry(qr/^RequirementSpecItem$/);
+
+  if (!$copied) {
+    return $self->js->error(t8("The clipboard does not contain anything that can be pasted here."))
+                    ->render($self);
+  }
+
+  $self->item($self->assign_requirement_spec_id_rec($copied->to_object));
+  my $req_spec = SL::DB::RequirementSpec->new(id => $::form->{requirement_spec_id})->load;
+  $self->sections($req_spec->sections);
+
+  if (($self->item->item_type ne 'section') && !@{ $self->sections }) {
+    return $self->js->error(t8("You cannot paste function blocks or sub function blocks if there is no section."))
+                    ->render($self);
+  }
+
+  $self->clicked_item($::form->{id} ? SL::DB::RequirementSpecItem->new(id => $::form->{id})->load : undef);
+
+  my ($new_parent_id, $insert_after) = $self->determine_paste_position;
+
+  # Store result in database.
+  $self->item->update_attributes(requirement_spec_id => $::form->{requirement_spec_id}, parent_id => $new_parent_id);
+  $self->item->add_to_list(position => 'after', reference => $insert_after) if $insert_after;
+
+  # Update the tree: create the node for all pasted objects.
+  $self->create_and_insert_node_rec($self->item, $new_parent_id, $insert_after);
+
+  # Pasting the very first section?
+  if (!@{ $self->sections }) {
+    my $html = $self->render('requirement_spec_item/_section', { output => 0 }, requirement_spec_item => $self->item);
+    $self->js->html('#column-content', $html)
+             ->jstree->select_node('#tree', '#fb-' . $self->item->id)
+  }
+
+  # Update the current view if required.
+  $self->insert_new_item_in_section_view($self->js) if $self->is_item_visible;
+
+  $self->invalidate_version->render($self);
+}
+
 #
 # filters
 #
 
-sub load_requirement_spec {
+sub check_auth {
   my ($self) = @_;
-  $self->requirement_spec(SL::DB::RequirementSpec->new(id => $::form->{requirement_spec_id})->load || die "No such requirement spec");
+  $::auth->assert('sales_quotation_edit');
 }
 
 sub load_requirement_spec_item {
   my ($self) = @_;
-  $self->item(SL::DB::RequirementSpecItem->new(id => $::form->{id})->load || die "No such requirement spec item");
+  $self->item(SL::DB::RequirementSpecItem->new(id => $::form->{id})->load);
 }
 
 #
@@ -376,21 +493,6 @@ sub format_exception {
   return join "\n", (split m/\n/, $@)[0..4];
 }
 
-sub init_visible_section {
-  my ($self)       = @_;
-
-  my $content_id   = $::form->{current_content_id};
-  my $content_type = $::form->{current_content_type};
-
-  return undef unless $content_id;
-  return undef unless $content_type =~ m/section|function-block/;
-
-  $self->visible_item(SL::DB::Manager::RequirementSpecItem->find_by(id => $content_id));
-  return undef unless $self->visible_item;
-
-  return $self->visible_section($self->visible_item->section);
-}
-
 sub init_complexities {
   my ($self) = @_;
 
@@ -403,28 +505,35 @@ sub init_risks {
   return SL::DB::Manager::RequirementSpecRisk->get_all_sorted;
 }
 
+sub init_js {
+  my ($self) = @_;
+  $self->js(SL::ClientJS->new);
+}
+
 sub replace_bottom {
-  my ($self, $js, $item_or_id) = @_;
+  my ($self, $item_or_id) = @_;
 
   my $item      = (ref($item_or_id) ? $item_or_id : SL::DB::RequirementSpecItem->new(id => $item_or_id))->load;
   my $id_prefix = $item->item_type eq 'function-block' ? '' : 'sub-';
   my $html      = $self->render('requirement_spec_item/_function_block_content_bottom', { output => 0 }, requirement_spec_item => $item, id_prefix => $id_prefix);
-  return $js->replaceWith('#' . $id_prefix . 'function-block-content-bottom-' . $item->id, $html);
+  return $self->js->replaceWith('#' . $id_prefix . 'function-block-content-bottom-' . $item->id, $html);
 }
 
 sub render_list {
-  my ($self, $js, $item, $item_to_select) = @_;
+  my ($self, $item, $item_to_select) = @_;
 
   my $html = $self->render('requirement_spec_item/_section', { output => 0 }, requirement_spec_item => $item);
-  $self->select_node($js->html('#column-content', $html), $item_to_select || $item);
+  $self->js->html('#column-content', $html);
+  $self->select_node($item_to_select || $item);
 }
 
 sub select_node {
-  my ($self, $js, $item) = @_;
+  my ($self, $item) = @_;
 
-  $js->val( '#current_content_type', $item->item_type)
-     ->val( '#current_content_id',   $item->id)
-     ->jstree->select_node('#tree', '#fb-' . $item->id);
+  $self->js
+    ->val( '#current_content_type', $item->item_type)
+    ->val( '#current_content_id',   $item->id)
+    ->jstree->select_node('#tree', '#fb-' . $item->id);
 }
 
 sub create_dependency_item {
@@ -437,8 +546,8 @@ sub create_dependencies {
 
   return map { [ $_->fb_number . ' ' . $_->title,
                  [ map { ( $self->create_dependency_item($_),
-                           map { $self->create_dependency_item($_, '->') } @{ $_->sorted_children })
-                       } @{ $_->sorted_children } ] ]
+                           map { $self->create_dependency_item($_, '->') } @{ $_->children_sorted })
+                       } @{ $_->children_sorted } ] ]
              } @{ $self->item->requirement_spec->sections };
 }
 
@@ -480,27 +589,41 @@ sub add_function_block {
     insert_after          => $insert_position eq 'insertAfter' ? $insert_reference : undef,
   );
 
-  my $js = SL::ClientJS->new;
-
   my $new_section = $self->item->section;
-  if (!$self->visible_section || ($self->visible_section->id != $new_section->id)) {
+  if (!$self->is_item_visible) {
     # Show section/item to edit if it is not visible.
 
     $html = $self->render('requirement_spec_item/_section', { output => 0 }, requirement_spec_item => $new_section);
-    $js->html('#column-content', $html)
-       ->val('#current_content_type', 'section')
-       ->val('#current_content_id',   $new_section->id)
-       ->jstree->select_node('#tree', '#fb-' . $new_section->id);
+    $self->js
+      ->html('#column-content', $html)
+      ->val('#current_content_type', 'section')
+      ->val('#current_content_id',   $new_section->id)
+      ->jstree->select_node('#tree', '#fb-' . $new_section->id);
   }
 
   # $::lxdebug->message(0, "alright! clicked ID " . $::form->{id} . " type $clicked_type new_type $new_type insert_pos $insert_position ref " . ($insert_reference // '<undef>') . " parent $parent_id display_ref $display_reference");
 
-  $js->action($insert_position, $html, $display_reference)
-     ->focus("#${id_base}_description");
+  $self->js->action($insert_position, $html, $display_reference)
+           ->focus("#${id_base}_description");
+
+  $self->js->show('#sub-function-block-container-' . $parent_id) if $new_type eq 'sub-function-block';
+
+  $self->js->render($self);
+}
+
+sub is_item_visible {
+  my ($self, $item) = @_;
 
-  $js->show('#sub-function-block-container-' . $parent_id) if $new_type eq 'sub-function-block';
+  $item ||= $self->item;
+  return $self->visible_section && ($self->visible_section->id == $item->section->id);
+}
+
+sub invalidate_version {
+  my ($self) = @_;
 
-  $js->render($self);
+  my $html   = $self->render('requirement_spec/_version', { output => 0 },
+                             requirement_spec => SL::DB::RequirementSpec->new(id => $::form->{requirement_spec_id} || $self->item->requirement_spec_id)->load);
+  return $self->js->html('#requirement_spec_version', $html);
 }
 
 1;