Wiederkehrende Rechnungen: Preis-Fix für einmalige Ausführung

[mfinanz.git] / SL / BackgroundJob / CreatePeriodicInvoices.pm

diff --git a/SL/BackgroundJob/CreatePeriodicInvoices.pm b/SL/BackgroundJob/CreatePeriodicInvoices.pm
index 48f41b48cacba331cb3607dc8ea823101898e7a0..6151ec158581d46c8bc6fe8e0221063a6b1a91bb 100644 (file)
--- a/SL/BackgroundJob/CreatePeriodicInvoices.pm
+++ b/SL/BackgroundJob/CreatePeriodicInvoices.pm
@@ -9,15 +9,18 @@ use DateTime::Format::Strptime;
 use English qw(-no_match_vars);
 use List::MoreUtils qw(uniq);
 
 use English qw(-no_match_vars);
 use List::MoreUtils qw(uniq);
 

+use SL::Common;

 use SL::DB::AuthUser;
 use SL::DB::Default;
 use SL::DB::Order;
 use SL::DB::Invoice;
 use SL::DB::PeriodicInvoice;
 use SL::DB::PeriodicInvoicesConfig;
 use SL::DB::AuthUser;
 use SL::DB::Default;
 use SL::DB::Order;
 use SL::DB::Invoice;
 use SL::DB::PeriodicInvoice;
 use SL::DB::PeriodicInvoicesConfig;

+use SL::File;

 use SL::Helper::CreatePDF qw(create_pdf find_template);
 use SL::Mailer;
 use SL::Util qw(trim);
 use SL::Helper::CreatePDF qw(create_pdf find_template);
 use SL::Mailer;
 use SL::Util qw(trim);

