Bug 3150: Move emails for sending cart and list contents to notices

[srvgit] / basket / sendbasket.pl

diff --git a/basket/sendbasket.pl b/basket/sendbasket.pl
index af47302..abd54de 100755 (executable)
--- a/basket/sendbasket.pl
+++ b/basket/sendbasket.pl
@@ -54,109 +54,64 @@ if ( $email_add ) {
             session_id => scalar $query->cookie('CGISESSID'),
             token  => scalar $query->param('csrf_token'),
         });
-    my $comment = $query->param('comment');
 
-    # Since we are already logged in, no need to check credentials again
-    # when loading a second template.
-    my $template2 = C4::Templates::gettemplate(
-        'basket/sendbasket.tt', 'intranet', $query,
-    );
+    my $patron = Koha::Patrons->find( $borrowernumber );
+    my $user_email = $patron->first_valid_email_address;
+
+    my $comment = $query->param('comment');
 
     my @bibs = split( /\//, $bib_list );
-    my @results;
     my $iso2709;
-    my $marcflavour = C4::Context->preference('marcflavour');
-    foreach my $biblionumber (@bibs) {
-        $template2->param( biblionumber => $biblionumber );
-
-        my $biblio           = Koha::Biblios->find( $biblionumber ) or next;
-        my $dat              = $biblio->unblessed;
-        my $record           = $biblio->metadata->record({ embed_items => 1 });
-        my $marcauthorsarray = $biblio->get_marc_contributors;
-        my $marcsubjctsarray = GetMarcSubjects( $record, $marcflavour );
-
-        my $hasauthors = 0;
-        if($dat->{'author'} || @$marcauthorsarray) {
-          $hasauthors = 1;
-        }
-       
-
-        $dat->{MARCSUBJCTS}    = $marcsubjctsarray;
-        $dat->{MARCAUTHORS}    = $marcauthorsarray;
-        $dat->{HASAUTHORS}     = $hasauthors;
-        $dat->{'biblionumber'} = $biblionumber;
-        $dat->{ITEM_RESULTS}   = $biblio->items->search_ordered;
-        my ( $host, $relatedparts ) = $biblio->get_marc_host;
-        $dat->{HOSTITEMENTRIES} = $host;
-        $dat->{RELATEDPARTS} = $relatedparts;
-
-        $iso2709 .= $record->as_usmarc();
-
-        push( @results, $dat );
-    }
-
-    my $resultsarray = \@results;
-    $template2->param(
-        BIBLIO_RESULTS => $resultsarray,
-        comment        => $comment
-    );
-
-    # Getting template result
-    my $template_res = $template2->output();
-    my $body;
-
-    my $subject;
-    # Analysing information and getting mail properties
-    if ( $template_res =~ /<SUBJECT>(?<subject>.*)<END_SUBJECT>/s ) {
-        $subject = $+{subject};
-        $subject =~ s|\n?(.*)\n?|$1|;
-    }
-    else {
-        $subject = "no subject";
-    }
 
-    my $email_header = "";
-    if ( $template_res =~ /<HEADER>(.*)<END_HEADER>/s ) {
-        $email_header = $1;
-        $email_header =~ s|\n?(.*)\n?|$1|;
+    foreach my $bib ( @bibs ) {
+        my $biblio = Koha::Biblios->find( $bib ) or next;
+        $iso2709 .= $biblio->metadata->record->as_usmarc();
     }
 
-    if ( $template_res =~ /<MESSAGE>(.*)<END_MESSAGE>/s ) {
-        $body = $1;
-        $body =~ s|\n?(.*)\n?|$1|;
-    }
-
-    my $THE_body = <<END_OF_BODY;
-$email_header
-$body
-END_OF_BODY
-
-    try {
+    if ( !defined $iso2709 ) {
+        carp "Error sending mail: empty basket";
+        $template->param( error => 1 );
+    } elsif ( !defined $user_email or $user_email eq '' ) {
+        carp "Error sending mail: sender's email address is invalid";
+        $template->param( error => 1 );
+    } else {
+        my %loops = (
+            biblio => \@bibs,
+        );
 
-        my $email = Koha::Email->create(
-            {
-                to      => $email_add,
-                subject => $subject,
-            }
+        my %substitute = (
+            comment => $comment,
         );
 
-        $email->text_body( $THE_body );
-        $email->attach(
-            Encode::encode( "UTF-8", $iso2709 ),
-            content_type => 'application/octet-stream',
-            name         => 'basket.iso2709',
-            disposition  => 'attachment',
+        my $letter = C4::Letters::GetPreparedLetter(
+            module => 'catalogue',
+            letter_code => 'CART',
+            lang => $patron->lang,
+            tables => {
+                borrowers => $borrowernumber,
+            },
+            message_transport_type => 'email',
+            loops => \%loops,
+            substitute => \%substitute,
         );
 
-        my $library = Koha::Patrons->find( $borrowernumber )->library;
-        $email->send_or_die({ transport => $library->smtp_server->transport });
-        $template->param( SENT => "1" );
-    }
-    catch {
-        carp "Error sending mail: $_";
-        $template->param( error => 1 );
-    };
+        my $attachment = {
+            filename => 'basket.iso2709',
+            type => 'application/octet-stream',
+            content => Encode::encode("UTF-8", $iso2709),
+        };
+
+        C4::Letters::EnqueueLetter({
+            letter => $letter,
+            message_transport_type => 'email',
+            borrowernumber => $patron->borrowernumber,
+            to_address => $email_add,
+            reply_address => $user_email,
+            attachments => [$attachment],
+        });
 
+        $template->param( SENT => 1 );
+    }
     $template->param( email_add => $email_add );
     output_html_with_http_headers $query, $cookie, $template->output;
 }