1293704 - Spell checker thinks "adoptee" is a misspelling

Status: RESOLVED FIXED

Description

[Matthew Spencer]

Created attachment 8779423 [details]
Spell checker thinks "adoptee" is a misspelling

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:47.0) Gecko/20100101 Firefox/47.0
Build ID: 20160623154057

Steps to reproduce:

1. Visit http://pastebin.mozilla.org/
2. Type "adoptee" into the text box






Actual results:

 Squiggly red underline indicating misspelling


Expected results:

No squiggly red underline indicating misspelling.

For reference for the spelling of "adoptee", please see http://www.merriam-webster.com/dictionary/adoptee or http://www.dictionary.com/browse/adoptee

(currently I am unable to install Xcode on my machine otherwise I would follow the MDN procedure for adding a new word to the en-US dictionary.)

Comment 1

[Jorg K (GMT+2)]

We should add this since it seems common enough, has a high SCOWL rating and is in en_US-large:
http://app.aspell.net/lookup?dict=en_US;words=adoptee

Comment 2

[Matthew Spencer]

Hi, I was finally able to get my dev environment set up on my computer and I am trying to add the word per instructions here: https://developer.mozilla.org/en-US/docs/Mozilla/Adding_a_new_word_to_the_en-US_dictionary

I am using mozilla-central via: hg clone https://hg.mozilla.org/mozilla-central

I am able to successfully add the word to "5-mozilla-added" but when I quit my editor and try to run the command "sh merge-dictionaries" as described in the above tutorial from the directory extensions/spellcheck/locales/en-US/hunspell/dictionary-sources it says: "sh: merge-dictionaries: No such file or directory"

I see that there is a "make-new-dict" and "install-new-dict" in the directory but not "merge-dictionaries". What am I missing here to merge the word I added and proceed with moving the revised dictionary file? Any help is greatly appreciated.

Comment 3

[Jorg K (GMT+2)]

To add a single word to the dictionary, add it to en-US.dic and don't forget to increase the count at the very top. This change here is a two-liner.

All those scripts are for bulk import of new dictionaries from the upstream source (SCOWL).

For your case, see
https://hg.mozilla.org/mozilla-central/diff/e42e07e4f668/extensions/spellcheck/locales/en-US/hunspell/en-US.dic
as an example of a single word addition.

Comment 4

[Jorg K (GMT+2)]

This
https://developer.mozilla.org/en-US/docs/Mozilla/Adding_a_new_word_to_the_en-US_dictionary
seems 100% out of date.

Comment 5

[Jorg K (GMT+2)]

Actually, you should add this line:
adoptee/MS
The suffixes M and S are for plural (adoptees) and possessive form (adoptee's) (or vice versa).

Comment 6

[Matthew Spencer]

Fantastic. Thank you for clarifying and sharing the example.

Comment 7

[Matthew Spencer]

Created attachment 8793850 [details] [diff] [review]
Proposed Patch - Spelling Checker Word Addition

My apologies but I was not able to submit patch via bzexport. Kept receiving the error: "abort: Local changes found; refresh first!" after running hg bzexport -e

Proposed patch for spelling checker has been edited per instructions in comments.

I would like to streamline this process for future additions. Any help or documentation would be greatly appreciated. Thank you for your patience

Comment 8

[Jorg K (GMT+2)]

You need to attach a patch for review, not the edited file.

You can use mercurial queues to create one:
hg qnew -m "Bug 1293704 - added adoptee/MS. r=ehsan" 1293704.patch.

Your reviewer will be Ehsan ([e-mail]).

Comment 9

[Matthew Spencer]

All set now. When I tried mercurial's qnew previously, I received "hg: unknown command 'qnew'" because I had mq disabled. Enabled mq and was able to create patch. Will attach patch file. Thanks!

Comment 10

[Matthew Spencer]

Created attachment 8793880 [details] [diff] [review]
Proposed Patch - Added adoptee/MS to spelling checker

Comment 11

[Jorg K (GMT+2)]

Comment on attachment 8793880 [details] [diff] [review]
Proposed Patch - Added adoptee/MS to spelling checker

Please check patches before attaching them for review.

Did you not see that you saved the file as UTF-8 where it was ANSI encoded before? You destroyed every single word with an accent, and you can see it in the patch, just look:
-Asunción/M
+Asunci�n/M

Comment 12

[Matthew Spencer]

Apologies - amateur mistake. My editor defaulted to UTF-8. Will resubmit

Comment 13

[Matthew Spencer]

Created attachment 8793967 [details] [diff] [review]
Proposed Patch - Added adoptee/MS to spelling checker

Updated proposed patch with correct encoding format. Thank you for your patience.

Comment 14

[Jorg K (GMT+2)]

Comment on attachment 8793967 [details] [diff] [review]
Proposed Patch - Added adoptee/MS to spelling checker

Yep ;-)

Comment 15

[Jorg K (GMT+2)]

Actually, you don't seem to have Mercurial set up correctly. To land the patch, it needs the author noted in it. My patches for example look have this:

# HG changeset patch
# User Jorg K <[e-mail]>
# Parent  5644818538de2413cce52551e32b025e6c7e352e
Bug 301712 - Remove uncommon proper names, remove unneeded words. r=ehsan

Another question. When this bug is done, maybe we can interest you in bug 1255327. That's a little harder since you'd have to add canonicalization/MS and also the verb, canonicalize which comes with the usual suffixes:
canonicalized, canonicalizes, canonicalizing.

Comment 16

[Matthew Spencer]

Hmmn - Looks like on my Global Config ("$HOME/.hgrc"): 
[ui]
username = m-r-m-s <[e-mail]>

but local repository at mozilla-central/.hg/hgrc is an example configuration file. I guess that is overriding global config? Will try and sort out and resubmit. 

Yes, I am interested in bug 1255327 once this bug is complete

Comment 17

[Jorg K (GMT+2)]

I have that in mercurial.ini

Comment 18

[:Ehsan Akhgari]

Comment on attachment 8793967 [details] [diff] [review]
Proposed Patch - Added adoptee/MS to spelling checker

Review of attachment 8793967 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

Comment 19

[Jorg K (GMT+2)]

(In reply to litewrite from comment #16)
> Will try and sort out and resubmit. 

Coming up?

When you do, you can add the review+ yourself and add as a comment:
Added user name to patch. Carrying forward Ehsan's r+.
Then you need to add the keyword "checkin-needed" so a sheriff will take the patch and commit it to the code base.

Comment 20

[Matthew Spencer]

Spent most of yesterday trying to figure out why User comments weren't being included in patch despite having the [ui] populated in my .hgrc file (mac os x equilvalent of the mercurial.ini). Still had no success. Added the User comments manually to patch, then did a "hg qrefresh". I will re-submit here shortly per your most recent instructions. Really appreciate all the help so far!

Comment 21

[Matthew Spencer]

Created attachment 8794547 [details] [diff] [review]
Proposed Patch - Added adoptee/MS to spelling checker

Added user name to patch. Carrying forward Ehsan's r+.

Comment 22

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e84d837b043e
added adoptee/MS. r=ehsan

Comment 23

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e84d837b043e