Closed (fixed)
Project:
Devel
Version:
8.x-1.x-dev
Component:
devel
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
23 May 2013 at 19:28 UTC
Updated:
14 Jun 2013 at 20:30 UTC
Jump to comment: Most recent file
I ran into this while testing #1560012: Port per-user issue notification email functionality to D7. It's minor, but annoying. If you use devel.mail.inc, you see stuff like this:
From: "admin (D7 Project)" <admin@example.com>
Date: Thu, 23 May 2013 15:23:17 -0400
...
To: admin@example.com
[Task] Yet another new title
Issue status update for:
http://localhost/d7project/node/30
Update issue:
http://localhost/d7project/node/30/edit
It should really look like this:
From: "admin (D7 Project)" <admin@example.com>
Date: Thu, 23 May 2013 15:23:17 -0400
...
To: admin@example.com
Subject: [Task] Yet another new title
Issue status update for:
http://localhost/d7project/node/30
Update issue:
http://localhost/d7project/node/30/edit
At first, I thought my email generation code was broken, but upon closer inspection, it was just how devel.mail.inc is writing this stuff to disk. :/
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 2002030-6.devel-mail-d8.patch | 1.19 KB | dww |
| #6 | 2002030-6.devel-mail-d7.patch | 1.01 KB | dww |
| #3 | 2002030-3.devel-mail-d7.patch | 939 bytes | dww |
| #1 | 2002030-1.devel-mail-d7.patch | 589 bytes | dww |
Comments
Comment #1
dwwAny objections? I can forward/back port to other branches if not.
Thanks,
-Derek
Comment #3
dwwHeh. ;) Fixed the test.
Comment #4
moshe weitzman commentedLooks fine to me. Will commit soon if nobody objects.
Comment #5
salvis"Subject" is a mail header and should not be translated. And it should be followed by an empty line, not dashes.
Yes, D8 first. :-)
Comment #6
dwwPatches for D8 and D7 that fix #5.
Comment #7
dwwp.s. D6 does this only via watchdog, and the bug isn't present there since there's actually a "Subject:" in the resulting log message.
Comment #8
dwwLet's see #6 against D8.
Comment #9
dww#6: 2002030-6.devel-mail-d8.patch queued for re-testing.
Comment #10
dwwNow can I commit these? ;)
Thanks,
-Derek
Comment #11
moshe weitzman commentedLooks good. Go ahead.
Comment #12
dwwGreat, thanks:
D8: http://drupalcode.org/project/devel.git/commit/59475bc
D7: http://drupalcode.org/project/devel.git/commit/dbb40b1
Cheers,
-Derek