FOR and FOREACH are completely interchangeable. I prefer the former, but either is fine.

I would suggest that you convince us (or yourself, at least) that $results contains what you expect, e.g. using Data::Dumper.

Update: it occurs to me that @sendoff is never defined. Did you really run this using strict? You don't seem to need $sendoff or @sendoff at all: just write

$template->param('results' => $results);
[download]

Yeah - printing the \@sendoff via Dumper displays what I need. I actually have a typo in what I posted, the actual code is

my $results = &return_values;
my @sendoff = @$results;
[download]