Closed Bug 1289504 Opened 8 years ago Closed 8 years ago

Changing review state in "Finish review" replaces content with HTML code in case of escaped strings are used

Categories

(MozReview Graveyard :: Review Board: User Interface, defect)

Product:
MozReview Graveyard
Component:
Review Board: User Interface
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1283024

People

(Reporter: whimboo, Unassigned)

Details

Today I was working on the following review and added two links to the review text area. When I flipped the review state flags the content got replaced by HTML code.

https://reviewboard.mozilla.org/r/67108/#review63982

Here is what I did:
1. Open https://reviewboard.mozilla.org/r/67108/diff/1#index_header
2. Clicked on Finish at the top
3. Add a link to the text area, e.g. https://dxr.mozilla.org/mozilla-central/search?q=platformName+path%3Atesting%2Fpuppeteer&redirect=true
4. Switch the review state

You will see that the content gets replaced by using <p> tags. Also the text gets HTML encoded.

Sidenote: If you discard or close the review, and reopen it afterward the text seems to be encoded each time again and again. I was able to see it once.
Looks like we had this on file, although it's been reported a lot more in the recent days.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.