Closed Bug 29211 Opened 25 years ago Closed 25 years ago

[nsHTMLToTXTSinkStream] URLs surrounded by quotes *and* angle brackets

Categories

(Core :: DOM: Serializers, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: BenB, Assigned: BenB)

Details

(Whiteboard: [PDT-]Patch available, waiting for review and check in)

Attachments

(1 file)

Reproduce:
Send a HTML mail with a link (a-tag).

Actual result:
URLs from href attribute surrounded by quotes *and* angle brackets.

Expected result:
URLs from href attribute surrounded only by angle brackets.

Proposed fix:
Make sure, the quotes aroung the href in the original HTML are not included.
Severity: normal → minor
Target Milestone: M15
Correction:
Reproduce:
Create a mail with a link (a-tag) using the HTML editor. Send it as plain text.
Fixed. akk, can you review and check in please?
Status: NEW → ASSIGNED
Component: Editor → DOM to Text Conversion
Whiteboard: Patch available, waiting for review and check in
The patch looks fine: I'll check it in in M15.  If I forget, please remind me
shortly after the tree opens ... or reassign this to me so that will be on my
list as a reminder.
Reassign to akk to "bug" her. Will reassign to me later, to keep track of bugs I
fixed.
Assignee: mozilla → akkana
Status: ASSIGNED → NEW
Okay, thanks.  I'll assign it back to you when I've checked in, since it's
really your fix.
Status: NEW → ASSIGNED
Low risk.
Keywords: beta1, patch
But, of course, no must-have.
Putting on PDT- radar for beta1.  will consider after branch.
Whiteboard: Patch available, waiting for review and check in → [PDT-]Patch available, waiting for review and check in
Checked in.  Ben, giving this back to you, you can mark it fixed.
Assignee: akkana → mozilla
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Thanks
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
whose gonna help me mark this bug verified-fixed???
Oh, the reproduction was wrong. You need to send the msg with "plain text" or

"both plain text and HTML". You just have to use the HTML editor. The bug is in

the plain text part. Sorry.



Jar's post <news://news.mozilla.org/38C2D4AE.9020706@netscape.com> shows a buggy

URL in the plain text part (view it with page source):



PDT+ in the =



status whiteboard =



<"http://bugzilla.mozilla.org/buglist.cgi?bug_status=3DNEW&bug_status=3DA=

SSIGNED&bug_status=3DREOPENED&email1=3D&emailtype1=3Dsubstring&emailassig=

ned_to1=3D1&email2=3D&emailtype2=3Dsubstring&emailreporter2=3D1&changedin=

=3D&votes=3D&chfieldfrom=3D&chfieldto=3DNow&chfieldvalue=3D&short_desc=3D=

&short_desc_type=3Dsubstring&long_desc=3D&long_desc_type=3Dsubstring&bug_=

file_loc=3D&bug_file_loc_type=3Dsubstring&status_whiteboard=3Dpdt%2B&stat=

us_whiteboard_type=3Dsubstring&newqueryname=3D&form_name=3Dquery&order=3D=

assign.login_name%2C%20bugs.priority%2C%20bugs.bug_severity">



(ignore the '=' and '=3D'). There should be no '"' (double quote) before 'http'

and after 'severity'.

verified in 3/14 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: