Merge branch 'test' of ../kivitendo-erp_20220811

[mfinanz.git] / SL / Common.pm

diff --git a/SL/Common.pm b/SL/Common.pm
index f1aa53c2d43c108e3d1ee8325f7b911d1055f12c..126a30ed846d45cabca592b5b2306c762c09c9db 100644 (file)
--- a/SL/Common.pm
+++ b/SL/Common.pm
@@ -24,6 +24,8 @@ use POSIX ();
 use Encode qw(decode);
 
 use SL::DBUtils;
 use Encode qw(decode);
 
 use SL::DBUtils;

+use SL::DB;
+use SL::HTML::Util;

 
 sub unique_id {
   my ($a, $b) = gettimeofday();
 
 sub unique_id {
   my ($a, $b) = gettimeofday();


@@ -54,7 +56,7 @@ sub retrieve_parts {
 
   my ($self, $myconfig, $form, $order_by, $order_dir) = @_;
 
 
   my ($self, $myconfig, $form, $order_by, $order_dir) = @_;
 

-  my $dbh = $form->dbconnect($myconfig);
+  my $dbh = SL::DB->client->dbh;

 
   my (@filter_values, $filter);
 
 
   my (@filter_values, $filter);
 


@@ -66,14 +68,14 @@ sub retrieve_parts {
   }
 
   if ($form->{no_assemblies}) {
   }
 
   if ($form->{no_assemblies}) {

-    $filter .= qq| AND (NOT COALESCE(assembly, FALSE))|;
+    $filter .= qq| AND (NOT part_type = 'assembly')|;

   }
   if ($form->{assemblies}) {
   }
   if ($form->{assemblies}) {

-    $filter .= qq| AND assembly=TRUE|;
+    $filter .= qq| AND part_type = 'assembly'|;

   }
 
   if ($form->{no_services}) {
   }
 
   if ($form->{no_services}) {

-    $filter .= qq| AND (inventory_accno_id is not NULL or assembly=TRUE)|;
+    $filter .= qq| AND NOT (part_type = 'service' OR part_type = 'assembly')|;

   }
 
   substr($filter, 1, 3) = "WHERE" if ($filter);
   }
 
   substr($filter, 1, 3) = "WHERE" if ($filter);


@@ -93,7 +95,6 @@ sub retrieve_parts {
     push(@{$parts}, $ref);
   }
   $sth->finish();
     push(@{$parts}, $ref);
   }
   $sth->finish();

-  $dbh->disconnect();

 
   $main::lxdebug->leave_sub();
 
 
   $main::lxdebug->leave_sub();
 


@@ -105,7 +106,7 @@ sub retrieve_customers_or_vendors {
 
   my ($self, $myconfig, $form, $order_by, $order_dir, $is_vendor, $allow_both) = @_;
 
 
   my ($self, $myconfig, $form, $order_by, $order_dir, $is_vendor, $allow_both) = @_;
 

-  my $dbh = $form->dbconnect($myconfig);
+  my $dbh = SL::DB->client->dbh;

 
   my (@filter_values, $filter);
   if ($form->{"name"}) {
 
   my (@filter_values, $filter);
   if ($form->{"name"}) {


@@ -156,7 +157,6 @@ sub retrieve_customers_or_vendors {
     push(@{$customers}, $ref);
   }
   $sth->finish();
     push(@{$customers}, $ref);
   }
   $sth->finish();

-  $dbh->disconnect();

 
   $main::lxdebug->leave_sub();
 
 
   $main::lxdebug->leave_sub();
 


@@ -168,7 +168,7 @@ sub retrieve_delivery_customer {
 
   my ($self, $myconfig, $form, $order_by, $order_dir) = @_;
 
 
   my ($self, $myconfig, $form, $order_by, $order_dir) = @_;
 

-  my $dbh = $form->dbconnect($myconfig);
+  my $dbh = SL::DB->client->dbh;

 
   my (@filter_values, $filter);
   if ($form->{"name"}) {
 
   my (@filter_values, $filter);
   if ($form->{"name"}) {


@@ -192,7 +192,6 @@ sub retrieve_delivery_customer {
     push(@{$delivery_customers}, $ref);
   }
   $sth->finish();
     push(@{$delivery_customers}, $ref);
   }
   $sth->finish();

-  $dbh->disconnect();

 
   $main::lxdebug->leave_sub();
 
 
   $main::lxdebug->leave_sub();
 


@@ -204,7 +203,7 @@ sub retrieve_vendor {
 
   my ($self, $myconfig, $form, $order_by, $order_dir) = @_;
 
 
   my ($self, $myconfig, $form, $order_by, $order_dir) = @_;
 

-  my $dbh = $form->dbconnect($myconfig);
+  my $dbh = SL::DB->client->dbh;

 
   my (@filter_values, $filter);
   if ($form->{"name"}) {
 
   my (@filter_values, $filter);
   if ($form->{"name"}) {


@@ -228,7 +227,6 @@ sub retrieve_vendor {
     push(@{$vendors}, $ref);
   }
   $sth->finish();
     push(@{$vendors}, $ref);
   }
   $sth->finish();

-  $dbh->disconnect();

 
   $main::lxdebug->leave_sub();
 
 
   $main::lxdebug->leave_sub();
 


@@ -314,7 +312,7 @@ sub get_vc_details {
 
   $vc = $vc eq "customer" ? "customer" : "vendor";
 
 
   $vc = $vc eq "customer" ? "customer" : "vendor";
 

-  my $dbh = $form->dbconnect($myconfig);
+  my $dbh = SL::DB->client->dbh;

 
   my $query;
 
 
   my $query;
 


@@ -334,7 +332,6 @@ sub get_vc_details {
   my $ref = selectfirst_hashref_query($form, $dbh, $query, $vc_id);
 
   if (!$ref) {
   my $ref = selectfirst_hashref_query($form, $dbh, $query, $vc_id);
 
   if (!$ref) {

-    $dbh->disconnect();

     $main::lxdebug->leave_sub();
     return 0;
   }
     $main::lxdebug->leave_sub();
     return 0;
   }


@@ -346,13 +343,16 @@ sub get_vc_details {
   $query = qq|SELECT * FROM shipto WHERE (trans_id = ?)|;
   $form->{SHIPTO} = selectall_hashref_query($form, $dbh, $query, $vc_id);
 
   $query = qq|SELECT * FROM shipto WHERE (trans_id = ?)|;
   $form->{SHIPTO} = selectall_hashref_query($form, $dbh, $query, $vc_id);
 

+  if ($vc eq 'customer') {
+    $query = qq|SELECT * FROM additional_billing_addresses WHERE (customer_id = ?)|;
+    $form->{ADDITIONAL_BILLING_ADDRESSES} = selectall_hashref_query($form, $dbh, $query, $vc_id);
+  }
+

   $query = qq|SELECT * FROM contacts WHERE (cp_cv_id = ?)|;
   $form->{CONTACTS} = selectall_hashref_query($form, $dbh, $query, $vc_id);
 
   # Only show default pricegroup for customer, not vendor, which is why this is outside the main query
   $query = qq|SELECT * FROM contacts WHERE (cp_cv_id = ?)|;
   $form->{CONTACTS} = selectall_hashref_query($form, $dbh, $query, $vc_id);
 
   # Only show default pricegroup for customer, not vendor, which is why this is outside the main query

-  ($form->{pricegroup}) = selectrow_query($form, $dbh, qq|SELECT pricegroup FROM pricegroup WHERE id = ?|, $form->{klass});
-
-  $dbh->disconnect();
+  ($form->{pricegroup}) = selectrow_query($form, $dbh, qq|SELECT pricegroup FROM pricegroup WHERE id = ?|, $form->{pricegroup_id});

 
   $main::lxdebug->leave_sub();
 
 
   $main::lxdebug->leave_sub();
 


@@ -366,7 +366,7 @@ sub get_shipto_by_id {
 
   $prefix ||= "";
 
 
   $prefix ||= "";
 

-  my $dbh = $form->dbconnect($myconfig);
+  my $dbh = SL::DB->client->dbh;

 
   my $query = qq|SELECT * FROM shipto WHERE shipto_id = ?|;
   my $ref   = selectfirst_hashref_query($form, $dbh, $query, $shipto_id);
 
   my $query = qq|SELECT * FROM shipto WHERE shipto_id = ?|;
   my $ref   = selectfirst_hashref_query($form, $dbh, $query, $shipto_id);


@@ -380,8 +380,6 @@ sub get_shipto_by_id {
   );
   $form->{"${prefix}shiptocvar_$_->{name}"} = $_->{value} for @{ $cvars };
 
   );
   $form->{"${prefix}shiptocvar_$_->{name}"} = $_->{value} for @{ $cvars };
 

-  $dbh->disconnect();
-

   $main::lxdebug->leave_sub();
 }
 
   $main::lxdebug->leave_sub();
 }
 


@@ -390,6 +388,8 @@ sub save_email_status {
 
   my ($self, $myconfig, $form) = @_;
 
 
   my ($self, $myconfig, $form) = @_;
 

+  return unless ($::instance_conf->get_email_journal);
+

   my ($table, $query, $dbh);
 
   if ($form->{script} eq 'oe.pl') {
   my ($table, $query, $dbh);
 
   if ($form->{script} eq 'oe.pl') {


@@ -407,33 +407,34 @@ sub save_email_status {
 
   return $main::lxdebug->leave_sub() if (!$form->{id} || !$table || !$form->{formname});
 
 
   return $main::lxdebug->leave_sub() if (!$form->{id} || !$table || !$form->{formname});
 

-  $dbh = $form->get_standard_dbh($myconfig);
-
-  my ($intnotes) = selectrow_query($form, $dbh, qq|SELECT intnotes FROM $table WHERE id = ?|, $form->{id});
+  SL::DB->client->with_transaction(sub {
+    $dbh = SL::DB->client->dbh;

 
 

-  $intnotes =~ s|\r||g;
-  $intnotes =~ s|\n$||;
+    my ($intnotes) = selectrow_query($form, $dbh, qq|SELECT intnotes FROM $table WHERE id = ?|, $form->{id});

 
 

-  $intnotes .= "\n\n" if ($intnotes);
+    $intnotes =~ s|\r||g;
+    $intnotes =~ s|\n$||;

 
 

-  my $cc  = $form->{cc}  ? $main::locale->text('Cc') . ": $form->{cc}\n"   : '';
-  my $bcc = $form->{bcc} ? $main::locale->text('Bcc') . ": $form->{bcc}\n" : '';
-  my $now = scalar localtime;
+    $intnotes .= "\n\n" if ($intnotes);

 
 

-  $intnotes .= $main::locale->text('[email]') . "\n"
-    . $main::locale->text('Date') . ": $now\n"
-    . $main::locale->text('To (email)') . ": $form->{email}\n"
-    . "${cc}${bcc}"
-    . $main::locale->text('Subject') . ": $form->{subject}\n\n"
-    . $main::locale->text('Message') . ": $form->{message}";
+    my $cc  = $form->{cc}  ? $main::locale->text('Cc') . ": $form->{cc}\n"   : '';
+    my $bcc = $form->{bcc} ? $main::locale->text('Bcc') . ": $form->{bcc}\n" : '';
+    my $now = scalar localtime;

 
 

-  $intnotes =~ s|\r||g;
+    $intnotes .= $main::locale->text('[email]') . "\n"
+      . $main::locale->text('Date') . ": $now\n"
+      . $main::locale->text('To (email)') . ": $form->{email}\n"
+      . "${cc}${bcc}"
+      . $main::locale->text('Subject') . ": $form->{subject}\n\n"
+      . $main::locale->text('Message') . ": " . SL::HTML::Util->strip($form->{message});

 
 

-  do_query($form, $dbh, qq|UPDATE $table SET intnotes = ? WHERE id = ?|, $intnotes, $form->{id});
+    $intnotes =~ s|\r||g;

 
 

-  $form->save_status($dbh);
+    do_query($form, $dbh, qq|UPDATE $table SET intnotes = ? WHERE id = ?|, $intnotes, $form->{id});

 
 

-  $dbh->commit();
+    $form->save_status($dbh);
+    1;
+  }) or do { die SL::DB->client->error };

 
   $main::lxdebug->leave_sub();
 }
 
   $main::lxdebug->leave_sub();
 }


@@ -546,16 +547,16 @@ sub copy_file_to_webdav_folder {
   foreach my $item (qw(tmpdir tmpfile type)){
     next if $form->{$item};
     $::lxdebug->message(LXDebug::WARN(), 'Missing parameter:' . $item);
   foreach my $item (qw(tmpdir tmpfile type)){
     next if $form->{$item};
     $::lxdebug->message(LXDebug::WARN(), 'Missing parameter:' . $item);

-    $::form->error($::locale->text("Missing parameter for WebDAV file copy"));
+    $::lxdebug->leave_sub();
+    return $::locale->text("Missing parameter for WebDAV file copy");

   }
 
   my ($webdav_folder, $document_name) =  get_webdav_folder($form);
 
   if (! $webdav_folder){
   }
 
   my ($webdav_folder, $document_name) =  get_webdav_folder($form);
 
   if (! $webdav_folder){

-    $::lxdebug->leave_sub();

     $::lxdebug->message(LXDebug::WARN(), 'Cannot check correct WebDAV folder');
     $::lxdebug->message(LXDebug::WARN(), 'Cannot check correct WebDAV folder');

-    $::form->error($::locale->text("Cannot check correct WebDAV folder"));
-    return undef;
+    $::lxdebug->leave_sub();
+    return $::locale->text("Cannot check correct WebDAV folder")

   }
 
   $complete_path =  File::Spec->catfile($form->{cwd},  $webdav_folder);
   }
 
   $complete_path =  File::Spec->catfile($form->{cwd},  $webdav_folder);


@@ -569,7 +570,11 @@ sub copy_file_to_webdav_folder {
     chdir($current_dir);
   }
 
     chdir($current_dir);
   }
 

-  opendir my $dh, $complete_path or die "Could not open $complete_path: $!";
+  my $dh;
+  if (!opendir $dh, $complete_path) {
+    $::lxdebug->leave_sub();
+    return "Could not open $complete_path: $!";
+  }

 
   my ($newest_name, $newest_time);
   while ( defined( my $file = readdir( $dh ) ) ) {
 
   my ($newest_name, $newest_time);
   while ( defined( my $file = readdir( $dh ) ) ) {


@@ -591,15 +596,19 @@ sub copy_file_to_webdav_folder {
     return;
   }
 
     return;
   }
 

+  $form->{attachment_filename} ||= $form->generate_attachment_filename;
+

   my $timestamp =  get_current_formatted_time();
   my $timestamp =  get_current_formatted_time();

-  my $new_file  =  File::Spec->catfile($form->{cwd}, $webdav_folder, $form->generate_attachment_filename());
+  my $new_file  =  File::Spec->catfile($form->{cwd}, $webdav_folder, $form->{attachment_filename});

   $new_file =~ s{(.*)\.}{$1$timestamp\.};
 
   if (!File::Copy::copy($current_file, $new_file)) {
     $::lxdebug->message(LXDebug::WARN(), "Copy file from $current_file to $new_file failed: $ERRNO");
   $new_file =~ s{(.*)\.}{$1$timestamp\.};
 
   if (!File::Copy::copy($current_file, $new_file)) {
     $::lxdebug->message(LXDebug::WARN(), "Copy file from $current_file to $new_file failed: $ERRNO");

-    $::form->error($::locale->text("Copy file from #1 to #2 failed: #3", $current_file, $new_file, $ERRNO));
+    $::lxdebug->leave_sub();
+    return $::locale->text("Copy file from #1 to #2 failed: #3", $current_file, $new_file, $ERRNO);

   }
 
   }
 

+  return;

   $::lxdebug->leave_sub();
 }
 
   $::lxdebug->leave_sub();
 }
 


@@ -639,6 +648,11 @@ C<full> (replace consecutive line feed/carriage return characters in
 the middle by a single space and remove tailing line feed/carriage
 return characters).
 
 the middle by a single space and remove tailing line feed/carriage
 return characters).
 

+=item C<save_email_status>
+
+Adds sending information to internal notes.
+Does nothing if the client config email_journal is enabled.
+

 =back
 
 =head1 BUGS
 =back
 
 =head1 BUGS