528650 - \n in the summary of listed addons

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect, P3)

Description

[Vishal Kamdar [:vish_moz]]

Attachment: addon summary

STR:
1.create a new addon of type 'listed'
2. In summary field type something on the first line and hit enter/return and
type on the second line.
3. click update description
4. go to the addon listing -> You will see that the \n in the addon summary. |n
gets added automatically to the summary when you hit enter and update
description.

Comment 1

[Wil Clouser [:clouserw]]

stephend:  we saw this bug yesterday and I thought moved it to 5.4.  Do you know the bug to dupe this to?

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

Reopening; this is what Vishal reported in https://bugzilla.mozilla.org/show_bug.cgi?id=528077#c3 (which was different from what *thought* I found).

Comment 4

[Fred Wenzel [:wenzel]]

Attachment: Patch, rev. 1

Argh, who ever thought mysql_real_escaping the input data in the dev controller was a good idea? Anywho, this adds a special case to also unescape line breaks properly. I'll be so happy when Zamboni comes around.

Comment 5

[Fred Wenzel [:wenzel]]

Thanks: r61037.

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on https://preview.addons.mozilla.org/en-US/firefox/addon/49154/.