The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Problem/Motivation
Some descriptions Test files are malformed. For example :
/**
* @file
* Contains Drupal\Component\Render\FormattableMarkup.
*/Miss a \.
core/lib/Drupal/Core/Render/Placeholder/SingleFlushStrategy.php
/**
* @file
* Contains \Drupal\Core\Render\Placeholder\SingleFlushStrategy
*/Doesn't end in a period (.)
Proposed resolution
- Add this missing \ to all
testsdefinitions - Add the missing . to the end of all @file declarations
RC Target evaluation
Only changes comments to make them consistent. Not disruptive at all.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | interdiff-26300304-6-9.txt | 3.15 KB | marvin_b8 |
| #10 | bad_file_description-2600304-9.patch | 17.77 KB | marvin_b8 |
Comments
Comment #2
goz commentedComment #3
anil280988 commentedHi Goz,
4 files still remaining which still misses a \. Added a new patch which include these 4 filles.
-- a/core/modules/node/src/Tests/NodeOwnerTest.php
-- a/core/modules/system/src/Tests/Field/FieldItemTest.php
-- a/core/modules/system/src/Tests/Field/FieldItemTest.php
-- a/core/modules/system/src/Tests/Form/ElementsAccessTest.php
Comment #4
anil280988 commentedChanged the status
Comment #6
marvin_b8 commentedremove already fixed elements.
Comment #7
justachris commentedWe should also handle the cases where the @file description does not end with a period (.) since there are only a few of those.
This grep should find those:
grep -R '* Contains \\D.*[^.]$' core/There are also cases where the namespace or classname in the description does not match the file contents, but will separate that out into a different issue.
Comment #8
justachris commentedAdds an example of missing trailing period to issue summary.
Comment #9
justachris commentedAnd adding rc eligible tag since this issue should only clean up documentation.
Comment #10
marvin_b8 commentedComment #11
marvin_b8 commentedComment #12
jhodgdonThanks!
Comment #14
xjmI used
git diff --color-wordsto confirm that this patch only adds leading slashes or periods to the@filedocblocks. Thanks for the cleanup! Committed and pushed to 8.0.x.