Conservancy/supporters
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

These two places should be $laspesSoon

Browse source 
Only debugging output is impacted here, but it really should be based on
the variable we're actually using in the next statement.
This commit is contained in:
Bradley M. Kuhn 2016-12-02 14:39:36 -08:00
parent 65070094ed
commit 98026dd16d
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
scripts/send-renewal-notices.plx
View file

@ -97,14 +97,14 @@ foreach my $supporterId (@supporterIds) {
if (defined $request) { if (defined $request) {
if (defined $request->{fulfillDate}) { if (defined $request->{fulfillDate}) {
print STDERR "$supporterId lapsed on $expiresOn but recorded as renewed on $request->{fulfillDate}\n" print STDERR "$supporterId lapsed on $expiresOn but recorded as renewed on $request->{fulfillDate}\n"
if ( ($isLapsed or $lapsesInOneWeek) and $VERBOSE); if ( ($isLapsed or $lapsesSoon) and $VERBOSE);
} else { } else {
print STDERR "$supporterId received this renewal notice already on $request->{requestDate}\n" print STDERR "$supporterId received this renewal notice already on $request->{requestDate}\n"
if $VERBOSE; if $VERBOSE;
} }
next; next;
} }
print STDERR "$supporterId skipped since he is not lapsed\n" if ( (not $isLapsed and not $lapsesInOneWeek) and $VERBOSE > 1); print STDERR "$supporterId skipped since he is not lapsed\n" if ( (not $isLapsed and not $lapsesSoon) and $VERBOSE > 1);
next unless $isLapsed or $lapsesSoon; next unless $isLapsed or $lapsesSoon;