+use SL::System::Process;

 
 sub create_job {
   $_[0]->create_standard_job('0 3 1 * *'); # first day of month at 3:00 am
 
 sub create_job {
   $_[0]->create_standard_job('0 3 1 * *'); # first day of month at 3:00 am


@@ -27,7 +30,8 @@ sub run {
   my $self        = shift;
   $self->{db_obj} = shift;
 
   my $self        = shift;
   $self->{db_obj} = shift;
 

-  $self->{job_errors} = [];
+  $self->{$_} = [] for qw(job_errors posted_invoices printed_invoices printed_failed emailed_invoices emailed_failed disabled_orders);
+

   if (!$self->{db_obj}->db->with_transaction(sub {
     1;                          # make Emacs happy
 
   if (!$self->{db_obj}->db->with_transaction(sub {
     1;                          # make Emacs happy
 


@@ -38,7 +42,7 @@ sub run {
       _log_msg("Periodic invoice configuration ID " . $config->id . " extended through " . $new_end_date->strftime('%d.%m.%Y') . "\n") if $new_end_date;
     }
 
       _log_msg("Periodic invoice configuration ID " . $config->id . " extended through " . $new_end_date->strftime('%d.%m.%Y') . "\n") if $new_end_date;
     }
 

-    my (@new_invoices, @invoices_to_print, @invoices_to_email, @disabled_orders);
+    my (@invoices_to_print, @invoices_to_email);

 
     _log_msg("Number of configs: " . scalar(@{ $configs}));
 
 
     _log_msg("Number of configs: " . scalar(@{ $configs}));
 


@@ -59,7 +63,7 @@ sub run {
 
         _log_msg("Invoice " . $data->{invoice}->invnumber . " posted for config ID " . $config->id . ", period start date " . $::locale->format_date(\%::myconfig, $date) . "\n");
 
 
         _log_msg("Invoice " . $data->{invoice}->invnumber . " posted for config ID " . $config->id . ", period start date " . $::locale->format_date(\%::myconfig, $date) . "\n");
 

-        push @new_invoices,      $data;
+        push @{ $self->{posted_invoices} }, $data->{invoice};

         push @invoices_to_print, $data if $config->print;
         push @invoices_to_email, $data if $config->send_email;
 
         push @invoices_to_print, $data if $config->print;
         push @invoices_to_email, $data if $config->send_email;
 


@@ -67,7 +71,7 @@ sub run {
         if ($inactive_ordnumber) {
           # disable one time configs and skip eventual invoices
           _log_msg("Order " . $inactive_ordnumber . " deavtivated \n");
         if ($inactive_ordnumber) {
           # disable one time configs and skip eventual invoices
           _log_msg("Order " . $inactive_ordnumber . " deavtivated \n");

-          push @disabled_orders, $inactive_ordnumber;
+          push @{ $self->{disabled_orders} }, $inactive_ordnumber;

           last;
         }
       }
           last;
         }
       }


@@ -76,12 +80,7 @@ sub run {
     foreach my $inv ( @invoices_to_print ) { $self->_print_invoice($inv); }
     foreach my $inv ( @invoices_to_email ) { $self->_email_invoice($inv); }
 
     foreach my $inv ( @invoices_to_print ) { $self->_print_invoice($inv); }
     foreach my $inv ( @invoices_to_email ) { $self->_email_invoice($inv); }
 

-    $self->_send_summary_email(
-      [ map { $_->{invoice} } @new_invoices      ],
-      [ map { $_->{invoice} } @invoices_to_print ],
-      [ map { $_->{invoice} } @invoices_to_email ],
-                               \@disabled_orders  ,
-    );
+    $self->_send_summary_email;

 
       1;
     })) {
 
       1;
     })) {


@@ -150,10 +149,7 @@ sub _replace_vars {
     $key               = $::locale->unquote_special_chars('html', $key) if $sub_fmt eq 'html';
     my $new_value;
 
     $key               = $::locale->unquote_special_chars('html', $key) if $sub_fmt eq 'html';
     my $new_value;
 

-    if (!$params{vars}->{$key}) {
-      $new_value = '';
-
-    } elsif ($format) {
+    if ($params{vars}->{$key} && $format) {

       $format    = $::locale->unquote_special_chars('html', $format) if $sub_fmt eq 'html';
 
       $new_value = DateTime::Format::Strptime->new(
       $format    = $::locale->unquote_special_chars('html', $format) if $sub_fmt eq 'html';
 
       $new_value = DateTime::Format::Strptime->new(


@@ -162,11 +158,15 @@ sub _replace_vars {
         time_zone   => 'local',
       )->format_datetime($params{vars}->{$key}->[0]);
 
         time_zone   => 'local',
       )->format_datetime($params{vars}->{$key}->[0]);
 

-    } else {
+    } elsif ($params{vars}->{$key}) {

       $new_value = $params{vars}->{$1}->[1]->($params{vars}->{$1}->[0]);
       $new_value = $params{vars}->{$1}->[1]->($params{vars}->{$1}->[0]);

+
+    } elsif ($params{invoice} && $params{invoice}->can($key)) {
+      $new_value = $params{invoice}->$key;

     }
 
     }
 

-    $new_value = $::locale->quote_special_chars('html', $new_value) if $sub_fmt eq 'html';
+    $new_value //= '';
+    $new_value   = $::locale->quote_special_chars('html', $new_value) if $sub_fmt eq 'html';

 
     $new_value;
 
 
     $new_value;
 


@@ -178,6 +178,8 @@ sub _replace_vars {
 sub _adjust_sellprices_for_period_lengths {
   my (%params) = @_;
 
 sub _adjust_sellprices_for_period_lengths {
   my (%params) = @_;
 

+  return if $params{config}->periodicity eq 'o';
+

   my $billing_len     = $params{config}->get_billing_period_length;
   my $order_value_len = $params{config}->get_order_value_period_length;
 
   my $billing_len     = $params{config}->get_billing_period_length;
   my $order_value_len = $params{config}->get_order_value_period_length;
 


@@ -225,10 +227,17 @@ sub _create_periodic_invoice {
 
     $invoice = SL::DB::Invoice->new_from($order);
 
 
     $invoice = SL::DB::Invoice->new_from($order);
 

+    my $tax_point = ($invoice->tax_point // $time_period_vars->{period_end_date}->[0])->clone;
+
+    while ($tax_point < $period_start_date) {
+      $tax_point->add(months => $config->get_billing_period_length);
+    }
+

     my $intnotes  = $invoice->intnotes ? $invoice->intnotes . "\n\n" : '';
     $intnotes    .= "Automatisch am " . $invdate->to_lxoffice . " erzeugte Rechnung";
 
     $invoice->assign_attributes(deliverydate => $period_start_date,
     my $intnotes  = $invoice->intnotes ? $invoice->intnotes . "\n\n" : '';
     $intnotes    .= "Automatisch am " . $invdate->to_lxoffice . " erzeugte Rechnung";
 
     $invoice->assign_attributes(deliverydate => $period_start_date,

+                                tax_point    => $tax_point,

                                 intnotes     => $intnotes,
                                 employee     => $order->employee, # new_from sets employee to import user
                                 direct_debit => $config->direct_debit,
                                 intnotes     => $intnotes,
                                 employee     => $order->employee, # new_from sets employee to import user
                                 direct_debit => $config->direct_debit,


@@ -290,14 +299,20 @@ sub _calculate_dates {
 }
 
 sub _send_summary_email {
 }
 
 sub _send_summary_email {

-  my ($self, $posted_invoices, $printed_invoices, $emailed_invoices,
-      $disabled_orders) = @_;
+  my ($self) = @_;

   my %config = %::lx_office_conf;
 
   my %config = %::lx_office_conf;
 

-  return if !$config{periodic_invoices} || !$config{periodic_invoices}->{send_email_to} || !scalar @{ $posted_invoices };
+  return if !$config{periodic_invoices} || !$config{periodic_invoices}->{send_email_to} || !scalar @{ $self->{posted_invoices} };
+
+  return if $config{periodic_invoices}->{send_for_errors_only} && !@{ $self->{printed_failed} } && !@{ $self->{emailed_failed} };
+
+  my $email = $config{periodic_invoices}->{send_email_to};
+  if ($email !~ m{\@}) {
+    my $user = SL::DB::Manager::AuthUser->find_by(login => $email);
+    $email   = $user ? $user->get_config_value('email') : undef;
+  }

 
 

-  my $user  = SL::DB::Manager::AuthUser->find_by(login => $config{periodic_invoices}->{send_email_to});
-  my $email = $user ? $user->get_config_value('email') : undef;
+  _log_msg("_send_summary_email: about to send to '" . ($email || '') . "'");

 
   return unless $email;
 
 
   return unless $email;
 


@@ -311,13 +326,10 @@ sub _send_summary_email {
 
   my $email_template = $config{periodic_invoices}->{email_template};
   my $filename       = $email_template || ( (SL::DB::Default->get->templates || "templates/webpages") . "/oe/periodic_invoices_email.txt" );
 
   my $email_template = $config{periodic_invoices}->{email_template};
   my $filename       = $email_template || ( (SL::DB::Default->get->templates || "templates/webpages") . "/oe/periodic_invoices_email.txt" );

-  my %params         = ( POSTED_INVOICES  => $posted_invoices,
-                         PRINTED_INVOICES => $printed_invoices,
-                         EMAILED_INVOICES => $emailed_invoices,
-                         DISABLED_ORDERS  => $disabled_orders );
+  my %params         = map { (uc($_) => $self->{$_}) } qw(posted_invoices printed_invoices printed_failed emailed_invoices emailed_failed disabled_orders);

 
   my $output;
 
   my $output;

-  $template->process($filename, \%params, \$output);
+  $template->process($filename, \%params, \$output) || die $template->error;

 
   my $mail              = Mailer->new;
   $mail->{from}         = $config{periodic_invoices}->{email_from};
 
   my $mail              = Mailer->new;
   $mail->{from}         = $config{periodic_invoices}->{email_from};


@@ -329,6 +341,49 @@ sub _send_summary_email {
   $mail->send;
 }
 
   $mail->send;
 }
 

+sub _store_pdf_in_webdav {
+  my ($self, $pdf_file_name, $invoice) = @_;
+
+  return unless $::instance_conf->get_webdav_documents;
+
+  my $form = Form->new('');
+
+  $form->{cwd}              = SL::System::Process->exe_dir;
+  $form->{tmpdir}           = ($pdf_file_name =~ m{(.+)/})[0];
+  $form->{tmpfile}          = ($pdf_file_name =~ m{.+/(.+)})[0];
+  $form->{format}           = 'pdf';
+  $form->{formname}         = 'invoice';
+  $form->{type}             = 'invoice';
+  $form->{vc}               = 'customer';
+  $form->{invnumber}        = $invoice->invnumber;
+  $form->{recipient_locale} = $invoice->language ? $invoice->language->template_code : '';
+
+  Common::copy_file_to_webdav_folder($form);
+}
+
+sub _store_pdf_in_filemanagement {
+  my ($self, $pdf_file, $invoice) = @_;
+
+  return unless $::instance_conf->get_doc_storage;
+
+  # create a form for generate_attachment_filename
+  my $form = Form->new('');
+  $form->{invnumber} = $invoice->invnumber;
+  $form->{type}      = 'invoice';
+  $form->{format}    = 'pdf';
+  $form->{formname}  = 'invoice';
+  $form->{language}  = '_' . $invoice->language->template_code if $invoice->language;
+  my $doc_name       = $form->generate_attachment_filename();
+
+  SL::File->save(object_id   => $invoice->id,
+                 object_type => 'invoice',
+                 mime_type   => 'application/pdf',
+                 source      => 'created',
+                 file_type   => 'document',
+                 file_name   => $doc_name,
+                 file_path   => $pdf_file);
+}
+

 sub _print_invoice {
   my ($self, $data) = @_;
 
 sub _print_invoice {
   my ($self, $data) = @_;
 


@@ -356,9 +411,11 @@ sub _print_invoice {
   $form->throw_on_error(sub {
     eval {
       $form->parse_template(\%::myconfig);
   $form->throw_on_error(sub {
     eval {
       $form->parse_template(\%::myconfig);

+      push @{ $self->{printed_invoices} }, $invoice;

       1;
     } or do {
       1;
     } or do {

-      push @{ $self->{job_errors} }, $EVAL_ERROR->getMessage;
+      push @{ $self->{job_errors} }, $EVAL_ERROR->error;
+      push @{ $self->{printed_failed} }, [ $invoice, $EVAL_ERROR->error ];

     };
   });
 }
     };
   });
 }


@@ -376,14 +433,18 @@ sub _email_invoice {
     grep { $_       }
     map  { trim($_) }
     (split(m{,}, $data->{config}->email_recipient_address),
     grep { $_       }
     map  { trim($_) }
     (split(m{,}, $data->{config}->email_recipient_address),

-     $data->{config}->email_recipient_contact ? ($data->{config}->email_recipient_contact->cp_email) : ());
+     $data->{config}->email_recipient_contact   ? ($data->{config}->email_recipient_contact->cp_email) : (),
+     $data->{invoice}->{customer}->invoice_mail ? ($data->{invoice}->{customer}->invoice_mail) : ()
+    );

 
   return unless @recipients;
 
 
   return unless @recipients;
 

+  my $language      = $data->{invoice}->language ? $data->{invoice}->language->template_code : undef;

   my %create_params = (
   my %create_params = (

-    template               => $self->find_template(name => 'invoice'),
+    template               => scalar($self->find_template(name => 'invoice', language => $language)),

     variables              => Form->new(''),
     return                 => 'file_name',
     variables              => Form->new(''),
     return                 => 'file_name',

+    record                 => $data->{invoice},

     variable_content_types => {
       longdescription => 'html',
       partnotes       => 'html',
     variable_content_types => {
       longdescription => 'html',
       partnotes       => 'html',


@@ -395,13 +456,18 @@ sub _email_invoice {
   $create_params{variables}->prepare_for_printing;
 
   my $pdf_file_name;
   $create_params{variables}->prepare_for_printing;
 
   my $pdf_file_name;

+  my $label = $language && Locale::is_supported($language) ? Locale->new($language)->text('Invoice') : $::locale->text('Invoice');

 
   eval {
     $pdf_file_name = $self->create_pdf(%create_params);
 
 
   eval {
     $pdf_file_name = $self->create_pdf(%create_params);
 

+    $self->_store_pdf_in_webdav        ($pdf_file_name, $data->{invoice});
+    $self->_store_pdf_in_filemanagement($pdf_file_name, $data->{invoice});
+

     for (qw(email_subject email_body)) {
       _replace_vars(
         object           => $data->{config},
     for (qw(email_subject email_body)) {
       _replace_vars(
         object           => $data->{config},

+        invoice          => $data->{invoice},

         vars             => $data->{time_period_vars},
         attribute        => $_,
         attribute_format => 'text'
         vars             => $data->{time_period_vars},
         attribute        => $_,
         attribute_format => 'text'


@@ -409,6 +475,7 @@ sub _email_invoice {
     }
 
     my $global_bcc = SL::DB::Default->get->global_bcc;
     }
 
     my $global_bcc = SL::DB::Default->get->global_bcc;

+    my $overall_error;

 
     for my $recipient (@recipients) {
       my $mail             = Mailer->new;
 
     for my $recipient (@recipients) {
       my $mail             = Mailer->new;


@@ -421,18 +488,25 @@ sub _email_invoice {
       $mail->{message}     = $data->{config}->email_body;
       $mail->{attachments} = [{
         path     => $pdf_file_name,
       $mail->{message}     = $data->{config}->email_body;
       $mail->{attachments} = [{
         path     => $pdf_file_name,

-        name     => sprintf('%s %s.pdf', $::locale->text('Invoice'), $data->{invoice}->invnumber),
+        name     => sprintf('%s %s.pdf', $label, $data->{invoice}->invnumber),

       }];
 
       my $error        = $mail->send;
 
       }];
 
       my $error        = $mail->send;
 

-      push @{ $self->{job_errors} }, $error if $error;
+      if ($error) {
+        push @{ $self->{job_errors} }, $error;
+        push @{ $self->{emailed_failed} }, [ $data->{invoice}, $error ];
+        $overall_error = 1;
+      }

     }
 
     }
 

+    push @{ $self->{emailed_invoices} }, $data->{invoice} unless $overall_error;
+

     1;
 
   } or do {
     push @{ $self->{job_errors} }, $EVAL_ERROR;
     1;
 
   } or do {
     push @{ $self->{job_errors} }, $EVAL_ERROR;

+    push @{ $self->{emailed_failed} }, [ $data->{invoice}, $EVAL_ERROR ];

   };
 
   unlink $pdf_file_name if $pdf_file_name;
   };
 
   unlink $pdf_file_name if $pdf_file_name;