From d8d2ca30e3780b4693ba42a0315e3f00f1884ebc Mon Sep 17 00:00:00 2001 From: "reed%reedloden.com" <> Date: Wed, 27 Dec 2006 09:12:35 +0000 Subject: Bug 364958 - "If user does not have a real name, just e-mail address should be displayed instead of " "" [p=reed r=bkor a=justdave] --- Bugzilla/BugMail.pm | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm index 6e0f57401..d6404f2eb 100644 --- a/Bugzilla/BugMail.pm +++ b/Bugzilla/BugMail.pm @@ -220,14 +220,16 @@ sub Send { my $diffheader = ""; my @diffparts; my $lastwho = ""; + my $fullwho; my @changedfields; foreach my $ref (@$diffs) { my ($who, $whoname, $what, $when, $old, $new, $attachid, $fieldname) = (@$ref); my $diffpart = {}; if ($who ne $lastwho) { $lastwho = $who; - $diffheader = "\n$whoname <$who" . Bugzilla->params->{'emailsuffix'} - . "> changed:\n\n"; + $fullwho = $whoname ? "$whoname <$who" . Bugzilla->params->{'emailsuffix'} . ">" : + "$who" . Bugzilla->params->{'emailsuffix'}; + $diffheader = "\n$fullwho changed:\n\n"; $diffheader .= FormatTriple("What ", "Removed", "Added"); $diffheader .= ('-' x 76) . "\n"; } @@ -691,9 +693,14 @@ sub prepare_comments { my $result = ""; foreach my $comment (@$raw_comments) { if ($count) { - $result .= "\n\n--- Comment #$count from " . $comment->{'name'} . " <" . - $comment->{'email'} . Bugzilla->params->{'emailsuffix'} . "> " . - format_time($comment->{'time'}) . " ---\n"; + $result .= "\n\n--- Comment #$count from "; + if ($comment->{'name'} eq $comment->{'email'}) { + $result .= $comment->{'email'} . Bugzilla->params->{'emailsuffix'}; + } else { + $result .= $comment->{'name'} . " <" . $comment->{'email'} . + Bugzilla->params->{'emailsuffix'} . ">"; + } + $result .= " " . format_time($comment->{'time'}) . " ---\n"; } # Format language specific comments. We don't update $comment->{'body'} # directly, otherwise it would grow everytime you call format_comment() -- cgit v1.2.3-24-g4f1b