Closed Bug 418290 Opened 16 years ago Closed 12 years ago

SEC_ERROR_REVOKED_CERTIFICATE message has typo

Categories

(Core :: General, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla13

People

(Reporter: kjoonlee, Assigned: capella)

Details

(Whiteboard: [good first bug])

Attachments

(1 file, 2 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3) Gecko/2008020514 Firefox/3.0b3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3) Gecko/2008020514 Firefox/3.0b3

The message for SEC_ERROR_REVOKED_CERTIFICATE has

> The page you are trying to view can not be shown [...]

This should say "cannot", not "can not".



Reproducible: Always

Steps to Reproduce:
1. Try to get to a site with a revoked certificate
2. Observe error message
3. Notice typo
Actual Results:  
The page you are trying to view can not be shown [...]

Expected Results:  
The page you are trying to view cannot be shown [...]
Assignee: nobody → kengert
Component: Security → Security: PSM
OS: Windows XP → All
Product: Firefox → Core
QA Contact: firefox → psm
Hardware: PC → All
Version: unspecified → Trunk
Oh well, the source has "can not" all over the place, but

http://www.askoxford.com/asktheexperts/faq/aboutspelling/cannot?view=uk

AskOxford says "can not" is also acceptable (although it isn't for me).
The "can not be shown" string is not contained in the security portion of the code, but in file netError.dtd, which is contained in 3 places:
- firefox
- camino
- dom

I'm changing the component of this bug to Firefox.
If any change is made, the other products should follow.

However, I think, if both forms "can not" and "cannot" are equally allowed, and if the source already mixes it (as you say), maybe we should not bother...?
Assignee: kengert → nobody
Status: UNCONFIRMED → NEW
Component: Security: PSM → General
Ever confirmed: true
Product: Core → Firefox
QA Contact: psm → general
As Kyungjun Lee points out, both forms are technically acceptable, but if someone wanted to build an omnibus patch that harmonized our strings here, that would be nifty.
Whiteboard: good first bug
However, AskOxford.com is a bit vague, and other sources which discourage "can not" (except in emphasis of "not") are readily found.

I'll see what I can do.
Whiteboard: good first bug → [good first bug]
http://lxr.mozilla.org/mozilla1.8.0/search?string=cannot
http://lxr.mozilla.org/mozilla1.8.0/search?string=can%20not

It seems both are used, although cannot is much more common. can not is mostly in comments.
This is a patch file created to change "can not" to "cannot" in netError.dtd file.  This is my first bug and I would like to experience the process of creating and submitting patches.
Attachment #391348 - Flags: review?(kaie)
Attachment #391348 - Flags: review?(kaie) → review?(kjoonlee)
Tommy, thank you very much for getting started to work on Mozilla!

As I suggested in comment 2, I would prefer a "complete" patch.
That is, due to technical reasons, this very same string is currently used in at least 3 areas in the Mozilla source tree.

Would you be able to produce a patch that changes all three occurrences?
I think all 3 files are named netError.dtd. One of them might not be in the standard Firefox tree, but live in a Camino directory (a mac browser based on Firefox).
Comment on attachment 391348 [details] [diff] [review]
Change wording from can not to cannot in netError.dtd file

r-

Patch is correct but incomplete.
Attachment #391348 - Flags: review?(kjoonlee) → review-
For whatever you fix, can you do a corresponding query on the l10n mxr's to see which localizations should get follow-up bugs filed? 1.9.2 branch might be good enough, if there are not extra entries for SeaMonkey/Thunderbird (that's on 1.9.1)
This is a patch for netError.dtd in /browser and in /dom. Not in Camino, because Camino is not in mozilla-central. On irc #developers, Pike advised opening a seperate bug for this issue in Camino.

