312507 - mailto:%20 messes up address fields

Status: RESOLVED FIXED

(MailNews Core :: Composition, defect)

Description

[Randall Schuh II]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4) Gecko/20050908 Firefox/1.4
Build Identifier: Thunderbird version 1.0.7 (20050923)   -  Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4) Gecko/20050908 Firefox/1.4

When the provided mailto link is clicked, a message appears with the correct
subject and body, but there is no empty To: field.  Furthermore, it was not
clear to me how to get a To: field.  I downloaded the latest version of T-Bird
1.0 and it still occurs.

Reproducible: Always

Steps to Reproduce:
1. In address bar of IE or Firefox, type the following
mailto:%20?Subject=Bloomberg%20news:%20%20Lockheed Martin to Cut Costs by Paring
Benefits to New Workers &body=%20Lockheed Martin to Cut Costs by Paring Benefits
to New Workers
%0D%0A%0D%0A%20http%3A//www.bloomberg.com/apps/news%3Fpid%3Demail_us%26refer=us%26sid%3DaARJ5k2bt76I

Alternatively, visit
http://www.bloomberg.com/apps/news?pid=10000103&sid=aARJ5k2bt76I&refer=us and
click on "E-Mail this story".

I had this problem on all the Bloomberg sites I visited.


Actual Results:  
Message appears with a To: field (I think an empty one), but disappears almost
faster than my eye could see.  The box where the To:, Cc:, or Bcc:  fields would
appear does not respond.  I even tried clicking Contacts and adding them, but
when I attempted to add contacts, there was no response.  I tried sending it
after adding a contact and it failed as having no To: recipient.

Expected Results:  
produce error or preferably provide empty To: field in the box between the From:
and Subject: boxes.

Comment 1

[zug_treno]

Also reproducible with Mozilla 1.8b1 and SeaMonkey 1.0a.

Comment 2

[Mike Cowperthwaite]

Reproduced with TB 1.5b1-1003 & TB 1.6a1-1003, Win2K.

Comment 3

[Magnus Melin [:mkmelin]]

Also linux branch builds.
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8b5) Gecko/20051015
Thunderbird/1.4.1 ID:2005101506

Can be reproduced with the simple mailto:%20

For me, the To: etc fields does not display at all. Only an empty area.

Comment 4

[Magnus Melin [:mkmelin]]

*** Bug 335228 has been marked as a duplicate of this bug. ***

Comment 7

[Bruno 'Aqualon' Escherl]

Attachment: Add an empty To: field, if there's no recipient

The patch looks at the number of recipients and not if msgTo contains any string. If the number is 0 like for the URL testcase, havePrimaryRecipient isn't set and an empty To: field is added, preventing the broken addressing widget.

The patch is for SeaMonkey and Thunderbird, but I only tested the SeaMonkey version.

Comment 8

[David :Bienvenu]

Comment on attachment 281116 [details] [diff] [review]
Add an empty To: field, if there's no recipient

thx, Bruno, that seems to work with Thunderbird as well.

You should probably ask a SM peer for a review on the SM part, or at least ask them if they're ok with my review :-)

Comment 9

[Karsten Düsterloh]

Landed on trunk.