500940 - "Ignore this check" broken with corner case

Status: RESOLVED FIXED

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

Description

[Wil Clouser [:clouserw]]

15:15 <@clouserw> Load Error: c9fd9d3 is not defined
15:15 <@clouserw> Source File: https://addons.mozilla.org/en-US/firefox/addon/10868
15:15 <@clouserw> er, yeah
15:16 <@clouserw> if you load that url ^^ with a not FF3.5b4 browser and you are logged in you can click "Ignore this check" where the download button should be
15:16 <@clouserw> I get a JS error when I try that.  anyone else?
15:19 <Mossop_away> Yeah
15:19 <Mossop_away> Error: missing ) after argument list
15:19 <Mossop_away> Source File: https://addons.mozilla.org/en-US/firefox/addon/10868
15:19 <Mossop_away> Line: 1, Column: 32
15:19 <Mossop_away> Source Code:
15:19 <Mossop_away> removeCompatibilityHint(71314-66b7b1ef);return false;
15:19 <Mossop_away> Perhaps there should be quotes around that
15:20 <@clouserw> thanks, I'll file a bug

Comment 1

[Ryan Doherty (:rdoherty)]

r53267