691211 - Incorrect encoding used when switching from guided to advanced bug entry form

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Nicolas Barbulesco]

Camino version 2.0.9 (1.9.0.19 2011091218). Mac OS X 10.6.8.

I go to https://bugzilla.mozilla.org and I submit a new bug. I enter this text :
“Move the tab to a new window” : do not reload the page
In the screen, my entered text appears fine.
In a next screen, I get this Summary field, not very pretty :
see screen photo http://screencast.com/t/R0F9mEnw66z .
This is incorrect. I must get the exact string I had entered.

Please fix that bug(s). In Camino or in Bugzilla ?
Thanks.

Nicolas

Comment 1

[Chris Lawson (gone)]

That's surely a "bug" in Bugzilla, although I'd argue that something might be broken in the encoding on your end, too, because I've never seen Bugzilla do that with curly quotes that I've entered.

Comment 2

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Bugzilla's ability to handle non-ASCII quotes in summaries has been a long-standing issue.

FWIW, it looks correct to me, though; perhaps there's an encoding mismatch on your end?

Comment 3

[Nicolas Barbulesco]

Now, similar scenarios, similar problems :

I enter this text :
“Save as…” : ask where to save

I get this Summary field, rather not pretty :
screen photo http://screencast.com/t/UzxgAjG6k

Here is my camino ;-) :
http://screencast.com/t/lKz5pkqFdSh

Nicolas

Comment 4

[Frédéric Buclin]

(In reply to Smokey Ardisson (back-ish; no bugmail - do not email) from comment #2)
> Bugzilla's ability to handle non-ASCII quotes in summaries has been a
> long-standing issue.

I don't know where this story comes from. Bugzilla supports UTF8 correctly since 3.2, i.e. several years ago.

Anyway, I cannot reproduce using Firefox 7.0.1. Please try with another browser too. I don't think this is an upstream bug, unless being reproducible with another browser. Maybe a bug in the bmo extension.

Comment 6

[Chris Lawson (gone)]

See comment 4 in bug 691212 for a possible explanation and STR. It does indeed sound like there's a bug somewhere in Bugzilla code.

Comment 7

[:glob ✱]

this only happens with some sequences, it took quite a bit of trial an error for me to find a broken sequence.

steps to reproduce:

1. visit https://bugzilla.mozilla.org/enter_bug.cgi?format=guided#h=dupes|Camino
2. enter "xx÷√" as a summary
3. click on "find similar issues"
4. click on "switch to the advanced bug entry form"

the summary is encoded with escape() as xx%F7%u221A, which isn't correct.  we should be using encodeURIComponent(), resulting in xx%C3%B7%E2%88%9A, which renders correctly on the advanced bug entry form.

Comment 8

[:glob ✱]

fixed:

Committing to: bzr+ssh://[e-mail]/bmo/4.0/
modified extensions/GuidedBugEntry/web/js/guided.js
Committed revision 7891.

Committing to: bzr+ssh://[e-mail]/bmo/4.2/
modified extensions/GuidedBugEntry/web/js/guided.js
Committed revision 7910.

note: the fix won't be visible on bugzilla.mozilla.org until the next code push.