Closed Bug 742538 Opened 9 years ago Closed 9 years ago
Removing Filelink's from message body causes subsequent Filelink insertions to fail
STR: 1) In either HTML or Plaintext, create a Filelink 2) Remove the Filelink from the message body 3) Create another Filelink What happens? No Filelink gets inserted into the message body. What's expected? If the root/list nodes are not found in the message body, we should re-insert them.
Comment on attachment 612593 [details] [diff] [review] Patch v1 very nice - my one nit would be naming the method "assertIDsFree" since "assert" in our code generally has a different meaning. Maybe clearNodeIDs? resetNodeIDs?
Attachment #612593 - Flags: review?(dbienvenu) → review+
Thanks! Renamed function to "_resetNodeIDs", and fixed some bitrot. -Mike
Attachment #612593 - Attachment is obsolete: true
Attachment #615364 - Flags: approval-comm-aurora?
Committed to comm-central as http://hg.mozilla.org/comm-central/rev/974245c3a39b
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0
Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/9a3c572d0568
Attachment #615364 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.