Problem/Motivation

// Tests exception in thrown when profile_fields tables does not exist. should be // Tests exception in thrown when profile_fields tables do not exist. in /core/modules/user/tests/src/Kernel/Migrate/d6/ProfileFieldCheckRequirementsTest.php and /core/modules/user/tests/src/Kernel/Migrate/d7/ProfileFieldCheckRequirementsTest.php.

Proposed resolution

Replace "does" to "do", because the word "tables" is plural.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

paulocs created an issue. See original summary.

paulocs’s picture

Assigned: paulocs » Unassigned
Status: Active » Needs review
FileSize
722 bytes

A patch for it.

paulocs’s picture

Issue summary: View changes
FileSize
1.41 KB

New patch that includes the /core/modules/user/tests/src/Kernel/Migrate/d7/ProfileFieldCheckRequirementsTest.php file

longwave’s picture

Status: Needs review » Needs work

"exception in thrown" doesn't make sense, this should be "exception is thrown"?

raman.b’s picture

Status: Needs work » Needs review
FileSize
1.41 KB
1.27 KB
longwave’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

  • xjm committed 743cb62 on 9.2.x
    Issue #3176036 by paulocs, raman.b, longwave: Replace "does" to "do" in...

  • xjm committed 7af06a3 on 9.1.x
    Issue #3176036 by paulocs, raman.b, longwave: Replace "does" to "do" in...

  • xjm committed 58639a7 on 9.0.x
    Issue #3176036 by paulocs, raman.b, longwave: Replace "does" to "do" in...

  • xjm committed 64154d9 on 8.9.x
    Issue #3176036 by paulocs, raman.b, longwave: Replace "does" to "do" in...
xjm’s picture

Title: Replace "does" to "do" in ProfileFieldCheckRequirementsTest.php » Docblock grammar fixes in ProfileFieldCheckRequirementsTest.php
Version: 9.1.x-dev » 8.9.x-dev

Thanks for the patch! We don't really have a way to scan for such grammatical errors.

Committed to 9.2.x, and backported to 9.1.x, 9.0.x, and 8.9.x as a patch-eligible documentation fix. Thanks!

xjm’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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