RT-Plugins/RT-Extension-ThreadByReference
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use sprintf for log4perl.

Browse source
This commit is contained in:
Harlan Lieberman-Berg 2015-12-21 16:20:50 -05:00
parent cacbc9182c
commit 615cac871f
1 changed files with 8 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
lib/RT/Interface/Email/TryThreading.pm
View file

@ -26,12 +26,12 @@ sub GetCurrentUser {
); );
if ($args{'Ticket'}->id) { if ($args{'Ticket'}->id) {
$RT::Logger->debug("Ticket %s already assigned. You don't need my help!", $RT::Logger->debug(sprintf("Ticket %s already assigned. You don't need my help!",
$args{'Ticket'}->id); $args{'Ticket'}->id));
return ($args{'CurrentUser'}, $args{'AuthLevel'}); return ($args{'CurrentUser'}, $args{'AuthLevel'});
} }
$RT::Logger->debug("Operating on queue %s", $args{'Queue'}); $RT::Logger->debug(sprintf("Operating on queue %s", $args{'Queue'}));
my @messageids = FetchPossibleHeaders($args{'Message'}); my @messageids = FetchPossibleHeaders($args{'Message'});
@ -57,11 +57,11 @@ sub GetCurrentUser {
} }
elsif (scalar(@tickets) > 1) { elsif (scalar(@tickets) > 1) {
$RT::Logger->warning("Email maps to more than one ticket."); $RT::Logger->warning("Email maps to more than one ticket.");
$RT::Logger->warning("Tickets: %s", @tickets); $RT::Logger->warning(sprintf("Tickets: %s", @tickets));
} }
# We have the ticket. Set it. # We have the ticket. Set it.
$RT::Logger->debug("Threading email in ticket %s", $tickets[0]); $RT::Logger->debug(sprintf("Threading email in ticket %s", $tickets[0]));
$args{'Ticket'}->Load($tickets[0]); $args{'Ticket'}->Load($tickets[0]);
return ($args{'CurrentUser'}, $args{'AuthLevel'}); return ($args{'CurrentUser'}, $args{'AuthLevel'});
@ -85,10 +85,10 @@ sub FetchPossibleHeaders {
$ref =~ /,?<([^>]+)>/; $ref =~ /,?<([^>]+)>/;
if ($1) { if ($1) {
push(@msgids, $1); push(@msgids, $1);
$RT::Logger->debug("Found reference: %s", $1); $RT::Logger->debug(sprintf("Found reference: %s", $1));
} }
else { else {
$RT::Logger->debug("Reference with borked syntax: %s", $ref); $RT::Logger->debug(sprintf("Reference with borked syntax: %s", $ref));
next; next;
} }
} }
@ -143,4 +143,5 @@ sub MessageIdToTicket {
return keys(%tickets); return keys(%tickets);
} }
1; 1;