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)
Core
DOM: Serializers
Tracking
()
VERIFIED
FIXED
M15
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.
Assignee | ||
Updated•25 years ago
|
Severity: normal → minor
Target Milestone: M15
Assignee | ||
Comment 1•25 years ago
|
||
Correction: Reproduce: Create a mail with a link (a-tag) using the HTML editor. Send it as plain text.
Assignee | ||
Comment 2•25 years ago
|
||
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
Assignee | ||
Comment 3•25 years ago
|
||
Comment 4•25 years ago
|
||
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.
Assignee | ||
Comment 5•25 years ago
|
||
Reassign to akk to "bug" her. Will reassign to me later, to keep track of bugs I fixed.
Assignee: mozilla → akkana
Status: ASSIGNED → NEW
Comment 6•25 years ago
|
||
Okay, thanks. I'll assign it back to you when I've checked in, since it's really your fix.
Status: NEW → ASSIGNED
Assignee | ||
Comment 8•25 years ago
|
||
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
Comment 10•25 years ago
|
||
Checked in. Ben, giving this back to you, you can mark it fixed.
Assignee: akkana → mozilla
Status: ASSIGNED → NEW
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 11•25 years ago
|
||
Thanks
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 12•25 years ago
|
||
whose gonna help me mark this bug verified-fixed???
Assignee | ||
Comment 13•25 years ago
|
||
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'.
You need to log in
before you can comment on or make changes to this bug.
Description
•