Problem with email attachment, 1 becomes 10 [Perl]

Prev: Some sort of scoping problem
Next: Removing HTML from text

From: M.O.B. i L. on 4 May 2008 20:07

I'm trying to write a program that can send out mails to companies where
I'm interested in working. These mails are the same except that the
company name is switched. My CV should be attached as a pdf-file.
The problem is that I don't get one attachment, but ten! Also the
message is repeated 10 times. If I change the program by uncommenting
the commented line in the function load() it works but then it doesn't
send the correct pdf-file. The CV is 129490 bytes. How could I change
this program so that there is only one attachment and no repeated messages?

In the program below some long lines might have been splitted.

#!/usr/bin/perl

# jobapplier.pl vN/A (Buggy!)
# Author: Mikael O. Bonnier, mikaelb(a)df.lth.se,
http://www.df.lth.se/~mikaelb/
# Copyright (C) 2008 by Mikael O. Bonnier, Lund, Sweden.
# License: GNU GPL v3 or later, http://www.gnu.org/licenses/gpl-3.0.txt

use strict;
use Email::Send;
use Email::Simple::Creator;
use Email::MIME::Modifier;
use DBI;
use MIME::Words qw/encode_mimewords/;
use File::stat;

my %settings = (
SMTP => 'mail.df.lth.se',
PORT => 588,
FROM_DOMAIN => 'cust.bredbandsbolaget.se',
DB => 'jobapplier_test',
DB_HOST => 'localhost',
FROM => 'mikaelb(a)df.lth.se',
SUBJECT => 'Intresseanm�lan som fysiker eller programutvecklare',
BODY =>
'Det skulle vara intressant att jobba hos er p� $cCompany.',
UPLDDIR => '../Documents/LyX',
FILE => 'MOB_CV.pdf',
);
@ARGV == 4
or die
'Provide username and password for the SMTP and the MySQL server,
respectively,'
. " on the command line.\n";
@settings{qw( USER PASS DB_USER DB_PASS)} = @ARGV;

my $dbh = DBI->connect( "DBI:mysql:$settings{DB}:$settings{DB_HOST}",
$settings{DB_USER}, $settings{DB_PASS} );

my $q = q|select cCompany, cEmail
from Contacts
order by cId asc|;
my $sth = $dbh->prepare($q);
$sth->execute;
my $as = [];
while ( my $hr = $sth->fetchrow_hashref ) {
push @{$as}, $hr;
}
$dbh->disconnect;

my $mailer = Email::Send->new(
{
mailer => 'SMTP::TLS',
mailer_args => [
Host => $settings{SMTP},
Port => $settings{PORT},
User => $settings{USER},
Password => $settings{PASS},
Hello => $settings{FROM_DOMAIN},
]
}
);

for my $row (@$as) {
my $body_text = $settings{BODY};
$body_text =~ s/\$cCompany/$row->{cCompany}/g;
my $email = Email::Simple->create(
header => [
From => $settings{FROM},
To => $row->{cEmail},
Subject => encode_mimewords( $settings{SUBJECT} ),
'Content-Type' => 'text/plain; charset="utf-8"',
],
body => $body_text,
);
my $body = load( $settings{UPLDDIR}, $settings{FILE} );
my $part = Email::MIME->new('');
$part->body_set($body);
$part->disposition_set('attachment');
$part->name_set( $settings{FILE} ); # Both needed?
$part->filename_set( $settings{FILE} ); # Both needed?
$part->encoding_set('base64');
$part->content_type_set('application/octet-stream');
my @more_parts = ($part);
$email = Email::MIME->new( $email->as_string );
$email->parts_add( \@more_parts );

eval { $mailer->send($email) };
die "Error sending email: $@" if $@;
}

sub load {
my ( $upldDir, $inFile ) = @_;
my $DNLDFILE;
open $DNLDFILE, '<', "$upldDir/$inFile";
binmode $DNLDFILE;
my $st = stat $DNLDFILE;
my $content;
read $DNLDFILE, $content, $st->size;
close $DNLDFILE;

# $content = 'ABC';
return $content;
}

__END__

From: Ben Bullock on 4 May 2008 21:59

On Mon, 05 May 2008 03:07:53 +0200, M.O.B. i L. wrote:

> I'm trying to write a program that can send out mails to companies where
> I'm interested in working. These mails are the same except that the
> company name is switched. My CV should be attached as a pdf-file. The
> problem is that I don't get one attachment, but ten! Also the message is
> repeated 10 times. If I change the program by uncommenting the commented
> line in the function load() it works but then it doesn't send the
> correct pdf-file. The CV is 129490 bytes. How could I change this
> program so that there is only one attachment and no repeated messages?

I don't know the answer to your question, but after reading it I wonder
if anyone else can solve it for you, so I'll post my advice instead. One
piece of advice which has been useful to me which I learnt from the
posting guidelines for this group is, before posting, to always try to
reduce the problem to the minimum possible, let's say ten or twenty lines
of code:

"First make a short (less than 20-30 lines) and *complete* program
that illustrates the problem you are having. People should be able
to run your program by copy/pasting the code from your article. (You
will find that doing this step very often reveals your problem
directly. Leading to an answer much more quickly and reliably than
posting to Usenet.)"

In the case of your program there are two big chunks: first, lots of
stuff involving reading lines from the database, which is where I suspect
the problem has occurred, probably some kind of slip-up, but I can't
prove this because I don't have the database available, and then there is
an almost separate piece of code which involves sending a mail with an
attachment, which might be where the bug happened, although I think it's
unlikely.

If I was going to seriously try to solve your problem, I would first of
all split the program into two pieces, one which gets those database
lines and just prints them to STDOUT, and another piece which sends just
one mail with some fixed parameters, written into the file itself rather
than pulled from a database, and see which one of these two pieces is
going wrong.

Then, if I still couldn't debug it myself, I might try posting whichever
of the two pieces the problem had occurred in to the newsgroup. But it's
very hard (for me at least) to work out what might have happened in your
program because of all the dependencies. As it says in the quote above,
if you can reduce the dependencies to a minimum piece of code which
anybody can run, you'll probably end up solving your own problem into the
bargain. I have repeatedly had problems in Perl which I thought were
incredibly difficult, ended up frustrated enough to want to post them
here, and in the business of reducing them into a short newsgroup post as
described, I ended up surprising myself by working out what the bug was
without having to post it.

Having said all that, perhaps someone will spot the bug in your code the
next minute after I post this, so who knows?

From: Bill H on 7 May 2008 06:14

On May 4, 9:07 pm, "M.O.B. i L." <mika...(a)df.lth.se> wrote:
> for my $row (@$as) {
> my $body_text = $settings{BODY};
> $body_text =~ s/\$cCompany/$row->{cCompany}/g;
> my $email = Email::Simple->create(
> header => [
> From => $settings{FROM},
> To => $row->{cEmail},
> Subject => encode_mimewords( $settings{SUBJECT} ),
> 'Content-Type' => 'text/plain; charset="utf-8"',
> ],
> body => $body_text,
> );
> my $body = load( $settings{UPLDDIR}, $settings{FILE} );
> my $part = Email::MIME->new('');
> $part->body_set($body);
> $part->disposition_set('attachment');
> $part->name_set( $settings{FILE} ); # Both needed?
> $part->filename_set( $settings{FILE} ); # Both needed?
> $part->encoding_set('base64');
> $part->content_type_set('application/octet-stream');
> my @more_parts = ($part);
> $email = Email::MIME->new( $email->as_string );
> $email->parts_add( \@more_parts );
>
> eval { $mailer->send($email) };
> die "Error sending email: $@" if $@;

I think your issue has to do with the "more_parts".

Bill H

|
Pages: 1
Prev: Some sort of scoping problem
Next: Removing HTML from text