Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#5 | interdiff_3-5.txt | 1.27 KB | raman.b |
#5 | 3176036-5.patch | 1.41 KB | raman.b |
#3 | 3176036-3.patch | 1.41 KB | paulocs |
#2 | 3176036-2.patch | 722 bytes | paulocs |
Comments
Comment #2
paulocsA patch for it.
Comment #3
paulocsNew patch that includes the /core/modules/user/tests/src/Kernel/Migrate/d7/ProfileFieldCheckRequirementsTest.php file
Comment #4
longwave"exception in thrown" doesn't make sense, this should be "exception is thrown"?
Comment #5
raman.b CreditAttribution: raman.b at OpenSense Labs commentedComment #6
longwaveThanks!
Comment #11
xjmThanks 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!
Comment #12
xjm