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.
There's a misspelling that slipped into head (Well, homonym confusion on my part)
// Now test with an error message to insure it is correctly passed
"insure" should be "ensure" there.
Issue tagged as novice and quick fix since this is an excellent case for someone just getting used git and making patch files to tackle.
Comment | File | Size | Author |
---|---|---|---|
#6 | corrected_spelling-2565671-6.patch | 599 bytes | joyceg |
Comments
Comment #2
Aki Tendo CreditAttribution: Aki Tendo commentedComment #3
joyceg CreditAttribution: joyceg commentedComment #4
joyceg CreditAttribution: joyceg commentedThis issue is not seen in the given file.
Comment #5
Aki Tendo CreditAttribution: Aki Tendo commentedJust checked, it's still there.
Line 177 of the file core/modules/simpletest/src/Tests/SimpleTestTest.php
For context, here's the surrounding code.
Note - this error got popped in only a couple days ago - to see it (and do development on Drupal in general) you need to check out core from git: https://www.drupal.org/project/drupal/git-instructions
The packaged beta releases only occur once a month, and they rapidly fall behind the repository HEAD.
Comment #6
joyceg CreditAttribution: joyceg commentedAdding the patch here.
Comment #7
joyceg CreditAttribution: joyceg commentedComment #8
Aki Tendo CreditAttribution: Aki Tendo commentedK, looks good. I'll go ahead and mark as RTBC now since we're only dealing with comment text. Congratulations on your first patch.
Comment #9
joyceg CreditAttribution: joyceg commentedThank you for your help Aki Tendo.
Comment #10
webchickCommitted and pushed to 8.0.x. Thanks!
Comment #12
webchickOops, also crediting Aki Tendo, and moving to Documentation component.