Problem/Motivation

Fix spelling of words only misspelled in tests, part 1

I found these words by applying #3379121: Skip spellcheck for 20 nonsense words that are parts of hashes and #3328741: Add a dictionary for Drupal-specific words and then searching core test for the words in the dictionary. And also avoided the more challenging words, like the one that have a 'able' suffix or an 'un prefix.

Steps to reproduce

Proposed resolution

Correct spelling for these words and remove word from dictionary.

  1. abcdefghijklmno
  2. adminlabel
  3. amet
  4. argaven
  5. barfoo
  6. bazinga
  7. bazqux
  8. bazs
  9. blist
  10. blub
  11. cagatio
  12. camelcase
  13. cfield
  14. childlist
  15. derp
  16. drivertestpqsql
  17. goldfinger
  18. heroless
  19. imagetest
  20. machinename
  21. magrathea
  22. makealternatemessage
  23. makedefaultmessage
  24. makewarningmessage
  25. moby
  26. multiwidgetfield
  27. parentlist
  28. roly
  29. rulesets
  30. uppercased

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3389283

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue summary: View changes

quietone opened merge request !4861

quietone’s picture

quietone commented
Issue summary: View changes
quietone’s picture

quietone commented
Status: Active » Needs review
quietone’s picture

quietone commented
Issue tags: +cspell error
quietone’s picture

quietone commented
quietone’s picture

quietone commented
Issue summary: View changes
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Reviewed the changes and they look good. Nothing doesn't not make sense.

xjm made their first commit to this issue’s fork.

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Needs work

Merge conflict after #3389286: Fix spelling of words only misspelled in tests, part 2.

quietone’s picture

quietone commented
Status: Needs work » Reviewed & tested by the community

Rebased and rebuilt the dictionary.

  • longwave committed 040f4f83 on 10.2.x 
    Issue #3389283 by quietone: Fix spelling of words only misspelled in...

  • longwave committed 36ab4565 on 11.x 
    Issue #3389283 by quietone: Fix spelling of words only misspelled in...
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 11.x and 10.2.x. Did not backport cleanly to 10.1.x, don't think it's worth the effort.

quietone closed merge request !4861

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.