455870 - Cannot send attachments with cli and file with accents (do urifixup in the command line handler).

Status: NEW

(SeaMonkey :: MailNews: Composition, enhancement)

Description

[Philip Chee]

+++ This bug was initially created as a clone of Bug #384160 +++

When you send a file from the desktop (gnome or kde), it triggers a command line which ressembles to :
thunderbird -compose "attachment='file:///home/mloiseleur/Desktop/aéa.txt'"

That's at least what I do. The compose windows then popup, but the mail cannot be sent. It says that the file is not present, but the file is here, and the attachment is viewable !

The problem here seems to be the encoding of the uri.


Reproducible: Always

Actual Results:  
The problem is that the mail cannot be send. There is a message saying the file is not here. 
After further investigation, we have seen that we need an encoded url in order to work. In my example, the problem go away with a call like this : 
thunderbird -compose "attachment='file://$path_with_accents'"
any file with accents is subject to this issue.

Expected Results:  
The mail should be sent normally, since the attachment is viewable in the GUI.
It's really strange to user and developpers who try to user thunderbird in CLI.

Comment 1

[Philip Chee]

From irc#seamonkey

<Ratty>	NeilAway, Mnyromyr: do we want bug 384160 as well?
<Mnyromyr>	RattyAway: yes, low hanging fruit, given the existence and size of the patch
<NeilAway>	hmm, that patch should try to do uri fixup
in fact, the command line handler should fix it up, so you can attach relative file names

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

(In reply to comment #1)
> <NeilAway>    hmm, that patch should try to do uri fixup
> in fact, the command line handler should fix it up, so you can attach relative
> file names

Before this falls by the wayside: I don't know what that means. The approach TB used <http://hg.mozilla.org/comm-central/rev/35e89cc8d796> is better than nothing and should be easy to port. Maybe there is a better way but the above is certainly too vague for someone like me to code it up.

Comment 3

[Philip Chee]

Oops I opened a new bug (Bug 617011) since I forgot all about this one.

> Before this falls by the wayside: I don't know what that means. The approach TB
> used <http://hg.mozilla.org/comm-central/rev/35e89cc8d796> is better than
> nothing and should be easy to port. Maybe there is a better way but the above
> is certainly too vague for someone like me to code it up.
Will be fixed in Bug 617011.

Leaving this open for the remaining part:

> <NeilAway>    hmm, that patch should try to do uri fixup
> in fact, the command line handler should fix it up, so you can attach relative file names