Open
Bug 323109
Opened 19 years ago
Updated 2 years ago
deleted attachment: "Deleted: " not localizable
Categories
(MailNews Core :: Internationalization, defect)
MailNews Core
Internationalization
Tracking
(Not tracked)
NEW
People
(Reporter: jhaar, Assigned: smontagu)
References
(Blocks 1 open bug)
Details
(Keywords: intl)
Attachments
(1 obsolete file)
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
When you delete an attachment, Thunderbird 1.5 replaces the attachment with MIME code like:
Content-Type: text/x-moz-deleted; name="Deleted: tibs-rescue.iso"
Content-Transfer-Encoding: 8bit
Content-Disposition: inline; filename="Deleted:tibs-rescue.iso"
X-Mozilla-Altered: AttachmentDeleted; date="Mon Nov 21 10:45:27 2005"
There's a slight mistake there: "Deleted: " vs "Deleted:"
Maybe that filename should be created as a variable first and then used for both lines?
Our AV triggers on that due to it looking like an attempt to pass an attachment through with different filenames...
Reproducible: Always
Comment 1•19 years ago
|
||
-> new, there is a missing space there.
Also, the string is not localizable (but the string for it is already in messenger.properties). Patch coming up.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Summary: deleted attachment feature created incorrect MIME messages → deleted attachment: "Deleted: " vs "Deleted:", not localizable
Comment 2•19 years ago
|
||
Attachment #210044 -
Flags: review?(bienvenu)
Comment 3•19 years ago
|
||
Comment on attachment 210044 [details] [diff] [review]
proposed fix
I'm not sure that just sticking a utf8 string in a content type name field is the right thing to do. Maybe it is, but cc'ing jshin
Attachment #210044 -
Flags: superreview?(bienvenu)
Attachment #210044 -
Flags: review?(jshin1987)
Attachment #210044 -
Flags: review?(bienvenu)
Comment 4•19 years ago
|
||
Comment on attachment 210044 [details] [diff] [review]
proposed fix
Sorry I missed this.
> (From update of attachment 210044 [details] [diff] [review] [edit])
> I'm not sure that just sticking a utf8 string in a content type name field is
> the right thing to do. Maybe it is, but cc'ing jshin
Per the *current* mail standard, we should use RFC 2231. The function to use is RFC2231ParmFolding at
http://lxr.mozilla.org/seamonkey/source/mailnews/compose/src/nsMsgCompUtils.cpp#1235
It's a static function so that we have to move it somewhere else.
Attachment #210044 -
Flags: review?(jshin1987)
Updated•19 years ago
|
Attachment #210044 -
Flags: superreview?(bienvenu)
Comment 6•18 years ago
|
||
The string is still not localizable and attachmentDeletePrefix from messenger.properties still not used. Added l10n@mozilla.com to CC.
Comment 7•18 years ago
|
||
(In reply to comment #1 and commnet #6)
blocking‑thunderbird2=?
Flags: blocking-thunderbird2?
Comment 8•18 years ago
|
||
Moving off bugs that didn't make the deadline for Thunderbird 2.
Flags: blocking-thunderbird2? → blocking-thunderbird2-
Target Milestone: --- → Thunderbird 3
Updated•18 years ago
|
QA Contact: front-end
Updated•17 years ago
|
Assignee: mscott → mkmelin+mozilla
Updated•15 years ago
|
Attachment #210044 -
Attachment is obsolete: true
Comment 10•15 years ago
|
||
The space after "Deleted:" was added in bug 329915.
Making it localizable is more work than I'd like to spend on this, so ->defaults
Assignee: mkmelin+mozilla → smontagu
Component: Mail Window Front End → Internationalization
Keywords: intl
Product: Thunderbird → MailNews Core
QA Contact: front-end → i18n
Hardware: x86 → All
Summary: deleted attachment: "Deleted: " vs "Deleted:", not localizable → deleted attachment: "Deleted: " not localizable
Target Milestone: Thunderbird 3 → ---
Comment 12•15 years ago
|
||
Another unlocalizable string next to it:
"You deleted an attachment from this message. The original MIME headers for the attachment were:"
Comment 13•15 years ago
|
||
Jesper, can you file a bug for this...
> Another unlocalizable string next to it:
> "You deleted an attachment from this message. The original MIME headers for the
> attachment were:"
Comment 14•15 years ago
|
||
(In reply to comment #13)
> Jesper, can you file a bug for this...
Isn't this the bug?
Comment 15•13 years ago
|
||
Simon Montagu (assignee), are you still working on this?
Otherwise, please assign to nobody.
Updated•13 years ago
|
Blocks: attachUXtracker
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•