Misspellings in schema files

I started with 'itok' which is only used in 4 files so that seemed reasonable to remove from the dictionary. And that is true for others with a small number of file changes. But then I found that 'linkset' is in json files so we can't add a comment there to ignore it. And 'datelist' is used in a lot of twig files and we haven't yet added cspell ignore lines there.

The MR shows the changes I made to that point. But now I think it would be better to move these to drupal.txt:

1. anyall
2. datelist - ~23 file include many twig files
3. daterange
4. filemime - About 35 usages in multiple subsystems and modules.
5. fulldate
6. groupby - ~19 files
7. groupwise
8. itok - 4 files
9. linkset Used in .json files
10. prerender
11. sitename
12. subquery - ~30 files
13. unpromote
14. unpublish
15. unsticky

and only change this one

1. fieldblock - this is in only in test files in layout_builder

We could make followup to change say 'datelist' to 'date_list' or 'anyall' to 'any_all' but does that add value?

If we stick to just those 2 (datelist and anyall) I think they could be handled here.

As far as the list to move to the drupal dictionary I think they all made sense, know personally some of those I've used often in conversation.

If we stick to just those 2 (datelist and anyall) I think they could be handled here.

Can you elaborate? There are no changes here for datelist.

Your last comment

We could make followup to change say 'datelist' to 'date_list' or 'anyall' to 'any_all' but does that add value?

Think they can be handled here.

But that doesn't answer the question if doing so adds value? Not to mention there will be BC to deal with.

To not hold this one up opened 📌 Determine value in update datelist and anyall Active

As stated in another one of these issues I don't think we should be adding these to config, as they are (pretty much) machine generated and don't care about retaining comments if they are recreated.

#3 makes a good point that most of these words look like they should be added to the Drupal dictionary.

I think these are all valid candidates:

• anyall (mostly because this will be hard to change)
• datelist (mostly because of the number of usages)
• daterange (this is a field type name, so a Drupalism)
• filemime (widely used)
• groupwise
• linkset (see https://www.rfc-editor.org/info/rfc9264)
• prerender
• subquery
• unpromote
• unpublish
• unsticky

Not so sure about itok (this is an abbreviation of "image token", but not a word) or fulldate, groupby or sitename (made up compound words).

#9. I moved the drupalisms to the drupal dictionary. Still need to work on the other words.

I am choosing to change the scope here and move these words to another issue. I'd like to explore these a bit more before deciding on a course of action and do not want to hold up progress here on that.

• datelist - ~23 file include many twig files
• filemime - About 35 usages in multiple subsystems and modules.
• linkset Used in .json files

It is important to notices that cspell fails on 'sitename' but the actual usage in core is 'sitename_title'. So, instead of adding sitename this adds 'sitename_title' to the drupal dictionary.

Formatting

Wonder if the title can actually be updated, maybe the issue summary some.

Appears there are changes to non schema files so not super clear.

It was just unpublish that is not in a schema file so I removed that from the MR.

anyall - core/modules/taxonomy/config/schema/taxonomy.views.schema.yml
autoplay - core/modules/file/config/schema/file.schema.yml
fieldblock - core/modules/layout_builder/tests/modules/layout_builder_fieldblock_test/config/schema/layout_builder_fieldblock_test.schema.yml
fulldate - core/modules/views/config/schema/views.argument.schema.yml
groupby - core/modules/views/config/schema/views.argument.schema.yml, core/modules/views/config/schema/views.filter.schema.yml
itok - core/modules/image/config/schema/image.schema.yml:
daterange - core/modules/datetime_range/config/schema/datetime_range.schema.yml
groupwise - core/modules/views/config/schema/views.relationship.schema.yml
prerender - core/modules/user/config/schema/user.views.schema.yml, core/modules/views/config/schema/views.field.schema.yml
sitename_title - core/modules/views/config/schema/views.display.schema.yml:
subquery - core/modules/views/config/schema/views.relationship.schema.yml:
unpromote - core/modules/node/config/schema/node.schema.yml
unsticky - core/modules/node/config/schema/node.schema.yml

Right but there are changes in non schema files too

Yes, that is true. These words are used in schema files and elsewhere. The common factor is that they are all in schema files. The title says 'used in schema files' not 'only used in schema files'.

Anyway title appears clearer and spelling changes seem fine.

Needs a rebase.

There were conflicts in both dictionaries. And in
core/modules/layout_builder/tests/modules/layout_builder_field_block_test/src/ContextProvider/FakeViewModeContext.php
core/modules/layout_builder/tests/modules/layout_builder_field_block_test/src/Plugin/Block/FieldBlock.php
due to adding declare strict types.

I then rebuilt both dictionaries to be sure.

Rebase seems good.

Rebased and rebuilt the dictionaries.

Some questions on the MR.

This one was a little confusing but believe questions have been answered, except maybe around itok but think ignoring should be fine for now.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

I have rebased this MR and encountered multiple conflicts. I carefully resolved all conflicts; however, this still requires further review. Therefore, I am moving it back to the "Needs Review" stage instead of "RTBC." Kindly review.

Using IS as a guide for what's being removed/moved don't think the MR lines up 100%

Working on it