This is my first patch, and although it is a very simple one, I would welcome all feedback, to get familiar with the patching process at mozilla.
Attachment #419677 - Flags: review?(kaie)
I see "can not" to "cannot", should the same thing happen for "could not" to "couldn't"?
(In reply to comment #11)
> I see "can not" to "cannot", should the same thing happen for "could not" to
> "couldn't"?

That's a different story, as far as I know... "could not" is formal whereas "couldn't" is considered informal in writing.
(In reply to comment #9)
> For whatever you fix, can you do a corresponding query on the l10n mxr's to see
> which localizations should get follow-up bugs filed? 1.9.2 branch might be good
> enough, if there are not extra entries for SeaMonkey/Thunderbird (that's on
> 1.9.1)

I checked, and there are several (see: http://mxr.mozilla.org/l10n-mozilla1.9.2/search?string=can+not&find=netError.dtd)

The file netError.dtd with "can not" is present in the following folders in the l10n branch:
eo
en-GB
ta
or
oc
ta-LK
af
pa-IN
x-testing

I guess since they are all in the same branch, there can be one bug filed to fix all of them? If so, I will file that bug and create a patch for them. Is this indeed the way to go?

When we fix this bug for Camino as well, is everything then covered, or should other cases like "couldn't" be taken care of as well? My feeling is that "could not" is fine, but I'm not a native English speaker, so I'm not in the best position to have an opinion on that.
(In reply to comment #13)
> I guess since they are all in the same branch, there can be one bug filed to
> fix all of them? If so, I will file that bug and create a patch for them. Is
> this indeed the way to go?
> 
> When we fix this bug for Camino as well, is everything then covered, or should
> other cases like "couldn't" be taken care of as well? My feeling is that "could
> not" is fine, but I'm not a native English speaker, so I'm not in the best
> position to have an opinion on that.

(In reply to comment #12)
> (In reply to comment #11)
> > I see "can not" to "cannot", should the same thing happen for "could not" to
> > "couldn't"?
> 
> That's a different story, as far as I know... "could not" is formal whereas
> "couldn't" is considered informal in writing.

I must say it warms my heart to see three non-native English speakers debating the relative propriety of could vs. couldn't; you would all fit in quite well in the Toronto office, where we spend more time than is strictly required teasing apart the vagaries of this stupid language.

I agree with Dao - "Could not" doesn't look wrong to the eye the way "can not" does. Let's fix "can not" and move along.  Thanks for the patch, Bas (and earlier, Tommy).
I filed bug 541055 for fixing this in Camino (with patch), and bug 541053 for fixing this in l10n (patch coming when my question there is answered).
http://l10n.mozilla.org/dashboard/bugzilla.html can help you with filing the various bugs. Use %(loc)s in any form field where you need the locale code, i.e.

[%(loc)s] typo...

for the summary and in the comment, http://mxr.mozilla.org/l10n-mozilla1.9.2/search?string=can+not&find=%(loc)s/.*/netError.dtd to generate a good mxr link for each locale.

For x-testing, there's no bug needed, that's just our test data to trigger builds every now and then, nobody is ever getting real builds off of that.
All the affected localizations have a bug now, and all of them have a patch. What should happen now?
Now it's up to the localizers to pick up those fixes and get them into released bits.

I un-CC-ed myself on purpose, fwiw, no need for me (and the folks that watch me) to get bugmail on those.
Okay, thanks, and sorry for the spam on those other bugs :)
Comment on attachment 419677 [details] [diff] [review]
Change wording from can not to cannot in netError.dtd file

r=kaie
Attachment #419677 - Flags: review?(kaie) → review+
   I'm reviewing the [Good first bugs] and wonder if this old item is resolvable? 

   I also found two additional references to netError.dtd files with "can not" that don't appear in the patch:

      b2g\locales\en-US\chrome\overrides
      mobile\locales\en-US\overrides
Full list of affected files:

http://mxr.mozilla.org/mozilla-central/search?string=can+not&find=&filter=properties|dtd
Product: Firefox → Core
QA Contact: general → general
Added the properties file to the four DTD files
Attachment #595345 - Flags: review?(dao)
Comment on attachment 595345 [details] [diff] [review]
five files changed ... old typos fixed

Thanks!
Attachment #595345 - Flags: review?(dao) → review+
Assignee: nobody → markcapella
Keywords: checkin-needed
Attachment #419677 - Attachment is obsolete: true
Attachment #391348 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/3696e562509d

Thank you for the patch! :-)
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.