Closed Bug 511216 Opened 15 years ago Closed 14 years ago

Assignee is blank in Whine emails

Categories

(Bugzilla :: Whining, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Bugzilla 3.4

People

(Reporter: reid, Assigned: mrbball)

Details

Attachments

(2 files, 3 obsolete files)

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
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.
Attachment #395140 - Flags: review?
Severity: normal → enhancement
OS: Linux → All
Priority: -- → P3
Hardware: x86 → All
Summary: Assignee value missing from whine mails → Show Assignee in Whine mails
Attachment #395140 - Flags: review? → review?(wicked)
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.
Attachment #407776 - Flags: review?(wicked)
Severity: enhancement → normal
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking3.4.5+
Priority: P3 → P1
Target Milestone: --- → Bugzilla 3.4
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.
Summary: Show Assignee in Whine mails → Assignee is blank in Whine emails
Assignee: whining → kar
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?
(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. :-)
Unbitrotten patch, due to bug 519142. I also removed the duplicated 'assigned_to' entry in whine.pl.
Attachment #395137 - Attachment is obsolete: true
Attachment #407776 - Attachment is obsolete: true
Attachment #419265 - Flags: review+
Attachment #407776 - Flags: review?(wicked)
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.
Attachment #395140 - Flags: review?(wicked) → review-
Backport of the patch for 3.6.
Attachment #395140 - Attachment is obsolete: true
Attachment #419266 - Flags: review+
Leaving Kent as the assignee as I simply removed bitrot.
Status: NEW → ASSIGNED
Flags: blocking3.6+
Flags: approval3.4+
Flags: approval+
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
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.