293253 - Reporter's privacy statement checkbox labeled very badly

Status: RESOLVED WONTFIX

(Other Applications Graveyard :: Reporter, defect)

Description

[James Ross]

The label for the privacy policy checkbox on the first page says:
  "Don't Show Privacy Statement Again"

Yet, I MUST check this to continue. It should say that I agree to the privacy
policy, as I think was the point. If it really does mean "don't show me this
again" then it should NOT affect whether I can continue.

Comment 1

[Robert Accettura [:raccettura]]

I'm thinking of changing it to:
I read and agree to the above privacy policy.

any objections?

I may land that tomorrow.

Comment 2

[Asa Dotzler [:asa]]

sounds good to me. a=asa

Comment 3

[Robert Accettura [:raccettura]]

Per discussion with mconnor, this is betterer than my earlier suggestion:

I have read and accept the above privacy policy.

Comment 4

[Robert Accettura [:raccettura]]

Checking in extensions/reporter/locales/en-US/chrome/reportWizard.dtd;
/cvsroot/mozilla/extensions/reporter/locales/en-US/chrome/reportWizard.dtd,v 
<--  reportWizard.dtd
new revision: 1.5; previous revision: 1.4
done

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

*** Bug 294489 has been marked as a duplicate of this bug. ***

Comment 6

[Axel Hecht [:Pike]]

Do we have more goodies like this in reporter?

Crappy code gives crappy localizations, of course we have at least one
localization a'la "Don't Show Privacy Statement Again" in our 1.5 release, see
bug 317624.
The entity is misnamed, that should be fixed on the trunk.

I'd really like to know if I need to hunt for more problems like this or if
we just have this in one place.

Comment 7

[Vidar Haarr (not reading bugmail)]

I found bug 317624 doing last-minute QA for nb-NO.

The reason we decided it was a showstopper for nb-NO was that the user doesn't currently accept the privacy policy. He might, in theory, sue the Mozilla Foundation after finding that the Reporter tool collects private information, saying "I never agreed to that."

It's a bit extreme, I know, but AFAIK it's a valid concern.

As painful as this might be, I think this calls for an audit of all 1.5 localizations.

Comment 8

[Robert Accettura [:raccettura]]

Fine, I'll tweak that entity on the trunk tomorrow morning.

I don't see how a string change in May can be causing problems in November.  The string is supposed to be localized based on the string alone, not the entity name.  FWIW I should be able to use an md5 hash to name a string, with no ill effects.

Based on bug 317624 (I rely on the translations within) it looks like nb-NO just never updated that string since sometime before May 15, 2005.  Nothing to do with the entity name.  I checked several other's using google translate, and they appeared to be updated (mainly looking for the word "agree" as that's the big change).  Looks like it was checked in at 2005-07-21 08:37.  IMHO it should be reviewed before checkin.

Comment 9

[Robert Accettura [:raccettura]]

Correction to above 2005-07-21 08:37 was a move, not initial checkin.

Comment 10

[Vidar Haarr (not reading bugmail)]

(In reply to comment #8)
> I don't see how a string change in May can be causing problems in November.

We never noticed that the string changed after translating the original one.

The best way to ensure that it gets on radar is to CC firefoxl10n or similar on relevant bugs.

Comment 11

[Robert Accettura [:raccettura]]

Attachment: rename entity

Comment 12

[Sylvestre Ledru [:Sylvestre]]

Reporter isn't a maintained project. Closing!