511216 - Assignee is blank in Whine emails

Status: RESOLVED FIXED

(Bugzilla :: Whining, defect, P1)

Description

[reid]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.0.10) Gecko/2009042315 Firefox/3.0.10
Build Identifier: 3.4.1

In my whine mails, the value of the Assignee field is missing.

Reproducible: Always

Comment 1

[reid]

Attachment: Sample whine e-mail demonstrating the missing value

Comment 2

[reid]

Attachment: patch against 3.4.1 using assignee real name in the mails

This patch uses the assignee real name for the whine mails.

It is against 3.4.1. I realize that that's not the right way to submit a patch, but I don't yet have a HEAD install and thought that since it's so small, it would be useful anyway. A cursory glance at HEAD makes me think that it would probably apply OK.

Feedback appreciated.

Comment 3

[Kent Rogers [:mrbball]]

Attachment: V2 - patch against tip, keeping assignee login name in the whine email

I just had this problem reported on my 3.4.1 installation.  

I see that the issue is that "map_assigned_to.login_name" now gets
mapped back to "assigned_to" by Search.pm::COLUMNS(), when the
select column list is built at line 927 (in Search.pm::init()).
Thus, that is the name that is used to build the "bug" arrays in
whine.pl::run_queries(), and ultimately the name that must be used
in the whine/mail.* templates.

This patch is against the tip and leaves login_name in the whine emails.

BTW, this is a bug, not an enhancement.  Assignee was there in 3.2.4
whine emails but is missing in 3.4.1 whine emails.

Comment 4

[Max Kanat-Alexander]

By the way, the reason that this bug was not given the appropriate priority before was that the summary described a solution, not a problem. Usually enhancements describe a solution, and bugs describe a problem.

Comment 5

[reid]

Max, my original summary was "Assignee value missing from whine mails" which seems quite similar to me to the current "Assignee is blank in Whine emails"... I don't understand how that describes a solution rather than a problem. Can you give any guidance on what I'm missing?

Comment 6

[Max Kanat-Alexander]

(In reply to comment #5)
> Max, my original summary was "Assignee value missing from whine mails" which
> seems quite similar to me to the current "Assignee is blank in Whine emails"...
> I don't understand how that describes a solution rather than a problem. Can you
> give any guidance on what I'm missing?

  Oh, you're indeed right, and I simply made an error here when I changed the field values. :-)

Comment 7

[Frédéric Buclin]

Attachment: patch for 3.6, v2.1

Unbitrotten patch, due to bug 519142. I also removed the duplicated 'assigned_to' entry in whine.pl.

Comment 8

[Frédéric Buclin]

Comment on attachment 395140 [details] [diff] [review]
patch against 3.4.1 using assignee real name in the mails

>-  Assignee: [%+ bug.$assignee_login_string %]
>+  Assignee: [%+ bug.assigned_to_realname %]

No reason to display the real name of assignees. We should keep login names.

Comment 9

[Frédéric Buclin]

Attachment: patch for 3.4, v2

Backport of the patch for 3.6.

Comment 10

[Frédéric Buclin]

Leaving Kent as the assignee as I simply removed bitrot.

Comment 11

[Frédéric Buclin]

tip:

Checking in whine.pl;
/cvsroot/mozilla/webtools/bugzilla/whine.pl,v  <--  whine.pl
new revision: 1.41; previous revision: 1.40
done
Checking in template/en/default/whine/mail.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/whine/mail.html.tmpl,v  <--  mail.html.tmpl
new revision: 1.10; previous revision: 1.9
done
Checking in template/en/default/whine/mail.txt.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/whine/mail.txt.tmpl,v  <--  mail.txt.tmpl
new revision: 1.10; previous revision: 1.9
done


3.4.4:

Checking in whine.pl;
/cvsroot/mozilla/webtools/bugzilla/whine.pl,v  <--  whine.pl
new revision: 1.38.2.2; previous revision: 1.38.2.1
done
Checking in template/en/default/whine/mail.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/whine/mail.html.tmpl,v  <--  mail.html.tmpl
new revision: 1.7.4.1; previous revision: 1.7
done
Checking in template/en/default/whine/mail.txt.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/whine/mail.txt.tmpl,v  <--  mail.txt.tmpl
new revision: 1.7.4.1; previous revision: 1.7
done