Attached patch fixes misspelled "availble", "unforetunately", "funtion" and several other words in Drupal core, basically in comments.

CommentFileSizeAuthor
#7 drupal_2083943_7.patch8.41 KBxano
#5 drupal_2083943_5.patch8.38 KBxano
drupal-grammar.patch9.33 KBinternetdevels

Comments

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Component: documentation » other

This only touches comments and one schema description, all looks good to me.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Component: other » documentation
Status: Needs review » Reviewed & tested by the community
xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Component: other » documentation

I agree with #1. This patch is good.

jhodgdon’s picture

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

Thanks! Yes, it all looks good, but we don't want to change things in core/vendor.

--- a/core/vendor/phpunit/phpunit-mock-objects/PHPUnit/Framework/MockObject/Builder/ParametersMatch.php
+++ b/core/vendor/phpunit/phpunit-mock-objects/PHPUnit/Framework/MockObject/Builder/ParametersMatch.php

You can file an upstream bug (maybe?) o that one.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Needs work » Needs review
StatusFileSize
new drupal_2083943_5.patch8.38 KB

I can't believe I missed that :X

jhodgdon’s picture

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

Looks good, thanks!

I noticed this:

+++ b/core/modules/user/lib/Drupal/user/Plugin/views/field/Mail.php
@@ -12,7 +12,7 @@
 
 /**
 /**
- * Field handler to provide acess control for the email field.
+ * Field handler to provide access control for the email field.

Extra credit if you remove that extra line... or maybe the committer can just take care of it. :)

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
StatusFileSize
new drupal_2083943_7.patch8.41 KB

Oooh, credits, nomnomnom!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, drupal_2083943_7.patch, failed testing.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs work » Needs review

#7: drupal_2083943_7.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal_2083943_7.patch, failed testing.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs work » Needs review

#7: drupal_2083943_7.patch queued for re-testing.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community
jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Title: Drupal core grammar minor fixes » Drupal core spelling minor fixes
Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 8.x.

Status: Fixed » Closed (fixed)

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