X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FRequirementSpecItem.pm;h=3129b43f013458458f3ea5b5a5bf7f4d9d07ff5d;hb=e8ac38a63f43c43b30cdf45bd877e39bf0fd41b1;hp=f3cbd0878163b4b5c1870e9660fc6716a6ae788c;hpb=e86e30de944c4bbe4b42f43d933e0a28a83408dd;p=kivitendo-erp.git diff --git a/SL/Controller/RequirementSpecItem.pm b/SL/Controller/RequirementSpecItem.pm index f3cbd0878..3129b43f0 100644 --- a/SL/Controller/RequirementSpecItem.pm +++ b/SL/Controller/RequirementSpecItem.pm @@ -4,6 +4,7 @@ use strict; use parent qw(SL::Controller::Base); +use Carp; use List::MoreUtils qw(apply); use List::Util qw(first); use Time::HiRes (); @@ -22,8 +23,8 @@ use Rose::Object::MakeMethods::Generic 'scalar --get_set_init' => [ qw(complexities risks) ], ); -__PACKAGE__->run_before('load_requirement_spec_item', only => [ qw(dragged_and_dropped ajax_update ajax_edit ajax_delete) ]); -__PACKAGE__->run_before('init_visible_section', only => [ qw(dragged_and_dropped ajax_list ajax_edit ajax_delete) ]); +__PACKAGE__->run_before('load_requirement_spec_item', only => [ qw(dragged_and_dropped ajax_update ajax_edit ajax_delete ajax_flag) ]); +__PACKAGE__->run_before('init_visible_section'); # # actions @@ -39,22 +40,27 @@ sub action_ajax_list { return $self->render($js); } - $self->item(SL::DB::RequirementSpecItem->new(id => $::form->{clicked_id})->load->get_section); + my $clicked_item = SL::DB::RequirementSpecItem->new(id => $::form->{clicked_id})->load; + $self->item($clicked_item->get_section); - $self->render_list($js, $self->item) if !$self->visible_section || ($self->visible_section->id != $self->item->id); + if (!$self->visible_section || ($self->visible_section->id != $self->item->id)) { + $self->render_list($js, $self->item, $clicked_item); + } else { + $self->select_node($js, $clicked_item); + } $self->render($js); } sub action_dragged_and_dropped { - my ($self) = @_; + my ($self) = @_; - my $position = $::form->{position} =~ m/^ (?: before | after | last ) $/x ? $::form->{position} : die "Unknown 'position' parameter"; - my $dropped_item = $::form->{dropped_id} ? SL::DB::RequirementSpecItem->new(id => $::form->{dropped_id})->load : undef; + my $position = $::form->{position} =~ m/^ (?: before | after | last ) $/x ? $::form->{position} : die "Unknown 'position' parameter"; + my $dropped_item = $::form->{dropped_id} ? SL::DB::RequirementSpecItem->new(id => $::form->{dropped_id})->load : undef; - my $visible_section_id = $self->visible_section ? $self->visible_section->id : undef; - my $old_parent_id = $self->item->parent_id; - my $old_type = $self->item->get_type; + my $old_visible_section = $self->visible_section ? $self->visible_section : undef; + my $old_parent_id = $self->item->parent_id; + my $old_type = $self->item->get_type; $self->item->db->do_transaction(sub { $self->item->remove_from_list; @@ -65,17 +71,27 @@ sub action_dragged_and_dropped { my $js = SL::ClientJS->new; $self->item(SL::DB::RequirementSpecItem->new(id => $self->item->id)->load); - my $new_section = $self->item->get_section; - my $new_type = $self->item->get_type; + my $new_section = $self->item->get_section; + my $new_type = $self->item->get_type; + my $new_visible_section = SL::DB::RequirementSpecItem->new(id => $self->visible_item->id)->load->get_section; + + return $self->render($js) if !$old_visible_section || ($new_type eq 'section'); - return $self->render($js) if !$visible_section_id || ($new_type eq 'section'); + # From here on $old_visible_section is definitely set. my $old_parent = SL::DB::RequirementSpecItem->new(id => $old_parent_id)->load; my $old_section = $old_parent->get_section; - # $::lxdebug->message(0, "old sec ID " . $old_section->id . " new " . $new_section->id . " visible $visible_section_id PARENT: old " . $old_parent->id . " new " . $self->item->parent_id . '/' . $self->item->parent->id); + # $::lxdebug->message(0, "old sec ID " . $old_section->id . " new " . $new_section->id . " old visible " . $old_visible_section->id . " new visible " . $new_visible_section->id + # . " PARENT: old " . $old_parent->id . " new " . $self->item->parent_id . '/' . $self->item->parent->id); + + 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) + ->render($self); + } - if ($visible_section_id == $old_section->id) { + 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); @@ -88,12 +104,12 @@ sub action_dragged_and_dropped { } } - if ($visible_section_id == $new_section->id) { + 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 = apply { s/-/_/g; $_ } $new_type; - my $html = "" . $self->render('requirement_spec_item/_' . $template, { output => 0 }, requirement_spec_item => $self->item); + 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) { @@ -112,10 +128,72 @@ sub action_dragged_and_dropped { $self->render($js); } -sub action_ajax_add { +sub action_ajax_add_section { + my ($self, %params) = @_; + + die "Missing parameter 'requirement_spec_id'" if !$::form->{requirement_spec_id}; + + $self->item(SL::DB::RequirementSpecItem->new(requirement_spec_id => $::form->{requirement_spec_id})); + + my $insert_after = $::form->{id} ? SL::DB::RequirementSpecItem->new(id => $::form->{id})->load->get_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 + ->remove('#new_section_form') + ->hide('#column-content > *') + ->appendTo($html, '#column-content') + ->focus('#new_section_title') + ->render($self); +} + +sub action_ajax_add_function_block { + my ($self, %params) = @_; + + return $self->add_function_block('function-block'); +} + +sub action_ajax_add_sub_function_block { + my ($self, %params) = @_; + + return $self->add_function_block('sub-function-block'); +} + +sub action_ajax_create { my ($self, %params) = @_; - die "TODO: add action"; + 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}; + + my @errors = $self->item(SL::DB::RequirementSpecItem->new(%{ $attributes }))->validate; + return $js->error(@errors)->render($self) if @errors; + + $self->item->save; + $self->item->add_to_list(position => 'after', reference => $insert_after) if $insert_after; + + my $type = $self->item->get_type; + + if ($type eq 'section') { + my $node = $self->presenter->requirement_spec_item_jstree_data($self->item); + return $self->render_list($js, $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); + } + + my $template = 'requirement_spec_item/_' . (apply { s/-/_/g; $_ } $type); + 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->replace_bottom($js, $self->item->parent) if $type eq 'sub-function-block'; + + $js->render($self); } sub action_ajax_edit { @@ -148,16 +226,7 @@ sub action_ajax_edit { } # Edit a function block or a sub function block - my $create_item = sub { - [ $_[0]->id, $self->presenter->truncate(join(' ', grep { $_ } ($_[1], $_[0]->fb_number, $_[0]->description))) ] - }; - my @dependencies = - map { [ $_->fb_number . ' ' . $_->title, - [ map { ( $create_item->($_), - map { $create_item->($_, '->') } @{ $_->sorted_children }) - } @{ $_->sorted_children } ] ] - } @{ $self->item->requirement_spec->sections }; - + my @dependencies = $self->create_dependencies; my @selected_dependencies = map { $_->id } @{ $self->item->dependencies }; my $html = $self->render('requirement_spec_item/_function_block_form', { output => 0 }, DEPENDENCIES => \@dependencies, SELECTED_DEPENDENCIES => \@selected_dependencies); @@ -178,7 +247,7 @@ sub action_ajax_update { my ($self, %params) = @_; my $js = SL::ClientJS->new; - my $prefix = $::form->{form_prefix} || 'function_block'; + my $prefix = $::form->{form_prefix} || die "Missing parameter 'form_prefix'"; my $attributes = $::form->{$prefix} || {}; foreach (qw(requirement_spec_id parent_id position)) { @@ -227,7 +296,8 @@ sub action_ajax_update { sub action_ajax_delete { my ($self) = @_; - my $js = SL::ClientJS->new; + my $js = SL::ClientJS->new; + my $full_list = $self->item->get_full_list; $self->item->delete; @@ -254,7 +324,7 @@ sub action_ajax_delete { $self->replace_bottom($js, $self->item->parent_id) if $type eq 'sub-function-block'; - if (1 == scalar @{ $self->item->get_full_list }) { + if (1 == scalar @{ $full_list }) { if ($type eq 'function-block') { $js->show('#section-list-empty'); } elsif ($type eq 'sub-function-block') { @@ -267,6 +337,19 @@ sub action_ajax_delete { ->render($self); } +sub action_ajax_flag { + my ($self) = @_; + + $self->item->update_attributes(is_flagged => !$self->item->is_flagged); + + my $is_visible = $self->visible_section && ($self->visible_section->id == $self->item->get_section->id); + + SL::ClientJS->new + ->action_if($is_visible, 'toggleClass', '#' . $self->item->get_type . '-' . $self->item->id, 'flagged') + ->toggleClass('#fb-' . $self->item->id, 'flagged') + ->render($self); +} + # # filters # @@ -330,13 +413,94 @@ sub replace_bottom { } sub render_list { - my ($self, $js, $item) = @_; + my ($self, $js, $item, $item_to_select) = @_; my $html = $self->render('requirement_spec_item/_section', { output => 0 }, requirement_spec_item => $item); - $js->html('#column-content', $html) - ->val( '#current_content_type', $item->get_type) + $self->select_node($js->html('#column-content', $html), $item_to_select || $item); +} + +sub select_node { + my ($self, $js, $item) = @_; + + $js->val( '#current_content_type', $item->get_type) ->val( '#current_content_id', $item->id) ->jstree->select_node('#tree', '#fb-' . $item->id); } +sub create_dependency_item { + my $self = shift; + [ $_[0]->id, $self->presenter->truncate(join(' ', grep { $_ } ($_[1], $_[0]->fb_number, $_[0]->description))) ]; +} + +sub create_dependencies { + my ($self) = @_; + + return map { [ $_->fb_number . ' ' . $_->title, + [ map { ( $self->create_dependency_item($_), + map { $self->create_dependency_item($_, '->') } @{ $_->sorted_children }) + } @{ $_->sorted_children } ] ] + } @{ $self->item->requirement_spec->sections }; +} + +sub add_function_block { + my ($self, $new_type) = @_; + + my $clicked_id = $::form->{id} || ($self->visible_item ? $self->visible_item->id : undef); + + die "Invalid new_type '$new_type'" if $new_type !~ m/^(?:sub-)?function-block$/; + die "Missing parameter 'id' and no visible item" if !$clicked_id; + die "Missing parameter 'requirement_spec_id'" if !$::form->{requirement_spec_id}; + + my $clicked_item = SL::DB::RequirementSpecItem->new(id => $clicked_id)->load; + my $clicked_type = $clicked_item->get_type; + + die "Invalid clicked_type '$clicked_type'" if $clicked_type !~ m/^(?: section | (?:sub-)? function-block )$/x; + + my $case = "${clicked_type}:${new_type}"; + + my ($insert_position, $insert_reference, $parent_id, $display_reference) + = $case eq 'section:function-block' ? ( 'appendTo', $clicked_item->id, $clicked_item->id, '#section-list' ) + : $case eq 'function-block:function-block' ? ( 'insertAfter', $clicked_item->id, $clicked_item->parent_id, '#function-block-' ) + : $case eq 'function-block:sub-function-block' ? ( 'appendTo' , $clicked_item->id, $clicked_item->id, '#sub-function-block-container-' ) + : $case eq 'sub-function-block:function-block' ? ( 'insertAfter', $clicked_item->parent_id, $clicked_item->parent->parent_id, '#function-block-' ) + : $case eq 'sub-function-block:sub-function-block' ? ( 'insertAfter', $clicked_item->id, $clicked_item->parent_id, '#sub-function-block-' ) + : die "Invalid combination of 'clicked_type (section)/new_type ($new_type)'"; + + $self->item(SL::DB::RequirementSpecItem->new(requirement_spec_id => $::form->{requirement_spec_id}, parent_id => $parent_id)); + + $display_reference .= $insert_reference if $display_reference =~ m/-$/; + my $id_base = join('_', 'new_function_block', Time::HiRes::gettimeofday(), int rand 1000000000000); + my $html = $self->render( + 'requirement_spec_item/_function_block_form', + { output => 0 }, + DEPENDENCIES => [ $self->create_dependencies ], + SELECTED_DEPENDENCIES => [], + requirement_spec_item => $self->item, + id_base => $id_base, + insert_after => $insert_position eq 'insertAfter' ? $insert_reference : undef, + ); + + my $js = SL::ClientJS->new; + + my $new_section = $self->item->get_section; + if (!$self->visible_section || ($self->visible_section->id != $new_section->id)) { + # 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); + } + + # $::lxdebug->message(0, "alright! clicked ID " . $::form->{id} . " type $clicked_type new_type $new_type insert_pos $insert_position ref " . ($insert_reference // '') . " parent $parent_id display_ref $display_reference"); + + $js->action($insert_position, $html, $display_reference) + ->focus("#${id_base}_description"); + + $js->show('#sub-function-block-container-' . $parent_id) if $new_type eq 'sub-function-block'; + + $js->render($self); +} + 1;