Returns a list of contacts (L<FS::contact> objects) for the customer. If
a list of contact classnums is given, returns only contacts in those
-classes. If the pseudo-classnum 'invoice' is given, returns contacts that
-are marked as invoice destinations. If '0' is given, also returns contacts
-with no class.
+classes. If '0' is given, also returns contacts with no class.
If no arguments are given, returns all contacts for the customer.
my $self = shift;
my $search = {
table => 'contact',
- select => 'contact.*, cust_contact.invoice_dest',
- addl_from => ' JOIN cust_contact USING (contactnum)',
- extra_sql => ' WHERE cust_contact.custnum = '.$self->custnum,
+ select => 'contact.*',
+ extra_sql => ' WHERE contact.custnum = '.$self->custnum,
};
my @orwhere;
my @classnums;
foreach (@_) {
- if ( $_ eq 'invoice' ) {
- push @orwhere, 'cust_contact.invoice_dest = \'Y\'';
- } elsif ( $_ eq '0' ) {
- push @orwhere, 'cust_contact.classnum is null';
+ if ( $_ eq '0' ) {
+ push @orwhere, 'contact.classnum is null';
} elsif ( /^\d+$/ ) {
push @classnums, $_;
} else {
}
if (@classnums) {
- push @orwhere, 'cust_contact.classnum IN ('.join(',', @classnums).')';
+ push @orwhere, 'contact.classnum IN ('.join(',', @classnums).')';
}
if (@orwhere) {
$search->{extra_sql} .= ' AND (' .
=item contact_list_email [ CLASSNUM, ... ]
Same as L</contact_list>, but returns email destinations instead of contact
-objects.
+objects. Also accepts 'invoice' as an argument, in which case this will also
+return the invoice email address if any.
=cut
sub contact_list_email {
my $self = shift;
- my @contacts = $self->contact_list(@_);
- my @emails;
- foreach my $contact (@contacts) {
- foreach my $contact_email ($contact->contact_email) {
- push @emails,
- $contact->firstlast . ' <' . $contact_email->emailaddress . '>';
+ my @classnums;
+ my $and_invoice;
+ foreach (@_) {
+ if (/^invoice$/) {
+ $and_invoice = 1;
+ } else {
+ push @classnums, $_;
+ }
+ }
+ my %emails;
+ # if the only argument passed was 'invoice' then no classnums are
+ # intended, so skip this.
+ if ( @classnums ) {
+ my @contacts = $self->contact_list(@classnums);
+ foreach my $contact (@contacts) {
+ foreach my $contact_email ($contact->contact_email) {
+ # unlike on 4.x, we have a separate list of invoice email
+ # destinations.
+ # make sure they're not redundant with contact emails
+ my $dest = $contact->firstlast . ' <' . $contact_email->emailaddress . '>';
+ $emails{ $contact_email->emailaddress } = $dest;
+ }
+ }
+ }
+ if ( $and_invoice ) {
+ foreach my $email ($self->invoicing_list_emailonly) {
+ my $dest = $self->name_short . ' <' . $email . '>';
+ $emails{ $email } ||= $dest;
}
}
- @emails;
+ values %emails;
}
=item referral_custnum_cust_main
# 3.x: false laziness with msg_template.pm; on 4.x, all email notices
# are generated from templates and this case goes away
my @classes;
- if ( $opt{'to_contact_classnum'} ) {
- my $classnum = $opt{'to_contact_classnum'};
- @classes = ref($classnum) ? @$classnum : split(',', $classnum);
+ if ( $to_contact_classnum ) {
+ @classes = ref($to_contact_classnum) ? @$to_contact_classnum : split(',', $to_contact_classnum);
}
if (!@classes) {
@classes = ( 'invoice' );
<INPUT TYPE="hidden" NAME="search" VALUE="<% encode_base64(nfreeze(\%search)) %>">
<INPUT TYPE="hidden" NAME="popup" VALUE="<% $popup %>">
<INPUT TYPE="hidden" NAME="url" VALUE="<% $url | h %>">
-<INPUT TYPE="hidden" NAME="to_contact_classnum" VALUE="<% join(',', @contact_classnum) %>">
% if ( $cgi->param('action') eq 'send' ) {
+ <INPUT TYPE="hidden" NAME="to_contact_classnum" VALUE="<% $cgi->param('to_contact_classnum') %>">
<FONT SIZE="+2">Sending notice</FONT>
<& /elements/progress-init.html,
% } elsif ( $cgi->param('action') eq 'preview' ) {
+ <INPUT TYPE="hidden" NAME="to_contact_classnum" VALUE="<% join(',', @contact_classnum) %>">
<FONT SIZE="+2">Preview notice</FONT>
% }
onchange => 'toggle(this)',
&>
<BR>
-% }
% # select destination contact classes
Send to contacts:
<& /elements/checkboxes.html,
($from, $subject, $html_body) = @message{'from', 'subject', 'html_body'};
}
- # contact_class_X params
- foreach my $param ( $cgi->multi_param ) {
+ # contact_class_X params in preview
+ foreach my $param ( $cgi->param ) {
if ( $param =~ /^contact_class_(\w+)$/ ) {
push @contact_classnum, $1;
if ( $1 eq 'invoice' ) {
}
}
}
+
}
+# and set up contact checkboxes for edit mode
my @contact_checkboxes = (
[ 'invoice' => { label => 'Invoice recipients' } ]
);