Thanks Greg - I've passed this on to the developer. When we give a full
release, this snippet should be in there :)
Gregory Neil Shapiro wrote:
>
> Some testing with the new 2.2 client code has turned up a couple of bugs in
> template handling:
>
> 1. mailfrom/mailto weren't set properly (code used $data{} instead of $in{}
> 2. period_text was using $data{} instead of $in{}
> 3. Builk orders would have all of the domains listed (null separated) in
> every message instead of one domain per message.
>
> The patch below fixes all three.
>
> --- cgi/reg_system.cgi~ Tue Sep 5 07:53:56 2000
> +++ cgi/reg_system.cgi Wed Sep 6 23:29:07 2000
> @@ -530,10 +530,11 @@
> send_email("$path_templates/message.txt",
> {
> %in,
> - mailfrom => $data{owner_email},
> + domain => $domain,
> + mailfrom => $in{owner_email},
> mailto => $ADMIN_EMAIL,
> id => $id,
> - period_text => $REG_PERIODS{$data{period}},
> + period_text => $REG_PERIODS{$in{period}},
> reg_type => $reg_type,
> });
> }
> @@ -543,7 +544,8 @@
> send_email("$path_templates/thankyou.txt",
> {
> %in,
> - mailto => $data{owner_email},
> + domain => $domain,
> + mailto => $in{owner_email},
> mailfrom => $ADMIN_EMAIL,
> id => $id,
> });
> @@ -660,7 +662,7 @@
> {
> %in,
> bulk_transfer_id =>
>$register_results->{attributes}->{bulk_transfer_id},
> - mailfrom => $data{owner_email},
> + mailfrom => $in{owner_email},
> mailto => $ADMIN_EMAIL,
> order_ids => $order_ids,
> order_errors => $order_errors,
> @@ -673,7 +675,7 @@
> {
> %in,
> bulk_transfer_id =>
>$register_results->{attributes}->{bulk_transfer_id},
> - mailto => $data{owner_email},
> + mailto => $in{owner_email},
> mailfrom => $ADMIN_EMAIL,
> order_ids => $order_ids,
> });
--
Charles Daminato
OpenSRS Support Manager
[EMAIL PROTECTED]
