X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FCsvImport.pm;h=bb1e8d6681cfdb6789756000755906212d75e5f0;hb=d1f6e5010a26a1cb109700ca2b97cf75d551f10a;hp=91ecb76d8e07c13706c77a63c19d57a4a1519a2e;hpb=8e258f81fb2bae3ca0014fe61bd5a9515b114b71;p=kivitendo-erp.git diff --git a/SL/Controller/CsvImport.pm b/SL/Controller/CsvImport.pm index 91ecb76d8..bb1e8d668 100644 --- a/SL/Controller/CsvImport.pm +++ b/SL/Controller/CsvImport.pm @@ -20,13 +20,14 @@ use SL::Controller::CsvImport::Inventory; use SL::Controller::CsvImport::Shipto; use SL::Controller::CsvImport::Project; use SL::Controller::CsvImport::Order; +use SL::Controller::CsvImport::DeliveryOrder; use SL::Controller::CsvImport::ARTransaction; use SL::JSON; use SL::Controller::CsvImport::BankTransaction; use SL::BackgroundJob::CsvImport; use SL::System::TaskServer; -use List::MoreUtils qw(none); +use List::MoreUtils qw(any none); use List::Util qw(min); use parent qw(SL::Controller::Base); @@ -163,7 +164,21 @@ sub action_report { $::form->error(t8('No report with id #1', $report_id)); } - my $num_rows = $self->{report}->numrows; + my $show_info_err = ($self->{report}->profile->get('full_preview', 0) == 1); + my $show_first_20 = ($self->{report}->profile->get('full_preview', 0) == 2); + + my $num_rows = 0; + if ($show_first_20) { + $num_rows = min($self->{report}->numrows, 20); + } elsif ($show_info_err) { + # count each status row only once + $num_rows = SL::DB::Manager::CsvImportReportStatus->get_all_count(query => [csv_import_report_id => $report_id], + select => ['row'], + distinct => 1,); + } else { + # show all + $num_rows = $self->{report}->numrows; + } # manual paginating, yuck my $page = $::form->{page} || 1; @@ -180,31 +195,34 @@ sub action_report { my $last_row_header = $self->{report_numheaders} - 1; my $first_row_data = $pages->{per_page} * ($pages->{page}-1) + $self->{report_numheaders}; my $last_row_data = min($pages->{per_page} * $pages->{page}, $num_rows) + $self->{report_numheaders} - 1; - $self->{display_rows} = [ - $first_row_header - .. - $last_row_header, - $first_row_data - .. - $last_row_data - ]; + + + $self->{display_rows} = []; + if ($show_info_err) { + my $limit = $last_row_data - $first_row_data + 1; + my $offset = $first_row_data - $self->{report_numheaders}; + my @err_rows = map { $_->row } @{SL::DB::Manager::CsvImportReportStatus->get_all(query => [csv_import_report_id => $report_id], + distinct => 1, + select => ['row'], + limit => $limit, + offset => $offset, + sort_by => 'row')}; + $self->{display_rows} = [ $first_row_header .. $last_row_header, + @err_rows ]; + + } else { + + $self->{display_rows} = [ $first_row_header .. $last_row_header, + $first_row_data .. $last_row_data ]; + } my @query = ( + row => $self->{display_rows}, csv_import_report_id => $report_id, - or => [ - and => [ - row => { ge => $first_row_header }, - row => { le => $last_row_header }, - ], - and => [ - row => { ge => $first_row_data }, - row => { le => $last_row_data }, - ] - ] ); - my $rows = SL::DB::Manager::CsvImportReportRow ->get_all(query => \@query); - my $status = SL::DB::Manager::CsvImportReportStatus->get_all(query => \@query); + my $rows = SL::DB::Manager::CsvImportReportRow ->get_all(query => \@query, sort_by => 'row'); + my $status = SL::DB::Manager::CsvImportReportStatus->get_all(query => \@query, sort_by => 'row'); $self->{num_errors} = SL::DB::Manager::CsvImportReportStatus->get_all_count(query => [csv_import_report_id => $report_id, type => 'errors']); $self->{report_rows} = $self->{report}->folded_rows(rows => $rows); @@ -289,7 +307,7 @@ sub check_auth { sub check_type { my ($self) = @_; - die "Invalid CSV import type" if none { $_ eq $::form->{profile}->{type} } qw(parts inventories customers_vendors addresses contacts projects orders bank_transactions ar_transactions); + die "Invalid CSV import type" if none { $_ eq $::form->{profile}->{type} } qw(parts inventories customers_vendors addresses contacts projects orders delivery_orders bank_transactions ar_transactions); $self->type($::form->{profile}->{type}); } @@ -336,11 +354,12 @@ sub render_inputs { : $self->type eq 'inventories' ? $::locale->text('CSV import: inventories') : $self->type eq 'projects' ? $::locale->text('CSV import: projects') : $self->type eq 'orders' ? $::locale->text('CSV import: orders') + : $self->type eq 'delivery_orders' ? $::locale->text('CSV import: delivery orders') : $self->type eq 'bank_transactions' ? $::locale->text('CSV import: bank transactions') : $self->type eq 'ar_transactions' ? $::locale->text('CSV import: ar transactions') : die; - if ($self->{type} eq 'customers_vendors' or $self->{type} eq 'orders' or $self->{type} eq 'ar_transactions' ) { + if ( any { $_ eq $self->{type} } qw(customers_vendors orders delivery_orders ar_transactions) ) { $self->all_taxzones(SL::DB::Manager::TaxZone->get_all_sorted(query => [ obsolete => 0 ])); }; @@ -704,6 +723,7 @@ sub init_worker { : $self->{type} eq 'inventories' ? SL::Controller::CsvImport::Inventory->new(@args) : $self->{type} eq 'projects' ? SL::Controller::CsvImport::Project->new(@args) : $self->{type} eq 'orders' ? SL::Controller::CsvImport::Order->new(@args) + : $self->{type} eq 'delivery_orders' ? SL::Controller::CsvImport::DeliveryOrder->new(@args) : $self->{type} eq 'bank_transactions' ? SL::Controller::CsvImport::BankTransaction->new(@args) : $self->{type} eq 'ar_transactions' ? SL::Controller::CsvImport::ARTransaction->new(@args) : die "Program logic error";